Passed
Pull Request — master (#161)
by Marcin
08:36 queued 06:13
created
src/ExceptionHandlers/HttpExceptionHandler.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -42,12 +42,12 @@
 block discarded – undo
42 42
 		$config = \array_replace($default_config, $user_config);
43 43
 
44 44
 		$http_code = $ex->getStatusCode();
45
-		$result = $config[ $http_code ] ?? null;
45
+		$result = $config[$http_code] ?? null;
46 46
 
47 47
 		// If we do not have dedicated entry fort this particular http_code,
48 48
 		// fall back to default value.
49 49
 		if ($result === null) {
50
-			$result = $config[ RB::KEY_DEFAULT ];
50
+			$result = $config[RB::KEY_DEFAULT];
51 51
 		}
52 52
 
53 53
 		// Some defaults to fall back to if not set in user config.
Please login to merge, or discard this patch.
src/ResponseBuilder.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -375,7 +375,7 @@
 block discarded – undo
375 375
 
376 376
 		if ($debug_data !== null) {
377 377
 			$debug_key = Config::get(RB::CONF_KEY_DEBUG_DEBUG_KEY, RB::KEY_DEBUG);
378
-			$response[ $debug_key ] = $debug_data;
378
+			$response[$debug_key] = $debug_data;
379 379
 		}
380 380
 
381 381
 		return $response;
Please login to merge, or discard this patch.
src/ExceptionHandlerHelper.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -45,11 +45,11 @@  discard block
 block discarded – undo
45 45
 		do {
46 46
 			if ($cfg === null) {
47 47
 				// Default handler MUST be present by design and always return something useful.
48
-				$cfg = self::getExceptionHandlerConfig()[ RB::KEY_DEFAULT ];
48
+				$cfg = self::getExceptionHandlerConfig()[RB::KEY_DEFAULT];
49 49
 			}
50 50
 
51
-			$handler = new $cfg[ RB::KEY_HANDLER ]();
52
-			$handler_result = $handler->handle($cfg[ RB::KEY_CONFIG ], $ex);
51
+			$handler = new $cfg[RB::KEY_HANDLER]();
52
+			$handler_result = $handler->handle($cfg[RB::KEY_CONFIG], $ex);
53 53
 			if ($handler_result !== null) {
54 54
 				$result = self::processException($ex, $handler_result);
55 55
 			} else {
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
 		$cfg = self::getExceptionHandlerConfig();
148 148
 
149 149
 		// This config entry is guaranted to exist. Enforced by tests.
150
-		$cfg = $cfg[ HttpException::class ][ RB::KEY_CONFIG ][ HttpResponse::HTTP_UNAUTHORIZED ];
150
+		$cfg = $cfg[HttpException::class][RB::KEY_CONFIG][HttpResponse::HTTP_UNAUTHORIZED];
151 151
 
152 152
 		return static::processException($exception, $cfg, HttpResponse::HTTP_UNAUTHORIZED);
153 153
 	}
@@ -277,13 +277,13 @@  discard block
 block discarded – undo
277 277
 
278 278
 			// check for exact class name match...
279 279
 			if (\array_key_exists($cls, $cfg)) {
280
-				$result = $cfg[ $cls ];
280
+				$result = $cfg[$cls];
281 281
 			} else {
282 282
 				// no exact match, then lets try with `instanceof`
283 283
 				// Config entries are already sorted by priority.
284 284
 				foreach (\array_keys($cfg) as $class_name) {
285 285
 					if ($ex instanceof $class_name) {
286
-						$result = $cfg[ $class_name ];
286
+						$result = $cfg[$class_name];
287 287
 						break;
288 288
 					}
289 289
 				}
Please login to merge, or discard this patch.
src/Converter.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 	    $result = null;
62 62
 
63 63
 	    $type = \gettype($data);
64
-	    $result = $this->primitives[ $type ] ?? null;
64
+	    $result = $this->primitives[$type] ?? null;
65 65
 	    if ($result === null) {
66 66
 		    throw new \InvalidArgumentException(sprintf('No data conversion mapping configured for "%s" primitive.', $type));
67 67
 	    }
@@ -93,13 +93,13 @@  discard block
 block discarded – undo
93 93
         $cls = \get_class($data);
94 94
         if (\is_string($cls)) {
95 95
 	        if (\array_key_exists($cls, $this->classes)) {
96
-		        $result = $this->classes[ $cls ];
96
+		        $result = $this->classes[$cls];
97 97
 		        $debug_result = 'exact config match';
98 98
 	        } else {
99 99
 		        // no exact match, then lets try with `instanceof`
100 100
 		        foreach (\array_keys($this->classes) as $class_name) {
101 101
 			        if ($data instanceof $class_name) {
102
-				        $result = $this->classes[ $class_name ];
102
+				        $result = $this->classes[$class_name];
103 103
 				        $debug_result = "subclass of {$class_name}";
104 104
 				        break;
105 105
 			        }
@@ -146,21 +146,21 @@  discard block
 block discarded – undo
146 146
 
147 147
 	    if ($result === null && \is_object($data)) {
148 148
 		    $cfg = $this->getClassMappingConfigOrThrow($data);
149
-		    $worker = new $cfg[ RB::KEY_HANDLER ]();
150
-		    $result = [$cfg[ RB::KEY_KEY ] => $worker->convert($data, $cfg)];
149
+		    $worker = new $cfg[RB::KEY_HANDLER]();
150
+		    $result = [$cfg[RB::KEY_KEY] => $worker->convert($data, $cfg)];
151 151
 	    }
152 152
 
153 153
 	    if ($result === null && \is_array($data)) {
154 154
 	        $cfg = $this->getPrimitiveMappingConfigOrThrow($data);
155 155
 
156 156
 		    $result = $this->convertArray($data);
157
-	        if (!Util::isArrayWithNonNumericKeys($data)){
158
-		        $result = [$cfg[ RB::KEY_KEY ] => $result];
157
+	        if (!Util::isArrayWithNonNumericKeys($data)) {
158
+		        $result = [$cfg[RB::KEY_KEY] => $result];
159 159
 	        }
160 160
         }
161 161
 
162
-	    if ( \is_bool($data) || \is_float($data) || \is_int($data) || \is_string($data)) {
163
-		    $result = [$this->getPrimitiveMappingConfigOrThrow($data)[ RB::KEY_KEY ] => $data];
162
+	    if (\is_bool($data) || \is_float($data) || \is_int($data) || \is_string($data)) {
163
+		    $result = [$this->getPrimitiveMappingConfigOrThrow($data)[RB::KEY_KEY] => $data];
164 164
 	    }
165 165
 
166 166
 	    return $result;
@@ -199,12 +199,12 @@  discard block
 block discarded – undo
199 199
 
200 200
         foreach ($data as $key => $val) {
201 201
             if (\is_array($val)) {
202
-                $data[ $key ] = $this->convertArray($val);
202
+                $data[$key] = $this->convertArray($val);
203 203
             } elseif (\is_object($val)) {
204 204
                 $cfg = $this->getClassMappingConfigOrThrow($val);
205
-                $worker = new $cfg[ RB::KEY_HANDLER ]();
205
+                $worker = new $cfg[RB::KEY_HANDLER]();
206 206
                 $converted_data = $worker->convert($val, $cfg);
207
-                $data[ $key ] = $converted_data;
207
+                $data[$key] = $converted_data;
208 208
             }
209 209
         }
210 210
 
Please login to merge, or discard this patch.