SiroDiaz / Routify

Routify/RouterParser.php

Doc Comments +3 added lines

@@ -20,6 +20,9 @@
      */
     const PARAMETER_IDENTIFIER = ':';
 
+    /**
+     * @param string $path
+     */
     public function __construct($path) {
         $this->path = $path;
     }

Spacing +11 added lines, -11 removed lines

@@ -65,14 +65,14 @@ 
      */
 
     public function countParams($pattern) {
-        if($this->hasParams($pattern) === false) {
+        if ($this->hasParams($pattern) === false) {
             return 0;
         }
 
         $pattern = str_split($pattern); // split string in characters
         $totalParams = 0;
-        for($i = 0; $i < count($pattern); $i++) {
-            if($pattern[$i] === self::PARAMETER_IDENTIFIER) {
+        for ($i = 0; $i < count($pattern); $i++) {
+            if ($pattern[$i] === self::PARAMETER_IDENTIFIER) {
                 $totalParams++;
             }
         }
@@ -90,15 +90,15 @@ 
      */
 
     public function getParams($pattern) {
-        if(!$this->hasParams($pattern)) {
+        if (!$this->hasParams($pattern)) {
             return [];
         }
 
         $pattern = explode('/', $pattern);
         $path = explode('/', $this->path);
         $params = [];
-        for($i = 0; $i < count($pattern); $i++) {
-            if(strpos($pattern[$i], self::PARAMETER_IDENTIFIER) !== false) {
+        for ($i = 0; $i < count($pattern); $i++) {
+            if (strpos($pattern[$i], self::PARAMETER_IDENTIFIER) !== false) {
                 $params[substr($pattern[$i], 1)] = $path[$i];
             }
         }
@@ -117,20 +117,20 @@ 
         $pattern = explode('/', $pattern);
         $path = explode('/', $this->path);
 
-        if(count($pattern) !== count($path)) {
+        if (count($pattern) !== count($path)) {
             return false;
         }
 
         $found = true;
         $index = 0;
-        while($found && $index < count($pattern)) {
+        while ($found && $index < count($pattern)) {
             $check = true;
-            if(strpos($pattern[$index], self::PARAMETER_IDENTIFIER) !== false) {
+            if (strpos($pattern[$index], self::PARAMETER_IDENTIFIER) !== false) {
                 $check = false;
             }
 
-            if($check) {
-                if (!preg_match('/^'. $pattern[$index] .'$/', $path[$index])) {
+            if ($check) {
+                if (!preg_match('/^'.$pattern[$index].'$/', $path[$index])) {
                     $found = false;
                 }
             }

Routify/Order.php

Spacing +6 added lines, -6 removed lines

@@ -44,12 +44,12 @@ 
 
     public function __construct($uri, $method, $response, array $middlewares = []) {
         $this->uri = $uri;
-        if(!in_array(mb_strtoupper($method), $this->availableRequestMethods)) {
+        if (!in_array(mb_strtoupper($method), $this->availableRequestMethods)) {
             throw new MethodException("The request method is invalid");
         }
         $this->method = $method;
         $this->response = $response;
-        if($this->isValidMiddleware($middlewares)) {
+        if ($this->isValidMiddleware($middlewares)) {
             $this->middlewares = $middlewares;
         }
     }
@@ -126,15 +126,15 @@ 
 
     private function isValidMiddleware($middleware) {
         // if there is not middlewares then it is valid
-        if(count($middleware) === 0) {
+        if (count($middleware) === 0) {
             return true;
         }
 
-        foreach($middleware as $key => $value) {
-            if(!in_array($key, $this->middlewareTypes)) {
+        foreach ($middleware as $key => $value) {
+            if (!in_array($key, $this->middlewareTypes)) {
                 throw new InvalidMiddlewareException("Only before and after middleware types are valid");
             }
-            if(!is_callable($value)) {
+            if (!is_callable($value)) {
                 throw new InvalidMiddlewareException("The middleware must be callable");
             }
         }

Routify/Router.php

Spacing +19 added lines, -19 removed lines

@@ -118,12 +118,12 @@ 
         $found = false;
         $counter = 0;
 
-        if(count($this->routes) === 0) {
+        if (count($this->routes) === 0) {
             return $found;
         }
 
-        while($found === false && $counter < count($this->routes)) {
-            if($this->routes[$counter]->getUri() === $uri && $this->routes[$counter]->getMethod() === $method) {
+        while ($found === false && $counter < count($this->routes)) {
+            if ($this->routes[$counter]->getUri() === $uri && $this->routes[$counter]->getMethod() === $method) {
                 $found = true;
             }
 
@@ -143,7 +143,7 @@ 
      */
 
     private function addRoute($uri, $method, $response, array $middleware = []) {
-        if($this->find($uri, $method)) {    // search if exists an apparition
+        if ($this->find($uri, $method)) {    // search if exists an apparition
             return false;
         }
 
@@ -159,13 +159,13 @@ 
      */
     private function getRequestHeaders() {
         // If getallheaders() is available, use that
-        if(function_exists('getallheaders')) {
+        if (function_exists('getallheaders')) {
             return getallheaders();
         }
         // Method getallheaders() not available: manually extract 'm
         $headers = [];
-        foreach(filter_input_array(INPUT_SERVER) as $key => $value) {
-            if((substr($key, 0, 5) == 'HTTP_') || ($key == 'CONTENT_TYPE') || ($key == 'CONTENT_LENGTH')) {
+        foreach (filter_input_array(INPUT_SERVER) as $key => $value) {
+            if ((substr($key, 0, 5) == 'HTTP_') || ($key == 'CONTENT_TYPE') || ($key == 'CONTENT_LENGTH')) {
                 $headers[str_replace([' ', 'Http'], ['-', 'HTTP'], ucwords(strtolower(str_replace('_', ' ', substr($key, 5)))))] = $value;
             }
         }
@@ -179,13 +179,13 @@ 
      */
 
     private function overrideMethod() {
-        if($this->requestMethod === 'HEAD') {
+        if ($this->requestMethod === 'HEAD') {
             $this->requestMethod = 'GET';
-        }elseif($this->requestMethod === 'POST' && filter_input(INPUT_POST, '_method') !== null) {
+        }elseif ($this->requestMethod === 'POST' && filter_input(INPUT_POST, '_method') !== null) {
             $this->requestMethod = filter_input(INPUT_POST, '_method');
-        }elseif($this->requestMethod === 'POST') {
+        }elseif ($this->requestMethod === 'POST') {
             $headers = $this->getRequestHeaders();
-            if(isset($headers['X-HTTP-Method-Override']) && in_array($headers['X-HTTP-Method-Override'], ['PUT', 'DELETE', 'PATCH'])) {
+            if (isset($headers['X-HTTP-Method-Override']) && in_array($headers['X-HTTP-Method-Override'], ['PUT', 'DELETE', 'PATCH'])) {
                 $this->requestMethod = $headers['X-HTTP-Method-Override'];
             }
         }
@@ -264,8 +264,8 @@ 
      */
 
     public function both($uri, $response, $methods = Method::GET, array $middleware = []) {
-        if(is_array($methods)) {
-            foreach($methods as $method) {
+        if (is_array($methods)) {
+            foreach ($methods as $method) {
                 $this->addRoute($uri, $method, $response, $middleware);
             }
         } else {
@@ -292,7 +292,7 @@ 
      */
 
     public function notFound($func) {
-        if(is_callable($func)) {
+        if (is_callable($func)) {
             $this->notFound = $func;
         }
     }
@@ -310,17 +310,17 @@ 
     public function run() {
         $found = false;
         $counter = 0;
-        while($found === false && $counter < count($this->routes)) {
-            if($this->routerParser->match($this->routes[$counter]->getUri()) && $this->routes[$counter]->getMethod() === $this->requestMethod) {
+        while ($found === false && $counter < count($this->routes)) {
+            if ($this->routerParser->match($this->routes[$counter]->getUri()) && $this->routes[$counter]->getMethod() === $this->requestMethod) {
                 $found = true;
             } else {
                 $counter++;
             }
         }
 
-        if($found) {
+        if ($found) {
             // run the before middleware if it exists
-            if($this->routes[$counter]->hasBefore()) {
+            if ($this->routes[$counter]->hasBefore()) {
                 call_user_func($this->routes[$counter]->getMiddlewares()['before']);
             }
 
@@ -328,7 +328,7 @@ 
             $response = call_user_func_array($this->routes[$counter]->getResponse(), $params);
 
             // run the after middleware if it exists
-            if($this->routes[$counter]->hasAfter()) {
+            if ($this->routes[$counter]->hasAfter()) {
                 call_user_func($this->routes[$counter]->getMiddlewares()['after']);
             }
 

Braces +2 added lines, -2 removed lines

@@ -181,9 +181,9 @@
     private function overrideMethod() {
         if($this->requestMethod === 'HEAD') {
             $this->requestMethod = 'GET';
-        }elseif($this->requestMethod === 'POST' && filter_input(INPUT_POST, '_method') !== null) {
+        } elseif($this->requestMethod === 'POST' && filter_input(INPUT_POST, '_method') !== null) {
             $this->requestMethod = filter_input(INPUT_POST, '_method');
-        }elseif($this->requestMethod === 'POST') {
+        } elseif($this->requestMethod === 'POST') {
             $headers = $this->getRequestHeaders();
             if(isset($headers['X-HTTP-Method-Override']) && in_array($headers['X-HTTP-Method-Override'], ['PUT', 'DELETE', 'PATCH'])) {
                 $this->requestMethod = $headers['X-HTTP-Method-Override'];