Fix the user update method

d0de62e0 · Nicolas Widart · 5efbec0c · d0de62e0
Commit d0de62e0 authored Nov 26, 2014 by Nicolas Widart
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 5 deletions

SentryUserRepository.php Modules/User/Repositories/Sentry/SentryUserRepository.php +5 -5

No files found.
--- a/Modules/User/Repositories/Sentry/SentryUserRepository.php
+++ b/Modules/User/Repositories/Sentry/SentryUserRepository.php
@@ -74,12 +74,12 @@ class SentryUserRepository implements UserRepository
    public function updateAndSyncRoles($userId, $data, $roles)
    {
        $user = Sentry::findUserById($userId);
-        $user = $user->update($data);
-        $user->save();
+        $user->update($data);
        if (!empty($roles)) {
-            $adminGroup = Sentry::findGroupByName($roles);
-            $user->removeGroup();
-            $user->addGroup($adminGroup);
+            foreach ($roles as $roleId) {
+                $group = Sentry::findGroupById($roleId);
+                $user->addGroup($group);
+            }
        }
    }