Merge pull request #2194 from erikn69/patch-6

Fix returning all roles instead of the assigned
spatie · Oct 19, 2022 · bc2227c · bc2227c
2 parents 5e8b069 + a096b9d
commit bc2227c
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/src/Traits/HasRoles.php b/src/Traits/HasRoles.php
@@ -189,6 +189,8 @@ public function syncRoles(...$roles)
      */
     public function hasRole($roles, string $guard = null): bool
     {
+        $this->loadMissing('roles');
+
         if (is_string($roles) && false !== strpos($roles, '|')) {
             $roles = $this->convertPipeToArray($roles);
         }
@@ -248,6 +250,8 @@ public function hasAnyRole(...$roles): bool
      */
     public function hasAllRoles($roles, string $guard = null): bool
     {
+        $this->loadMissing('roles');
+
         if (is_string($roles) && false !== strpos($roles, '|')) {
             $roles = $this->convertPipeToArray($roles);
         }
@@ -282,6 +286,8 @@ public function hasAllRoles($roles, string $guard = null): bool
      */
     public function hasExactRoles($roles, string $guard = null): bool
     {
+        $this->loadMissing('roles');
+
         if (is_string($roles) && false !== strpos($roles, '|')) {
             $roles = $this->convertPipeToArray($roles);
         }
@@ -311,6 +317,8 @@ public function getDirectPermissions(): Collection
 
     public function getRoleNames(): Collection
     {
+        $this->loadMissing('roles');
+
         return $this->roles->pluck('name');
     }