diff --git a/app/Http/Controllers/Auth/RegisterController.php b/app/Http/Controllers/Auth/RegisterController.php index ebc31b21d..53fabce55 100644 --- a/app/Http/Controllers/Auth/RegisterController.php +++ b/app/Http/Controllers/Auth/RegisterController.php @@ -79,7 +79,7 @@ class RegisterController extends Controller $role = Role::query()->where('id', '=', $data['roles_id'])->first(); if ($role !== null) { - $user->assignRole($role->name); + $user->syncRoles([$role->name]); if ($user->can('view console')) { $user->givePermissionTo('view console'); } diff --git a/app/Models/User.php b/app/Models/User.php index da2bdcea4..6b1dbbc89 100644 --- a/app/Models/User.php +++ b/app/Models/User.php @@ -396,7 +396,7 @@ class User extends Authenticatable $roleName = $roleQuery->name; $user = self::find($uid); - $user->assignRole($roleName); + $user->syncRoles([$roleName]); return self::find($uid)->update(['roles_id' => $role]); } @@ -436,7 +436,7 @@ class User extends Authenticatable } foreach (self::query()->whereDate('rolechangedate', '<', $now)->get() as $expired) { $expired->update(['roles_id' => self::ROLE_USER, 'rolechangedate' => null]); - $expired->syncRoles('User'); + $expired->syncRoles(['User']); SendAccountExpiredEmail::dispatch($expired)->onQueue('emails'); } } diff --git a/app/Observers/UserServiceObserver.php b/app/Observers/UserServiceObserver.php index fdfcb4088..46f3e2d11 100644 --- a/app/Observers/UserServiceObserver.php +++ b/app/Observers/UserServiceObserver.php @@ -28,7 +28,7 @@ class UserServiceObserver $roleData = Role::query()->where('id', $user->roles_id); $rateLimit = $roleData->value('rate_limit'); $roleName = $roleData->value('name'); - $user->assignRole($roleName); + $user->syncRoles([$roleName]); $user->update( [ 'api_token' => md5(Password::getRepository()->createNewToken()), diff --git a/misc/run_me_once/updateUserRoles.php b/misc/run_me_once/updateUserRoles.php index ce5f7a5f4..bfe22f43a 100644 --- a/misc/run_me_once/updateUserRoles.php +++ b/misc/run_me_once/updateUserRoles.php @@ -62,10 +62,10 @@ foreach ($oldRoles as $oldRole) { foreach ($users as $user) { if ($user->role !== null && $user->hasRole($user->role->name) === false) { - $user->assignRole($user->role->name); + $user->syncRoles([$user->role->name]); echo 'Role: '.$user->role->name.' assigned to user: '.$user->username.PHP_EOL; } elseif ($user->role === null) { - $user->assignRole('User'); + $user->syncRoles(['User']); } else { echo 'User '.$user->username.' already has the role: '.$user->role->name.PHP_EOL; }