Completed
Pull Request — master (#61)
by Tomas
02:14
created
src/Authorization/BearerTokenAuthorization.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -45,12 +45,12 @@  discard block
 block discarded – undo
45 45
         }
46 46
 
47 47
         $result = $this->tokenRepository->validToken($token);
48
-        if (!$result) {
48
+        if ( ! $result) {
49 49
             $this->errorMessage = 'Token doesn\'t exists or isn\'t active';
50 50
             return false;
51 51
         }
52 52
 
53
-        if (!$this->isValidIp($this->tokenRepository->ipRestrictions($token))) {
53
+        if ( ! $this->isValidIp($this->tokenRepository->ipRestrictions($token))) {
54 54
             $this->errorMessage = 'Invalid IP';
55 55
             return false;
56 56
         }
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
         list($range, $netmask) = explode('/', $range, 2);
114 114
         $range_decimal = ip2long($range);
115 115
         $ipDecimal = ip2long($ip);
116
-        $wildcard_decimal = pow(2, (32 - (int)$netmask)) - 1;
116
+        $wildcard_decimal = pow(2, (32 - (int) $netmask)) - 1;
117 117
         $netmask_decimal = ~ $wildcard_decimal;
118 118
         return (($ipDecimal & $netmask_decimal) === ($range_decimal & $netmask_decimal));
119 119
     }
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
      */
127 127
     private function readAuthorizationToken(): ?string
128 128
     {
129
-        if (!isset($_SERVER['HTTP_AUTHORIZATION'])) {
129
+        if ( ! isset($_SERVER['HTTP_AUTHORIZATION'])) {
130 130
             $this->errorMessage = 'Authorization header HTTP_Authorization is not set';
131 131
             return null;
132 132
         }
Please login to merge, or discard this patch.
src/ValidationResult/ValidationResult.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@
 block discarded – undo
16 16
 
17 17
     public function __construct(string $status, array $errors = [])
18 18
     {
19
-        if (!in_array($status, [self::STATUS_OK, self::STATUS_ERROR], true)) {
19
+        if ( ! in_array($status, [self::STATUS_OK, self::STATUS_ERROR], true)) {
20 20
             throw new InvalidArgumentException($status . ' is not valid validation result status');
21 21
         }
22 22
 
Please login to merge, or discard this patch.
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/Presenters/ApiPresenter.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
                 }
89 89
                 $outputValidatorErrors[] = $validationResult->getErrors();
90 90
             }
91
-            if (!$outputValid) {
91
+            if ( ! $outputValid) {
92 92
                 $response = new JsonApiResponse(500, ['status' => 'error', 'message' => 'Internal server error', 'details' => $outputValidatorErrors]);
93 93
             }
94 94
             $code = $response->getCode();
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
 
128 128
     private function checkAuth(ApiAuthorizationInterface $authorization): bool
129 129
     {
130
-        if (!$authorization->authorized()) {
130
+        if ( ! $authorization->authorized()) {
131 131
             $this->getHttpResponse()->setCode(Response::S403_FORBIDDEN);
132 132
             $this->sendResponse(new JsonResponse(['status' => 'error', 'message' => $authorization->getErrorMessage()]));
133 133
             return false;
@@ -207,11 +207,11 @@  discard block
 block discarded – undo
207 207
 
208 208
     private function getRequestDomain(): ?string
209 209
     {
210
-        if (!filter_input(INPUT_SERVER, 'HTTP_REFERER')) {
210
+        if ( ! filter_input(INPUT_SERVER, 'HTTP_REFERER')) {
211 211
             return null;
212 212
         }
213 213
         $refererParsedUrl = parse_url(filter_input(INPUT_SERVER, 'HTTP_REFERER'));
214
-        if (!(isset($refererParsedUrl['scheme']) && isset($refererParsedUrl['host']))) {
214
+        if ( ! (isset($refererParsedUrl['scheme']) && isset($refererParsedUrl['host']))) {
215 215
             return null;
216 216
         }
217 217
         $url = $refererParsedUrl['scheme'] . '://' . $refererParsedUrl['host'];
Please login to merge, or discard this patch.