View GitHub Repository framework

Schedule Inspection
Subscribe
Passed
Push — master ( 261c41...4db453 )
by Kirill
06:16 queued 11s
created
Status
Category
Toggle Patches
src/Security/src/Command/RolePermissionsCommand.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -30,16 +30,16 @@  discard block
 
 block discarded – undo
30 30 
     {
31 31 
         $role = $this->argument('role');
32 32
33 
-        if ($role !== null && !$rbac->hasRole($role)) {
33 
+        if ($role !== null && !$rbac->hasRole($role)){
34 34 
             throw new CommandException('Unknown role provided');
35 35 
         }
36 36
37 
-        if ($role !== null) {
37 
+        if ($role !== null){
38 38 
             $rows = $this->getRolePermissions($role, $rbac);
39 
-        } else {
39 
+        }else{
40 40 
             $rows = [];
41 41
42 
-            foreach ($rbac->getRoles() as $role) {
42 
+            foreach ($rbac->getRoles() as $role){
43 43 
                 /** @noinspection SlowArrayOperationsInLoopInspection */
44 44 
                 $rows = array_merge(
45 45 
                     $this->getRolePermissions($role, $rbac),
 
@@ -67,7 +67,7 @@  discard block
 
 block discarded – undo
67 67 
     {
68 68 
         $permissions = [];
69 69
70 
-        foreach ($rbac->getPermissions($role) as $permission => $rule) {
70 
+        foreach ($rbac->getPermissions($role) as $permission => $rule){
71 71 
             $permissions[] = [
72 72 
                 'role' => $role,
73 73 
                 'permission' => $permission,
Please login to merge, or discard this patch.
Braces   +11 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -30,16 +30,21 @@  discard block
 
 block discarded – undo
30 30 
     {
31 31 
         $role = $this->argument('role');
32 32
33 
-        if ($role !== null && !$rbac->hasRole($role)) {
33 
+        if ($role !== null && !$rbac->hasRole($role))
34 
+        {
34 35 
             throw new CommandException('Unknown role provided');
35 36 
         }
36 37
37 
-        if ($role !== null) {
38 
+        if ($role !== null)
39 
+        {
38 40 
             $rows = $this->getRolePermissions($role, $rbac);
39 
-        } else {
41 
+        }
42 
+        else
43 
+        {
40 44 
             $rows = [];
41 45
42 
-            foreach ($rbac->getRoles() as $role) {
46 
+            foreach ($rbac->getRoles() as $role)
47 
+            {
43 48 
                 /** @noinspection SlowArrayOperationsInLoopInspection */
44 49 
                 $rows = array_merge(
45 50 
                     $this->getRolePermissions($role, $rbac),
 
@@ -67,7 +72,8 @@  discard block
 
 block discarded – undo
67 72 
     {
68 73 
         $permissions = [];
69 74
70 
-        foreach ($rbac->getPermissions($role) as $permission => $rule) {
75 
+        foreach ($rbac->getPermissions($role) as $permission => $rule)
76 
+        {
71 77 
             $permissions[] = [
72 78 
                 'role' => $role,
73 79 
                 'permission' => $permission,
Please login to merge, or discard this patch.