From 60c1031cc10e3f8dcf4250853616f74f1f6c1464 Mon Sep 17 00:00:00 2001
From: Leonid Andreev <leonid@hmdc.harvard.edu>
Date: Wed, 24 May 2023 10:27:32 -0400
Subject: [PATCH] a couple of extra fixes. #8889

---
 src/main/java/edu/harvard/iq/dataverse/api/Dataverses.java | 3 ++-
 src/test/java/edu/harvard/iq/dataverse/api/FilesIT.java    | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/main/java/edu/harvard/iq/dataverse/api/Dataverses.java b/src/main/java/edu/harvard/iq/dataverse/api/Dataverses.java
index 06747a9f142..9a09a298ac5 100644
--- a/src/main/java/edu/harvard/iq/dataverse/api/Dataverses.java
+++ b/src/main/java/edu/harvard/iq/dataverse/api/Dataverses.java
@@ -588,7 +588,7 @@ public Response deleteDataverse(@Context ContainerRequestContext crc, @PathParam
     @AuthRequired
     @Path("{identifier}/attribute/{attribute}")
     public Response updateAttribute(@Context ContainerRequestContext crc, @PathParam("identifier") String identifier,
-                                    @PathParam("attribute") String attribute, @NotNull String value) {
+                                    @PathParam("attribute") String attribute, @QueryParam("value") String value) {
         try {
             Dataverse collection = findDataverseOrDie(identifier);
             User user = getRequestUser(crc);
@@ -622,6 +622,7 @@ public Response updateAttribute(@Context ContainerRequestContext crc, @PathParam
                  */
                 case "filePIDsEnabled":
                     collection.setFilePIDsEnabled(parseBooleanOrDie(value));
+                    break;
                 default:
                     return badRequest("'" + attribute + "' is not a supported attribute");
             }
diff --git a/src/test/java/edu/harvard/iq/dataverse/api/FilesIT.java b/src/test/java/edu/harvard/iq/dataverse/api/FilesIT.java
index 8b0afab01d6..476cf16854c 100644
--- a/src/test/java/edu/harvard/iq/dataverse/api/FilesIT.java
+++ b/src/test/java/edu/harvard/iq/dataverse/api/FilesIT.java
@@ -2079,7 +2079,7 @@ public void testFilePIDsBehavior() {
         
         // And confirm that the file didn't get a PID:
         
-        fileInfoResponse = UtilIT.getFileData(origFileId.toString(), apiToken);
+        fileInfoResponse = UtilIT.getFileData(newFileId.toString(), apiToken);
         fileInfoResponse.then().assertThat().statusCode(OK.getStatusCode());
         fileInfoResponseString = fileInfoResponse.body().asString();
         msg(fileInfoResponseString);