djsmithme / Router

Src/Router/Dispatcher/Dispatcher.php

Spacing +4 added lines, -4 removed lines

@@ -116,7 +116,7 @@ 
 
         try {
             $resolvedHandler['controller'] = $this->_findClass($controllerMethod[0]);
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             throw new DispatchException($e->getMessage());
         }
 
@@ -167,7 +167,7 @@ 
 
         try {
             $this->_getParamsFromVariableName($args);
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             $this->_getParamsFromTypeHint($args);
         }
 
@@ -293,12 +293,12 @@ 
         try {
             $this->_checkWhiteList($controller, $this->options);
             $this->_checkBlackList($controller, $this->options);
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             unset($controllerObj);
             throw new DispatchException($e->getMessage());
         }
 
-        if (!method_exists($controllerObj,$method)){
+        if (!method_exists($controllerObj, $method)) {
             throw new DispatchException('Controller method does not exist');
         }
 

Tests/Router/RouteCollectionTest.php

Spacing +7 added lines, -7 removed lines

@@ -33,9 +33,9 @@ 
     {
         $handler = 'my-handler';
         $routes = $this->collection;
-        $routes->group('/path', function () use ($routes, $handler) {
+        $routes->group('/path', function() use ($routes, $handler) {
             $routes->addRoute(['GET', 'POST'], '/foo', $handler, ['global']);
-            $routes->group('/sub-dir', function () use ($routes, $handler) {
+            $routes->group('/sub-dir', function() use ($routes, $handler) {
                 $routes->addRoute(['GET'], '/sub-page', $handler, ['sub-dir']);
             });
         });
@@ -175,7 +175,7 @@ 
     public function testMergeCollection()
     {
         $handler = 'handler::string';
-        $names = ['routes','names'];
+        $names = ['routes', 'names'];
         $collection = new RouteCollection();
         $collection->addRoute('GET', '/path', $handler, $names);
         $collection->addRoute('GET', '/another', $handler, $names);
@@ -194,7 +194,7 @@ 
     {
         $this->setExpectedException(RouteException::class);
         $handler = 'handler::string';
-        $names = ['routes','names'];
+        $names = ['routes', 'names'];
         $collection = new RouteCollection();
         $collection->addRoute('GET', '/path', $handler, $names);
         $collection->addRoute('GET', '/another', $handler, $names);
@@ -240,7 +240,7 @@ 
         $rawPattern = '/mypattern/';
         $expectedPattern = '/' . \ltrim($rawPattern, '/');
         $expected = \implode('', $groups) . $expectedPattern;
-        $expected = \rtrim($expected,'/');
+        $expected = \rtrim($expected, '/');
 
         $this->assertEquals($expected, $collection->formatRoutePattern($rawPattern));
     }
@@ -250,11 +250,11 @@ 
      */
     public function testMergeGroupNames()
     {
-        $expected = ['route-name','group-name'];
+        $expected = ['route-name', 'group-name'];
         $handler = 'myhandler';
         $routes = $this->collection;
 
-        $routes->group('/path', function () use ($routes, $handler) {
+        $routes->group('/path', function() use ($routes, $handler) {
             $routes->addRoute(['GET', 'POST'], '/foo', $handler, ['route-name']);
         }, ['group-name']);
 

Src/Router/Adaptor/RSymfonyAdaptor.php

Spacing +2 added lines, -2 removed lines

@@ -86,7 +86,7 @@ 
     public function getSymfonyRouteCollection(RouteCollectionInterface $routes)
     {
 
-        if (!$this->options['cacheDisabled'] && file_exists($this->options['cacheFile'])){
+        if (!$this->options['cacheDisabled'] && file_exists($this->options['cacheFile'])) {
             //get cached route collection.
         }
 
@@ -99,7 +99,7 @@ 
             array('month' => '[0-9]{4}-[0-9]{2}'), // requirements
             array(), // options
             '', // host
-            array('https','https'), // schemes
+            array('https', 'https'), // schemes
             array('GET') // methods
         );
 

Src/Router/Adaptor/FastRouteAdaptor.php

Spacing +2 added lines, -2 removed lines

@@ -135,7 +135,7 @@ 
         $this->options = $options;
 
         return \FastRoute\cachedDispatcher(
-            function (RouteCollector $fastRouteCollector) use ($routes) {
+            function(RouteCollector $fastRouteCollector) use ($routes) {
                 /**
                  * @var $route RouteInterface
                  */
@@ -228,7 +228,7 @@ 
             }
             $routes = (array)require $cacheFile;
             foreach ($routes[0] as $routeMethod => $routeData) {
-                foreach ($routeData as $routePattern => $routeContent){
+                foreach ($routeData as $routePattern => $routeContent) {
                     $routePath = \rtrim($context, '/') . DIRECTORY_SEPARATOR . \ltrim($routePattern, '/');
                     $collection->addRoute($routeMethod, $routePath, $routeContent['content'], $routeContent['name']);
                 }