jetfirephp / routing

src/Routing/Dispatcher/ControllerDispatcher.php

Braces +14 added lines, -8 removed lines

@@ -50,12 +50,17 @@ 
         }
         if($count == count($this->route->getParameters())) {
             $dependencies = array_merge($dependencies, ($this->route->getParameters() == '') ? [] : $this->route->getParameters());
-            if ($this->response->getStatusCode() == 202)
-                $this->response->setStatusCode(200);
-            if (is_array($content = $reflectionMethod->invokeArgs($this->getController(), $dependencies))) $this->route->addTarget('data', $content);
-            elseif (!is_null($content)) $this->response->setContent($content);
-        }else
-            $this->response->setStatusCode(404);
+            if ($this->response->getStatusCode() == 202) {

+                            $this->response->setStatusCode(200);

+            }

+            if (is_array($content = $reflectionMethod->invokeArgs($this->getController(), $dependencies))) {

+                $this->route->addTarget('data', $content);

+            } elseif (!is_null($content)) {

+                $this->response->setContent($content);

+            }

+        } else {

+                    $this->response->setStatusCode(404);

+        }

     }
 
 
@@ -66,8 +71,9 @@ 
     private function getController()
     {
         $reflector = new ReflectionClass($this->route->getTarget('controller'));
-        if (!$reflector->isInstantiable())
-            throw new \Exception('Target [' . $this->route->getTarget('controller') . '] is not instantiable.');
+        if (!$reflector->isInstantiable()) {

+                    throw new \Exception('Target [' . $this->route->getTarget('controller') . '] is not instantiable.');

+        }

         $constructor = $reflector->getConstructor();
         if (is_null($constructor)) {
             $class = $this->route->getTarget('controller');

Spacing +6 added lines, -6 removed lines

@@ -27,7 +27,7 @@ 
     /**
      * @param Route $route
      */
-    public function __construct(Route $route,ResponseInterface $response)
+    public function __construct(Route $route, ResponseInterface $response)
     {
         $this->route = $route;
         $this->response = $response;
@@ -48,13 +48,13 @@ 
                 ? $count++
                 : $dependencies[] = call_user_func_array($this->route->getTarget('di'), [$arg->getClass()->name]);
         }
-        if($count == count($this->route->getParameters()) || ($this->route->getParameters() == '' && $count == 0)) {
+        if ($count == count($this->route->getParameters()) || ($this->route->getParameters() == '' && $count == 0)) {
             $dependencies = array_merge($dependencies, ($this->route->getParameters() == '') ? [] : $this->route->getParameters());
             if ($this->response->getStatusCode() == 202)
                 $this->response->setStatusCode(200);
             if (is_array($content = $reflectionMethod->invokeArgs($this->getController(), $dependencies))) $this->route->addTarget('data', $content);
             elseif (!is_null($content)) $this->response->setContent($content);
-        }else
+        } else
             $this->response->setStatusCode(404);
     }
 
@@ -67,17 +67,17 @@ 
     {
         $reflector = new ReflectionClass($this->route->getTarget('controller'));
         if (!$reflector->isInstantiable())
-            throw new \Exception('Target [' . $this->route->getTarget('controller') . '] is not instantiable.');
+            throw new \Exception('Target ['.$this->route->getTarget('controller').'] is not instantiable.');
         $constructor = $reflector->getConstructor();
         if (is_null($constructor)) {
             $class = $this->route->getTarget('controller');
-            return call_user_func_array($this->route->getTarget('di'),[$class]);
+            return call_user_func_array($this->route->getTarget('di'), [$class]);
         }
         $dependencies = $constructor->getParameters();
         $arguments = [];
         foreach ($dependencies as $dep) {
             $class = $dep->getClass()->name;
-            array_push($arguments, call_user_func_array($this->route->getTarget('di'),[$class]));
+            array_push($arguments, call_user_func_array($this->route->getTarget('di'), [$class]));
         }
         return $reflector->newInstanceArgs($arguments);
     }

src/Routing/Middleware.php

Spacing +12 added lines, -12 removed lines

@@ -46,7 +46,7 @@ 
                 if (is_array($blocks)) {
                     foreach ($blocks as $block)
                         if (class_exists($block))
-                            if($this->callHandler($block) === false) return false;
+                            if ($this->callHandler($block) === false) return false;
                 }
                 elseif (is_string($blocks) && class_exists($blocks))
                     return $this->callHandler($blocks);
@@ -63,10 +63,10 @@ 
             $ctrl = str_replace('\\', '/', $this->router->route->getTarget('controller'));
             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)){
+                if (is_array($classes)) {
                     foreach ($classes as $class)
-                        if($this->callHandler($class) === false)return false;
-                }elseif(is_string($classes))
+                        if ($this->callHandler($class) === false)return false;
+                }elseif (is_string($classes))
                     return $this->callHandler($classes);
             }
         }
@@ -81,10 +81,10 @@ 
         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)){
+                if (is_array($classes)) {
                     foreach ($classes as $class)
-                        if($this->callHandler($class) === false)return false;
-                }elseif(is_string($classes))
+                        if ($this->callHandler($class) === false)return false;
+                }elseif (is_string($classes))
                     return $this->callHandler($classes);
             }
         return null;
@@ -94,8 +94,8 @@ 
      * @param $class
      * @return mixed
      */
-    private function callHandler($class){
-        $instance = call_user_func($this->router->getConfig()['di'],$class);
+    private function callHandler($class) {
+        $instance = call_user_func($this->router->getConfig()['di'], $class);
         if (method_exists($instance, 'handle')) {
             $reflectionMethod = new ReflectionMethod($instance, 'handle');
             $dependencies = [];
@@ -112,8 +112,8 @@ 
      * @param $class
      * @return Route|RouteCollection|Router|mixed
      */
-    private function getClass($class){
-        switch($class){
+    private function getClass($class) {
+        switch ($class) {
             case 'JetFire\Routing\Route':
                 return $this->router->route;
             case 'JetFire\Routing\Router':
@@ -121,7 +121,7 @@ 
             case 'JetFire\Routing\RouteCollection':
                 return $this->router->collection;
             default:
-                return call_user_func_array($this->router->getConfig()['di'],[$class]);
+                return call_user_func_array($this->router->getConfig()['di'], [$class]);
         }
     }
 }

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);
         }