Completed
Push — master ( 5d4ad8...2aa3b5 )
by Sinnarasa
04:30
created
src/Routing/Response.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -166,8 +166,9 @@
 block discarded – undo
166 166
      */
167 167
     public function sendHeaders()
168 168
     {
169
-        foreach($this->headers as $key => $content)
170
-            header($key.' : '.$content);
169
+        foreach($this->headers as $key => $content) {
170
+                    header($key.' : '.$content);
171
+        }
171 172
         http_response_code($this->getStatusCode());
172 173
         return $this;
173 174
     }
Please login to merge, or discard this patch.
src/Routing/Route.php 1 patch
Braces   +18 added lines, -8 removed lines patch added patch discarded remove patch
@@ -52,10 +52,18 @@  discard block
 block discarded – undo
52 52
      */
53 53
     public function set($args = [])
54 54
     {
55
-        if (isset($args['name'])) $this->name = $args['name'];
56
-        if (isset($args['callback'])) $this->callback = $args['callback'];
57
-        if (isset($args['target'])) $this->target = $args['target'];
58
-        if (isset($args['detail'])) $this->detail = $args['detail'];
55
+        if (isset($args['name'])) {
56
+            $this->name = $args['name'];
57
+        }
58
+        if (isset($args['callback'])) {
59
+            $this->callback = $args['callback'];
60
+        }
61
+        if (isset($args['target'])) {
62
+            $this->target = $args['target'];
63
+        }
64
+        if (isset($args['detail'])) {
65
+            $this->detail = $args['detail'];
66
+        }
59 67
     }
60 68
 
61 69
     /**
@@ -145,8 +153,9 @@  discard block
 block discarded – undo
145 153
      */
146 154
     public function getTarget($key = null)
147 155
     {
148
-        if (!is_null($key))
149
-            return isset($this->target[$key]) ? $this->target[$key] : '';
156
+        if (!is_null($key)) {
157
+                    return isset($this->target[$key]) ? $this->target[$key] : '';
158
+        }
150 159
         return empty($this->target) ? '' : $this->target;
151 160
     }
152 161
 
@@ -165,8 +174,9 @@  discard block
 block discarded – undo
165 174
      */
166 175
     public function hasTarget($key = null)
167 176
     {
168
-        if (!is_null($key))
169
-            return isset($this->target[$key]) ? true : false;
177
+        if (!is_null($key)) {
178
+                    return isset($this->target[$key]) ? true : false;
179
+        }
170 180
         return empty($this->target) ? false : true;
171 181
     }
172 182
 
Please login to merge, or discard this patch.
src/Routing/Dispatcher/ClosureDispatcher.php 1 patch
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -42,8 +42,11 @@
 block discarded – undo
42 42
             $this->response->setHeaders(['Content-Type' => 'text/html']);
43 43
         }
44 44
         $params = ($this->route->getParameters() == '') ? [] : $this->route->getParameters();
45
-        if(is_array($content = call_user_func_array($this->route->getTarget('closure'), $params))) $this->route->addTarget('data',$content);
46
-        elseif(!is_null($content)) $this->response->setContent($content);
45
+        if(is_array($content = call_user_func_array($this->route->getTarget('closure'), $params))) {
46
+            $this->route->addTarget('data',$content);
47
+        } elseif(!is_null($content)) {
48
+            $this->response->setContent($content);
49
+        }
47 50
     }
48 51
 
49 52
 }
Please login to merge, or discard this patch.
src/Routing/Router.php 1 patch
Braces   +22 added lines, -12 removed lines patch added patch discarded remove patch
@@ -80,8 +80,9 @@  discard block
 block discarded – undo
80 80
      * @param object|array $matcher
81 81
      */
82 82
     public function setMatcher($matcher){
83
-        if(is_object($matcher))
84
-            $matcher = [$matcher];
83
+        if(is_object($matcher)) {
84
+                    $matcher = [$matcher];
85
+        }
85 86
         $this->matcher = $matcher;
86 87
     }
87 88
 
@@ -98,7 +99,9 @@  discard block
 block discarded – undo
98 99
     public function run()
