Completed
Pull Request — master (#107)
by Michal
15:08
created
src/Params/ParamsProcessor.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -21,11 +21,11 @@
 block discarded – undo
21 21
     {
22 22
         foreach ($this->params as $param) {
23 23
             $validationResult = $param->validate();
24
-            if (!$validationResult->isOk()) {
24
+            if ( ! $validationResult->isOk()) {
25 25
                 $this->errors[$param->getKey()] = $validationResult->getErrors();
26 26
             }
27 27
         }
28
-        return !empty($this->errors);
28
+        return ! empty($this->errors);
29 29
     }
30 30
 
31 31
     public function getErrors(): array
Please login to merge, or discard this patch.
src/Params/JsonInputParam.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@
 block discarded – undo
40 40
             return new ValidationResult(ValidationResult::STATUS_ERROR, [json_last_error_msg()]);
41 41
         }
42 42
 
43
-        if (!$value && $this->isRequired() === self::OPTIONAL) {
43
+        if ( ! $value && $this->isRequired() === self::OPTIONAL) {
44 44
             return new ValidationResult(ValidationResult::STATUS_OK);
45 45
         }
46 46
 
Please login to merge, or discard this patch.
src/Output/JsonOutput.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
     public function validate(ResponseInterface $response): ValidationResultInterface
27 27
     {
28
-        if (!$response instanceof JsonApiResponse) {
28
+        if ( ! $response instanceof JsonApiResponse) {
29 29
             return new ValidationResult(ValidationResult::STATUS_ERROR);
30 30
         }
31 31
         if ($this->code !== $response->getCode()) {
Please login to merge, or discard this patch.
src/Misc/ConsoleRequest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -205,7 +205,7 @@
 block discarded – undo
205 205
     {
206 206
         $result = [];
207 207
         foreach ($values as $key => $value) {
208
-            if (!is_array($value)) {
208
+            if ( ! is_array($value)) {
209 209
                 $result[$key] = $value;
210 210
                 continue;
211 211
             }
Please login to merge, or discard this patch.
src/Output/RedirectOutput.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -11,7 +11,7 @@
 block discarded – undo
11 11
 {
12 12
     public function validate(ResponseInterface $response): ValidationResultInterface
13 13
     {
14
-        if (!$response instanceof RedirectResponse) {
14
+        if ( ! $response instanceof RedirectResponse) {
15 15
             return new ValidationResult(ValidationResult::STATUS_ERROR);
16 16
         }
17 17
         if ($this->code !== $response->getCode()) {
Please login to merge, or discard this patch.
src/Authorization/BasicAuthentication.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@
 block discarded – undo
29 29
     {
30 30
         $urlScript = $this->httpRequest->getUrl();
31 31
         $authentication = $this->authentications[$urlScript->getUser()] ?? null;
32
-        if (!$authentication) {
32
+        if ( ! $authentication) {
33 33
             return false;
34 34
         }
35 35
         return $authentication === $urlScript->getPassword();
Please login to merge, or discard this patch.
src/Presenters/ApiPresenter.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
         $paramsProcessor = new ParamsProcessor($handler->params());
83 83
         if ($paramsProcessor->isError()) {
84 84
             $this->response->setCode(Response::S400_BAD_REQUEST);
85
-            if (!Debugger::$productionMode) {
85
+            if ( ! Debugger::$productionMode) {
86 86
                 $response = new JsonResponse(['status' => 'error', 'message' => 'wrong input', 'detail' => $paramsProcessor->getErrors()]);
87 87
             } else {
88 88
                 $response = new JsonResponse(['status' => 'error', 'message' => 'wrong input']);
@@ -103,12 +103,12 @@  discard block
 block discarded – undo
103 103
                 }
104 104
                 $outputValidatorErrors[] = $validationResult->getErrors();
105 105
             }
106
-            if (!$outputValid) {
106
+            if ( ! $outputValid) {
107 107
                 $response = new JsonApiResponse(Response::S500_INTERNAL_SERVER_ERROR, ['status' => 'error', 'message' => 'Internal server error', 'details' => $outputValidatorErrors]);
108 108
             }
109 109
             $code = $response->getCode();
110 110
         } catch (Throwable $exception) {
111
-            if (!Debugger::$productionMode) {
111
+            if ( ! Debugger::$productionMode) {
112 112
                 $response = new JsonApiResponse(Response::S500_INTERNAL_SERVER_ERROR, ['status' => 'error', 'message' => 'Internal server error', 'detail' => $exception->getMessage()]);
113 113
             } else {
114 114
                 $response = new JsonApiResponse(Response::S500_INTERNAL_SERVER_ERROR, ['status' => 'error', 'message' => 'Internal server error']);
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
 
143 143
     private function checkAuth(ApiAuthorizationInterface $authorization): ?IResponse
144 144
     {
145
-        if (!$authorization->authorized()) {
145
+        if ( ! $authorization->authorized()) {
146 146
             $this->response->setCode(Response::S403_FORBIDDEN);
147 147
             return new JsonResponse(['status' => 'error', 'message' => $authorization->getErrorMessage()]);
148 148
         }
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
     private function checkRateLimit(RateLimitInterface $rateLimit): ?IResponse
153 153
     {
154 154
         $rateLimitResponse = $rateLimit->check();
155
-        if (!$rateLimitResponse) {
155
+        if ( ! $rateLimitResponse) {
156 156
             return null;
157 157
         }
158 158
 
@@ -160,12 +160,12 @@  discard block
 block discarded – undo
160 160
         $remaining = $rateLimitResponse->getRemaining();
161 161
         $retryAfter = $rateLimitResponse->getRetryAfter();
162 162
 
163
-        $this->response->addHeader('X-RateLimit-Limit', (string)$limit);
164
-        $this->response->addHeader('X-RateLimit-Remaining', (string)$remaining);
163
+        $this->response->addHeader('X-RateLimit-Limit', (string) $limit);
164
+        $this->response->addHeader('X-RateLimit-Remaining', (string) $remaining);
165 165
 
166 166
         if ($remaining === 0) {
167 167
             $this->response->setCode(Response::S429_TOO_MANY_REQUESTS);
168
-            $this->response->addHeader('Retry-After', (string)$retryAfter);
168
+            $this->response->addHeader('Retry-After', (string) $retryAfter);
169 169
             return $rateLimitResponse->getErrorResponse() ?: new JsonResponse(['status' => 'error', 'message' => 'Too many requests. Retry after ' . $retryAfter . ' seconds.']);
170 170
         }
171 171
         return null;
@@ -227,11 +227,11 @@  discard block
 block discarded – undo
227 227
 
228 228
     private function getRequestDomain(): ?string
229 229
     {
230
-        if (!filter_input(INPUT_SERVER, 'HTTP_REFERER')) {
230
+        if ( ! filter_input(INPUT_SERVER, 'HTTP_REFERER')) {
231 231
             return null;
232 232
         }
233 233
         $refererParsedUrl = parse_url(filter_input(INPUT_SERVER, 'HTTP_REFERER'));
234
-        if (!(isset($refererParsedUrl['scheme']) && isset($refererParsedUrl['host']))) {
234
+        if ( ! (isset($refererParsedUrl['scheme']) && isset($refererParsedUrl['host']))) {
235 235
             return null;
236 236
         }
237 237
         $url = $refererParsedUrl['scheme'] . '://' . $refererParsedUrl['host'];
Please login to merge, or discard this patch.
src/Authorization/CookieApiKeyAuthentication.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@
 block discarded – undo
20 20
     protected function readAuthorizationToken(): ?string
21 21
     {
22 22
         $apiKey = $_COOKIE[$this->cookieName] ?? null;
23
-        if (!$apiKey) {
23
+        if ( ! $apiKey) {
24 24
             $this->errorMessage = 'API key is not set';
25 25
             return null;
26 26
         }
Please login to merge, or discard this patch.
src/Authorization/BearerTokenAuthorization.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -9,7 +9,7 @@
 block discarded – undo
9 9
 
10 10
 class BearerTokenAuthorization extends TokenAuthorization
11 11
 {
12
-   /**
12
+    /**
13 13
      * BearerTokenAuthorization constructor.
14 14
      *
15 15
      * @param BearerTokenRepositoryInterface $tokenRepository
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
      */
29 29
     protected function readAuthorizationToken(): ?string
30 30
     {
31
-        if (!isset($_SERVER['HTTP_AUTHORIZATION'])) {
31
+        if ( ! isset($_SERVER['HTTP_AUTHORIZATION'])) {
32 32
             $this->errorMessage = 'Authorization header HTTP_Authorization is not set';
33 33
             return null;
34 34
         }
Please login to merge, or discard this patch.