Merge pull request #325 from Wilt/patch-2

Improvements InMemoryRoleProvider

Author: bakura10

src/ZfcRbac/Role/InMemoryRoleProvider.php

@@ -18,6 +18,7 @@
 
 namespace ZfcRbac\Role;
 
+use Rbac\Role\RoleInterface;
 use Rbac\Role\HierarchicalRole;
 use Rbac\Role\Role;
 
@@ -42,6 +43,15 @@
 class InMemoryRoleProvider implements RoleProviderInterface
 {
     /**
+     * Role storage
+     *
+     * @var array
+     */
+    private $roles = [];
+
+    /**
+     * Roles config
+     *
      * @var array
      */
     private $rolesConfig = [];
@@ -62,34 +72,50 @@ public function getRoles(array $roleNames)
         $roles = [];
 
         foreach ($roleNames as $roleName) {
-            // If no config, we create a simple role with no permission
-            if (!isset($this->rolesConfig[$roleName])) {
-                $roles[] = new Role($roleName);
-                continue;
-            }
-
-            $roleConfig = $this->rolesConfig[$roleName];
+            $roles[] = $this->getRole($roleName);
+        }
+        return $roles;
+    }
 
-            if (isset($roleConfig['children'])) {
-                $role       = new HierarchicalRole($roleName);
-                $childRoles = (array) $roleConfig['children'];
+    /**
+     * Get role by role name
+     *
+     * @param $roleName
+     * @return RoleInterface
+     */
+    protected function getRole($roleName)
+    {
+        if (isset($this->roles[$roleName])) {
+            return $this->roles[$roleName];
+        }
 
-                foreach ($this->getRoles($childRoles) as $childRole) {
-                    $role->addChild($childRole);
-                }
-            } else {
-                $role = new Role($roleName);
-            }
+        // If no config, we create a simple role with no permission
+        if (!isset($this->rolesConfig[$roleName])) {
+            $role = new Role($roleName);
+            $this->roles[$roleName] = $role;
+            return $role;
+        }
 
-            $permissions = isset($roleConfig['permissions']) ? $roleConfig['permissions'] : [];
+        $roleConfig = $this->rolesConfig[$roleName];
 
-            foreach ($permissions as $permission) {
-                $role->addPermission($permission);
+        if (isset($roleConfig['children'])) {
+            $role = new HierarchicalRole($roleName);
+            $childRoles = (array)$roleConfig['children'];
+            foreach ($childRoles as $childRole) {
+                $childRole = $this->getRole($childRole);
+                $role->addChild($childRole);
             }
+        } else {
+            $role = new Role($roleName);
+        }
 
-            $roles[] = $role;
+        $permissions = isset($roleConfig['permissions']) ? $roleConfig['permissions'] : [];
+        foreach ($permissions as $permission) {
+            $role->addPermission($permission);
         }
 
-        return $roles;
+        $this->roles[$roleName] = $role;
+
+        return $role;
     }
 }

tests/ZfcRbacTest/Role/InMemoryRoleProviderTest.php

@@ -63,5 +63,8 @@ public function testInMemoryProvider()
         $this->assertEquals('guest', $guestRole->getName());
         $this->assertFalse($guestRole->hasPermission('write'));
         $this->assertFalse($guestRole->hasPermission('delete'));
+
+        $this->assertSame($adminRole->getChildren()['member'], $memberRole);
+        $this->assertSame($memberRole->getChildren()['guest'], $guestRole);
     }
 }