Stops overwriting roles with empty array

When the user already exists and has roles, they are now not overwritten with empty arrays.
365Werk · Feb 24, 2022 · 0a09a2a · 0a09a2a
1 parent 9214509
commit 0a09a2a
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/src/JwtAuthRoles.php b/src/JwtAuthRoles.php
@@ -153,28 +153,26 @@ public static function authUser(object $request)
                 $user[config('jwtauthroles.userId')] = $claims->sub;
                 if (config('jwtauthroles.storeRoles')){
                     $user->roles = json_encode($claims->roles);
-                } else {
-                    $user->roles = json_encode(array());
+                } elseif (!$user->exists) {
+                    $user->roles = json_encode([]);
                 }
                 $user->claims = json_encode($claims);
                 $user->save();
             } else {
                 $user = JwtUser::where(config('jwtauthroles.userId'), '=', $claims->sub)->firstOrFail();
                 if (config('jwtauthroles.storeRoles')){
                     $user->roles = json_encode($claims->roles);
-                } else {
-                    $user->roles = json_encode(array());
                 }
                 $user->claims = json_encode($claims);
                 $user->save();
             }
         } else {
             $user = new JwtUser;
             $user->uuid = $claims->sub;
-            if (config('jwtauthroles.storeRoles')){
+            if (config('jwtauthroles.storeRoles') && isset($claims->roles)){
                 $user->roles = $claims->roles;
             } else {
-                $user->roles = json_encode(array());
+                $user->roles = json_encode([]);
             }
             $user->claims = $claims;
         }