Updated image metadata endpoint to include opusId

AtlasOfLivingAustralia · Dec 11, 2023 · a87c340 · a87c340
1 parent df8031e
commit a87c340
Show file tree

Hide file tree

Showing 7 changed files with 13 additions and 14 deletions.
diff --git a/gradle.properties b/gradle.properties
@@ -1,4 +1,4 @@
-profileHubVersion=4.2
+profileHubVersion=4.2.1-SNAPSHOT
 grailsVersion=5.2.4
 grailsGradlePluginVersion=5.2.3
 groovyVersion=3.0.11

diff --git a/grails-app/assets/javascripts/profileEditor/directives/imageUpload.js b/grails-app/assets/javascripts/profileEditor/directives/imageUpload.js
@@ -103,7 +103,7 @@ profileEditor.directive('imageUpload', function ($browser, $http, config) {
                   $scope.metadata.created = util.formatLocalDate($scope.metadata.created);
                   $cacheFactory.get('$http').removeAll();
 
-                  profileService.saveImageMetadata($scope.image.imageId, $scope.metadata).then(function(data) {
+                  profileService.saveImageMetadata($scope.opusId, $scope.image.imageId, $scope.metadata).then(function(data) {
                       if (angular.isDefined($scope.callbackHandler)) {
                           $scope.callbackHandler(data);
                       }

diff --git a/grails-app/assets/javascripts/profileEditor/services/ProfileService.js b/grails-app/assets/javascripts/profileEditor/services/ProfileService.js
@@ -586,12 +586,12 @@ profileEditor.service('profileService', function ($http, util, $cacheFactory, co
             return util.toStandardPromise(future);
         },
 
-        saveImageMetadata: function(imageId, data) {
+        saveImageMetadata: function(opusId, imageId, data) {
             $log.debug("Saving image metadata: " + imageId);
             var future = null;
-            if (imageId) {
+            if (opusId && imageId) {
                 future = enqueue(function() {
-                    return $http.post(util.contextRoot() + "/image/" + imageId + "/metadata", data)
+                    return $http.post(util.contextRoot() + "/opus/" + opusId + "/image/" + imageId + "/metadata", data)
                 });
             }
 

diff --git a/grails-app/controllers/au/org/ala/profile/hub/ProfileController.groovy b/grails-app/controllers/au/org/ala/profile/hub/ProfileController.groovy
@@ -172,10 +172,10 @@ class ProfileController extends BaseController {
     @Secured(role = ROLE_PROFILE_EDITOR)
     def updateLocalImageMetadata() {
         def metadata = request.getJSON()
-        if (!params.imageId || !metadata) {
-            badRequest "imageId or metadata is required"
+        if (!params.opusId || !params.imageId || !metadata) {
+            badRequest "opusId, imageId and metadata is required"
         } else {
-            handle imageService.updateLocalImageMetadata(params.imageId, metadata)
+            handle imageService.updateLocalImageMetadata(params.opusId, params.imageId, metadata)
         }
     }
 

diff --git a/grails-app/controllers/au/org/ala/profile/hub/UrlMappings.groovy b/grails-app/controllers/au/org/ala/profile/hub/UrlMappings.groovy
@@ -122,6 +122,7 @@ class UrlMappings {
 
         "/opus/$opusId/statistics" controller: "statistics", action: [GET: "index"]
         "/opus/$opusId/image/$filename" controller: "opus", action: [GET: "downloadImage", DELETE: "deleteImage"]
+        "/opus/$opusId/image/$imageId/metadata" controller: "profile", action: [POST: "updateLocalImageMetadata"]
         "/opus/$opusId/image" controller: "opus", action: [POST: "uploadImage", PUT: "uploadImage"]
 
         "/opus/$opusId/update" controller: "opus", action: [GET: "edit", POST: "updateOpus"]
@@ -187,8 +188,6 @@ class UrlMappings {
         "/publication/$pubId" controller: "profile", action: [GET: "getPublication"]
         "/publication/$pubId/json" controller: "profile", action: [GET: "getPublicationJson"]
 
-        "/image/$imageId/metadata" controller: "profile", action: [POST: "updateLocalImageMetadata"]
-
         "/" controller: "opus", action: [GET: "index"]
 
         "/logout/logout" controller: "logout", action: "logout"

diff --git a/grails-app/services/au/org/ala/profile/hub/ImageService.groovy b/grails-app/services/au/org/ala/profile/hub/ImageService.groovy
@@ -668,8 +668,8 @@ class ImageService {
         return ImageOption.byName(displayOptionStr, defaultOption) == ImageOption.INCLUDE
     }
 
-    def updateLocalImageMetadata(String imageId, Map metadata) {
-        webService.post("${grailsApplication.config.profile.service.url}/image/${encPath(imageId)}/metadata", metadata, [:], ContentType.APPLICATION_JSON, true, false, profileService.getCustomHeaderWithUserId())
+    def updateLocalImageMetadata(String opusId, String imageId, Map metadata) {
+        webService.post("${grailsApplication.config.profile.service.url}/opus/${opusId}/image/${encPath(imageId)}/metadata", metadata, [:], ContentType.APPLICATION_JSON, true, false, profileService.getCustomHeaderWithUserId())
     }
 
     def publishPrivateImage(String opusId, String profileId, String imageId) {

diff --git a/src/test/js/specs/services/ProfileServiceSpec.js b/src/test/js/specs/services/ProfileServiceSpec.js
@@ -428,9 +428,9 @@ describe("ProfileService tests", function () {
 
     it("should invoke the updateLocalImageMetadata service on the context root when saveImageMetadata is called", function() {
         var data = {creator: "one", createdDate: new Date(), rights: "rights", licence: "licence"};
-        service.saveImageMetadata('abcdefghijlkmnop', data);
+        service.saveImageMetadata('opusId', 'abcdefghijlkmnop', data);
 
-        http.expectPOST("/someContext/image/abcdefghijlkmnop/metadata", data).respond("bla");
+        http.expectPOST("/someContext/opus/opusId/image/abcdefghijlkmnop/metadata", data).respond("bla");
     });
 
     it("should invoke GET request when getImageMetadata is called", function() {