Completed
Push — master ( 855079...1a7984 )
by Sinnarasa
02:36
created
src/Routing/Response.php 2 patches
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -7,7 +7,7 @@  discard block
 block discarded – undo
7 7
  * Class Response
8 8
  * @package JetFire\Routing
9 9
  */
10
-class Response implements ResponseInterface{
10
+class Response implements ResponseInterface {
11 11
 
12 12
     /**
13 13
      * @var array
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
     public static $statusTexts = array(
54 54
         100 => 'Continue',
55 55
         101 => 'Switching Protocols',
56
-        102 => 'Processing',            // RFC2518
56
+        102 => 'Processing', // RFC2518
57 57
         200 => 'OK',
58 58
         201 => 'Created',
59 59
         202 => 'Accepted',
@@ -61,9 +61,9 @@  discard block
 block discarded – undo
61 61
         204 => 'No Content',
62 62
         205 => 'Reset Content',
63 63
         206 => 'Partial Content',
64
-        207 => 'Multi-Status',          // RFC4918
65
-        208 => 'Already Reported',      // RFC5842
66
-        226 => 'IM Used',               // RFC3229
64
+        207 => 'Multi-Status', // RFC4918
65
+        208 => 'Already Reported', // RFC5842
66
+        226 => 'IM Used', // RFC3229
67 67
         300 => 'Multiple Choices',
68 68
         301 => 'Moved Permanently',
69 69
         302 => 'Found',
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
         304 => 'Not Modified',
72 72
         305 => 'Use Proxy',
73 73
         307 => 'Temporary Redirect',
74
-        308 => 'Permanent Redirect',    // RFC7238
74
+        308 => 'Permanent Redirect', // RFC7238
75 75
         400 => 'Bad Request',
76 76
         401 => 'Unauthorized',
77 77
         402 => 'Payment Required',
@@ -90,26 +90,26 @@  discard block
 block discarded – undo
90 90
         415 => 'Unsupported Media Type',
91 91
         416 => 'Range Not Satisfiable',
92 92
         417 => 'Expectation Failed',
93
-        418 => 'I\'m a teapot',                                               // RFC2324
94
-        422 => 'Unprocessable Entity',                                        // RFC4918
95
-        423 => 'Locked',                                                      // RFC4918
96
-        424 => 'Failed Dependency',                                           // RFC4918
97
-        425 => 'Reserved for WebDAV advanced collections expired proposal',   // RFC2817
98
-        426 => 'Upgrade Required',                                            // RFC2817
99
-        428 => 'Precondition Required',                                       // RFC6585
100
-        429 => 'Too Many Requests',                                           // RFC6585
101
-        431 => 'Request Header Fields Too Large',                             // RFC6585
93
+        418 => 'I\'m a teapot', // RFC2324
94
+        422 => 'Unprocessable Entity', // RFC4918
95
+        423 => 'Locked', // RFC4918
96
+        424 => 'Failed Dependency', // RFC4918
97
+        425 => 'Reserved for WebDAV advanced collections expired proposal', // RFC2817
98
+        426 => 'Upgrade Required', // RFC2817
99
+        428 => 'Precondition Required', // RFC6585
100
+        429 => 'Too Many Requests', // RFC6585
101
+        431 => 'Request Header Fields Too Large', // RFC6585
102 102
         500 => 'Internal Server Error',
103 103
         501 => 'Not Implemented',
104 104
         502 => 'Bad Gateway',
105 105
         503 => 'Service Unavailable',
106 106
         504 => 'Gateway Timeout',
107 107
         505 => 'HTTP Version Not Supported',
108
-        506 => 'Variant Also Negotiates (Experimental)',                      // RFC2295
109
-        507 => 'Insufficient Storage',                                        // RFC4918
110
-        508 => 'Loop Detected',                                               // RFC5842
111
-        510 => 'Not Extended',                                                // RFC2774
112
-        511 => 'Network Authentication Required',                             // RFC6585
108
+        506 => 'Variant Also Negotiates (Experimental)', // RFC2295
109
+        507 => 'Insufficient Storage', // RFC4918
110
+        508 => 'Loop Detected', // RFC5842
111
+        510 => 'Not Extended', // RFC2774
112
+        511 => 'Network Authentication Required', // RFC6585
113 113
     );
114 114
 
115 115
     /**
@@ -144,8 +144,8 @@  discard block
 block discarded – undo
144 144
     public function __toString()
145 145
     {
146 146
         return
147
-            sprintf('HTTP/%s %s %s', $this->version, $this->statusCode, $this->statusText)."\r\n".
148
-            $this->headers."\r\n".
147
+            sprintf('HTTP/%s %s %s', $this->version, $this->statusCode, $this->statusText) . "\r\n" .
148
+            $this->headers . "\r\n" .
149 149
             $this->getContent();
150 150
     }
151 151
 
@@ -166,8 +166,8 @@  discard block
 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
         http_response_code($this->getStatusCode());
172 172
         return $this;
173 173
     }
Please login to merge, or discard this 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/Dispatcher/TemplateDispatcher.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
     /**
37 37
      * @param Route $route
38 38
      */
39
-    public function __construct(Route $route,ResponseInterface $response)
39
+    public function __construct(Route $route, ResponseInterface $response)
40 40
     {
41 41
         $this->route = $route;
42 42
         $this->response = $response;
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
     /**
59 59
      * @param $extension
60 60
      */
61
-    public function setContentType($extension){
61
+    public function setContentType($extension) {
62 62
         $this->response->setStatusCode(200);
63 63
         isset($this->types[$extension])
64 64
             ? $this->response->setHeaders(['Content-Type' => $this->types[$extension]])
Please login to merge, or discard this patch.
Braces   +6 added lines, -5 removed lines patch added patch discarded remove patch
@@ -47,11 +47,12 @@
 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 57
             require($this->route->getTarget('template'));
57 58
             $this->response->setContent(ob_get_clean());
Please login to merge, or discard this patch.
src/Routing/Dispatcher/DispatcherInterface.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@
 block discarded – undo
15 15
     /**
16 16
      * @param Route $route
17 17
      */
18
-    public function __construct(Route $route,ResponseInterface $response);
18
+    public function __construct(Route $route, ResponseInterface $response);
19 19
 
20 20
     /**
21 21
      * @return mixed
Please login to merge, or discard this patch.
src/Routing/Dispatcher/ControllerDispatcher.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
     /**
28 28
      * @param Route $route
29 29
      */
30
-    public function __construct(Route $route,ResponseInterface $response)
30
+    public function __construct(Route $route, ResponseInterface $response)
31 31
     {
32 32
         $this->route = $route;
33 33
         $this->response = $response;
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
         foreach ($reflectionMethod->getParameters() as $arg) {
46 46
             if (!is_null($arg->getClass())) {
47 47
                 $class = $arg->getClass()->name;
48
-                array_unshift($dependencies, call_user_func_array($this->route->getTarget('di'),[$class]));
48
+                array_unshift($dependencies, call_user_func_array($this->route->getTarget('di'), [$class]));
49 49
             }
50 50
         }
51 51
         if ($this->response->getStatusCode() == 202)
@@ -66,13 +66,13 @@  discard block
 block discarded – undo
66 66
         $constructor = $reflector->getConstructor();
67 67
         if (is_null($constructor)) {
68 68
             $class = $this->route->getTarget('controller');
69
-            return call_user_func_array($this->route->getTarget('di'),[$class]);
69
+            return call_user_func_array($this->route->getTarget('di'), [$class]);
70 70
         }
71 71
         $dependencies = $constructor->getParameters();
72 72
         $arguments = [];
73 73
         foreach ($dependencies as $dep) {
74 74
             $class = $dep->getClass()->name;
75
-            array_push($arguments, call_user_func_array($this->route->getTarget('di'),[$class]));
75
+            array_push($arguments, call_user_func_array($this->route->getTarget('di'), [$class]));
76 76
         }
77 77
         return $reflector->newInstanceArgs($arguments);
78 78
     }
Please login to merge, or discard this patch.
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -48,8 +48,9 @@  discard block
 block discarded – undo
48 48
                 array_unshift($dependencies, call_user_func_array($this->route->getTarget('di'),[$class]));
49 49
             }
50 50
         }
51
-        if ($this->response->getStatusCode() == 202)
52
-            $this->response->setStatusCode(200);
51
+        if ($this->response->getStatusCode() == 202) {
52
+                    $this->response->setStatusCode(200);
53
+        }
53 54
         return $this->response->setContent($reflectionMethod->invokeArgs($this->getController(), $dependencies));
54 55
     }
55 56
 
@@ -61,8 +62,9 @@  discard block
 block discarded – undo
61 62
     private function getController()
62 63
     {
63 64
         $reflector = new ReflectionClass($this->route->getTarget('controller'));
64
-        if (!$reflector->isInstantiable())
65
-            throw new \Exception('Target [' . $this->route->getTarget('controller') . '] is not instantiable.');
65
+        if (!$reflector->isInstantiable()) {
66
+                    throw new \Exception('Target [' . $this->route->getTarget('controller') . '] is not instantiable.');
67
+        }
66 68
         $constructor = $reflector->getConstructor();
67 69
         if (is_null($constructor)) {
68 70
             $class = $this->route->getTarget('controller');
Please login to merge, or discard this patch.
src/Routing/Dispatcher/ClosureDispatcher.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
     /**
26 26
      * @param Route $route
27 27
      */
28
-    public function __construct(Route $route,ResponseInterface $response)
28
+    public function __construct(Route $route, ResponseInterface $response)
29 29
     {
30 30
         $this->route = $route;
31 31
         $this->response = $response;
Please login to merge, or discard this patch.
src/Routing/Route.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
      */
128 128
     public function setDetail($detail)
129 129
     {
130
-        $this->detail = array_merge($detail,$this->detail);
130
+        $this->detail = array_merge($detail, $this->detail);
131 131
     }
132 132
 
133 133
     /**
@@ -173,8 +173,8 @@  discard block
 block discarded – undo
173 173
     /**
174 174
      * @return array
175 175
      */
176
-    public function getData(){
177
-        return (isset($this->getDetail()['data']) && is_array($this->getDetail()['data']))?$this->getDetail()['data']:[];
176
+    public function getData() {
177
+        return (isset($this->getDetail()['data']) && is_array($this->getDetail()['data'])) ? $this->getDetail()['data'] : [];
178 178
     }
179 179
 
180 180
     /**
Please login to merge, or discard this 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/Router.php 2 patches
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -46,13 +46,13 @@  discard block
 block discarded – undo
46 46
      * @param ResponseInterface $response
47 47
      * @param Route $route
48 48
      */
49
-    public function __construct(RouteCollection $collection,ResponseInterface $response = null,Route $route = null)
49
+    public function __construct(RouteCollection $collection, ResponseInterface $response = null, Route $route = null)
50 50
     {
51 51
         $this->collection = $collection;
52
-        $this->response = is_null($response)? new Response() : $response;
52
+        $this->response = is_null($response) ? new Response() : $response;
53 53
         $this->response->setStatusCode(404);
54
-        $this->route = is_null($route)? new Route() : $route;
55
-        $this->config['di'] = function($class){
54
+        $this->route = is_null($route) ? new Route() : $route;
55
+        $this->config['di'] = function($class) {
56 56
             return new $class;
57 57
         };
58 58
     }
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
     /**
77 77
      * @param string $matcher
78 78
      */
79
-    public function addMatcher($matcher){
79
+    public function addMatcher($matcher) {
80 80
         $this->config['matcher'][] = $matcher;
81 81
     }
82 82
 
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
     public function callTarget()
124 124
     {
125 125
         $target = $this->route->getTarget('dispatcher');
126
-        if(!empty($target)) {
126
+        if (!empty($target)) {
127 127
             $this->dispatcher = new $target($this->route, $this->response);
128 128
             return call_user_func([$this->dispatcher, 'call']);
129 129
         }
@@ -157,12 +157,12 @@  discard block
 block discarded – undo
157 157
     {
158 158
         if (isset($this->route->getDetail()['response_templates']) && isset($this->route->getDetail()['response_templates'][$code = $this->response->getStatusCode()])) {
159 159
             $this->route->setCallback($this->route->getDetail()['response_templates'][$code]);
160
-            if(!isset($this->config['matcherInstance']))
160
+            if (!isset($this->config['matcherInstance']))
161 161
                 foreach ($this->config['matcher'] as $matcher)
162 162
                     $this->config['matcherInstance'][$matcher] = new $matcher($this);
163
-            foreach($this->config['matcherInstance'] as $instance) {
163
+            foreach ($this->config['matcherInstance'] as $instance) {
164 164
                 foreach (call_user_func([$instance, 'getMatcher']) as $match)
165
-                    if (call_user_func([$instance, $match])){ $this->callTarget(); break; }
165
+                    if (call_user_func([$instance, $match])) { $this->callTarget(); break; }
166 166
             }
167 167
             $this->response->setStatusCode($code);
168 168
         }
Please login to merge, or discard this patch.
Braces   +17 added lines, -10 removed lines patch added patch discarded remove patch
@@ -86,7 +86,9 @@  discard block
 block discarded – undo
86 86
     public function run()
87 87
     {
88 88
         $this->setUrl();
89
-        if ($this->config['generateRoutesPath']) $this->collection->generateRoutesPath();
89
+        if ($this->config['generateRoutesPath']) {
90
+            $this->collection->generateRoutesPath();
91
+        }
90 92
         if ($this->match()) {
91 93
             $this->handle();
92 94
             $this->callTarget();
@@ -99,8 +101,9 @@  discard block
 block discarded – undo
99 101
      */
100 102
     public function setUrl($url = null)
101 103
     {
102
-        if (is_null($url))
103
-            $url = (isset($_GET['url'])) ? $_GET['url'] : substr(str_replace(str_replace('/index.php', '', $_SERVER['SCRIPT_NAME']), '', $_SERVER['REQUEST_URI']), 1);
104
+        if (is_null($url)) {
105
+                    $url = (isset($_GET['url'])) ? $_GET['url'] : substr(str_replace(str_replace('/index.php', '', $_SERVER['SCRIPT_NAME']), '', $_SERVER['REQUEST_URI']), 1);
106
+        }
104 107
         $this->route->setUrl('/' . trim(explode('?', $url)[0], '/'));
105 108
     }
106 109
 
@@ -111,8 +114,9 @@  discard block
 block discarded – undo
111 114
     {
112 115
         foreach ($this->config['matcher'] as $matcher) {
113 116
             $this->config['matcherInstance'][$matcher] = new $matcher($this);
114
-            if (call_user_func([$this->config['matcherInstance'][$matcher], 'match']))
115
-                return true;
117
+            if (call_user_func([$this->config['matcherInstance'][$matcher], 'match'])) {
118
+                            return true;
119
+            }
116 120
         }
117 121
         return false;
118 122
     }
@@ -157,12 +161,15 @@  discard block
 block discarded – undo
157 161
     {
158 162
         if (isset($this->route->getDetail()['response_templates']) && isset($this->route->getDetail()['response_templates'][$code = $this->response->getStatusCode()])) {
159 163
             $this->route->setCallback($this->route->getDetail()['response_templates'][$code]);
160
-            if(!isset($this->config['matcherInstance']))
161
-                foreach ($this->config['matcher'] as $matcher)
162
-                    $this->config['matcherInstance'][$matcher] = new $matcher($this);
164
+            if(!isset($this->config['matcherInstance'])) {
165
+                            foreach ($this->config['matcher'] as $matcher)
166
+                    $this->config['matcherInstance'][$matcher] = new $matcher($this);
167
+            }
163 168
             foreach($this->config['matcherInstance'] as $instance) {
164
-                foreach (call_user_func([$instance, 'getMatcher']) as $match)
165
-                    if (call_user_func([$instance, $match])){ $this->callTarget(); break; }
169
+                foreach (call_user_func([$instance, 'getMatcher']) as $match) {
170
+                                    if (call_user_func([$instance, $match])){ $this->callTarget();
171
+                }
172
+                break; }
166 173
             }
167 174
             $this->response->setStatusCode($code);
168 175
         }
Please login to merge, or discard this patch.
src/Routing/Matcher/ArrayMatcher.php 3 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@
 block discarded – undo
64 64
     }
65 65
 
66 66
     /**
67
-     * @param $regex
67
+     * @param string $regex
68 68
      * @return bool
69 69
      */
70 70
     private function routeMatch($regex)
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
     /**
25 25
      * @var array
26 26
      */
27
-    private $matcher = ['matchClosure','matchController','matchTemplate'];
27
+    private $matcher = ['matchClosure', 'matchController', 'matchTemplate'];
28 28
 
29 29
     /**
30 30
      * @var array
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
     /**
47 47
      * @param string $matcher
48 48
      */
49
-    public function addMatcher($matcher){
49
+    public function addMatcher($matcher) {
50 50
         $this->matcher[] = $matcher;
51 51
     }
52 52
 
@@ -61,8 +61,8 @@  discard block
 block discarded – undo
61 61
     /**
62 62
      * @param array $dispatcher
63 63
      */
64
-    public function setDispatcher($dispatcher = []){
65
-        $this->dispatcher = array_merge($dispatcher,$this->dispatcher);
64
+    public function setDispatcher($dispatcher = []) {
65
+        $this->dispatcher = array_merge($dispatcher, $this->dispatcher);
66 66
     }
67 67
 
68 68
     /**
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
      */
123 123
     private function generateTarget()
124 124
     {
125
-        if(is_callable($this->request['path'])){
125
+        if (is_callable($this->request['path'])) {
126 126
             $this->router->route->setCallback($this->request['path']);
127 127
             $this->router->route->setDetail($this->request);
128 128
             $this->matchClosure();
@@ -135,11 +135,11 @@  discard block
 block discarded – undo
135 135
                     ? $this->router->route->setCallback($this->request['path']['use'])
136 136
                     : $this->router->route->setCallback($this->request['path']);
137 137
             $this->router->route->setDetail($this->request);
138
-            if($this->validMethod()) {
139
-                foreach($this->matcher as $matcher)
140
-                    call_user_func([$this,$matcher]);
138
+            if ($this->validMethod()) {
139
+                foreach ($this->matcher as $matcher)
140
+                    call_user_func([$this, $matcher]);
141 141
                 $this->router->response->setStatusCode(202);
142
-            }else
142
+            } else
143 143
                 $this->router->response->setStatusCode(405);
144 144
         }
145 145
         return $this->router->route->hasTarget();
@@ -181,7 +181,7 @@  discard block
 block discarded – undo
181 181
             $index = isset($this->request['index']) ? $this->request['index'] : 0;
182 182
             $class = (class_exists($routes[0]))
183 183
                 ? $routes[0]
184
-                : $this->router->collection->getRoutes()['namespace_'.$index].$routes[0];
184
+                : $this->router->collection->getRoutes()['namespace_' . $index] . $routes[0];
185 185
             if (!class_exists($class))
186 186
                 throw new \Exception('Class "' . $class . '." is not found');
187 187
             if (method_exists($class, $routes[1])) {
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
             $extension = explode('.', $path);
210 210
             $extension = end($extension);
211 211
             $index = isset($this->request['index']) ? $this->request['index'] : 0;
212
-            $block = $this->router->collection->getRoutes('path_'.$index);
212
+            $block = $this->router->collection->getRoutes('path_' . $index);
213 213
             if (in_array('.' . $extension, $this->router->getConfig()['viewExtension'])) {
214 214
                 if (is_file($block . $path)) {
215 215
                     $target = $block . $path;
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
                 throw new \Exception('Template file "' . $path . '" is not found in "' . $block . '"');
226 226
             } else {
227 227
                 foreach ($this->router->getConfig()['viewExtension'] as $ext) {
228
-                    if (is_file($block . $path . $ext)){
228
+                    if (is_file($block . $path . $ext)) {
229 229
                         $target = $block . $path . $ext;
230 230
                         $extension = explode('.', $ext);
231 231
                         $extension = end($extension);
@@ -239,7 +239,7 @@  discard block
 block discarded – undo
239 239
                         return true;
240 240
                     }
241 241
                 }
242
-                throw new \Exception('Template file "' . $path . '" is not found in "' .$block . '"');
242
+                throw new \Exception('Template file "' . $path . '" is not found in "' . $block . '"');
243 243
             }
244 244
         }
245 245
         return false;
Please login to merge, or discard this patch.
Braces   +31 added lines, -18 removed lines patch added patch discarded remove patch
@@ -78,8 +78,9 @@  discard block
 block discarded – undo
78 78
                 $this->request['path'] = $dependencies;
79 79
                 $this->request['index'] = $i;
80 80
                 $this->request['route'] = preg_replace_callback('#:([\w]+)#', [$this, 'paramMatch'], '/' . trim(trim($this->request['prefix'], '/') . '/' . trim($route, '/'), '/'));
81
-                if ($this->routeMatch('#^' . $this->request['route'] . '$#'))
82
-                    return $this->generateTarget();
81
+                if ($this->routeMatch('#^' . $this->request['route'] . '$#')) {
82
+                                    return $this->generateTarget();
83
+                }
83 84
             }
84 85
         }
85 86
         unset($this->request);
@@ -107,8 +108,9 @@  discard block
 block discarded – undo
107 108
     {
108 109
         if (substr($this->request['route'], -1) == '*') {
109 110
             $pos = strpos($this->request['route'], '*');
110
-            if (substr($this->router->route->getUrl(), 0, $pos) == substr($this->request['route'], 0, $pos))
111
-                if (isset($this->request)) return true;
111
+            if (substr($this->router->route->getUrl(), 0, $pos) == substr($this->request['route'], 0, $pos)) {
112
+                            if (isset($this->request)) return true;
113
+            }
112 114
         }
113 115
         if (preg_match($regex, $this->router->route->getUrl(), $this->request['parameters'])) {
114 116
             array_shift($this->request['parameters']);
@@ -128,19 +130,26 @@  discard block
 block discarded – undo
128 130
             $this->matchClosure();
129 131
             $this->router->response->setStatusCode(202);
130 132
         } else {
131
-            if (isset($this->request['path']['name'])) $this->router->route->setName($this->request['path']['name']);
132
-            if (isset($this->request['path']['method'])) $this->request['path']['method'] = is_array($this->request['path']['method']) ? $this->request['path']['method'] : [$this->request['path']['method']];
133
-            if (isset($this->request['path']))
134
-                (is_array($this->request['path']) && isset($this->request['path']['use']))
133
+            if (isset($this->request['path']['name'])) {
134
+                $this->router->route->setName($this->request['path']['name']);
135
+            }
136
+            if (isset($this->request['path']['method'])) {
137
+                $this->request['path']['method'] = is_array($this->request['path']['method']) ? $this->request['path']['method'] : [$this->request['path']['method']];
138
+            }
139
+            if (isset($this->request['path'])) {
140
+                            (is_array($this->request['path']) && isset($this->request['path']['use']))
135 141
                     ? $this->router->route->setCallback($this->request['path']['use'])
136
-                    : $this->router->route->setCallback($this->request['path']);
142
+                    : $this->router->route->setCallback($this->request['path']);
143
+            }
137 144
             $this->router->route->setDetail($this->request);
138 145
             if($this->validMethod()) {
139
-                foreach($this->matcher as $matcher)
140
-                    call_user_func([$this,$matcher]);
146
+                foreach($this->matcher as $matcher) {
147
+                                    call_user_func([$this,$matcher]);
148
+                }
141 149
                 $this->router->response->setStatusCode(202);
142
-            }else
143
-                $this->router->response->setStatusCode(405);
150
+            } else {
151
+                            $this->router->response->setStatusCode(405);
152
+            }
144 153
         }
145 154
         return $this->router->route->hasTarget();
146 155
     }
@@ -150,8 +159,9 @@  discard block
 block discarded – undo
150 159
      */
151 160
     public function validMethod()
152 161
     {
153
-        if (!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest')
154
-            return (isset($this->request['path']['ajax']) && $this->request['path']['ajax'] === true) ? true : false;
162
+        if (!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest') {
163
+                    return (isset($this->request['path']['ajax']) && $this->request['path']['ajax'] === true) ? true : false;
164
+        }
155 165
         $method = (isset($this->router->route->getDetail()['path']['method'])) ? $this->router->route->getDetail()['path']['method'] : ['GET'];
156 166
         return (in_array($this->router->route->getMethod(), $method)) ? true : false;
157 167
     }
@@ -177,13 +187,16 @@  discard block
 block discarded – undo
177 187
     {
178 188
         if (!$this->router->route->hasTarget() && strpos($this->router->route->getCallback(), '@') !== false) {
179 189
             $routes = explode('@', $this->router->route->getCallback());
180
-            if (!isset($routes[1])) $routes[1] = 'index';
190
+            if (!isset($routes[1])) {
191
+                $routes[1] = 'index';
192
+            }
181 193
             $index = isset($this->request['index']) ? $this->request['index'] : 0;
182 194
             $class = (class_exists($routes[0]))
183 195
                 ? $routes[0]
184 196
                 : $this->router->collection->getRoutes()['namespace_'.$index].$routes[0];
185
-            if (!class_exists($class))
186
-                throw new \Exception('Class "' . $class . '." is not found');
197
+            if (!class_exists($class)) {
198
+                            throw new \Exception('Class "' . $class . '." is not found');
199
+            }
187 200
             if (method_exists($class, $routes[1])) {
188 201
                 $this->router->route->setTarget([
189 202
                     'dispatcher' => $this->dispatcher['matchController'],
Please login to merge, or discard this patch.
src/Routing/Matcher/UriMatcher.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
     /**
21 21
      * @var array
22 22
      */
23
-    private $matcher = ['matchController','matchTemplate'];
23
+    private $matcher = ['matchController', 'matchTemplate'];
24 24
 
25 25
     private $dispatcher = [
26 26
         'matchTemplate' => 'JetFire\Routing\Dispatcher\TemplateDispatcher',
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
     /**
39 39
      * @param string $matcher
40 40
      */
41
-    public function addMatcher($matcher){
41
+    public function addMatcher($matcher) {
42 42
         $this->matcher[] = $matcher;
43 43
     }
44 44
 
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
      */
56 56
     public function setDispatcher($dispatcher = [])
57 57
     {
58
-        $this->dispatcher = array_merge($dispatcher,$this->dispatcher);
58
+        $this->dispatcher = array_merge($dispatcher, $this->dispatcher);
59 59
     }
60 60
 
61 61
     /**
@@ -63,8 +63,8 @@  discard block
 block discarded – undo
63 63
      */
64 64
     public function match()
65 65
     {
66
-        foreach($this->matcher as $matcher){
67
-            if(call_user_func([$this,$matcher])) {
66
+        foreach ($this->matcher as $matcher) {
67
+            if (call_user_func([$this, $matcher])) {
68 68
                 $this->router->response->setStatusCode(202);
69 69
                 return true;
70 70
             }
@@ -79,9 +79,9 @@  discard block
 block discarded – undo
79 79
     {
80 80
         foreach ($this->router->getConfig()['viewExtension'] as $extension) {
81 81
             for ($i = 0; $i < $this->router->collection->countRoutes; ++$i) {
82
-                $url = explode('/', str_replace($this->router->collection->getRoutes('prefix_' . $i), '',$this->router->route->getUrl()));
82
+                $url = explode('/', str_replace($this->router->collection->getRoutes('prefix_' . $i), '', $this->router->route->getUrl()));
83 83
                 $end = array_pop($url);
84
-                $url = implode('/', array_map('ucwords', $url)).'/'.$end;
84
+                $url = implode('/', array_map('ucwords', $url)) . '/' . $end;
85 85
                 if (is_file(($template = rtrim($this->router->collection->getRoutes('path_' . $i), '/') . $url . $extension))) {
86 86
                     $this->router->route->setTarget([
87 87
                         'dispatcher' => $this->dispatcher['matchTemplate'],
@@ -104,11 +104,11 @@  discard block
 block discarded – undo
104 104
     {
105 105
         $routes = array_slice(explode('/', $this->router->route->getUrl()), 1);
106 106
         $i = 0;
107
-        do{
108
-            $route =  ('/' . $routes[0] == $this->router->collection->getRoutes('prefix_' . $i)) ? array_slice($routes, 1) : $routes;
107
+        do {
108
+            $route = ('/' . $routes[0] == $this->router->collection->getRoutes('prefix_' . $i)) ? array_slice($routes, 1) : $routes;
109 109
             if (isset($route[0])) {
110
-                $class =  (class_exists($this->router->collection->getRoutes('namespace_' . $i). ucfirst($route[0]) . 'Controller'))
111
-                    ? $this->router->collection->getRoutes('namespace_' . $i). ucfirst($route[0]) . 'Controller'
110
+                $class = (class_exists($this->router->collection->getRoutes('namespace_' . $i) . ucfirst($route[0]) . 'Controller'))
111
+                    ? $this->router->collection->getRoutes('namespace_' . $i) . ucfirst($route[0]) . 'Controller'
112 112
                     : ucfirst($route[0]) . 'Controller';
113 113
                 if (isset($route[1]) && method_exists($class, $route[1])) {
114 114
                     $this->router->route->setTarget([
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
                 }
124 124
             }
125 125
             ++$i;
126
-        }while($i < $this->router->collection->countRoutes);
126
+        }while ($i < $this->router->collection->countRoutes);
127 127
         return false;
128 128
     }
129 129
 
Please login to merge, or discard this patch.