SimLibaud / SilRouteSecurityBundle

Tests/Twig/RouteSecurityExtensionTest.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
         $twig_functions = $routeSecurityExtension->getFunctions();
         $this->assertEquals(3, count($twig_functions));
 
-        $functions =  ['hasUserAccessToRoute', 'hasUserAccessToRoutes', 'hasUserAccessAtLeastOneRoute'];
+        $functions = ['hasUserAccessToRoute', 'hasUserAccessToRoutes', 'hasUserAccessAtLeastOneRoute'];
         foreach ($twig_functions as $index => $twig_function) {
             $this->assertTrue(in_array($twig_function->getName(), $functions));
         }

Tests/Security/AccessControlTest.php

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
         $routeToRoleConverter = $this->createMock(NamingStrategyInterface::class);
         $routeToRoleConverter
             ->method('generateRoleForRoute')
-            ->will($this->returnCallback(function ($route) {
+            ->will($this->returnCallback(function($route) {
                 return 'ROLE_'.strtoupper($route);
             }));
         $configuration = [

Tests/Listener/AccessControlListenerTest.php

Spacing +1 added lines, -1 removed lines

@@ -167,7 +167,7 @@
             ->expects($this->once())
             ->method('dispatch')
             ->with(AccessDeniedToRouteEvent::ON_ACCESS_DENIED_TO_ROUTE, $this->isInstanceOf(AccessDeniedToRouteEvent::class))
-            ->will($this->returnCallback(function ($name, $access_denied_event) use ($response) {
+            ->will($this->returnCallback(function($name, $access_denied_event) use ($response) {
                 $access_denied_event->setResponse($response);
             }));
 

Security/AccessControl.php

Spacing +1 added lines, -1 removed lines

@@ -135,7 +135,7 @@
         $all_secured_routes = [];
         $configured_routes = array_keys($this->router->getRouteCollection()->all());
 
-        foreach ($configured_routes as $configured_route) if (true === $this->isRouteSecure($configured_route)){
+        foreach ($configured_routes as $configured_route) if (true === $this->isRouteSecure($configured_route)) {
             $all_secured_routes[] = $configured_route;
         }
 

Braces +4 added lines, -2 removed lines

@@ -135,8 +135,10 @@
         $all_secured_routes = [];
         $configured_routes = array_keys($this->router->getRouteCollection()->all());
 
-        foreach ($configured_routes as $configured_route) if (true === $this->isRouteSecure($configured_route)){
-            $all_secured_routes[] = $configured_route;
+        foreach ($configured_routes as $configured_route) {

+            if (true === $this->isRouteSecure($configured_route)){
+            $all_secured_routes[] = $configured_route;

+        }

         }
 
         return $all_secured_routes;

Tests/Role/RolesProviderTest.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
         $routeToRoleConverter
             ->expects($this->exactly(count($secured_routes)))
             ->method('generateRoleForRoute')
-            ->will($this->returnCallback(function ($route) {
+            ->will($this->returnCallback(function($route) {
                 return 'ROLE_'.strtoupper($route);
             }));
         $rolesProvider = new RolesProvider($routeToRoleConverter, $secured_routes);