#700 make sure only access option has changed

AtlasOfLivingAustralia · Oct 6, 2023 · 175ce3b · 175ce3b
1 parent f7f6f44
commit 175ce3b
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 18 deletions.
diff --git a/grails-app/assets/javascripts/profileEditor/controllers/UserAccessController.js b/grails-app/assets/javascripts/profileEditor/controllers/UserAccessController.js
@@ -80,21 +80,23 @@ profileEditor.controller('UserAccessController', function (messageService, util,
         });
     };
 
-    self.privateModeChanged = function(form) {
+    self.privateModeChanged = function(form, isOptionChanged) {
         if (self.opus.privateCollection) {
             self.roles.push(userRole);
         } else {
-            var hasRole_USER = self.users.find(it=>it.role === 'ROLE_USER')
-            if (hasRole_USER) {
-                self.users = self.users.filter(it => it.role !== 'ROLE_USER')
-                var data = {privateCollection: self.opus.privateCollection, authorities: self.users};
-                var promise = profileService.updateUsers(self.opusId, data);
-                promise.then(function () {
-                    form.$setPristine();
-                    messageService.success("User access has been successfully updated.");
-                }, function () {
-                    messageService.alert("An error has occurred while updating user access.");
-                });
+            if (isOptionChanged) {
+                var hasRole_USER = self.users.find(it => it.role === 'ROLE_USER')
+                if (hasRole_USER) {
+                    self.users = self.users.filter(it => it.role !== 'ROLE_USER')
+                    var data = {privateCollection: self.opus.privateCollection, authorities: self.users};
+                    var promise = profileService.updateUsers(self.opusId, data);
+                    promise.then(function () {
+                        form.$setPristine();
+                        messageService.success("User access has been successfully updated.");
+                    }, function () {
+                        messageService.alert("An error has occurred while updating user access.");
+                    });
+                }
             }
             self.roles.splice(4, 1)
         }
@@ -124,10 +126,10 @@ profileEditor.controller('UserAccessController', function (messageService, util,
     }
 
     self.reset = function (form) {
-        loadOpus(form, true);
+        loadOpus(form);
     };
 
-    function loadOpus(form, isReset) {
+    function loadOpus(form) {
         if (!self.opusId) {
             return;
         }
@@ -143,9 +145,7 @@ profileEditor.controller('UserAccessController', function (messageService, util,
                     popupateUserDetails(user);
                 });
 
-                if (isReset) {
-                    self.privateModeChanged(form);
-                }
+                self.privateModeChanged();
 
                 if (form) {
                     form.$setPristine();

diff --git a/grails-app/views/opus/_editAccessControl.gsp b/grails-app/views/opus/_editAccessControl.gsp
@@ -21,7 +21,7 @@
                         <div class="checkbox padding-bottom-1">
                             <label for="privateCollection" class="inline-label">
                                 <input id="privateCollection" type="checkbox" name="privateCollection"
-                                    ng-change="userCtrl.privateModeChanged(UserForm)"
+                                    ng-change="userCtrl.privateModeChanged(UserForm, true)"
                                        ng-model="userCtrl.opus.privateCollection" ng-false-value="false">
                                 Make this collection private
                             </label>