djsmithme / Router

Src/Router/RouteCollection.php

Spacing +3 added lines, -3 removed lines

@@ -231,8 +231,8 @@ 
 
         $route = \implode('', $this->group) . $routePattern;
 
-        if (substr($route, -1) === '/' && strlen($route) > 1){
-            $route = rtrim($route,'/');
+        if (substr($route, -1) === '/' && strlen($route) > 1) {
+            $route = rtrim($route, '/');
         }
 
         return $route;
@@ -254,7 +254,7 @@ 
                     )
                 );
                 $new->collection[] = $addRoute;
-            } catch (\Exception $e) {
+            }catch (\Exception $e) {
                 if ($throw) {
                     throw new RouteException($e->getMessage());
                 }

Src/Router/Loaders/YamlLoader.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
                         $collection->addRoute($routeMethod, $route['path'], $route['handler'], $route['names']);
                     }
                 }
-            } catch (ParseException $e) {
+            }catch (ParseException $e) {
                 throw new RouterException($e->getMessage());
             }
 

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

Unused Use Statements -2 removed lines

@@ -14,8 +14,6 @@
 use Ds\Router\Interfaces\RouteInterface;
 use Ds\Router\Interfaces\SerializerInterface;
 use Ds\Router\RouterResponse;
-
-
 use Symfony\Component\Routing\Matcher\UrlMatcher;
 use Symfony\Component\Routing\RequestContext;
 use Symfony\Component\Routing\RouteCollection;

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/SymfonyAdaptor.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 use Ds\Router\Interfaces\RouteInterface;
 use Ds\Router\Interfaces\SerializerInterface;
 use Ds\Router\RouterResponse;
-
 use Symfony\Component\Routing\Matcher\UrlMatcher;
 use Symfony\Component\Routing\RequestContext;
 use Symfony\Component\Routing\Route;