jetfirephp / routing

src/Routing/Middleware.php

Braces +35 added lines, -26 removed lines

@@ -29,9 +29,10 @@ 
      */
     public function globalMiddleware()
     {
-        if (isset($this->router->collection->middleware['global_middleware']))
-            foreach ($this->router->collection->middleware['global_middleware'] as $class)
-                if (class_exists($class)) return $this->callHandler($class);
+        if (isset($this->router->collection->middleware['global_middleware'])) {

+                    foreach ($this->router->collection->middleware['global_middleware'] as $class)
+                if (class_exists($class)) return $this->callHandler($class);

+        }

         return null;
     }
 
@@ -40,16 +41,18 @@ 
      */
     public function blockMiddleware()
     {
-        if (isset($this->router->collection->middleware['block_middleware']))
-            if (isset($this->router->collection->middleware['block_middleware'][$this->router->route->getTarget('block')])) {
-                $blocks = $this->router->collection->middleware['block_middleware'][$this->router->route->getTarget('block')];
+        if (isset($this->router->collection->middleware['block_middleware'])) {

+                    if (isset($this->router->collection->middleware['block_middleware'][$this->router->route->getTarget('block')])) {
+                $blocks = $this->router->collection->middleware['block_middleware'][$this->router->route->getTarget('block')];

+        }

                 if (is_array($blocks)) {
-                    foreach ($blocks as $block)
-                        if (class_exists($block))
-                            if($this->callHandler($block) === false) return false;
-                }
-                elseif (is_string($blocks) && class_exists($blocks))
-                    return $this->callHandler($blocks);
+                    foreach ($blocks as $block) {

+                                            if (class_exists($block))
+                            if($this->callHandler($block) === false) return false;

+                    }

+                } elseif (is_string($blocks) && class_exists($blocks)) {

+                                    return $this->callHandler($blocks);

+                }

             }
         return null;
     }
@@ -64,10 +67,12 @@ 
             if (isset($this->router->collection->middleware['class_middleware'][$ctrl]) && class_exists($this->router->route->getTarget('controller'))) {
                 $classes = $this->router->collection->middleware['class_middleware'][$ctrl];
                 if(is_array($classes)){
-                    foreach ($classes as $class)
-                        if($this->callHandler($class) === false)return false;
-                }elseif(is_string($classes))
-                    return $this->callHandler($classes);
+                    foreach ($classes as $class) {

+                                            if($this->callHandler($class) === false)return false;

+                    }

+                } elseif(is_string($classes)) {

+                                    return $this->callHandler($classes);

+                }

             }
         }
         return null;
@@ -78,14 +83,17 @@ 
      */
     public function routeMiddleware()
     {
-        if (isset($this->router->collection->middleware['route_middleware']))
-            if (isset($this->router->route->getPath()['middleware']) && class_exists($this->router->collection->middleware['route_middleware'][$this->router->route->getPath()['middleware']])) {
-                $classes = $this->router->collection->middleware['route_middleware'][$this->router->route->getPath()['middleware']];
+        if (isset($this->router->collection->middleware['route_middleware'])) {

+                    if (isset($this->router->route->getPath()['middleware']) && class_exists($this->router->collection->middleware['route_middleware'][$this->router->route->getPath()['middleware']])) {
+                $classes = $this->router->collection->middleware['route_middleware'][$this->router->route->getPath()['middleware']];

+        }

                 if(is_array($classes)){
-                    foreach ($classes as $class)
-                        if($this->callHandler($class) === false)return false;
-                }elseif(is_string($classes))
-                    return $this->callHandler($classes);
+                    foreach ($classes as $class) {

+                                            if($this->callHandler($class) === false)return false;

+                    }

+                } elseif(is_string($classes)) {

+                                    return $this->callHandler($classes);

+                }

             }
         return null;
     }
@@ -99,9 +107,10 @@ 
         if (method_exists($instance, 'handle')) {
             $reflectionMethod = new ReflectionMethod($instance, 'handle');
             $dependencies = [];
-            foreach ($reflectionMethod->getParameters() as $arg)
-                if (!is_null($arg->getClass()))
-                    $dependencies[] = $this->getClass($arg->getClass()->name);
+            foreach ($reflectionMethod->getParameters() as $arg) {

+                            if (!is_null($arg->getClass()))
+                    $dependencies[] = $this->getClass($arg->getClass()->name);

+            }

             $dependencies = array_merge($dependencies, [$this->router->route]);
             return $reflectionMethod->invokeArgs($instance, $dependencies);
         }

src/Routing/Router.php

Braces +25 added lines, -12 removed lines

@@ -81,8 +81,9 @@ 
      * @param object|array $matcher
      */
     public function setMatcher($matcher){
-        if(is_object($matcher))
-            $matcher = [$matcher];
+        if(is_object($matcher)) {

+                    $matcher = [$matcher];

+        }

         $this->matcher = $matcher;
     }
 
@@ -99,8 +100,12 @@ 
     public function run()
     {
         $this->setUrl();
-        if ($this->config['generateRoutesPath']) $this->collection->generateRoutesPath();
-        if ($this->match()) $this->callTarget();
+        if ($this->config['generateRoutesPath']) {

+            $this->collection->generateRoutesPath();

+        }

+        if ($this->match()) {

+            $this->callTarget();

+        }

         $this->callResponse();
     }
 
@@ -109,8 +114,9 @@ 
      */
     public function setUrl($url = null)
     {
-        if (is_null($url))
-            $url = (isset($_GET['url'])) ? $_GET['url'] : substr(str_replace(str_replace('/index.php', '', $_SERVER['SCRIPT_NAME']), '', $_SERVER['REQUEST_URI']), 1);
+        if (is_null($url)) {

+                    $url = (isset($_GET['url'])) ? $_GET['url'] : substr(str_replace(str_replace('/index.php', '', $_SERVER['SCRIPT_NAME']), '', $_SERVER['REQUEST_URI']), 1);

+        }

         $this->route->setUrl('/' . trim(explode('?', $url)[0], '/'));
     }
 
@@ -120,7 +126,9 @@ 
     public function match()
     {
         foreach ($this->matcher as $key => $matcher) {
-            if (call_user_func([$this->matcher[$key], 'match'])) return true;
+            if (call_user_func([$this->matcher[$key], 'match'])) {

+                return true;

+            }

         }
         return false;
     }
@@ -155,11 +163,16 @@ 
         if (isset($this->route->getDetail()['response_templates']) && isset($this->route->getDetail()['response_templates'][$code = $this->response->getStatusCode()])) {
             $this->route->setCallback($this->route->getDetail()['response_templates'][$code]);
             $matcher = null;
-            foreach($this->matcher as $instance) if($instance instanceof ArrayMatcher) $matcher = $instance;
-            if(is_null($matcher))$matcher = new ArrayMatcher($this);
-            foreach (call_user_func([$matcher, 'getResolver']) as $match)
-                if (is_array($target = call_user_func_array([$matcher, $match], [$this->route->getCallback()]))){
-                    call_user_func_array([$matcher, 'setTarget'],[$target]);
+            foreach($this->matcher as $instance) {

+                if($instance instanceof ArrayMatcher) $matcher = $instance;

+            }

+            if(is_null($matcher)) {

+                $matcher = new ArrayMatcher($this);

+            }

+            foreach (call_user_func([$matcher, 'getResolver']) as $match) {

+                            if (is_array($target = call_user_func_array([$matcher, $match], [$this->route->getCallback()]))){
+                    call_user_func_array([$matcher, 'setTarget'],[$target]);

+            }

                     $this->callTarget();
                     break;
                 }