Issue 48499: Use preferred SecurityPolicyManager.savePolicy() variant (#178)

Author: labkey-jeckels

mGAP/src/org/labkey/mgap/mGAPController.java

@@ -434,7 +434,7 @@ public Object execute(ApproveUserRequestsForm form, BindException errors) throws
                     }
                 }
 
-                SecurityPolicyManager.savePolicy(policy);
+                SecurityPolicyManager.savePolicy(policy, getUser());
 
                 transaction.commit();
             }

mcc/src/org/labkey/mcc/MccController.java

@@ -572,31 +572,31 @@ public boolean handlePost(Object o, BindException errors) throws Exception
                 {
                     MutableSecurityPolicy policy = new MutableSecurityPolicy(requestContainer.getPolicy());
                     policy.addRoleAssignment(requestGroup, RoleManager.getRole(MccRequesterRole.class));
-                    SecurityPolicyManager.savePolicy(policy);
+                    SecurityPolicyManager.savePolicy(policy, getUser());
                 }
 
                 Group reviewGroup = GroupManager.getGroup(ContainerManager.getRoot(), MccManager.REQUEST_REVIEW_GROUP_NAME, GroupEnumType.SITE);
                 if (!requestContainer.getPolicy().getAssignedRoles(reviewGroup).contains(RoleManager.getRole(MccRabReviewerRole.class)))
                 {
                     MutableSecurityPolicy policy = new MutableSecurityPolicy(requestContainer.getPolicy());
                     policy.addRoleAssignment(reviewGroup, RoleManager.getRole(MccRabReviewerRole.class));
-                    SecurityPolicyManager.savePolicy(policy);
+                    SecurityPolicyManager.savePolicy(policy, getUser());
                 }
 
                 Group finalGroup = GroupManager.getGroup(ContainerManager.getRoot(), MccManager.FINAL_REVIEW_GROUP_NAME, GroupEnumType.SITE);
                 if (!requestContainer.getPolicy().getAssignedRoles(finalGroup).contains(RoleManager.getRole(MccFinalReviewerRole.class)))
                 {
                     MutableSecurityPolicy policy = new MutableSecurityPolicy(requestContainer.getPolicy());
                     policy.addRoleAssignment(finalGroup, RoleManager.getRole(MccFinalReviewerRole.class));
-                    SecurityPolicyManager.savePolicy(policy);
+                    SecurityPolicyManager.savePolicy(policy, getUser());
                 }
 
                 Group adminGroup = GroupManager.getGroup(ContainerManager.getRoot(), MccManager.ADMIN_GROUP_NAME, GroupEnumType.SITE);
                 if (!requestContainer.getPolicy().getAssignedRoles(adminGroup).contains(RoleManager.getRole(MccDataAdminRole.class)))
                 {
                     MutableSecurityPolicy policy = new MutableSecurityPolicy(requestContainer.getPolicy());
                     policy.addRoleAssignment(adminGroup, RoleManager.getRole(MccDataAdminRole.class));
-                    SecurityPolicyManager.savePolicy(policy);
+                    SecurityPolicyManager.savePolicy(policy, getUser());
                 }
             }
 
@@ -608,7 +608,7 @@ public boolean handlePost(Object o, BindException errors) throws Exception
                 {
                     MutableSecurityPolicy policy = new MutableSecurityPolicy(dataContainer.getPolicy());
                     policy.addRoleAssignment(adminGroup, RoleManager.getRole(MccDataAdminRole.class));
-                    SecurityPolicyManager.savePolicy(policy);
+                    SecurityPolicyManager.savePolicy(policy, getUser());
                 }
             }