Passed
Pull Request — master (#1345)
by Michael
12:41
created
src/Handler/HandlerRegistry.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -23,10 +23,10 @@  discard block
 block discarded – undo
23 23
 
24 24
         switch ($direction) {
25 25
             case GraphNavigatorInterface::DIRECTION_DESERIALIZATION:
26
-                return 'deserialize' . $type . 'From' . $format;
26
+                return 'deserialize'.$type.'From'.$format;
27 27
 
28 28
             case GraphNavigatorInterface::DIRECTION_SERIALIZATION:
29
-                return 'serialize' . $type . 'To' . $format;
29
+                return 'serialize'.$type.'To'.$format;
30 30
 
31 31
             default:
32 32
                 throw new LogicException(sprintf('The direction %s does not exist; see GraphNavigatorInterface::DIRECTION_??? constants.', json_encode($direction)));
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
     public function registerSubscribingHandler(SubscribingHandlerInterface $handler): void
42 42
     {
43 43
         foreach ($handler->getSubscribingMethods() as $methodData) {
44
-            if (!isset($methodData['type'], $methodData['format'])) {
44
+            if ( ! isset($methodData['type'], $methodData['format'])) {
45 45
                 throw new RuntimeException(sprintf('For each subscribing method a "type" and "format" attribute must be given, but only got "%s" for %s.', implode('" and "', array_keys($methodData)), \get_class($handler)));
46 46
             }
47 47
 
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
      */
71 71
     public function getHandler(int $direction, string $typeName, string $format)
72 72
     {
73
-        if (!isset($this->handlers[$direction][$typeName][$format])) {
73
+        if ( ! isset($this->handlers[$direction][$typeName][$format])) {
74 74
             return null;
75 75
         }
76 76
 
Please login to merge, or discard this patch.
src/Handler/LazyHandlerRegistry.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
      */
27 27
     public function __construct($container, array $handlers = [])
28 28
     {
29
-        if (!$container instanceof PsrContainerInterface && !$container instanceof ContainerInterface) {
29
+        if ( ! $container instanceof PsrContainerInterface && ! $container instanceof ContainerInterface) {
30 30
             throw new InvalidArgumentException(sprintf('The container must be an instance of %s or %s (%s given).', PsrContainerInterface::class, ContainerInterface::class, \is_object($container) ? \get_class($container) : \gettype($container)));
31 31
         }
32 32
 
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
             return $this->initializedHandlers[$direction][$typeName][$format];
53 53
         }
54 54
 
55
-        if (!isset($this->handlers[$direction][$typeName][$format])) {
55
+        if ( ! isset($this->handlers[$direction][$typeName][$format])) {
56 56
             return null;
57 57
         }
58 58
 
Please login to merge, or discard this patch.
src/Metadata/PropertyMetadata.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -157,22 +157,22 @@  discard block
 block discarded – undo
157 157
             $class = $this->getReflection()->getDeclaringClass();
158 158
 
159 159
             if (empty($getter)) {
160
-                if ($class->hasMethod('get' . $this->name) && $class->getMethod('get' . $this->name)->isPublic()) {
161
-                    $getter = 'get' . $this->name;
162
-                } elseif ($class->hasMethod('is' . $this->name) && $class->getMethod('is' . $this->name)->isPublic()) {
163
-                    $getter = 'is' . $this->name;
164
-                } elseif ($class->hasMethod('has' . $this->name) && $class->getMethod('has' . $this->name)->isPublic()) {
165
-                    $getter = 'has' . $this->name;
160
+                if ($class->hasMethod('get'.$this->name) && $class->getMethod('get'.$this->name)->isPublic()) {
161
+                    $getter = 'get'.$this->name;
162
+                } elseif ($class->hasMethod('is'.$this->name) && $class->getMethod('is'.$this->name)->isPublic()) {
163
+                    $getter = 'is'.$this->name;
164
+                } elseif ($class->hasMethod('has'.$this->name) && $class->getMethod('has'.$this->name)->isPublic()) {
165
+                    $getter = 'has'.$this->name;
166 166
                 } else {
167
-                    throw new InvalidMetadataException(sprintf('There is neither a public %s method, nor a public %s method, nor a public %s method in class %s. Please specify which public method should be used for retrieving the value of the property %s.', 'get' . ucfirst($this->name), 'is' . ucfirst($this->name), 'has' . ucfirst($this->name), $this->class, $this->name));
167
+                    throw new InvalidMetadataException(sprintf('There is neither a public %s method, nor a public %s method, nor a public %s method in class %s. Please specify which public method should be used for retrieving the value of the property %s.', 'get'.ucfirst($this->name), 'is'.ucfirst($this->name), 'has'.ucfirst($this->name), $this->class, $this->name));
168 168
                 }
169 169
             }
170 170
 
171
-            if (empty($setter) && !$this->readOnly) {
172
-                if ($class->hasMethod('set' . $this->name) && $class->getMethod('set' . $this->name)->isPublic()) {
173
-                    $setter = 'set' . $this->name;
171
+            if (empty($setter) && ! $this->readOnly) {
172
+                if ($class->hasMethod('set'.$this->name) && $class->getMethod('set'.$this->name)->isPublic()) {
173
+                    $setter = 'set'.$this->name;
174 174
                 } else {
175
-                    throw new InvalidMetadataException(sprintf('There is no public %s method in class %s. Please specify which public method should be used for setting the value of the property %s.', 'set' . ucfirst($this->name), $this->class, $this->name));
175
+                    throw new InvalidMetadataException(sprintf('There is no public %s method in class %s. Please specify which public method should be used for setting the value of the property %s.', 'set'.ucfirst($this->name), $this->class, $this->name));
176 176
                 }
177 177
             }
178 178
         }
@@ -191,7 +191,7 @@  discard block
 block discarded – undo
191 191
         return is_array($type)
192 192
             && 'array' === $type['name']
193 193
             && isset($type['params'][0])
194
-            && !isset($type['params'][1]);
194
+            && ! isset($type['params'][1]);
195 195
     }
196 196
 
197 197
     public static function isCollectionMap(?array $type = null): bool
Please login to merge, or discard this patch.
src/Metadata/Driver/NullDriver.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
     public function loadMetadataForClass(\ReflectionClass $class): ?BaseClassMetadata
14 14
     {
15 15
         $classMetadata = new ClassMetadata($name = $class->name);
16
-        $fileResource =  $class->getFilename();
16
+        $fileResource = $class->getFilename();
17 17
         if (false !== $fileResource) {
18 18
             $classMetadata->fileResources[] = $fileResource;
19 19
         }
Please login to merge, or discard this patch.
src/Serializer.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 
115 115
     private function getNavigator(int $direction): GraphNavigatorInterface
116 116
     {
117
-        if (!isset($this->graphNavigators[$direction])) {
117
+        if ( ! isset($this->graphNavigators[$direction])) {
118 118
             throw new RuntimeException(
119 119
                 sprintf(
120 120
                     'Can not find a graph navigator for the direction "%s".',
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
             ? $this->serializationVisitors
133 133
             : $this->deserializationVisitors;
134 134
 
135
-        if (!isset($factories[$format])) {
135
+        if ( ! isset($factories[$format])) {
136 136
             throw new UnsupportedFormatException(
137 137
                 sprintf(
138 138
                     'The format "%s" is not supported for %s.',
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
         $result = $this->visit($navigator, $visitor, $context, $data, 'json', $type);
197 197
         $result = $this->convertArrayObjects($result);
198 198
 
199
-        if (!\is_array($result)) {
199
+        if ( ! \is_array($result)) {
200 200
             throw new RuntimeException(sprintf(
201 201
                 'The input data of type "%s" did not convert to an array, but got a result of type "%s".',
202 202
                 \is_object($data) ? \get_class($data) : \gettype($data),
Please login to merge, or discard this patch.
src/Context.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
             $this->addExclusionStrategy(new VersionExclusionStrategy($this->attributes['version']));
80 80
         }
81 81
 
82
-        if (!empty($this->attributes['max_depth_checks'])) {
82
+        if ( ! empty($this->attributes['max_depth_checks'])) {
83 83
             $this->addExclusionStrategy(new DepthExclusionStrategy());
84 84
         }
85 85
 
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
 
133 133
     final protected function assertMutable(): void
134 134
     {
135
-        if (!$this->initialized) {
135
+        if ( ! $this->initialized) {
136 136
             return;
137 137
         }
138 138
 
@@ -208,7 +208,7 @@  discard block
 block discarded – undo
208 208
     {
209 209
         $metadata = $this->metadataStack->pop();
210 210
 
211
-        if (!$metadata instanceof PropertyMetadata) {
211
+        if ( ! $metadata instanceof PropertyMetadata) {
212 212
             throw new RuntimeException('Context metadataStack not working well');
213 213
         }
214 214
     }
@@ -217,7 +217,7 @@  discard block
 block discarded – undo
217 217
     {
218 218
         $metadata = $this->metadataStack->pop();
219 219
 
220
-        if (!$metadata instanceof ClassMetadata) {
220
+        if ( ! $metadata instanceof ClassMetadata) {
221 221
             throw new RuntimeException('Context metadataStack not working well');
222 222
         }
223 223
     }
@@ -232,7 +232,7 @@  discard block
 block discarded – undo
232 232
      */
233 233
     public function getCurrentPath(): array
234 234
     {
235
-        if (!$this->metadataStack) {
235
+        if ( ! $this->metadataStack) {
236 236
             return [];
237 237
         }
238 238
 
Please login to merge, or discard this patch.
src/AbstractVisitor.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
             throw new NonStringCastableTypeException($value);
57 57
         }
58 58
 
59
-        if (is_object($value) && !method_exists($value, '__toString')) {
59
+        if (is_object($value) && ! method_exists($value, '__toString')) {
60 60
             throw new NonStringCastableTypeException($value);
61 61
         }
62 62
     }
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
      */
70 70
     protected function assertValueCanBeCastToInt($value): void
71 71
     {
72
-        if (is_object($value) && !$value instanceof \SimpleXMLElement) {
72
+        if (is_object($value) && ! $value instanceof \SimpleXMLElement) {
73 73
             throw new NonIntCastableTypeException($value);
74 74
         }
75 75
     }
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
      */
83 83
     protected function assertValueCanCastToFloat($value): void
84 84
     {
85
-        if (is_object($value) && !$value instanceof \SimpleXMLElement) {
85
+        if (is_object($value) && ! $value instanceof \SimpleXMLElement) {
86 86
             throw new NonFloatCastableTypeException($value);
87 87
         }
88 88
     }
Please login to merge, or discard this patch.
src/Handler/DateHandler.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
                     'type' => $type,
48 48
                     'format' => $format,
49 49
                     'direction' => GraphNavigatorInterface::DIRECTION_SERIALIZATION,
50
-                    'method' => 'serialize' . $type,
50
+                    'method' => 'serialize'.$type,
51 51
                 ];
52 52
             }
53 53
 
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
                 'type' => 'DateTimeInterface',
56 56
                 'direction' => GraphNavigatorInterface::DIRECTION_DESERIALIZATION,
57 57
                 'format' => $format,
58
-                'method' => 'deserializeDateTimeFrom' . ucfirst($format),
58
+                'method' => 'deserializeDateTimeFrom'.ucfirst($format),
59 59
             ];
60 60
         }
61 61
 
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
      */
224 224
     private function parseDateTime($data, array $type, bool $immutable = false): \DateTimeInterface
225 225
     {
226
-        $timezone = !empty($type['params'][1]) ? new \DateTimeZone($type['params'][1]) : $this->defaultTimezone;
226
+        $timezone = ! empty($type['params'][1]) ? new \DateTimeZone($type['params'][1]) : $this->defaultTimezone;
227 227
         $formats = $this->getDeserializationFormats($type);
228 228
 
229 229
         $formatTried = [];
@@ -248,7 +248,7 @@  discard block
 block discarded – undo
248 248
         throw new RuntimeException(sprintf(
249 249
             'Invalid datetime "%s", expected one of the format %s.',
250 250
             $data,
251
-            '"' . implode('", "', $formatTried) . '"'
251
+            '"'.implode('", "', $formatTried).'"'
252 252
         ));
253 253
     }
254 254
 
@@ -289,15 +289,15 @@  discard block
 block discarded – undo
289 289
         $format = 'P';
290 290
 
291 291
         if (0 < $dateInterval->y) {
292
-            $format .= $dateInterval->y . 'Y';
292
+            $format .= $dateInterval->y.'Y';
293 293
         }
294 294
 
295 295
         if (0 < $dateInterval->m) {
296
-            $format .= $dateInterval->m . 'M';
296
+            $format .= $dateInterval->m.'M';
297 297
         }
298 298
 
299 299
         if (0 < $dateInterval->d) {
300
-            $format .= $dateInterval->d . 'D';
300
+            $format .= $dateInterval->d.'D';
301 301
         }
302 302
 
303 303
         if (0 < $dateInterval->h || 0 < $dateInterval->i || 0 < $dateInterval->s) {
@@ -305,15 +305,15 @@  discard block
 block discarded – undo
305 305
         }
306 306
 
307 307
         if (0 < $dateInterval->h) {
308
-            $format .= $dateInterval->h . 'H';
308
+            $format .= $dateInterval->h.'H';
309 309
         }
310 310
 
311 311
         if (0 < $dateInterval->i) {
312
-            $format .= $dateInterval->i . 'M';
312
+            $format .= $dateInterval->i.'M';
313 313
         }
314 314
 
315 315
         if (0 < $dateInterval->s) {
316
-            $format .= $dateInterval->s . 'S';
316
+            $format .= $dateInterval->s.'S';
317 317
         }
318 318
 
319 319
         if ('P' === $format) {
Please login to merge, or discard this patch.
src/GraphNavigator/DeserializationGraphNavigator.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -171,11 +171,11 @@  discard block
 block discarded – undo
171 171
                 $metadata = $this->metadataFactory->getMetadataForClass($type['name']);
172 172
                 \assert($metadata instanceof ClassMetadata);
173 173
 
174
-                if ($metadata->usingExpression && !$this->expressionExclusionStrategy) {
174
+                if ($metadata->usingExpression && ! $this->expressionExclusionStrategy) {
175 175
                     throw new ExpressionLanguageRequiredException(sprintf('To use conditional exclude/expose in %s you must configure the expression language.', $metadata->name));
176 176
                 }
177 177
 
178
-                if (!empty($metadata->discriminatorMap) && $type['name'] === $metadata->discriminatorBaseClass) {
178
+                if ( ! empty($metadata->discriminatorMap) && $type['name'] === $metadata->discriminatorBaseClass) {
179 179
                     $metadata = $this->resolveMetadata($data, $metadata);
180 180
                 }
181 181
 
@@ -234,7 +234,7 @@  discard block
 block discarded – undo
234 234
     {
235 235
         $typeValue = $this->visitor->visitDiscriminatorMapProperty($data, $metadata);
236 236
 
237
-        if (!isset($metadata->discriminatorMap[$typeValue])) {
237
+        if ( ! isset($metadata->discriminatorMap[$typeValue])) {
238 238
             throw new LogicException(sprintf(
239 239
                 'The type value "%s" does not exist in the discriminator map of class "%s". Available types: %s',
240 240
                 $typeValue,
Please login to merge, or discard this patch.