Passed
Pull Request — master (#3)
by Vince
03:02
created
src/core/endpoints/map.php 2 patches
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -76,14 +76,14 @@  discard block
 block discarded – undo
76 76
         /**
77 77
          * Check if a custom directory was set in the Responsible API options
78 78
          */
79
-        if( (isset($options['classRoute']) && !empty($options['classRoute'])) && 
79
+        if ((isset($options['classRoute']) && !empty($options['classRoute'])) && 
80 80
             (isset($options['classRoute']['directory']) && isset($options['classRoute']['namespace']))
81 81
         ) {
82 82
             $customService = $this->options['classRoute'];
83 83
             $directory = $customService['directory'];
84 84
             $middleware = $customService['namespace'];
85 85
 
86
-        }else {
86
+        } else {
87 87
             $middleware = 'responsible';
88 88
 
89 89
             $endpoint = str_replace(
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
                 if (class_exists($child) && method_exists($child, 'register')) {
142 142
                     self::$middleWareClass = new $child;
143 143
                     $this->registry[$point] = self::$middleWareClass->register();
144
-                }else{
144
+                } else {
145 145
                     (new exception\errorException)
146 146
                         ->setOptions($this->getOptions())
147 147
                         ->message("Class Error:: class {$child} needs to exist. See documentation on setting up a service.")
@@ -232,16 +232,16 @@  discard block
 block discarded – undo
232 232
                  * @var array
233 233
                  */
234 234
                 if (array_search($endpoint, $this->registry[$api]) !== false) {
235
-                    if( method_exists($this->NAMESPACE_ENDPOINTS[$api], 'scope') ) {
235
+                    if (method_exists($this->NAMESPACE_ENDPOINTS[$api], 'scope')) {
236 236
                         $classScope = (new $this->NAMESPACE_ENDPOINTS[$api])->scope();
237 237
                         $position = array_search($endpoint, $this->registry[$api]);
238 238
                         
239
-                        if( is_array($classScope) && isset($classScope[$position]) ) {
239
+                        if (is_array($classScope) && isset($classScope[$position])) {
240 240
                             $endpointSettings['model']['scope'] = $classScope[$position];
241 241
 
242
-                        }else{
242
+                        } else {
243 243
 
244
-                            if( !is_array($classScope) ) {
244
+                            if (!is_array($classScope)) {
245 245
                                 $endpointSettings['model']['scope'] = $classScope;
246 246
                             }
247 247
                         }
@@ -292,16 +292,16 @@  discard block
 block discarded – undo
292 292
 
293 293
                         $scope = 'private';
294 294
 
295
-                        if( method_exists($this->NAMESPACE_ENDPOINTS[$api], 'scope') ) {
295
+                        if (method_exists($this->NAMESPACE_ENDPOINTS[$api], 'scope')) {
296 296
                             $classScope = (new $this->NAMESPACE_ENDPOINTS[$api])->scope();
297 297
                             $position = array_search($path, $this->registry[$api]);
298 298
                             
299
-                            if( is_array($classScope) && isset($classScope[$position]) ) {
299
+                            if (is_array($classScope) && isset($classScope[$position])) {
300 300
                                 $scope = $classScope[$position];
301 301
 
302
-                            }else{
302
+                            } else {
303 303
 
304
-                                if( !is_array($classScope) ) {
304
+                                if (!is_array($classScope)) {
305 305
                                     $scope = $classScope;
306 306
                                 }
307 307
                             }
Please login to merge, or discard this patch.
Braces   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
             $directory = $customService['directory'];
84 84
             $middleware = $customService['namespace'];
85 85
 
86
-        }else {
86
+        } else {
87 87
             $middleware = 'responsible';
88 88
 
89 89
             $endpoint = str_replace(
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
                 if (class_exists($child) && method_exists($child, 'register')) {
142 142
                     self::$middleWareClass = new $child;
143 143
                     $this->registry[$point] = self::$middleWareClass->register();
144
-                }else{
144
+                } else{
145 145
                     (new exception\errorException)
146 146
                         ->setOptions($this->getOptions())
147 147
                         ->message("Class Error:: class {$child} needs to exist. See documentation on setting up a service.")
@@ -239,7 +239,7 @@  discard block
 block discarded – undo
239 239
                         if( is_array($classScope) && isset($classScope[$position]) ) {
240 240
                             $endpointSettings['model']['scope'] = $classScope[$position];
241 241
 
242
-                        }else{
242
+                        } else{
243 243
 
244 244
                             if( !is_array($classScope) ) {
245 245
                                 $endpointSettings['model']['scope'] = $classScope;
@@ -299,7 +299,7 @@  discard block
 block discarded – undo
299 299
                             if( is_array($classScope) && isset($classScope[$position]) ) {
300 300
                                 $scope = $classScope[$position];
301 301
 
302
-                            }else{
302
+                            } else{
303 303
 
304 304
                                 if( !is_array($classScope) ) {
305 305
                                     $scope = $classScope;
Please login to merge, or discard this patch.
src/responsible.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
         // Authenticate the API connections
145 145
         try {
146 146
             $this->server->authenticate();
147
-        }catch (responsibleException | \Exception $e) {
147
+        } catch (responsibleException | \Exception $e) {
148 148
             self::$response = $e->getMessage();
149 149
             return;
150 150
         }
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
                 $this->getRateLimit(),
156 156
                 $this->getRateWindow()
157 157
             );
158
-        }catch (responsibleException | \Exception $e) {
158
+        } catch (responsibleException | \Exception $e) {
159 159
             self::$response = $e->getMessage();
160 160
             return;
161 161
         }
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
         // Build the APIs internal router
164 164
         try {
165 165
             $this->server->route($route);
166
-        }catch (responsibleException | \Exception $e) {
166
+        } catch (responsibleException | \Exception $e) {
167 167
             self::$response = $e->getMessage();
168 168
             return;
169 169
         }
@@ -372,7 +372,7 @@  discard block
 block discarded – undo
372 372
         $getJWT = (isset($options['getJWT']) && is_bool($options['getJWT']))
373 373
         ? $options['getJWT'] : true;
374 374
 
375
-        $getSecretAppend = (isset($options['secret']) && ($options['secret'] == 'append') )
375
+        $getSecretAppend = (isset($options['secret']) && ($options['secret'] == 'append'))
376 376
         ? $options['secret'] : false;
377 377
 
378 378
         return (new user\user)
Please login to merge, or discard this patch.
Braces   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
         // Authenticate the API connections
145 145
         try {
146 146
             $this->server->authenticate();
147
-        }catch (responsibleException | \Exception $e) {
147
+        } catch (responsibleException | \Exception $e) {
148 148
             self::$response = $e->getMessage();
149 149
             return;
150 150
         }
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
                 $this->getRateLimit(),
156 156
                 $this->getRateWindow()
157 157
             );
158
-        }catch (responsibleException | \Exception $e) {
158
+        } catch (responsibleException | \Exception $e) {
159 159
             self::$response = $e->getMessage();
160 160
             return;
161 161
         }
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
         // Build the APIs internal router
164 164
         try {
165 165
             $this->server->route($route);
166
-        }catch (responsibleException | \Exception $e) {
166
+        } catch (responsibleException | \Exception $e) {
167 167
             self::$response = $e->getMessage();
168 168
             return;
169 169
         }
Please login to merge, or discard this patch.
src/core/server.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -153,7 +153,7 @@  discard block
 block discarded – undo
153 153
             $this->header = new headers\header;
154 154
             $this->header->setOptions($options);
155 155
 
156
-            if (empty((array)$this->header->getMethod())) {
156
+            if (empty((array) $this->header->getMethod())) {
157 157
                 $this->header->requestMethod();
158 158
             }
159 159
         }
@@ -206,7 +206,7 @@  discard block
 block discarded – undo
206 206
      * [getOptions Get the stored Responsible API options]
207 207
      * @return array|null
208 208
      */
209
-    public function getOptions():?array
209
+    public function getOptions(): ?array
210 210
     {
211 211
         return $this->options;
212 212
     }
@@ -263,7 +263,7 @@  discard block
 block discarded – undo
263 263
         }
264 264
 
265 265
         if (is_null($key) || $key == '') {
266
-            if( !is_null($response) ) {
266
+            if (!is_null($response)) {
267 267
                 $this->RESPONSE['response'] = $response;
268 268
             }
269 269
             $this->RESPONSE = array_merge($responseHeader, $this->RESPONSE);
@@ -280,7 +280,7 @@  discard block
 block discarded – undo
280 280
      */
281 281
     public function getResponse()
282 282
     {
283
-        if(isset($this->RESPONSE['response']['response'])) {
283
+        if (isset($this->RESPONSE['response']['response'])) {
284 284
             $this->RESPONSE['response'] = $this->RESPONSE['response']['response'];
285 285
         }
286 286
         return $this->RESPONSE;
@@ -310,7 +310,7 @@  discard block
 block discarded – undo
310 310
     public function authenticate()
311 311
     {
312 312
         $options = $this->getOptions();
313
-        $route = (isset($options['route']) && !empty($options['route']) ) ? $options['route'] : '';
313
+        $route = (isset($options['route']) && !empty($options['route'])) ? $options['route'] : '';
314 314
 
315 315
         $this->endpoints->baseApiRoot(dirname(__DIR__));
316 316
         $this->endpoints->register();
@@ -321,7 +321,7 @@  discard block
 block discarded – undo
321 321
         $this->router = $router->route($route);
322 322
         $endpoint = $this->endpoints->isEndpoint($router->getApi(), $router->getPath());
323 323
 
324
-        if(isset($endpoint->model['scope'])) {
324
+        if (isset($endpoint->model['scope'])) {
325 325
             $_REQUEST['scope'] = $endpoint->model['scope'];
326 326
             $this->header->setData($_REQUEST);
327 327
         }
@@ -406,7 +406,7 @@  discard block
 block discarded – undo
406 406
         /**
407 407
          * Check if theres a request payload sent
408 408
          */
409
-        if(isset($_REQUEST['payload'])) {
409
+        if (isset($_REQUEST['payload'])) {
410 410
             $router->setRequestBody($_REQUEST['payload']);
411 411
         }
412 412
         $router->setPostBody($this->header->getBody());
@@ -417,11 +417,11 @@  discard block
 block discarded – undo
417 417
         /**
418 418
          * Check the access scope
419 419
          */
420
-        if( !isset($this->router->endpoint->model['scope']) ) {
420
+        if (!isset($this->router->endpoint->model['scope'])) {
421 421
             $this->router->endpoint->model['scope'] = 'private';
422 422
         }
423 423
 
424
-        if( isset($this->header->getMethod()->data['scope']) && 
424
+        if (isset($this->header->getMethod()->data['scope']) && 
425 425
             ($this->header->getMethod()->data['scope'] == 'anonymous')
426 426
         ) {
427 427
             $this->router->endpoint->model['scope'] = 'anonymous';
@@ -502,7 +502,7 @@  discard block
 block discarded – undo
502 502
         $corsAllowed = ($this->getOptions()['cors']) ?? false;
503 503
         $isCorsRequest = ($_SERVER['HTTP_ORIGIN']) ?? false;
504 504
         $this->header->requestType($this->getRequestType());
505
-        $this->header->setHeaders($corsAllowed&&$isCorsRequest);
505
+        $this->header->setHeaders($corsAllowed && $isCorsRequest);
506 506
 
507 507
         /**
508 508
          * Output the response if any
Please login to merge, or discard this patch.
src/core/exception/errorException.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -157,7 +157,7 @@
 block discarded – undo
157 157
         
158 158
         $headers = new headers\header;
159 159
         $headers->setOptions($options);
160
-        $headers->setHeaders($corsAllowed&&$isCorsRequest);
160
+        $headers->setHeaders($corsAllowed && $isCorsRequest);
161 161
 
162 162
         http_response_code($this->ERROR_STATE['ERROR_CODE']);
163 163
 
Please login to merge, or discard this patch.
src/core/headers/headerAuth.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -256,7 +256,7 @@
 block discarded – undo
256 256
     {
257 257
         $corsAllowed = ($this->getOptions()['cors']) ?? false;
258 258
         $isCorsRequest = ($_SERVER['HTTP_ORIGIN']) ?? false;
259
-        $this->setHeaders($corsAllowed&&$isCorsRequest);
259
+        $this->setHeaders($corsAllowed && $isCorsRequest);
260 260
 
261 261
         $this->setHeader('HTTP/1.1', array(
262 262
             'Unauthorized',
Please login to merge, or discard this patch.
src/core/route/base.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@
 block discarded – undo
55 55
     {
56 56
         $scriptName = ltrim($_SERVER['SCRIPT_NAME']);
57 57
         $uri = array_values(array_filter(explode('/', $_SERVER['SCRIPT_NAME'])));
58
-        return '/'.implode('/', array_slice($uri, 0, 1)).'/';
58
+        return '/' . implode('/', array_slice($uri, 0, 1)) . '/';
59 59
     }
60 60
 
61 61
     /**
Please login to merge, or discard this patch.
src/core/headers/header.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -487,6 +487,6 @@
 block discarded – undo
487 487
             return '';
488 488
         }
489 489
 
490
-        return ','.implode(',', $this->additionalCORSHeaders);
490
+        return ',' . implode(',', $this->additionalCORSHeaders);
491 491
     }
492 492
 }
Please login to merge, or discard this patch.
src/core/route/router.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -198,7 +198,7 @@
 block discarded – undo
198 198
             ),
199 199
         );
200 200
 
201
-        $this->setRoutes((object)$routesArray);
201
+        $this->setRoutes((object) $routesArray);
202 202
 
203 203
         return $this->getRoutes();
204 204
     }
Please login to merge, or discard this patch.