Passed
Push — master ( 82fa8b...914114 )
by Henri
01:17
created
src/Router.php 2 patches
Indentation   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -71,26 +71,26 @@  discard block
 block discarded – undo
71 71
 
72 72
     public function set($url ,$walking , string $protocol): Router
73 73
     {
74
-		$url = (substr($url,0,1) !=='/' and strlen($url) > 0) ? "/{$url}" : $url;
74
+        $url = (substr($url,0,1) !=='/' and strlen($url) > 0) ? "/{$url}" : $url;
75 75
 
76
-    	foreach($this->routers as $key => $value){
77
-    		if( md5($this->prefix . $value['url'] . $value['protocol'] ) === md5( $url . $protocol ) ){
76
+        foreach($this->routers as $key => $value){
77
+            if( md5($this->prefix . $value['url'] . $value['protocol'] ) === md5( $url . $protocol ) ){
78 78
                 throw new Exception("There is already a route with the url {$url} and with the {$protocol} protocol configured.");
79 79
             }
80 80
         }
81 81
         
82 82
         $this->checkTypeRole($walking);
83 83
 
84
-		$route = [
85
-			'url' => $this->prefix.$url,
86
-			'role' => $walking,
87
-			'protocol' => $protocol,
84
+        $route = [
85
+            'url' => $this->prefix.$url,
86
+            'role' => $walking,
87
+            'protocol' => $protocol,
88 88
             'filters' => null,
89 89
             'where' => null,
90 90
             'group' => self::getInstance()->group
91
-		];
91
+        ];
92 92
 
93
-		$this->routers[] = $route;		
93
+        $this->routers[] = $route;		
94 94
         
95 95
         return self::getInstance();
96 96
     }
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
 
190 190
         $instance->getInstance()->byName($routeName);
191 191
 
192
-		$currentProtocol = $instance->getInstance()->getProtocol();
192
+        $currentProtocol = $instance->getInstance()->getProtocol();
193 193
 
194 194
         foreach(array_reverse($instance->getInstance()->routers) as $r => $route){
195 195
 
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
 
208 208
             $routeRequest = $instance->getInstance()->explodeRoute((substr($_SERVER['REQUEST_URI'],strlen($_SERVER['REQUEST_URI'])-1,1) === '/') , $_SERVER['REQUEST_URI']);
209 209
 
210
-	        if($instance->getInstance()->checkNumparams($routeLoop, $routeRequest) || !$instance->getInstance()->checkParameters($routeLoop, $routeRequest)){
210
+            if($instance->getInstance()->checkNumparams($routeLoop, $routeRequest) || !$instance->getInstance()->checkParameters($routeLoop, $routeRequest)){
211 211
                 continue;
212 212
             }
213 213
 
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
         
224 224
         $instance->getInstance()->currentRoute = null;
225 225
 
226
-	    throw new Exception('Page not found.',404);
226
+        throw new Exception('Page not found.',404);
227 227
     }
228 228
 
229 229
     public static function filter($filters): Router
Please login to merge, or discard this patch.
Spacing   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
 
5 5
 use Exception;
6 6
 
7
-class Router{
7
+class Router {
8 8
     use Helper, CheckTrait, CheckWhere;
9 9
 
10 10
     private static $instance = null;
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
 
22 22
     public static function create(): Router
23 23
     {
24
-        if(!self::getInstance()->instanced){
24
+        if (!self::getInstance()->instanced) {
25 25
             self::getInstance()->checkConfig();
26 26
             self::getInstance()->import(ROUTER_CONFIG['path']);
27 27
             self::getInstance()->instanced = true;
@@ -69,12 +69,12 @@  discard block
 block discarded – undo
69 69
         return self::getInstance()->set($uri, $walking, $protocol);
70 70
     }
71 71
 
72
-    public function set($url ,$walking , string $protocol): Router
72
+    public function set($url, $walking, string $protocol): Router
73 73
     {
74
-		$url = (substr($url,0,1) !=='/' and strlen($url) > 0) ? "/{$url}" : $url;
74
+		$url = (substr($url, 0, 1) !== '/' and strlen($url) > 0) ? "/{$url}" : $url;
75 75
 
76
-    	foreach($this->routers as $key => $value){
77
-    		if( md5($this->prefix . $value['url'] . $value['protocol'] ) === md5( $url . $protocol ) ){
76
+    	foreach ($this->routers as $key => $value) {
77
+    		if (md5($this->prefix.$value['url'].$value['protocol']) === md5($url.$protocol)) {
78 78
                 throw new Exception("There is already a route with the url {$url} and with the {$protocol} protocol configured.");
79 79
             }
80 80
         }
@@ -95,9 +95,9 @@  discard block
 block discarded – undo
95 95
         return self::getInstance();
96 96
     }
97 97
 
98
-    public static function group(string $prefix,$callback): Router
98
+    public static function group(string $prefix, $callback): Router
99 99
     {
100
-        self::getInstance()->prefix = (substr($prefix,0,1) !== '/') ? "/{$prefix}" : $prefix;
100
+        self::getInstance()->prefix = (substr($prefix, 0, 1) !== '/') ? "/{$prefix}" : $prefix;
101 101
         self::getInstance()->group = sha1(date('d/m/Y h:m:i'));
102 102
         $callback();
103 103
         self::getInstance()->group = null;
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
 
109 109
     public static function where(): Router
110 110
     {
111
-        if(self::getInstance()->lastReturn){
111
+        if (self::getInstance()->lastReturn) {
112 112
             throw new Exception("It is not possible to define parameter tests for groups of routes.");
113 113
         }
114 114
 
@@ -119,13 +119,13 @@  discard block
 block discarded – undo
119 119
         $data = (count($data) > 1) ? [$data[0] => $data[1]] : $data[0];
120 120
         
121 121
         $route = end(self::getInstance()->routers);
122
-        $routeURI = explode('/',$route['url']);
122
+        $routeURI = explode('/', $route['url']);
123 123
         $params = [];
124
-        foreach($routeURI as $part){
125
-            if(substr($part,0,1) === '{' && substr($part,-1) === '}'){
126
-                $param = substr($part,1,-1);
124
+        foreach ($routeURI as $part) {
125
+            if (substr($part, 0, 1) === '{' && substr($part, -1) === '}') {
126
+                $param = substr($part, 1, -1);
127 127
 
128
-                self::getInstance()->checkExistParam($param,$data);
128
+                self::getInstance()->checkExistParam($param, $data);
129 129
 
130 130
                 $params[$param] = $data[$param];
131 131
             }
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
 
134 134
         self::getInstance()->checkWhereParams($params);
135 135
 
136
-        $route['where'] = (is_array($route['where'])) ? array_merge($route['where'],$params) : $params;
136
+        $route['where'] = (is_array($route['where'])) ? array_merge($route['where'], $params) : $params;
137 137
 
138 138
         self::getInstance()->routers[count(self::getInstance()->routers)-1] = $route;
139 139
 
@@ -142,14 +142,14 @@  discard block
 block discarded – undo
142 142
 
143 143
     public static function name(string $name): Router
144 144
     {
145
-        if(self::getInstance()->lastReturn){
145
+        if (self::getInstance()->lastReturn) {
146 146
             throw new Exception("There is no reason to call a {$name} route group.");
147 147
         }
148 148
 
149 149
         $currentRoute = end(self::getInstance()->routers);
150 150
 
151
-        foreach(self::getInstance()->routers as $key => $value){
152
-            if(array_key_exists($name, self::getInstance()->routers)){
151
+        foreach (self::getInstance()->routers as $key => $value) {
152
+            if (array_key_exists($name, self::getInstance()->routers)) {
153 153
                 throw new Exception("There is already a route with the name {$name} configured.");
154 154
             }
155 155
         }
@@ -165,15 +165,15 @@  discard block
 block discarded – undo
165 165
 
166 166
     private function byName(?string $routName)
167 167
     {
168
-        if(!is_null($routName)){
168
+        if (!is_null($routName)) {
169 169
             $currentProtocol = $this->getProtocol();
170 170
 
171 171
             $this->checkName($routName);
172 172
     
173 173
             $route = $this->routers[$routName];
174 174
     
175
-            if(!$this->checkProtocol($route['protocol'], $currentProtocol)){
176
-                throw new Exception('Page not found.',404);
175
+            if (!$this->checkProtocol($route['protocol'], $currentProtocol)) {
176
+                throw new Exception('Page not found.', 404);
177 177
             }
178 178
     
179 179
             $this->checkFiltering($route);
@@ -191,27 +191,27 @@  discard block
 block discarded – undo
191 191
 
192 192
 		$currentProtocol = $instance->getInstance()->getProtocol();
193 193
 
194
-        foreach(array_reverse($instance->getInstance()->routers) as $r => $route){
194
+        foreach (array_reverse($instance->getInstance()->routers) as $r => $route) {
195 195
 
196 196
             $instance->getInstance()->currentRoute = $route;
197 197
 
198
-            if(!$instance->getInstance()->checkProtocol($route['protocol'], $currentProtocol)){
198
+            if (!$instance->getInstance()->checkProtocol($route['protocol'], $currentProtocol)) {
199 199
                 continue;
200 200
             }
201 201
 
202 202
             $instance->getInstance()->hasProtocol($route, $currentProtocol);
203 203
 
204
-            $routeLoop = $instance->getInstance()->explodeRoute( (substr($route['url'],strlen($route['url'])-1,1) === '/') , $route['url']);
204
+            $routeLoop = $instance->getInstance()->explodeRoute((substr($route['url'], strlen($route['url'])-1, 1) === '/'), $route['url']);
205 205
             
206 206
             $_SERVER['REQUEST_URI'] = (array_key_exists('REQUEST_URI', $_SERVER)) ? $_SERVER['REQUEST_URI'] : '';
207 207
 
208
-            $routeRequest = $instance->getInstance()->explodeRoute((substr($_SERVER['REQUEST_URI'],strlen($_SERVER['REQUEST_URI'])-1,1) === '/') , $_SERVER['REQUEST_URI']);
208
+            $routeRequest = $instance->getInstance()->explodeRoute((substr($_SERVER['REQUEST_URI'], strlen($_SERVER['REQUEST_URI'])-1, 1) === '/'), $_SERVER['REQUEST_URI']);
209 209
 
210
-	        if($instance->getInstance()->checkNumparams($routeLoop, $routeRequest) || !$instance->getInstance()->checkParameters($routeLoop, $routeRequest)){
210
+	        if ($instance->getInstance()->checkNumparams($routeLoop, $routeRequest) || !$instance->getInstance()->checkParameters($routeLoop, $routeRequest)) {
211 211
                 continue;
212 212
             }
213 213
 
214
-            if($instance->getInstance()->checkWhere($route, $routeRequest)){
214
+            if ($instance->getInstance()->checkWhere($route, $routeRequest)) {
215 215
 
216 216
                 $instance->getInstance()->checkFiltering($route);
217 217
 
@@ -223,18 +223,18 @@  discard block
 block discarded – undo
223 223
         
224 224
         $instance->getInstance()->currentRoute = null;
225 225
 
226
-	    throw new Exception('Page not found.',404);
226
+	    throw new Exception('Page not found.', 404);
227 227
     }
228 228
 
229 229
     public static function filter($filters): Router
230 230
     {
231
-        if(self::getInstance()->lastReturn !== null){
231
+        if (self::getInstance()->lastReturn !== null) {
232 232
             $currentGroup = end(self::getInstance()->routers)['group'];
233 233
 
234 234
             foreach (self::getInstance()->routers as $key => $value) {
235 235
 
236
-                if($value['group'] === $currentGroup){
237
-                    $currentRoute = self::getInstance()->addFilter(self::getInstance()->routers[$key],$filters);
236
+                if ($value['group'] === $currentGroup) {
237
+                    $currentRoute = self::getInstance()->addFilter(self::getInstance()->routers[$key], $filters);
238 238
                     self::getInstance()->routers[$key] = $currentRoute;
239 239
                 }
240 240
 
@@ -245,24 +245,24 @@  discard block
 block discarded – undo
245 245
             return self::getInstance();
246 246
         }
247 247
         
248
-        self::getInstance()->routers[count(self::getInstance()->routers)-1] = self::getInstance()->addFilter(end(self::getInstance()->routers),$filters);
248
+        self::getInstance()->routers[count(self::getInstance()->routers)-1] = self::getInstance()->addFilter(end(self::getInstance()->routers), $filters);
249 249
         return self::getInstance();
250 250
     }
251 251
 
252 252
     public static function addFilter(array $route, $filter): array
253 253
     {
254
-        if(is_null($route['filters'])){
254
+        if (is_null($route['filters'])) {
255 255
             $route['filters'] = $filter;
256 256
             return $route;
257 257
         }
258 258
 
259 259
         $filters = (is_array($filter)) ? $filter : [0 => $filter];
260 260
 
261
-        if(is_array($route['filters'])){
261
+        if (is_array($route['filters'])) {
262 262
             foreach ($route['filters'] as $key => $value) {
263 263
                 $filters[] = $value;
264 264
             }
265
-        }else{
265
+        }else {
266 266
             $filters[] = $route['filters'];
267 267
         }
268 268
 
Please login to merge, or discard this patch.
src/CheckWhere.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -4,25 +4,25 @@  discard block
 block discarded – undo
4 4
 
5 5
 use Exception;
6 6
 
7
-trait CheckWhere{
7
+trait CheckWhere {
8 8
         
9 9
     protected function checkWhereParam($data)
10 10
     {
11
-        if(count($data) === 0){
11
+        if (count($data) === 0) {
12 12
             throw new Exception('It is necessary to define a condition to be tested.');
13 13
         }
14 14
     }
15 15
 
16 16
     protected function checkExistParam($param, $data)
17 17
     {
18
-        if(!array_key_exists($param,$data)){
18
+        if (!array_key_exists($param, $data)) {
19 19
             throw new Exception('Test parameter not available on the route in question.');
20 20
         }
21 21
     }
22 22
 
23 23
     protected function checkWhereParams($params)
24 24
     {
25
-        if(count($params) === 0){
25
+        if (count($params) === 0) {
26 26
             throw new Exception('The route in question has no parameters to be tested.');
27 27
         }
28 28
     }
@@ -31,18 +31,18 @@  discard block
 block discarded – undo
31 31
     {
32 32
         $pass = true;
33 33
 
34
-        if(!is_array($route['where'])){
34
+        if (!is_array($route['where'])) {
35 35
             return $pass;
36 36
         }
37 37
 
38
-        $routeURI = explode('/',$route['url']);
38
+        $routeURI = explode('/', $route['url']);
39 39
         $params = [];
40
-        foreach($routeURI as $p => $part){
41
-            if(substr($part,0,1) === '{' && substr($part,-1) === '}'){
42
-                $param = substr($part,1,-1);
40
+        foreach ($routeURI as $p => $part) {
41
+            if (substr($part, 0, 1) === '{' && substr($part, -1) === '}') {
42
+                $param = substr($part, 1, -1);
43 43
                 $params[$param] = $route['where'][$param];
44 44
 
45
-                if(!preg_match("/^{$params[$param]}$/",$request[$p])){
45
+                if (!preg_match("/^{$params[$param]}$/", $request[$p])) {
46 46
                     $pass = false;
47 47
                 }
48 48
             }
Please login to merge, or discard this patch.