burzum / php-router

src/Host.php

Spacing +3 added lines, -3 removed lines

@@ -216,7 +216,7 @@ 
             return false;
         }
 
-        if (!preg_match('~^' . $this->getRegex() . '$~', $host, $matches)) {
+        if (!preg_match('~^'.$this->getRegex().'$~', $host, $matches)) {
             $hostVariables = null;
             return false;
         }
@@ -242,8 +242,8 @@ 
             $options['host'] = $this->_link($this->getToken(), $params);
         }
 
-        $scheme = $options['scheme'] !== '*' ? $options['scheme'] . '://' : '//';
-        return $scheme . $options['host'];
+        $scheme = $options['scheme'] !== '*' ? $options['scheme'].'://' : '//';
+        return $scheme.$options['host'];
     }
 
     /**

spec/Suite/Host.spec.php

Spacing +20 added lines, -20 removed lines

@@ -7,13 +7,13 @@ 
 use Lead\Router\Exception\RouterException;
 use Lead\Router\Host;
 
-describe("Host", function () {
+describe("Host", function() {
 
 
-    describe("->scheme()", function () {
+    describe("->scheme()", function() {
 
 
-        it("gets/sets the scheme", function () {
+        it("gets/sets the scheme", function() {
 
 
             $host = new Host();
@@ -22,10 +22,10 @@ 
             expect($host->getScheme())->toBe('https');
         });
     });
-    describe("->pattern()", function () {
+    describe("->pattern()", function() {
 
 
-        it("gets/sets the pattern", function () {
+        it("gets/sets the pattern", function() {
 
 
             $host = new Host();
@@ -37,7 +37,7 @@ 
                 'domain'    => false
             ]);
         });
-        it("updates the regex", function () {
+        it("updates the regex", function() {
 
 
             $host = new Host();
@@ -46,7 +46,7 @@ 
             expect($host->setPattern('foo.{domain}.baz'))->toBe($host);
             expect($host->getRegex())->toBe('foo\\.([^.]+)\\.baz');
         });
-        it("updates the variables", function () {
+        it("updates the variables", function() {
 
 
             $host = new Host();
@@ -60,10 +60,10 @@ 
             ]);
         });
     });
-    describe("->match()", function () {
+    describe("->match()", function() {
 
 
-        it("returns `true` when host & scheme matches", function () {
+        it("returns `true` when host & scheme matches", function() {
 
 
             $host = new Host(['pattern' => 'foo.{domain}.bar', 'scheme' => 'https']);
@@ -72,7 +72,7 @@ 
             expect($host->match(['scheme' => 'https', 'host' => 'biz.bar.baz'], $variables))->toBe(false);
             expect($host->match(['scheme' => 'http', 'host' => 'foo.baz.bar'], $variables))->toBe(false);
         });
-        it("returns `true` when host matches with a wildcard as host's scheme", function () {
+        it("returns `true` when host matches with a wildcard as host's scheme", function() {
 
 
             $host = new Host(['pattern' => 'foo.{domain}.bar', 'scheme' => 'https']);
@@ -80,7 +80,7 @@ 
             expect($variables)->toBe(['domain' => 'baz']);
             expect($host->match(['scheme' => '*', 'host' => 'biz.baz.bar'], $variables))->toBe(false);
         });
-        it("returns `true` when host matches with a wildcard as request's scheme", function () {
+        it("returns `true` when host matches with a wildcard as request's scheme", function() {
 
 
             $host = new Host(['pattern' => 'foo.{domain}.bar', 'scheme' => '*']);
@@ -88,7 +88,7 @@ 
             expect($variables)->toBe(['domain' => 'baz']);
             expect($host->match(['scheme' => 'https', 'host' => 'biz.baz.bar'], $variables))->toBe(false);
         });
-        it("returns `true` when scheme matches with a wildcard as host's pattern", function () {
+        it("returns `true` when scheme matches with a wildcard as host's pattern", function() {
 
 
             $host = new Host(['pattern' => '*', 'scheme' => 'http']);
@@ -96,7 +96,7 @@ 
             expect($variables)->toBe([]);
             expect($host->match(['scheme' => 'https', 'host' => 'foo.baz.bar'], $variables))->toBe(false);
         });
-        it("returns `true` when scheme matches with a wildcard as request's pattern", function () {
+        it("returns `true` when scheme matches with a wildcard as request's pattern", function() {
 
 
             $host = new Host(['pattern' => 'foo.{domain}.bar', 'scheme' => 'http']);
@@ -105,24 +105,24 @@ 
             expect($host->match(['scheme' => 'https', 'host' => '*'], $variables))->toBe(false);
         });
     });
-    describe("->link()", function () {
+    describe("->link()", function() {
 
 
-        it("builds an host link", function () {
+        it("builds an host link", function() {
 
 
             $host = new Host(['pattern' => 'www.{domain}.com', 'scheme' => 'https']);
             $link = $host->link(['domain' => 'example']);
             expect($link)->toBe('https://www.example.com');
         });
-        it("builds a scheme less host link", function () {
+        it("builds a scheme less host link", function() {
 
 
             $host = new Host(['pattern' => 'www.{domain}.com']);
             $link = $host->link(['domain' => 'example']);
             expect($link)->toBe('//www.example.com');
         });
-        it("overrides scheme when passed as parameter", function () {
+        it("overrides scheme when passed as parameter", function() {
 
 
             $host = new Host(['pattern' => 'www.{domain}.com']);
@@ -131,7 +131,7 @@ 
             ]);
             expect($link)->toBe('http://www.example.com');
         });
-        it("processes complex pattern", function () {
+        it("processes complex pattern", function() {
 
 
             $host = new Host(['pattern' => '[{subdomains}.]*{domain}.com', 'scheme' => 'https']);
@@ -145,10 +145,10 @@ 
                 'domain' => 'example'
             ]))->toBe('https://a.b.c.example.com');
         });
-        it("throws an exception when variables are missing", function () {
+        it("throws an exception when variables are missing", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 $host = new Host(['pattern' => 'www.{domain}.com']);
                 $link = $host->link();

spec/Suite/Parser.spec.php

Spacing +26 added lines, -26 removed lines

@@ -7,13 +7,13 @@ 
 use Lead\Router\Exception\ParserException;
 use Lead\Router\Parser;
 
-describe("Parser", function () {
+describe("Parser", function() {
 
 
-    describe("::tokenize()", function () {
+    describe("::tokenize()", function() {
 
 
-        it("parses an empty url", function () {
+        it("parses an empty url", function() {
 
 
             $result = Parser::tokenize('');
@@ -25,7 +25,7 @@ 
                 'tokens'   => []
             ]);
         });
-        it("parses a static url", function () {
+        it("parses a static url", function() {
 
 
             $result = Parser::tokenize('/test');
@@ -37,7 +37,7 @@ 
                 'tokens'   => ['/test']
             ]);
         });
-        it("parses an url with a variable", function () {
+        it("parses an url with a variable", function() {
 
 
             $result = Parser::tokenize('/test/{param}');
@@ -55,7 +55,7 @@ 
                 ]
             ]);
         });
-        it("parses an url with several variables", function () {
+        it("parses an url with several variables", function() {
 
 
             $result = Parser::tokenize('/test/{param1}/test2/{param2}');
@@ -78,7 +78,7 @@ 
                 ]
             ]);
         });
-        it("parses an url with a variable with a custom regex", function () {
+        it("parses an url with a variable with a custom regex", function() {
 
 
             $result = Parser::tokenize('/test/{param:\d+}');
@@ -96,7 +96,7 @@ 
                 ]
             ]);
         });
-        it("parses an url with an optional segment", function () {
+        it("parses an url with an optional segment", function() {
 
 
             $result = Parser::tokenize('/test[opt]');
@@ -119,7 +119,7 @@ 
                 ]
             ]);
         });
-        it("parses an optional segment", function () {
+        it("parses an optional segment", function() {
 
 
             $result = Parser::tokenize('[test]');
@@ -141,7 +141,7 @@ 
                 ]
             ]);
         });
-        it("parses an optional segment inside a route definition", function () {
+        it("parses an optional segment inside a route definition", function() {
 
 
             $result = Parser::tokenize('/test[/opt]/required');
@@ -165,7 +165,7 @@ 
                 ]
             ]);
         });
-        it("parses an url with an optional variable", function () {
+        it("parses an url with an optional variable", function() {
 
 
             $result = Parser::tokenize('/test[/{param}]');
@@ -192,7 +192,7 @@ 
                 ]
             ]);
         });
-        it("parses an url with a variable with a prefix and a suffix inside an optional segment", function () {
+        it("parses an url with a variable with a prefix and a suffix inside an optional segment", function() {
 
 
             $result = Parser::tokenize('/test[/:{param}:]');
@@ -220,7 +220,7 @@ 
                 ]
             ]);
         });
-        it("parses an url with a variable and an optional segment", function () {
+        it("parses an url with a variable and an optional segment", function() {
 
 
             $result = Parser::tokenize('/{param}[opt]');
@@ -247,7 +247,7 @@ 
                 ]
             ]);
         });
-        it("parses nested segments", function () {
+        it("parses nested segments", function() {
 
 
             $result = Parser::tokenize('/test[/{name}[/{id:[0-9]+}]]');
@@ -287,25 +287,25 @@ 
                 ]
             ]);
         });
-        it("throws an exception when there's a missing closing square bracket", function () {
+        it("throws an exception when there's a missing closing square bracket", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 Parser::tokenize('/test[opt');
             };
             expect($closure)->toThrow(ParserException::squareBracketMismatch());
-            $closure = function () {
+            $closure = function() {
 
                 Parser::tokenize('/test[opt[opt2]');
             };
             expect($closure)->toThrow(ParserException::squareBracketMismatch());
         });
     });
-    describe("::compile()", function () {
+    describe("::compile()", function() {
 
 
-        it("compiles a tokens structure", function () {
+        it("compiles a tokens structure", function() {
 
 
             $token = Parser::tokenize('/test[/{name}[/{id:[0-9]+}]]');
@@ -314,7 +314,7 @@ 
                 '/test(?:/([^/]+)(?:/([0-9]+))?)?', ['name' => false, 'id' => false]
             ]);
         });
-        it("compiles a tokens structure with repeatable patterns", function () {
+        it("compiles a tokens structure with repeatable patterns", function() {
 
 
             $tokens = Parser::tokenize('/test[/{name}[/{id:[0-9]+}]*]');
@@ -323,28 +323,28 @@ 
                 '/test(?:/([^/]+)((?:/[0-9]+)*))?', ['name' => false, 'id' => '/{id:[0-9]+}']
             ]);
         });
-        it("throws an exception when a placeholder is present several time", function () {
+        it("throws an exception when a placeholder is present several time", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 Parser::compile(Parser::tokenize('/test/{var}/{var}'));
             };
             expect($closure)->toThrow(ParserException::duplicatePlaceholder('var'));
         });
-        it("throws an exception when a placeholder is present several time through different segments", function () {
+        it("throws an exception when a placeholder is present several time through different segments", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 Parser::compile(Parser::tokenize('/test/{var}[/{var}]'));
             };
             expect($closure)->toThrow(ParserException::duplicatePlaceholder('var'));
         });
-        it("throws an exception when multiple placeholder are present in repeatable segments", function () {
+        it("throws an exception when multiple placeholder are present in repeatable segments", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 Parser::compile(Parser::tokenize('/test[/{var1}/{var2}]*'));
             };

src/Scope.php

Spacing +4 added lines, -4 removed lines

@@ -94,12 +94,12 @@ 
         $scope = $this->scope;
 
         if (!empty($options['name'])) {
-            $options['name'] = $scope['name'] ? $scope['name'] . '.' . $options['name'] : $options['name'];
+            $options['name'] = $scope['name'] ? $scope['name'].'.'.$options['name'] : $options['name'];
         }
 
         if (!empty($options['prefix'])) {
-            $options['prefix'] = $scope['prefix'] . trim($options['prefix'], '/');
-            $options['prefix'] = $options['prefix'] ? $options['prefix'] . '/' : '';
+            $options['prefix'] = $scope['prefix'].trim($options['prefix'], '/');
+            $options['prefix'] = $options['prefix'] ? $options['prefix'].'/' : '';
         }
 
         if (isset($options['persist'])) {
@@ -107,7 +107,7 @@ 
         }
 
         if (isset($options['namespace'])) {
-            $options['namespace'] = $scope['namespace'] . trim($options['namespace'], '\\') . '\\';
+            $options['namespace'] = $scope['namespace'].trim($options['namespace'], '\\').'\\';
         }
 
         return $options + $scope;

src/Parser.php

Indentation +7 added lines, -7 removed lines

@@ -70,13 +70,13 @@
 \}
 EOD;
 /**
-     * Tokenizes a route pattern. Optional segments are identified by square brackets.
-     *
-     * @param string $pattern A route pattern
-     * @param string $delimiter The path delimiter.
-     * @param array The tokens structure root node.
-     * @return array
-     */
+ * Tokenizes a route pattern. Optional segments are identified by square brackets.
+ *
+ * @param string $pattern A route pattern
+ * @param string $delimiter The path delimiter.
+ * @param array The tokens structure root node.
+ * @return array
+ */
     public static function tokenize(string $pattern, string $delimiter = '/'): array
     {
         // Checks if the pattern has some optional segments.

Spacing +6 added lines, -6 removed lines

@@ -80,7 +80,7 @@ 
     public static function tokenize(string $pattern, string $delimiter = '/'): array
     {
         // Checks if the pattern has some optional segments.
-        if (count(preg_split('~' . static::PLACEHOLDER_REGEX . '(*SKIP)(*F)|\[~x', $pattern)) > 1) {
+        if (count(preg_split('~'.static::PLACEHOLDER_REGEX.'(*SKIP)(*F)|\[~x', $pattern)) > 1) {
             $tokens = static::_tokenizePattern($pattern, $delimiter);
         } else {
             $tokens = static::tokenizeSegment($pattern, $delimiter);
@@ -143,7 +143,7 @@ 
         $tokens = [];
         $index = 0;
         $path = '';
-        if (preg_match_all('~' . static::PLACEHOLDER_REGEX . '()~x', $pattern, $matches, PREG_OFFSET_CAPTURE | PREG_SET_ORDER)) {
+        if (preg_match_all('~'.static::PLACEHOLDER_REGEX.'()~x', $pattern, $matches, PREG_OFFSET_CAPTURE | PREG_SET_ORDER)) {
             foreach ($matches as $match) {
                 $offset = $match[0][1];
                 $path .= substr($pattern, $index, $offset - $index);
@@ -154,7 +154,7 @@ 
                 }
 
                 $variable = $match[1][0];
-                $capture = $match[2][0] ?: '[^' . $delimiter . ']+';
+                $capture = $match[2][0] ?: '[^'.$delimiter.']+';
                 $tokens[] = [
                     'name'      => $variable,
                     'pattern'   => $capture
@@ -259,9 +259,9 @@ 
                     if (count($rule[1]) > 1) {
                         throw ParserException::placeholderExceeded();
                     }
-                    $regex .= '((?:' . $rule[0] . ')' . $child['greedy'] . ')';
+                    $regex .= '((?:'.$rule[0].')'.$child['greedy'].')';
                 } elseif ($child['optional']) {
-                    $regex .= '(?:' . $rule[0] . ')?';
+                    $regex .= '(?:'.$rule[0].')?';
                 }
                 foreach ($rule[1] as $name => $pattern) {
                     if (isset($variables[$name])) {
@@ -279,7 +279,7 @@ 
                     $regex .= $child['pattern'];
                 } else {
                     $variables[$name] = false;
-                    $regex .= '(' . $child['pattern'] . ')';
+                    $regex .= '('.$child['pattern'].')';
                 }
             }
         }

src/RouteInterface.php

Indentation +11 added lines, -11 removed lines

@@ -81,17 +81,17 @@
     public function match($request, &$variables = null, &$hostVariables = null): bool;
 
     /**
-      * Returns the route's link.
-      *
-      * @param  array $params  The route parameters.
-      * @param  array $options Options for generating the proper prefix. Accepted values are:
-      *                        - `'absolute'` _boolean_: `true` or `false`. - `'scheme'`
-      *                        _string_ : The scheme. - `'host'`     _string_ : The host
-      *                        name. - `'basePath'` _string_ : The base path. - `'query'`
-      *                        _string_ : The query string. - `'fragment'` _string_ : The
-      *                        fragment string.
-      * @return string          The link.
-      */
+     * Returns the route's link.
+     *
+     * @param  array $params  The route parameters.
+     * @param  array $options Options for generating the proper prefix. Accepted values are:
+     *                        - `'absolute'` _boolean_: `true` or `false`. - `'scheme'`
+     *                        _string_ : The scheme. - `'host'`     _string_ : The host
+     *                        name. - `'basePath'` _string_ : The base path. - `'query'`
+     *                        _string_ : The query string. - `'fragment'` _string_ : The
+     *                        fragment string.
+     * @return string          The link.
+     */
     public function link(array $params = [], array $options = []): string;
 
     /**

src/Route.php

Spacing +24 added lines, -24 removed lines

@@ -180,7 +180,7 @@ 
         $this->setMethods($config['methods']);
         $this->setScope($config['scope']);
         $this->setPattern($config['pattern']);
-        $this->setMiddleware((array)$config['middleware']);
+        $this->setMiddleware((array) $config['middleware']);
     }
 
     /**
@@ -191,7 +191,7 @@ 
      */
     public function setMiddleware(array $middleware)
     {
-        $this->_middleware = (array)$middleware;
+        $this->_middleware = (array) $middleware;
 
         return $this;
     }
@@ -371,7 +371,7 @@ 
     {
         $this->prefix = trim($prefix, '/');
         if ($this->prefix) {
-            $this->prefix = '/' . $this->prefix;
+            $this->prefix = '/'.$this->prefix;
         }
 
         return $this;
@@ -440,7 +440,7 @@ 
      */
     public function setMethods($methods): self
     {
-        $methods = $methods ? (array)$methods : [];
+        $methods = $methods ? (array) $methods : [];
         $methods = array_map('strtoupper', $methods);
         $methods = array_fill_keys($methods, true);
 
@@ -463,7 +463,7 @@ 
      */
     public function allow($methods = [])
     {
-        $methods = $methods ? (array)$methods : [];
+        $methods = $methods ? (array) $methods : [];
         $methods = array_map('strtoupper', $methods);
         $methods = array_fill_keys($methods, true) + $this->methods;
 
@@ -523,10 +523,10 @@ 
         $this->variables = null;
 
         if (!$pattern || $pattern[0] !== '[') {
-            $pattern = '/' . trim($pattern, '/');
+            $pattern = '/'.trim($pattern, '/');
         }
 
-        $this->pattern = $this->prefix . $pattern;
+        $this->pattern = $this->prefix.$pattern;
 
         return $this;
     }
@@ -640,9 +640,9 @@ 
             }
         }
 
-        $path = '/' . trim($path, '/');
+        $path = '/'.trim($path, '/');
 
-        if (!preg_match('~^' . $this->regex() . '$~', $path, $matches)) {
+        if (!preg_match('~^'.$this->regex().'$~', $path, $matches)) {
             return false;
         }
         $variables = $this->_buildVariables($matches);
@@ -674,7 +674,7 @@ 
             } else {
                 $token = $parser::tokenize($pattern, '/');
                 $rule = $parser::compile($token);
-                if (preg_match_all('~' . $rule[0] . '~', $values[$i], $parts)) {
+                if (preg_match_all('~'.$rule[0].'~', $values[$i], $parts)) {
                     foreach ($parts[1] as $value) {
                         if (strpos($value, '/') !== false) {
                             $variables[$name][] = explode('/', $value);
@@ -705,7 +705,7 @@ 
 
         $generator = $this->middleware();
 
-        $next = function () use ($request, $response, $generator, &$next) {
+        $next = function() use ($request, $response, $generator, &$next) {
             $handler = $generator->current();
             $generator->next();
 
@@ -733,7 +733,7 @@ 
             }
         }
 
-        yield function () {
+        yield function() {
             $handler = $this->handler();
             if ($handler === null) {
                 return null;
@@ -781,7 +781,7 @@ 
 
         $options = array_filter(
             $options,
-            function ($value) {
+            function($value) {
                 return $value !== '*';
             }
         );
@@ -793,24 +793,24 @@ 
 
         $basePath = trim($options['basePath'], '/');
         if ($basePath) {
-            $basePath = '/' . $basePath;
+            $basePath = '/'.$basePath;
         }
         $link = isset($link) ? ltrim($link, '/') : '';
-        $link = $basePath . ($link ? '/' . $link : $link);
-        $query = $options['query'] ? '?' . $options['query'] : '';
-        $fragment = $options['fragment'] ? '#' . $options['fragment'] : '';
+        $link = $basePath.($link ? '/'.$link : $link);
+        $query = $options['query'] ? '?'.$options['query'] : '';
+        $fragment = $options['fragment'] ? '#'.$options['fragment'] : '';
 
         if ($options['absolute']) {
             if ($this->host !== null) {
-                $link = $this->host->link($params, $options) . "{$link}";
+                $link = $this->host->link($params, $options)."{$link}";
             } else {
-                $scheme = !empty($options['scheme']) ? $options['scheme'] . '://' : '//';
+                $scheme = !empty($options['scheme']) ? $options['scheme'].'://' : '//';
                 $host = isset($options['host']) ? $options['host'] : 'localhost';
                 $link = "{$scheme}{$host}{$link}";
             }
         }
 
-        return $link . $query . $fragment;
+        return $link.$query.$fragment;
     }
 
     /**
@@ -831,7 +831,7 @@ 
             if (isset($child['tokens'])) {
                 if ($child['repeat']) {
                     $name = $child['repeat'];
-                    $values = isset($params[$name]) && $params[$name] !== null ? (array)$params[$name] : [];
+                    $values = isset($params[$name]) && $params[$name] !== null ? (array) $params[$name] : [];
                     if (!$values && !$child['optional']) {
                         throw new RouterException("Missing parameters `'{$name}'` for route: `'{$this->name}#{$this->pattern}'`.");
                     }
@@ -858,12 +858,12 @@ 
                 $parts = [];
             }
             foreach ($parts as $key => $value) {
-                $parts[$key] = rawurlencode((string)$value);
+                $parts[$key] = rawurlencode((string) $value);
             }
             $value = join('/', $parts);
 
-            if (!preg_match('~^' . $child['pattern'] . '$~', $value)) {
-                throw new RouterException("Expected `'" . $child['name'] . "'` to match `'" . $child['pattern'] . "'`, but received `'" . $value . "'`.");
+            if (!preg_match('~^'.$child['pattern'].'$~', $value)) {
+                throw new RouterException("Expected `'".$child['name']."'` to match `'".$child['pattern']."'`, but received `'".$value."'`.");
             }
             $link .= $value;
         }

spec/Suite/Route.spec.php

Spacing +44 added lines, -44 removed lines

@@ -11,13 +11,13 @@ 
 use Lead\Router\Router;
 use Lead\Router\Route;
 
-describe("Route", function () {
+describe("Route", function() {
 
 
-    describe("->pattern()", function () {
+    describe("->pattern()", function() {
 
 
-        it("gets/sets the pattern", function () {
+        it("gets/sets the pattern", function() {
 
 
             $route = new Route();
@@ -29,7 +29,7 @@ 
                 'paths' => '/{paths}'
             ]);
         });
-        it("updates the regex", function () {
+        it("updates the regex", function() {
 
 
             $route = new Route();
@@ -38,7 +38,7 @@ 
             expect($route->setPattern('/foo/baz/{id}[/{paths}]*'))->toBe($route);
             expect($route->Regex())->toBe('/foo/baz/([^/]+)((?:/[^/]+)*)');
         });
-        it("updates the variables", function () {
+        it("updates the variables", function() {
 
 
             $route = new Route();
@@ -54,10 +54,10 @@ 
             ]);
         });
     });
-    describe("->scope()", function () {
+    describe("->scope()", function() {
 
 
-        it("gets/sets route scope", function () {
+        it("gets/sets route scope", function() {
 
 
             $scope = new Scope();
@@ -66,17 +66,17 @@ 
             expect($route->Scope())->toBe($scope);
         });
     });
-    describe("->methods()", function () {
+    describe("->methods()", function() {
 
 
-        it("gets/sets route methods", function () {
+        it("gets/sets route methods", function() {
 
 
             $route = new Route();
             expect($route->setMethods(['POST', 'PUT']))->toBe($route);
             expect($route->Methods())->toBe(['POST', 'PUT']);
         });
-        it("formats method names", function () {
+        it("formats method names", function() {
 
 
             $route = new Route();
@@ -84,17 +84,17 @@ 
             expect($route->Methods())->toBe(['POST', 'PUT']);
         });
     });
-    describe("->allow()", function () {
+    describe("->allow()", function() {
 
 
-        it("adds some extra allowed methods", function () {
+        it("adds some extra allowed methods", function() {
 
 
             $route = new Route(['methods' => []]);
             expect($route->allow(['POST', 'PUT']))->toBe($route);
             expect($route->Methods())->toBe(['POST', 'PUT']);
         });
-        it("formats newly allowed method names", function () {
+        it("formats newly allowed method names", function() {
 
 
             $route = new Route(['methods' => []]);
@@ -102,39 +102,39 @@ 
             expect($route->Methods())->toBe(['POST', 'PUT']);
         });
     });
-    describe("->apply()", function () {
+    describe("->apply()", function() {
 
 
-        it("applies middlewares", function () {
+        it("applies middlewares", function() {
 
 
             $r = new Router();
-            $route = $r->bind('foo/bar', function ($route) {
+            $route = $r->bind('foo/bar', function($route) {
 
                 return 'A';
-            })->apply(function ($request, $response, $next) {
+            })->apply(function($request, $response, $next) {
 
-                return '1' . $next() . '1';
-            })->apply(function ($request, $response, $next) {
+                return '1'.$next().'1';
+            })->apply(function($request, $response, $next) {
 
-                return '2' . $next() . '2';
+                return '2'.$next().'2';
             });
             $route = $r->route('foo/bar');
             $actual = $route->dispatch();
             expect($actual)->toBe('21A12');
         });
     });
-    describe("->link()", function () {
+    describe("->link()", function() {
 
 
-        it("creates relative links", function () {
+        it("creates relative links", function() {
 
 
             $route = new Route(['pattern' => '/foo/{bar}']);
             $link = $route->link(['bar' => 'baz']);
             expect($link)->toBe('/foo/baz');
         });
-        it("supports optionnal parameters", function () {
+        it("supports optionnal parameters", function() {
 
 
             $route = new Route(['pattern' => '/foo[/{bar}]']);
@@ -143,7 +143,7 @@ 
             $link = $route->link(['bar' => 'baz']);
             expect($link)->toBe('/foo/baz');
         });
-        it("supports multiple optionnal parameters", function () {
+        it("supports multiple optionnal parameters", function() {
 
 
             $route = new Route(['pattern' => '/file[/{paths}]*']);
@@ -152,14 +152,14 @@ 
             $link = $route->link(['paths' => ['some', 'file', 'path']]);
             expect($link)->toBe('/file/some/file/path');
         });
-        it("merges default params", function () {
+        it("merges default params", function() {
 
 
             $route = new Route(['pattern' => '/foo/{bar}', 'params' => ['bar' => 'baz']]);
             $link = $route->link();
             expect($link)->toBe('/foo/baz');
         });
-        it("creates absolute links with custom base path", function () {
+        it("creates absolute links with custom base path", function() {
 
 
             $route = new Route([
@@ -176,7 +176,7 @@ 
             ]);
             expect($link)->toBe('https://www.example.com/app/foo/baz');
         });
-        it("allows host and scheme overriding", function () {
+        it("allows host and scheme overriding", function() {
 
 
             $route = new Route([
@@ -195,7 +195,7 @@ 
             ]);
             expect($link)->toBe('http://www.overrided.com/app/foo/baz');
         });
-        it("supports repeatable parameter placeholders as an array", function () {
+        it("supports repeatable parameter placeholders as an array", function() {
 
 
             $route = new Route(['pattern' => 'post[/{id}]*']);
@@ -207,7 +207,7 @@ 
             expect($route->link(['id' => ['123']]))->toBe('/post/123');
             expect($route->link(['id' => ['123', '456', '789']]))->toBe('/post/123/456/789');
         });
-        it("supports route with multiple optional segments", function () {
+        it("supports route with multiple optional segments", function() {
 
 
             $route = new Route(['pattern' => '[{relation}/{rid:[^/:][^/]*}/]post[/{id:[^/:][^/]*}][/:{action}]']);
@@ -222,7 +222,7 @@ 
             $link = $route->link(['relation' => 'user', 'rid' => 5, 'action' => 'edit', 'id' => 12]);
             expect($link)->toBe('/user/5/post/12/:edit');
         });
-        it("supports route with complex repeatable optional segments", function () {
+        it("supports route with complex repeatable optional segments", function() {
 
 
             $route = new Route(['pattern' => '[{relations:[^/]+/[^/:][^/]*}/]*post[/{id:[^/:][^/]*}][/:{action}]']);
@@ -237,40 +237,40 @@ 
             $link = $route->link(['relations' => [['user', 5]], 'action' => 'edit', 'id' => 12]);
             expect($link)->toBe('/user/5/post/12/:edit');
         });
-        it("throws an exception for missing variables", function () {
+        it("throws an exception for missing variables", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 $route = new Route(['pattern' => 'post[/{id}]+']);
                 echo $route->link([]);
             };
             expect($closure)->toThrow(new RouterException("Missing parameters `'id'` for route: `'#/post[/{id}]+'`."));
         });
-        it("throws an exception when a variable doesn't match its capture pattern", function () {
+        it("throws an exception when a variable doesn't match its capture pattern", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 $route = new Route(['pattern' => 'post/{id:[0-9]{3}}']);
                 $route->link(['id' => '1234']);
             };
             expect($closure)->toThrow(new RouterException("Expected `'id'` to match `'[0-9]{3}'`, but received `'1234'`."));
         });
-        it("throws an exception when a an array is provided for a non repeatable parameter placeholder", function () {
+        it("throws an exception when a an array is provided for a non repeatable parameter placeholder", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 $route = new Route(['pattern' => 'post/{id}']);
                 $route->link(['id' => ['123', '456']]);
             };
             expect($closure)->toThrow(new RouterException("Expected `'id'` to match `'[^/]+'`, but received `'123/456'`."));
         });
-        it("throws an exception when one element of an array doesn't match the capture pattern", function () {
+        it("throws an exception when one element of an array doesn't match the capture pattern", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 $route = new Route(['pattern' => 'post[/{id:[0-9]{3}}]+']);
                 $route->link(['id' => ['123', '456', '78']]);
@@ -278,17 +278,17 @@ 
             expect($closure)->toThrow(new RouterException("Expected `'id'` to match `'[0-9]{3}'`, but received `'78'`."));
         });
     });
-    describe("->dispatch()", function () {
+    describe("->dispatch()", function() {
 
 
-        it("passes route as argument of the handler function", function () {
+        it("passes route as argument of the handler function", function() {
 
 
             $r = new Router();
             $r->get(
                 'foo/{var1}[/{var2}]',
                 ['host' => '{subdomain}.{domain}.bar'],
-                function ($route, $response) {
+                function($route, $response) {
 
                     return array_merge([$response], array_values($route->params));
                 }
@@ -301,13 +301,13 @@ 
             $actual = $route->dispatch($response);
             expect($actual)->toBe([$response, 'foo', 'biz', '25', 'bar']);
         });
-        it("throws an exception on non valid routes", function () {
+        it("throws an exception on non valid routes", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 $r = new Router();
-                $r->get('foo', function () {
+                $r->get('foo', function() {
                 });
                 $route = $r->route('bar');
                 $route->dispatch();

src/Router.php

Spacing +5 added lines, -5 removed lines

@@ -183,7 +183,7 @@ 
     public function setBasePath(string $basePath): self
     {
         $basePath = trim($basePath, '/');
-        $this->basePath = $basePath ? '/' . $basePath : '';
+        $this->basePath = $basePath ? '/'.$basePath : '';
         return $this;
     }
 
@@ -220,7 +220,7 @@ 
             $options['host'] = $this->hosts[$scheme][$host];
         }
 
-        $patternKey = md5($options['pattern'] . '-' . $options['name']);
+        $patternKey = md5($options['pattern'].'-'.$options['name']);
         if (isset($this->pattern[$scheme][$host][$patternKey])) {
             $route = $this->pattern[$scheme][$host][$patternKey];
         } else {
@@ -280,7 +280,7 @@ 
             $options['host'] = $this->hosts[$scheme][$host];
         }
 
-        $patternKey = md5($options['pattern'] . '-' . $options['name']);
+        $patternKey = md5($options['pattern'].'-'.$options['name']);
         if (isset($this->pattern[$scheme][$host][$patternKey])) {
             $instance = $this->pattern[$scheme][$host][$patternKey];
         } else {
@@ -293,7 +293,7 @@ 
             $this->pattern[$scheme][$host][$patternKey] = $instance;
         }
 
-        $methods = $options['methods'] ? (array)$options['methods'] : [];
+        $methods = $options['methods'] ? (array) $options['methods'] : [];
         $instance->allow($methods);
         foreach ($methods as $method) {
             $this->routes[$scheme][$host][strtoupper($method)][] = $instance;
@@ -411,7 +411,7 @@ 
             $request = $parsed + $request;
         }
 
-        $request['path'] = (ltrim((string)strtok($request['path'], '?'), '/'));
+        $request['path'] = (ltrim((string) strtok($request['path'], '?'), '/'));
         $request['method'] = strtoupper($request['method']);
 
         return $request;