changed setAssignments to work with comma separated string

hiqdev · Nov 10, 2016 · c015235 · c015235
1 parent 84c6b5a
commit c015235
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/SetterTrait.php b/src/SetterTrait.php
@@ -109,11 +109,14 @@ public function setAssignment($item, $userId)
 
     /**
      * Assigns items to a user.
-     * @param array $items
+     * @param string|array $items
      * @param string|integer $userId
      */
-    public function setAssignments(array $items, $userId)
+    public function setAssignments($items, $userId)
     {
+        if (is_string($items)) {
+            $items = explode(',', $items);
+        }
         foreach ($items as $item) {
             $this->setAssignment($item, $userId);
         }

diff --git a/tests/unit/CheckAccessTrait.php b/tests/unit/CheckAccessTrait.php
@@ -76,7 +76,7 @@ public function testPermission()
 
     public function testMighty()
     {
-        $this->auth->setAssignments(['role:admin', 'role:manager', 'bill.create', 'domain.freeze'], 'user:mighty');
+        $this->auth->setAssignments('role:admin,role:manager,bill.create,domain.freeze', 'user:mighty');
 
         $this->assertTrue ($this->auth->checkAccess('user:mighty', 'support'));
         $this->assertTrue ($this->auth->checkAccess('user:mighty', 'manage'));