Fixes userGroup membership form xibosignage/xibo#1653

PeterMis · Oct 24, 2018 · cd51f9f · cd51f9f
1 parent 9d7776a
commit cd51f9f
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/lib/Controller/UserGroup.php b/lib/Controller/UserGroup.php
@@ -683,6 +683,7 @@ public function assignUser($groupId)
         $this->getLog()->debug('Assign User for groupId %d', $groupId);
 
         $group = $this->userGroupFactory->getById($groupId);
+        $group->load();
 
         if (!$this->isEditable($group))
             throw new AccessDeniedException();
@@ -699,6 +700,7 @@ public function assignUser($groupId)
                 throw new AccessDeniedException(__('Access Denied to User'));
 
             $group->assignUser($user);
+            $group->save(['validate' => false]);
         }
 
         // Check to see if unassign has been provided.
@@ -714,9 +716,9 @@ public function assignUser($groupId)
                 throw new AccessDeniedException(__('Access Denied to User'));
 
             $group->unassignUser($user);
+            $group->save(['validate' => false]);
         }
 
-        $group->save(['validate' => false]);
 
         // Return
         $this->getState()->hydrate([