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 +5 added lines, -5 removed lines

@@ -35,7 +35,7 @@ 
         $this->uri = $uri;
         $this->method = $method;
         $this->response = $response;
-        if($this->isValidMiddleware($middlewares)) {
+        if ($this->isValidMiddleware($middlewares)) {
             $this->middlewares = $middlewares;
         }
     }
@@ -112,15 +112,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 +12 added lines, -12 removed lines

@@ -104,12 +104,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;
             }
 
@@ -129,7 +129,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;
         }
 
@@ -212,8 +212,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 {
@@ -228,7 +228,7 @@ 
      */
 
     public function notFound($func) {
-        if(is_callable($func)) {
+        if (is_callable($func)) {
             $this->notFound = $func;
         }
     }
@@ -246,17 +246,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']);
             }
 
@@ -264,7 +264,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']);
             }