jetfirephp / routing

src/Routing/Router.php

Braces +22 added lines, -12 removed lines

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

+                    $matcher = [$matcher];

+        }

         $this->matcher = $matcher;
     }
 
@@ -98,7 +99,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();
@@ -111,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], '/'));
     }
 
@@ -122,8 +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;
     }
@@ -170,11 +175,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;
                 }