99 100
     {
100 101
         $this->setUrl();
101
-        if ($this->config['generateRoutesPath']) $this->collection->generateRoutesPath();
102
+        if ($this->config['generateRoutesPath']) {
103
+            $this->collection->generateRoutesPath();
104
+        }
102 105
         if ($this->match()) {
103 106
             $this->handle();
104 107
             $this->callTarget();
@@ -111,8 +114,9 @@  discard block
 block discarded – undo
111 114
      */
112 115
     public function setUrl($url = null)
113 116
     {
114
-        if (is_null($url))
115
-            $url = (isset($_GET['url'])) ? $_GET['url'] : substr(str_replace(str_replace('/index.php', '', $_SERVER['SCRIPT_NAME']), '', $_SERVER['REQUEST_URI']), 1);
117
+        if (is_null($url)) {
118
+                    $url = (isset($_GET['url'])) ? $_GET['url'] : substr(str_replace(str_replace('/index.php', '', $_SERVER['SCRIPT_NAME']), '', $_SERVER['REQUEST_URI']), 1);
119
+        }
116 120
         $this->route->setUrl('/' . trim(explode('?', $url)[0], '/'));
117 121
     }
118 122
 
@@ -122,8 +126,9 @@  discard block
 block discarded – undo
122 126
     public function match()
123 127
     {
124 128
         foreach ($this->matcher as $key => $matcher) {
125
-            if (call_user_func([$this->matcher[$key], 'match']))
126
-                return true;
129
+            if (call_user_func([$this->matcher[$key], 'match'])) {
130
+                            return true;
131
+            }
127 132
         }
128 133
         return false;
129 134
     }
@@ -170,11 +175,16 @@  discard block
 block discarded – undo
170 175
         if (isset($this->route->getDetail()['response_templates']) && isset($this->route->getDetail()['response_templates'][$code = $this->response->getStatusCode()])) {
171 176
             $this->route->setCallback($this->route->getDetail()['response_templates'][$code]);
172 177
             $matcher = null;
173
-            foreach($this->matcher as $instance) if($instance instanceof ArrayMatcher) $matcher = $instance;
174
-            if(is_null($matcher))$matcher = new ArrayMatcher($this);
175
-            foreach (call_user_func([$matcher, 'getResolver']) as $match)
176
-                if (is_array($target = call_user_func_array([$matcher, $match], [$this->route->getCallback()]))){
177
-                    call_user_func_array([$matcher, 'setTarget'],[$target]);
178
+            foreach($this->matcher as $instance) {
179
+                if($instance instanceof ArrayMatcher) $matcher = $instance;
180
+            }
181
+            if(is_null($matcher)) {
182
+                $matcher = new ArrayMatcher($this);
183
+            }
184
+            foreach (call_user_func([$matcher, 'getResolver']) as $match) {
185
+                            if (is_array($target = call_user_func_array([$matcher, $match], [$this->route->getCallback()]))){
186
+                    call_user_func_array([$matcher, 'setTarget'],[$target]);
187
+            }
178 188
                     $this->callTarget();
179 189
                     break;
180 190
                 }
Please login to merge, or discard this patch.
src/Routing/Dispatcher/TemplateDispatcher.php 1 patch
Braces   +12 added lines, -7 removed lines patch added patch discarded remove patch
@@ -47,14 +47,19 @@
 block discarded – undo
47 47
      */
48 48
     public function call()
49 49
     {
50
-        if ($this->response->getStatusCode() == 202)
51
-            $this->setContentType($this->route->getTarget('extension'));
52
-        if (isset($this->route->getTarget('callback')[$this->route->getTarget('extension')]))
53
-            $this->response->setContent(call_user_func_array($this->route->getTarget('callback')[$this->route->getTarget('extension')], [$this->route]));
54
-        else {
50
+        if ($this->response->getStatusCode() == 202) {
51
+                    $this->setContentType($this->route->getTarget('extension'));
52
+        }
53
+        if (isset($this->route->getTarget('callback')[$this->route->getTarget('extension')])) {
54
+                    $this->response->setContent(call_user_func_array($this->route->getTarget('callback')[$this->route->getTarget('extension')], [$this->route]));
55
+        } else {
55 56
             ob_start();
56
-            if(isset($this->route->getTarget()['data']))extract($this->route->getTarget('data'));
57
-            if(isset($this->route->getParams()['data']))extract($this->route->getParams()['data']);
57
+            if(isset($this->route->getTarget()['data'])) {
58
+                extract($this->route->getTarget('data'));
59
+            }
60
+            if(isset($this->route->getParams()['data'])) {
61
+                extract($this->route->getParams()['data']);
62
+            }
58 63
             require($this->route->getTarget('template'));
59 64
             $this->response->setContent(ob_get_clean());
60 65
         }
Please login to merge, or discard this patch.
src/Routing/RouteCollection.php 1 patch
Braces   +63 added lines, -33 removed lines patch added patch discarded remove patch
@@ -33,7 +33,9 @@  discard block
 block discarded – undo
33 33
      */
34 34
     public function __construct($routes = null, $options = [])
35 35
     {
36
-        if (!is_null($routes) || !empty($options)) $this->addRoutes($routes, $options);
36
+        if (!is_null($routes) || !empty($options)) {
37
+            $this->addRoutes($routes, $options);
38
+        }
37 39
     }
38 40
 
39 41
     /**
@@ -43,10 +45,16 @@  discard block
 block discarded – undo
43 45
     public function addRoutes($routes = null, $options = [])
44 46
     {
45 47
         if (!is_null($routes) && !is_array($routes)) {
46
-            if (strpos($routes, '.php') === false) $routes = trim($routes, '/') . '/';
47
-            if (is_file($routes . '/routes.php') && is_array($routesFile = include $routes . '/routes.php')) $routes = $routesFile;
48
-            elseif (is_file($routes) && is_array($routesFile = include $routes)) $routes = $routesFile;
49
-            else throw new \InvalidArgumentException('Argument for "' . get_called_class() . '" constructor is not recognized. Expected argument array or file containing array but "' . $routes . '" given');
48
+            if (strpos($routes, '.php') === false) {
49
+                $routes = trim($routes, '/') . '/';
50
+            }
51
+            if (is_file($routes . '/routes.php') && is_array($routesFile = include $routes . '/routes.php')) {
52
+                $routes = $routesFile;
53
+            } elseif (is_file($routes) && is_array($routesFile = include $routes)) {
54
+                $routes = $routesFile;
55
+            } else {
56
+                throw new \InvalidArgumentException('Argument for "' . get_called_class() . '" constructor is not recognized. Expected argument array or file containing array but "' . $routes . '" given');
57
+            }
50 58
         }
51 59
         $this->routes['routes_' . $this->countRoutes] = is_array($routes) ? $routes : [];
52 60
         $this->setRoutes($options, $this->countRoutes);
@@ -59,8 +67,9 @@  discard block
 block discarded – undo
59 67
      */
60 68
     public function getRoutes($key = null)
61 69
     {
62
-        if (!is_null($key))
63
-            return isset($this->routes[$key]) ? $this->routes[$key] : '';
70
+        if (!is_null($key)) {
71
+                    return isset($this->routes[$key]) ? $this->routes[$key] : '';
72
+        }
64 73
         return $this->routes;
65 74
     }
66 75
 
@@ -71,12 +80,18 @@  discard block
 block discarded – undo
71 80
     {
72 81
         if (is_array($args)) {
73 82
             $nbrArgs = count($args);
74
-            for ($i = 0; $i < $nbrArgs; ++$i)
75
-                $this->routes['prefix_' . $i] = '/' . trim($args[$i], '/');
76
-        } elseif (is_string($args))
77
-            for ($i = 0; $i < $this->countRoutes; ++$i)
78
-                $this->routes['prefix_' . $i] = '/' . trim($args, '/');
79
-        if ($this->countRoutes == 0) $this->countRoutes++;
83
+            for ($i = 0; $i < $nbrArgs; ++$i) {
84
+                            $this->routes['prefix_' . $i] = '/' . trim($args[$i], '/');
85
+            }
86
+        } elseif (is_string($args)) {
87
+                    for ($i = 0;
88
+        }
89
+        $i < $this->countRoutes; ++$i) {
90
+                            $this->routes['prefix_' . $i] = '/' . trim($args, '/');
91
+            }
92
+        if ($this->countRoutes == 0) {
93
+            $this->countRoutes++;
94
+        }
80 95
     }
81 96
 
82 97
     /**
@@ -88,10 +103,14 @@  discard block
 block discarded – undo
88 103
         for ($i = 0; $i < $nbrArgs; ++$i) {
89 104
             if (is_array($args[$i])) {
90 105
                 $this->setRoutes($args[$i], $i);
91
-                if (!isset($this->routes['routes_' . $i])) $this->routes['routes_' . $i] = [];
106
+                if (!isset($this->routes['routes_' . $i])) {
107
+                    $this->routes['routes_' . $i] = [];
108
+                }
92 109
             }
93 110
         }
94
-        if ($this->countRoutes == 0) $this->countRoutes++;
111
+        if ($this->countRoutes == 0) {
112
+            $this->countRoutes++;
113
+        }
95 114
     }
96 115
 
97 116
     /**
@@ -113,12 +132,16 @@  discard block
 block discarded – undo
113 132
      */
114 133
     public function setMiddleware($middleware)
115 134
     {
116
-        if (is_string($middleware)) $middleware = rtrim($middleware, '/');
117
-        if (is_array($middleware))
118
-            $this->middleware = $middleware;
119
-        elseif (is_file($middleware) && is_array($mid = include $middleware))
120
-            $this->middleware = $mid;
121
-        else throw new \InvalidArgumentException('Accepted argument for setMiddleware are array and array file');
135
+        if (is_string($middleware)) {
136
+            $middleware = rtrim($middleware, '/');
137
+        }
138
+        if (is_array($middleware)) {
139
+                    $this->middleware = $middleware;
140
+        } elseif (is_file($middleware) && is_array($mid = include $middleware)) {
141
+                    $this->middleware = $mid;
142
+        } else {
143
+            throw new \InvalidArgumentException('Accepted argument for setMiddleware are array and array file');
144
+        }
122 145
     }
123 146
 
124 147
     /**
@@ -127,19 +150,21 @@  discard block
 block discarded – undo
127 150
     public function generateRoutesPath()
128 151
     {
129 152
         $root = (isset($_SERVER['REQUEST_SCHEME'])?$_SERVER['REQUEST_SCHEME']:'http') . '://' . ($domain = (isset($_SERVER['HTTP_HOST']) ? $_SERVER['HTTP_HOST'] : $_SERVER['SERVER_NAME'])) . str_replace('/index.php', '', $_SERVER['SCRIPT_NAME']);
130
-        if (strpos($domain, ($new_domain = $this->getDomain($root))) !== false)
131
-            $root = str_replace($domain, $new_domain, $root);
153
+        if (strpos($domain, ($new_domain = $this->getDomain($root))) !== false) {
154
+                    $root = str_replace($domain, $new_domain, $root);
155
+        }
132 156
         $count = 0;
133 157
         for ($i = 0; $i < $this->countRoutes; ++$i) {
134 158
             $prefix = (isset($this->routes['prefix_' . $i])) ? $this->routes['prefix_' . $i] : '';
135
-            if (isset($this->routes['routes_' . $i]))
136
-                foreach ($this->routes['routes_' . $i] as $route => $dependencies) {
159
+            if (isset($this->routes['routes_' . $i])) {
160
+                            foreach ($this->routes['routes_' . $i] as $route => $dependencies) {
137 161
                     if (is_array($dependencies) && isset($dependencies['use']))
138
-                        $use = (is_callable($dependencies['use'])) ? 'closure-' . $count : trim($dependencies['use'], '/');
139
-                    elseif (!is_array($dependencies))
140
-                        $use = (is_callable($dependencies)) ? 'closure-' . $count : trim($dependencies, '/');
141
-                    else
142
-                        $use = $route;
162
+                        $use = (is_callable($dependencies['use'])) ? 'closure-' . $count : trim($dependencies['use'], '/');
163
+            } elseif (!is_array($dependencies)) {
164
+                                            $use = (is_callable($dependencies)) ? 'closure-' . $count : trim($dependencies, '/');
165
+                    } else {
166
+                                            $use = $route;
167
+                    }
143 168
                     if (isset($route[0]) && $route[0] == '/') {
144 169
                         (!is_callable($dependencies) && isset($dependencies['name'])) ? $this->routesByName[$use . '#' . $dependencies['name']] = $root . $prefix . $route : $this->routesByName[$use] = $root . $prefix . $route;
145 170
                     } else {
@@ -176,9 +201,14 @@  discard block
 block discarded – undo
176 201
         foreach ($this->routesByName as $key => $route) {
177 202
             $param = explode('#', $key);
178 203
             $route = str_replace('{subdomain}', $subdomain, $route);
179
-            foreach ($params as $key2 => $value) $route = str_replace(':' . $key2, $value, $route);
180
-            if ($param[0] == trim($name, '/')) return $route;
181
-            else if (isset($param[1]) && $param[1] == $name) return $route;
204
+            foreach ($params as $key2 => $value) {
205
+                $route = str_replace(':' . $key2, $value, $route);
206
+            }
207
+            if ($param[0] == trim($name, '/')) {
208
+                return $route;
209
+            } else if (isset($param[1]) && $param[1] == $name) {
210
+                return $route;
211
+            }
182 212
         }
183 213
         return null;
184 214
     }
Please login to merge, or discard this patch.
src/Routing/Matcher/UriMatcher.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -90,8 +90,9 @@
 block discarded – undo
90 90
         foreach($this->resolver as $resolver){
91 91
             if(is_array($target = call_user_func([$this,$resolver]))) {
92 92
                 $this->setTarget($target);
93
-                if($this->router->middleware->globalMiddleware() === false || $this->router->middleware->blockMiddleware() === false || $this->router->middleware->classMiddleware() === false || $this->router->middleware->routeMiddleware() === false)
94
-                    return null;
93
+                if($this->router->middleware->globalMiddleware() === false || $this->router->middleware->blockMiddleware() === false || $this->router->middleware->classMiddleware() === false || $this->router->middleware->routeMiddleware() === false) {
94
+                                    return null;
95
+                }
95 96
                 $this->router->response->setStatusCode(202);
96 97
                 return true;
97 98
             }
Please login to merge, or discard this patch.
src/Routing/Dispatcher/ControllerDispatcher.php 1 patch
Braces   +14 added lines, -8 removed lines patch added patch discarded remove patch
@@ -50,12 +50,17 @@  discard block
 block discarded – undo
50 50
         }
51 51
         if($count == count($this->route->getParameters())) {
52 52
             $dependencies = array_merge($dependencies, ($this->route->getParameters() == '') ? [] : $this->route->getParameters());
53
-            if ($this->response->getStatusCode() == 202)
54
-                $this->response->setStatusCode(200);
55
-            if (is_array($content = $reflectionMethod->invokeArgs($this->getController(), $dependencies))) $this->route->addTarget('data', $content);
56
-            elseif (!is_null($content)) $this->response->setContent($content);
57
-        }else
58
-            $this->response->setStatusCode(404);
53
+            if ($this->response->getStatusCode() == 202) {
54
+                            $this->response->setStatusCode(200);
55
+            }
56
+            if (is_array($content = $reflectionMethod->invokeArgs($this->getController(), $dependencies))) {
57
+                $this->route->addTarget('data', $content);
58
+            } elseif (!is_null($content)) {
59
+                $this->response->setContent($content);
60
+            }
61
+        } else {
62
+                    $this->response->setStatusCode(404);
63
+        }
59 64
     }
60 65
 
61 66
 
@@ -66,8 +71,9 @@  discard block
 block discarded – undo
66 71
     private function getController()
67 72
     {
68 73
         $reflector = new ReflectionClass($this->route->getTarget('controller'));
69
-        if (!$reflector->isInstantiable())
70
-            throw new \Exception('Target [' . $this->route->getTarget('controller') . '] is not instantiable.');
74
+        if (!$reflector->isInstantiable()) {
75
+                    throw new \Exception('Target [' . $this->route->getTarget('controller') . '] is not instantiable.');
76
+        }
71 77
         $constructor = $reflector->getConstructor();
72 78
         if (is_null($constructor)) {
73 79
             $class = $this->route->getTarget('controller');
Please login to merge, or discard this patch.
src/Routing/Matcher/ArrayMatcher.php 1 patch
Braces   +46 added lines, -26 removed lines patch added patch discarded remove patch
@@ -102,7 +102,9 @@  discard block
 block discarded – undo
102 102
                     $this->request['route'] = preg_replace_callback('#:([\w]+)#', [$this, 'paramMatch'], '/' . trim(trim($this->request['prefix'], '/') . '/' . trim($route, '/'), '/'));
103 103
                     if ($this->routeMatch('#^' . $this->request['route'] . '$#')) {
104 104
                         $this->setCallback();
105
-                        if(!is_null($response = $this->generateTarget())) return $response;
105
+                        if(!is_null($response = $this->generateTarget())) {
106
+                            return $response;
107
+                        }
106 108
                     }
107 109
                 }
108 110
             }
@@ -117,10 +119,12 @@  discard block
 block discarded – undo
117 119
     private function checkSubdomain($route){
118 120
         $url = (isset($_SERVER['REQUEST_SCHEME']) ? $_SERVER['REQUEST_SCHEME'] : 'http') . '://' . ($host = (isset($_SERVER['HTTP_HOST']) ? $_SERVER['HTTP_HOST'] : $_SERVER['SERVER_NAME']));
119 121
         $domain = $this->router->collection->getDomain($url);
120
-        if(!empty($this->request['subdomain']) && $route[0] == '/') $route = trim($this->request['subdomain'],'.').'.'.$domain.$route;
122
+        if(!empty($this->request['subdomain']) && $route[0] == '/') {
123
+            $route = trim($this->request['subdomain'],'.').'.'.$domain.$route;
124
+        }
121 125
         if($route[0] == '/'){
122 126
             return ($host != $domain) ? false : true;
123
-        }elseif($route[0] != '/' && $host != $domain){
127
+        } elseif($route[0] != '/' && $host != $domain){
124 128
             $route = substr($route, 0, strpos($route, "/"));
125 129
             $route = str_replace('{host}', $domain, $route);
126 130
             $route = preg_replace_callback('#{subdomain}#', [$this, 'subdomainMatch'], $route);
@@ -165,8 +169,9 @@  discard block
 block discarded – undo
165 169
     {
166 170
         if (substr($this->request['route'], -1) == '*') {
167 171
             $pos = strpos($this->request['route'], '*');
168
-            if (substr($this->router->route->getUrl(), 0, $pos) == substr($this->request['route'], 0, $pos) && isset($this->request['params']))
169
-                return true;
172
+            if (substr($this->router->route->getUrl(), 0, $pos) == substr($this->request['route'], 0, $pos) && isset($this->request['params'])) {
173
+                            return true;
174
+            }
170 175
         }
171 176
         if (preg_match($regex, $this->router->route->getUrl(), $this->request['parameters'])) {
172 177
             array_shift($this->request['parameters']);
@@ -184,8 +189,9 @@  discard block
 block discarded – undo
184 189
             foreach ($this->resolver as $resolver) {
185 190
                 if (is_array($target = call_user_func_array([$this, $resolver], [$this->router->route->getCallback()]))) {
186 191
                     $this->setTarget($target);
187
-                    if ($this->router->middleware->globalMiddleware() === false || $this->router->middleware->blockMiddleware() === false || $this->router->middleware->classMiddleware() === false || $this->router->middleware->routeMiddleware() === false)
188
-                        return null;
192
+                    if ($this->router->middleware->globalMiddleware() === false || $this->router->middleware->blockMiddleware() === false || $this->router->middleware->classMiddleware() === false || $this->router->middleware->routeMiddleware() === false) {
193
+                                            return null;
194
+                    }
189 195
                     $this->router->response->setStatusCode(202);
190 196
                     return true;
191 197
                 }
@@ -212,14 +218,18 @@  discard block
 block discarded – undo
212 218
      */
213 219
     private function setCallback(){
214 220
         if (isset($this->request['params'])) {
215
-            if(is_callable($this->request['params']))
216
-                $this->router->route->setCallback($this->request['params']);
217
-            else {
221
+            if(is_callable($this->request['params'])) {
222
+                            $this->router->route->setCallback($this->request['params']);
223
+            } else {
218 224
                 (is_array($this->request['params']) && isset($this->request['params']['use']))
219 225
                     ? $this->router->route->setCallback($this->request['params']['use'])
220 226
                     : $this->router->route->setCallback($this->request['params']);
221
-                if (isset($this->request['params']['name'])) $this->router->route->setName($this->request['params']['name']);
222
-                if (isset($this->request['params']['method'])) $this->request['params']['method'] = is_array($this->request['params']['method']) ? $this->request['params']['method'] : [$this->request['params']['method']];
227
+                if (isset($this->request['params']['name'])) {
228
+                    $this->router->route->setName($this->request['params']['name']);
229
+                }
230
+                if (isset($this->request['params']['method'])) {
231
+                    $this->request['params']['method'] = is_array($this->request['params']['method']) ? $this->request['params']['method'] : [$this->request['params']['method']];
232
+                }
223 233
             }
224 234
         }
225 235
     }
@@ -229,9 +239,12 @@  discard block
 block discarded – undo
229 239
      */
230 240
     public function validMethod()
231 241
     {
232
-        if(is_callable($this->request['params']))return true;
233
-        if (!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest')
234
-            return (isset($this->request['params']['ajax']) && $this->request['params']['ajax'] === true) ? true : false;
242
+        if(is_callable($this->request['params'])) {
243
+            return true;
244
+        }
245
+        if (!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest') {
246
+                    return (isset($this->request['params']['ajax']) && $this->request['params']['ajax'] === true) ? true : false;
247
+        }
235 248
         $method = (isset($this->request['params']['method'])) ? $this->request['params']['method'] : ['GET'];
236 249
         return (in_array($this->router->route->getMethod(), $method)) ? true : false;
237 250
     }
@@ -295,20 +308,25 @@  discard block
 block discarded – undo
295 308
     {
296 309
         if (is_string($callback) && strpos($callback, '@') !== false) {
297 310
             $routes = explode('@', $callback);
298
-            if (!isset($routes[1])) $routes[1] = 'index';
311
+            if (!isset($routes[1])) {
312
+                $routes[1] = 'index';
313
+            }
299 314
             if ($routes[1] == '{method}'){
300 315
                 $this->request['parameters'] = explode('/',str_replace(str_replace('*','',$this->request['route']),'',$this->router->route->getUrl()));
301 316
                 $routes[1] = $this->request['parameters'][0];
302 317
                 array_shift($this->request['parameters']);
303
-                if(preg_match('/[A-Z]/', $routes[1])) return false;
318
+                if(preg_match('/[A-Z]/', $routes[1])) {
319
+                    return false;
320
+                }
304 321
                 $routes[1] = lcfirst(str_replace('-','',ucwords($routes[1],'-')));
305 322
             }
306 323
             $index = isset($this->request['collection_index']) ? $this->request['collection_index'] : 0;
307 324
             $class = (class_exists($routes[0]))
308 325
                 ? $routes[0]
309 326
                 : $this->router->collection->getRoutes()['ctrl_namespace_' . $index] . $routes[0];
310
-            if (!class_exists($class))
311
-                throw new \Exception('Class "' . $class . '." is not found');
327
+            if (!class_exists($class)) {
328
+                            throw new \Exception('Class "' . $class . '." is not found');
329
+            }
312 330
             if (method_exists($class, $routes[1])) {
313 331
                 return [
314 332
                     'dispatcher' => $this->dispatcher['isController'],
@@ -317,8 +335,9 @@  discard block
 block discarded – undo
317 335
                     'action' => $routes[1]
318 336
                 ];
319 337
             }
320
-            if (!strpos($callback, '{method}') !== false)
321
-                throw new \Exception('The required method "' . $routes[1] . '" is not found in "' . $class . '"');
338
+            if (!strpos($callback, '{method}') !== false) {
339
+                            throw new \Exception('The required method "' . $routes[1] . '" is not found in "' . $class . '"');
340
+            }
322 341
         }
323 342
         return false;
324 343
     }
@@ -336,9 +355,9 @@  discard block
 block discarded – undo
336 355
             $index = isset($this->request['collection_index']) ? $this->request['collection_index'] : 0;
337 356
             $viewDir = $this->router->collection->getRoutes('view_dir_' . $index);
338 357
             $target = null;
339
-            if (in_array('.' . $extension, $this->router->getConfig()['templateExtension']) && (is_file($fullPath = $viewDir . $path) || is_file($fullPath = $path)))
340
-                $target = $fullPath;
341
-            else {
358
+            if (in_array('.' . $extension, $this->router->getConfig()['templateExtension']) && (is_file($fullPath = $viewDir . $path) || is_file($fullPath = $path))) {
359
+                            $target = $fullPath;
360
+            } else {
342 361
                 foreach ($this->router->getConfig()['templateExtension'] as $ext) {
343 362
                     if (is_file($fullPath = $viewDir . $path . $ext) || is_file($fullPath = $path . $ext)) {
344 363
                         $target = $fullPath;
@@ -347,8 +366,9 @@  discard block
 block discarded – undo
347 366
                     }
348 367
                 }
349 368
             }
350
-            if(is_null($target))
351
-                throw new \Exception('Template file "' . $path . '" is not found in "' . $viewDir . '"');
369
+            if(is_null($target)) {
370
+                            throw new \Exception('Template file "' . $path . '" is not found in "' . $viewDir . '"');
371
+            }
352 372
             return [
353 373
                 'dispatcher' => $this->dispatcher['isTemplate'],
354 374
                 'template'   => $target,
Please login to merge, or discard this patch.