GitHub Issue 1300: BulkUpgradeGroupAction scoping changes

Author: labkey-jeckels

core/src/org/labkey/core/security/SecurityApiActions.java

@@ -1106,6 +1106,12 @@ public ApiResponse execute(GroupForm form, BindException errors)
                 throw new UnauthorizedException("You do not have permission to modify site-wide groups.");
             }
 
+            // A project group must belong to the current container
+            if (_group.getContainer() != null && !container.getId().equals(_group.getContainer()))
+            {
+                throw new UnauthorizedException("The specified group does not belong to this project.");
+            }
+
             SecurityController.verifyUserCanModifyGroup(_group, getUser());
 
             Map<String, String> memberErrors = new HashMap<>();
@@ -1219,6 +1225,11 @@ private void addOrReplaceMembers(GroupForm form, Map<String, List<UserPrincipal>
 
                     if (principal == null && member.getEmail() != null) // create the user
                     {
+                        if (!getContainer().hasPermission(getUser(), AddUserPermission.class))
+                        {
+                            memberErrors.put(member.getEmail(), "You do not have permission to create new users.");
+                            continue;
+                        }
                         try
                         {
                             SecurityManager.NewUserStatus status = SecurityManager.addUser(new ValidEmail(member.getEmail()), getUser());