jetfirephp / routing

src/Routing/Match/RoutesMatch.php

Spacing +12 added lines, -12 removed lines

@@ -24,7 +24,7 @@ 
     /**
      * @var array
      */
-    private $matcher = ['matchClosure','matchController','matchTemplate'];
+    private $matcher = ['matchClosure', 'matchController', 'matchTemplate'];
 
     /**
      * @param Router $router
@@ -37,7 +37,7 @@ 
     /**
      * @param string $matcher
      */
-    public function addMatcher($matcher){
+    public function addMatcher($matcher) {
         $this->matcher[] = $matcher;
     }
 
@@ -106,7 +106,7 @@ 
      */
     private function generateTarget()
     {
-        if(is_callable($this->request['path'])){
+        if (is_callable($this->request['path'])) {
             $this->router->route->setCallback($this->request['path']);
             $this->router->route->setDetail($this->request);
             $this->matchClosure();
@@ -119,11 +119,11 @@ 
                     ? $this->router->route->setCallback($this->request['path']['use'])
                     : $this->router->route->setCallback($this->request['path']);
             $this->router->route->setDetail($this->request);
-            if($this->validMethod()) {
-                foreach($this->matcher as $matcher)
-                    call_user_func([$this,$matcher]);
+            if ($this->validMethod()) {
+                foreach ($this->matcher as $matcher)
+                    call_user_func([$this, $matcher]);
                 $this->router->route->setResponse(['code' => 202, 'message' => 'Accepted']);
-            }else
+            } else
                 $this->router->route->setResponse(['code' => 405, 'message' => 'Method Not Allowed']);
         }
         return $this->router->route->hasTarget();
@@ -165,7 +165,7 @@ 
             $index = isset($this->request['index']) ? $this->request['index'] : 0;
             $class = (class_exists($routes[0]))
                 ? $routes[0]
-                : $this->router->collection->getRoutes()['namespace_'.$index].$routes[0];
+                : $this->router->collection->getRoutes()['namespace_' . $index] . $routes[0];
             if (!class_exists($class))
                 throw new \Exception('Class "' . $class . '." is not found');
             if (method_exists($class, $routes[1])) {
@@ -193,7 +193,7 @@ 
             $extension = explode('.', $path);
             $extension = end($extension);
             $index = isset($this->request['index']) ? $this->request['index'] : 0;
-            $block = $this->router->collection->getRoutes('path_'.$index);
+            $block = $this->router->collection->getRoutes('path_' . $index);
             if (in_array('.' . $extension, $this->router->getConfig()['viewExtension'])) {
                 if (is_file($block . $path)) {
                     $target = $block . $path;
@@ -209,13 +209,13 @@ 
                 throw new \Exception('Template file "' . $path . '" is not found in "' . $block . '"');
             } else {
                 foreach ($this->router->getConfig()['viewExtension'] as $ext) {
-                    if (is_file($block . $path . $ext)){
+                    if (is_file($block . $path . $ext)) {
                         $target = $block . $path . $ext;
-                        $this->router->route->setTarget(['dispatcher' => 'JetFire\Routing\Dispatcher\TemplateDispatcher', 'template' => $target,'block' => $block,  'extension' => str_replace('.', '', $ext),'callback' => $this->router->getConfig()['viewCallback']]);
+                        $this->router->route->setTarget(['dispatcher' => 'JetFire\Routing\Dispatcher\TemplateDispatcher', 'template' => $target, 'block' => $block, 'extension' => str_replace('.', '', $ext), 'callback' => $this->router->getConfig()['viewCallback']]);
                         return true;
                     }
                 }
-                throw new \Exception('Template file "' . $path . '" is not found in "' .$block . '"');
+                throw new \Exception('Template file "' . $path . '" is not found in "' . $block . '"');
             }
         }
         return false;

Braces +31 added lines, -18 removed lines

@@ -62,8 +62,9 @@ 
                 $this->request['path'] = $dependencies;
                 $this->request['index'] = $i;
                 $this->request['route'] = preg_replace_callback('#:([\w]+)#', [$this, 'paramMatch'], '/' . trim(trim($this->request['prefix'], '/') . '/' . trim($route, '/'), '/'));
-                if ($this->routeMatch('#^' . $this->request['route'] . '$#'))
-                    return $this->generateTarget();
+                if ($this->routeMatch('#^' . $this->request['route'] . '$#')) {

+                                    return $this->generateTarget();

+                }

             }
         }
         unset($this->request);
@@ -91,8 +92,9 @@ 
     {
         if (substr($this->request['route'], -1) == '*') {
             $pos = strpos($this->request['route'], '*');
-            if (substr($this->router->route->getUrl(), 0, $pos) == substr($this->request['route'], 0, $pos))
-                if (isset($this->request)) return true;
+            if (substr($this->router->route->getUrl(), 0, $pos) == substr($this->request['route'], 0, $pos)) {

+                            if (isset($this->request)) return true;

+            }

         }
         if (preg_match($regex, $this->router->route->getUrl(), $this->request['parameters'])) {
             array_shift($this->request['parameters']);
@@ -112,19 +114,26 @@ 
             $this->matchClosure();
             $this->router->route->setResponse(['code' => 202, 'message' => 'Accepted']);
         } else {
-            if (isset($this->request['path']['name'])) $this->router->route->setName($this->request['path']['name']);
-            if (isset($this->request['path']['method'])) $this->request['path']['method'] = is_array($this->request['path']['method']) ? $this->request['path']['method'] : [$this->request['path']['method']];
-            if (isset($this->request['path']))
-                (is_array($this->request['path']) && isset($this->request['path']['use']))
+            if (isset($this->request['path']['name'])) {

+                $this->router->route->setName($this->request['path']['name']);

+            }

+            if (isset($this->request['path']['method'])) {

+                $this->request['path']['method'] = is_array($this->request['path']['method']) ? $this->request['path']['method'] : [$this->request['path']['method']];

+            }

+            if (isset($this->request['path'])) {

+                            (is_array($this->request['path']) && isset($this->request['path']['use']))
                     ? $this->router->route->setCallback($this->request['path']['use'])
-                    : $this->router->route->setCallback($this->request['path']);
+                    : $this->router->route->setCallback($this->request['path']);

+            }

             $this->router->route->setDetail($this->request);
             if($this->validMethod()) {
-                foreach($this->matcher as $matcher)
-                    call_user_func([$this,$matcher]);
+                foreach($this->matcher as $matcher) {

+                                    call_user_func([$this,$matcher]);

+                }

                 $this->router->route->setResponse(['code' => 202, 'message' => 'Accepted']);
-            }else
-                $this->router->route->setResponse(['code' => 405, 'message' => 'Method Not Allowed']);
+            } else {

+                            $this->router->route->setResponse(['code' => 405, 'message' => 'Method Not Allowed']);

+            }

         }
         return $this->router->route->hasTarget();
     }
@@ -134,8 +143,9 @@ 
      */
     public function validMethod()
     {
-        if (!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest')
-            return (isset($this->request['path']['ajax']) && $this->request['path']['ajax'] === true) ? true : false;
+        if (!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest') {

+                    return (isset($this->request['path']['ajax']) && $this->request['path']['ajax'] === true) ? true : false;

+        }

         $method = (isset($this->router->route->getDetail()['path']['method'])) ? $this->router->route->getDetail()['path']['method'] : ['GET'];
         return (in_array($this->router->route->getMethod(), $method)) ? true : false;
     }
@@ -161,13 +171,16 @@ 
     {
         if (!$this->router->route->hasTarget() && strpos($this->router->route->getCallback(), '@') !== false) {
             $routes = explode('@', $this->router->route->getCallback());
-            if (!isset($routes[1])) $routes[1] = 'index';
+            if (!isset($routes[1])) {

+                $routes[1] = 'index';

+            }

             $index = isset($this->request['index']) ? $this->request['index'] : 0;
             $class = (class_exists($routes[0]))
                 ? $routes[0]
                 : $this->router->collection->getRoutes()['namespace_'.$index].$routes[0];
-            if (!class_exists($class))
-                throw new \Exception('Class "' . $class . '." is not found');
+            if (!class_exists($class)) {

+                            throw new \Exception('Class "' . $class . '." is not found');

+            }

             if (method_exists($class, $routes[1])) {
                 $this->router->route->setTarget([
                     'dispatcher' => 'JetFire\Routing\Dispatcher\ControllerDispatcher',

src/Routing/Match/SmartMatch.php

Spacing +11 added lines, -11 removed lines

@@ -20,7 +20,7 @@ 
     /**
      * @var array
      */
-    private $matcher = ['matchTemplate','matchController'];
+    private $matcher = ['matchTemplate', 'matchController'];
 
     /**
      * @param Router $router
@@ -33,7 +33,7 @@ 
     /**
      * @param string $matcher
      */
-    public function addMatcher($matcher){
+    public function addMatcher($matcher) {
         $this->matcher[] = $matcher;
     }
 
@@ -50,8 +50,8 @@ 
      */
     public function match()
     {
-        foreach($this->matcher as $matcher){
-            if(call_user_func([$this,$matcher])) {
+        foreach ($this->matcher as $matcher) {
+            if (call_user_func([$this, $matcher])) {
                 $this->router->route->setResponse(['code' => 202, 'message' => 'Accepted']);
                 return true;
             }
@@ -66,9 +66,9 @@ 
     {
         foreach ($this->router->getConfig()['viewExtension'] as $extension) {
             for ($i = 0; $i < $this->router->collection->countRoutes; ++$i) {
-                $url = explode('/', str_replace($this->router->collection->getRoutes('prefix_' . $i), '',$this->router->route->getUrl()));
+                $url = explode('/', str_replace($this->router->collection->getRoutes('prefix_' . $i), '', $this->router->route->getUrl()));
                 $end = array_pop($url);
-                $url = implode('/', array_map('ucwords', $url)).'/'.$end;
+                $url = implode('/', array_map('ucwords', $url)) . '/' . $end;
                 if (is_file(($template = rtrim($this->router->collection->getRoutes('path_' . $i), '/') . $url . $extension))) {
                     $this->router->route->setTarget([
                         'dispatcher' => 'JetFire\Routing\Dispatcher\TemplateDispatcher',
@@ -91,12 +91,12 @@ 
     {
         $routes = array_slice(explode('/', $this->router->route->getUrl()), 1);
         $i = 0;
-        do{
-            $route =  ('/' . $routes[0] == $this->router->collection->getRoutes('prefix_' . $i))
+        do {
+            $route = ('/' . $routes[0] == $this->router->collection->getRoutes('prefix_' . $i))
                 ? array_slice($routes, 1) : $routes;
             if (isset($route[0])) {
-                $class =  (class_exists($this->router->collection->getRoutes('namespace_' . $i). ucfirst($route[0]) . 'Controller'))
-                    ? $this->router->collection->getRoutes('namespace_' . $i). ucfirst($route[0]) . 'Controller'
+                $class = (class_exists($this->router->collection->getRoutes('namespace_' . $i) . ucfirst($route[0]) . 'Controller'))
+                    ? $this->router->collection->getRoutes('namespace_' . $i) . ucfirst($route[0]) . 'Controller'
                     : ucfirst($route[0]) . 'Controller';
                 if (isset($route[1]) && method_exists($class, $route[1])) {
                     $this->router->route->setTarget([
@@ -110,7 +110,7 @@ 
                 }
             }
             ++$i;
-        }while($i < $this->router->collection->countRoutes);
+        }while ($i < $this->router->collection->countRoutes);
         return false;
     }
 

src/Routing/Router.php

Spacing +3 added lines, -3 removed lines

@@ -44,7 +44,7 @@ 
     {
         $this->collection = $collection;
         $this->route = new Route();
-        $this->config['di'] = function($class){
+        $this->config['di'] = function($class) {
             return new $class;
         };
     }
@@ -139,9 +139,9 @@ 
     {
         if (isset($this->route->getResponse()['templates']) && isset($this->route->getResponse()['templates'][$this->route->getResponse('code')])) {
             $this->route->setCallback($this->route->getResponse()['templates'][$this->route->getResponse('code')]);
-            foreach($this->config['matcherInstance'] as $matcher) {
+            foreach ($this->config['matcherInstance'] as $matcher) {
                 foreach (call_user_func([$matcher, 'getMatcher']) as $match)
-                    if (call_user_func([$matcher, $match])){ $this->callTarget(); break; }
+                    if (call_user_func([$matcher, $match])) { $this->callTarget(); break; }
             }
         }
         http_response_code($this->route->getResponse('code'));

Braces +13 added lines, -7 removed lines

@@ -71,7 +71,9 @@ 
     public function run()
     {
         $this->setUrl();
-        if ($this->config['generateRoutesPath']) $this->collection->generateRoutesPath();
+        if ($this->config['generateRoutesPath']) {

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

+        }

         if ($this->match()) {
             $this->handle();
             $this->callTarget();
@@ -84,8 +86,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], '/'));
     }
 
@@ -96,8 +99,9 @@ 
     {
         foreach ($this->config['matcher'] as $matcher) {
             $this->config['matcherInstance'][$matcher] = new $matcher($this);
-            if (call_user_func([$this->config['matcherInstance'][$matcher], 'match']))
-                return true;
+            if (call_user_func([$this->config['matcherInstance'][$matcher], 'match'])) {

+                            return true;

+            }

         }
         return false;
     }
@@ -140,8 +144,10 @@ 
         if (isset($this->route->getResponse()['templates']) && isset($this->route->getResponse()['templates'][$this->route->getResponse('code')])) {
             $this->route->setCallback($this->route->getResponse()['templates'][$this->route->getResponse('code')]);
             foreach($this->config['matcherInstance'] as $matcher) {
-                foreach (call_user_func([$matcher, 'getMatcher']) as $match)
-                    if (call_user_func([$matcher, $match])){ $this->callTarget(); break; }
+                foreach (call_user_func([$matcher, 'getMatcher']) as $match) {

+                                    if (call_user_func([$matcher, $match])){ $this->callTarget();

+                }

+                break; }
             }
         }
         http_response_code($this->route->getResponse('code'));