[Security] Optimize RoleHierarchy's buildRoleMap method

enleur · fabpot · commit c3b68b0d2879 · 2016-09-06T17:21:44.000-07:00
diff --git a/src/Symfony/Component/Security/Core/Role/RoleHierarchy.php b/src/Symfony/Component/Security/Core/Role/RoleHierarchy.php
@@ -65,9 +65,17 @@ protected function buildRoleMap()
                 }
 
                 $visited[] = $role;
-                $this->map[$main] = array_unique(array_merge($this->map[$main], $this->hierarchy[$role]));
-                $additionalRoles = array_merge($additionalRoles, array_diff($this->hierarchy[$role], $visited));
+
+                foreach ($this->hierarchy[$role] as $roleToAdd) {
+                    $this->map[$main][] = $roleToAdd;
+                }
+
+                foreach (array_diff($this->hierarchy[$role], $visited) as $additionalRole) {
+                    $additionalRoles[] = $additionalRole;
+                }
             }
+
+            $this->map[$main] = array_unique($this->map[$main]);
         }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -65,9 +65,17 @@ protected function buildRoleMap()`
`65`	`65`	`}`
`66`	`66`
`67`	`67`	`$visited[] = $role;`
`68`		`- $this->map[$main] = array_unique(array_merge($this->map[$main], $this->hierarchy[$role]));`
`69`		`- $additionalRoles = array_merge($additionalRoles, array_diff($this->hierarchy[$role], $visited));`
	`68`	`+`
	`69`	`+ foreach ($this->hierarchy[$role] as $roleToAdd) {`
	`70`	`+ $this->map[$main][] = $roleToAdd;`
	`71`	`+ }`
	`72`	`+`
	`73`	`+ foreach (array_diff($this->hierarchy[$role], $visited) as $additionalRole) {`
	`74`	`+ $additionalRoles[] = $additionalRole;`
	`75`	`+ }`
`70`	`76`	`}`
	`77`	`+`
	`78`	`+ $this->map[$main] = array_unique($this->map[$main]);`
`71`	`79`	`}`
`72`	`80`	`}`
`73`	`81`	`}`