fix duplicates on sync (#2574)

Author: erikn69

src/Traits/HasPermissions.php

@@ -370,9 +370,10 @@ private function collectPermissions(...$permissions): array
                     return $array;
                 }
 
-                $this->ensureModelSharesGuard($permission);
-
-                $array[] = $permission->getKey();
+                if (! in_array($permission->getKey(), $array)) {
+                    $this->ensureModelSharesGuard($permission);
+                    $array[] = $permission->getKey();
+                }
 
                 return $array;
             }, []);

src/Traits/HasRoles.php

@@ -129,9 +129,10 @@ private function collectRoles(...$roles): array
                     return $array;
                 }
 
-                $this->ensureModelSharesGuard($role);
-
-                $array[] = $role->getKey();
+                if (! in_array($role->getKey(), $array)) {
+                    $this->ensureModelSharesGuard($role);
+                    $array[] = $role->getKey();
+                }
 
                 return $array;
             }, []);

tests/HasPermissionsTest.php

@@ -541,6 +541,16 @@ public function it_can_sync_multiple_permissions()
         $this->assertFalse($this->testUser->hasDirectPermission('edit-news'));
     }
 
+    /** @test */
+    public function it_can_avoid_sync_duplicated_permissions()
+    {
+        $this->testUser->syncPermissions('edit-articles', 'edit-blog', 'edit-blog');
+
+        $this->assertTrue($this->testUser->hasDirectPermission('edit-articles'));
+
+        $this->assertTrue($this->testUser->hasDirectPermission('edit-blog'));
+    }
+
     /** @test */
     public function it_can_sync_multiple_permissions_by_id()
     {

tests/HasRolesTest.php

@@ -259,6 +259,16 @@ public function it_can_sync_roles_from_a_string_on_a_permission()
         $this->assertTrue($this->testUserPermission->hasRole('testRole2'));
     }
 
+    /** @test */
+    public function it_can_avoid_sync_duplicated_roles()
+    {
+        $this->testUser->syncRoles('testRole', 'testRole', 'testRole2');
+
+        $this->assertTrue($this->testUser->hasRole('testRole'));
+
+        $this->assertTrue($this->testUser->hasRole('testRole2'));
+    }
+
     /** @test */
     public function it_can_sync_multiple_roles()
     {