Passed
Pull Request — master (#1606)
by Ivan
05:28
created
src/Construction/DoctrineObjectConstructor.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
     {
69 69
         $objectManager = $this->managerRegistry->getManagerForClass($metadata->name);
70 70
 
71
-        if (!$objectManager) {
71
+        if ( ! $objectManager) {
72 72
             // No ObjectManager found, proceed with normal deserialization
73 73
             return $this->fallbackConstructor->construct($visitor, $metadata, $data, $type, $context);
74 74
         }
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
         }
83 83
 
84 84
         // Managed entity, check for proxy load
85
-        if (!is_array($data) && !(is_object($data) && 'SimpleXMLElement' === get_class($data))) {
85
+        if ( ! is_array($data) && ! (is_object($data) && 'SimpleXMLElement' === get_class($data))) {
86 86
             \assert($objectManager instanceof EntityManagerInterface || $objectManager instanceof DocumentManagerInterface);
87 87
 
88 88
             // Single identifier, load proxy
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
 
96 96
         foreach ($classMetadata->getIdentifierFieldNames() as $name) {
97 97
             // Avoid calling objectManager->find if some identification properties are excluded
98
-            if (!isset($metadata->propertyMetadata[$name])) {
98
+            if ( ! isset($metadata->propertyMetadata[$name])) {
99 99
                 return $this->fallbackConstructor->construct($visitor, $metadata, $data, $type, $context);
100 100
             }
101 101
 
@@ -106,9 +106,9 @@  discard block
 block discarded – undo
106 106
                 return $this->fallbackConstructor->construct($visitor, $metadata, $data, $type, $context);
107 107
             }
108 108
 
109
-            if (is_array($data) && !array_key_exists($propertyMetadata->serializedName, $data)) {
109
+            if (is_array($data) && ! array_key_exists($propertyMetadata->serializedName, $data)) {
110 110
                 return $this->fallbackConstructor->construct($visitor, $metadata, $data, $type, $context);
111
-            } elseif (is_object($data) && !property_exists($data, $propertyMetadata->serializedName)) {
111
+            } elseif (is_object($data) && ! property_exists($data, $propertyMetadata->serializedName)) {
112 112
                 return $this->fallbackConstructor->construct($visitor, $metadata, $data, $type, $context);
113 113
             }
114 114
 
Please login to merge, or discard this patch.
src/Metadata/Driver/NullDriver.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
     public function loadMetadataForClass(\ReflectionClass $class): ?BaseClassMetadata
26 26
     {
27 27
         $classMetadata = new ClassMetadata($name = $class->name);
28
-        $fileResource =  $class->getFilename();
28
+        $fileResource = $class->getFilename();
29 29
         if (false !== $fileResource) {
30 30
             $classMetadata->fileResources[] = $fileResource;
31 31
         }
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
 
38 38
             $propertyMetadata = new PropertyMetadata($name, $property->getName());
39 39
 
40
-            if (!$propertyMetadata->serializedName) {
40
+            if ( ! $propertyMetadata->serializedName) {
41 41
                 $propertyMetadata->serializedName = $this->namingStrategy->translateName($propertyMetadata);
42 42
             }
43 43
 
Please login to merge, or discard this patch.
src/Metadata/Driver/YamlDriver.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
      */
71 71
     public function getAllClassNames(): array
72 72
     {
73
-        if (!$this->locator instanceof AdvancedFileLocatorInterface) {
73
+        if ( ! $this->locator instanceof AdvancedFileLocatorInterface) {
74 74
             throw new RuntimeException(
75 75
                 sprintf(
76 76
                     'Locator "%s" must be an instance of "AdvancedFileLocatorInterface".',
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
     {
94 94
         $config = Yaml::parseFile($file, Yaml::PARSE_CONSTANT);
95 95
 
96
-        if (!isset($config[$name = $class->name])) {
96
+        if ( ! isset($config[$name = $class->name])) {
97 97
             throw new InvalidMetadataException(
98 98
                 sprintf('Expected metadata for class %s to be defined in %s.', $class->name, $file),
99 99
             );
@@ -130,9 +130,9 @@  discard block
 block discarded – undo
130 130
                     );
131 131
                     unset($propertySettings['exp']);
132 132
                 } else {
133
-                    if (!$class->hasMethod($methodName)) {
133
+                    if ( ! $class->hasMethod($methodName)) {
134 134
                         throw new InvalidMetadataException(
135
-                            'The method ' . $methodName . ' not found in class ' . $class->name,
135
+                            'The method '.$methodName.' not found in class '.$class->name,
136 136
                         );
137 137
                     }
138 138
 
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
             }
145 145
         }
146 146
 
147
-        if (!$excludeAll) {
147
+        if ( ! $excludeAll) {
148 148
             foreach ($class->getProperties() as $property) {
149 149
                 if ($property->class !== $name || (isset($property->info) && $property->info['class'] !== $name)) {
150 150
                     continue;
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
 
153 153
                 $pName = $property->getName();
154 154
                 $propertiesMetadata[] = new PropertyMetadata($name, $pName);
155
-                $propertiesData[] =  !empty($config['properties']) && true === array_key_exists($pName, $config['properties'])
155
+                $propertiesData[] = ! empty($config['properties']) && true === array_key_exists($pName, $config['properties'])
156 156
                     ? (array) $config['properties'][$pName]
157 157
                     : null;
158 158
             }
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
                     || isset($propertiesData[$propertyKey]);
165 165
 
166 166
                 $pConfig = $propertiesData[$propertyKey];
167
-                if (!empty($pConfig)) {
167
+                if ( ! empty($pConfig)) {
168 168
                     if (isset($pConfig['exclude'])) {
169 169
                         $isExclude = (bool) $pConfig['exclude'];
170 170
                     }
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
                     }
195 195
 
196 196
                     if (isset($pConfig['expose_if'])) {
197
-                        $pMetadata->excludeIf = $this->parseExpression('!(' . $pConfig['expose_if'] . ')');
197
+                        $pMetadata->excludeIf = $this->parseExpression('!('.$pConfig['expose_if'].')');
198 198
                     }
199 199
 
200 200
                     if (isset($pConfig['serialized_name'])) {
@@ -302,7 +302,7 @@  discard block
 block discarded – undo
302 302
                     }
303 303
                 }
304 304
 
305
-                if (!$pMetadata->serializedName) {
305
+                if ( ! $pMetadata->serializedName) {
306 306
                     $pMetadata->serializedName = $this->namingStrategy->translateName($pMetadata);
307 307
                 }
308 308
 
@@ -311,12 +311,12 @@  discard block
 block discarded – undo
311 311
                     $metadata->isMap = $metadata->isMap || PropertyMetadata::isCollectionMap($pMetadata->type);
312 312
                 }
313 313
 
314
-                if (!empty($pConfig) && !empty($pConfig['name'])) {
314
+                if ( ! empty($pConfig) && ! empty($pConfig['name'])) {
315 315
                     $pMetadata->name = (string) $pConfig['name'];
316 316
                 }
317 317
 
318 318
                 if (
319
-                    (ExclusionPolicy::NONE === $exclusionPolicy && !$isExclude)
319
+                    (ExclusionPolicy::NONE === $exclusionPolicy && ! $isExclude)
320 320
                     || (ExclusionPolicy::ALL === $exclusionPolicy && $isExpose)
321 321
                 ) {
322 322
                     $metadata->addPropertyMetadata($pMetadata);
@@ -361,7 +361,7 @@  discard block
 block discarded – undo
361 361
 
362 362
     private function addClassProperties(ClassMetadata $metadata, array $config): void
363 363
     {
364
-        if (isset($config['custom_accessor_order']) && !isset($config['accessor_order'])) {
364
+        if (isset($config['custom_accessor_order']) && ! isset($config['accessor_order'])) {
365 365
             $config['accessor_order'] = 'custom';
366 366
         }
367 367
 
@@ -391,11 +391,11 @@  discard block
 block discarded – undo
391 391
             if (isset($config['discriminator']['disabled']) && true === $config['discriminator']['disabled']) {
392 392
                 $metadata->discriminatorDisabled = true;
393 393
             } else {
394
-                if (!isset($config['discriminator']['field_name'])) {
394
+                if ( ! isset($config['discriminator']['field_name'])) {
395 395
                     throw new InvalidMetadataException('The "field_name" attribute must be set for discriminators.');
396 396
                 }
397 397
 
398
-                if (!isset($config['discriminator']['map']) || !is_array($config['discriminator']['map'])) {
398
+                if ( ! isset($config['discriminator']['map']) || ! is_array($config['discriminator']['map'])) {
399 399
                     throw new InvalidMetadataException(
400 400
                         'The "map" attribute must be set, and be an array for discriminators.',
401 401
                     );
@@ -432,7 +432,7 @@  discard block
 block discarded – undo
432 432
     {
433 433
         if (is_string($config)) {
434 434
             $config = [$config];
435
-        } elseif (!is_array($config)) {
435
+        } elseif ( ! is_array($config)) {
436 436
             throw new InvalidMetadataException(
437 437
                 sprintf(
438 438
                     'callback methods expects a string, or an array of strings that represent method names, but got %s.',
@@ -443,7 +443,7 @@  discard block
 block discarded – undo
443 443
 
444 444
         $methods = [];
445 445
         foreach ($config as $name) {
446
-            if (!$class->hasMethod($name)) {
446
+            if ( ! $class->hasMethod($name)) {
447 447
                 throw new InvalidMetadataException(
448 448
                     sprintf('The method %s does not exist in class %s.', $name, $class->name),
449 449
                 );
Please login to merge, or discard this patch.
src/Type/Parser.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
     {
38 38
         $this->lexer->moveNext();
39 39
 
40
-        if (!$this->lexer->token) {
40
+        if ( ! $this->lexer->token) {
41 41
             throw new SyntaxError(
42 42
                 'Syntax error, unexpected end of stream',
43 43
             );
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
             }
60 60
 
61 61
             return $this->visitSimpleType();
62
-        } elseif (!$this->root && Lexer::T_ARRAY_START === $this->lexer->token->type) {
62
+        } elseif ( ! $this->root && Lexer::T_ARRAY_START === $this->lexer->token->type) {
63 63
             return $this->visitArrayType();
64 64
         }
65 65
 
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
         $this->match(Lexer::T_TYPE_START);
88 88
 
89 89
         $params = [];
90
-        if (!$this->lexer->isNextToken(Lexer::T_TYPE_END)) {
90
+        if ( ! $this->lexer->isNextToken(Lexer::T_TYPE_END)) {
91 91
             while (true) {
92 92
                 $params[] = $this->visit();
93 93
 
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
          */
116 116
 
117 117
         $params = [];
118
-        if (!$this->lexer->isNextToken(Lexer::T_ARRAY_END)) {
118
+        if ( ! $this->lexer->isNextToken(Lexer::T_ARRAY_END)) {
119 119
             while (true) {
120 120
                 $params[] = $this->visit();
121 121
                 if ($this->lexer->isNextToken(Lexer::T_ARRAY_END)) {
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
 
134 134
     private function match(int $token): void
135 135
     {
136
-        if (!$this->lexer->lookahead) {
136
+        if ( ! $this->lexer->lookahead) {
137 137
             throw new SyntaxError(
138 138
                 sprintf('Syntax error, unexpected end of stream, expected %s', $this->getConstant($token)),
139 139
             );
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::class,
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
             $methods[] = [
@@ -231,7 +231,7 @@  discard block
 block discarded – undo
231 231
      */
232 232
     private function parseDateTime($data, array $type, bool $immutable = false): \DateTimeInterface
233 233
     {
234
-        $timezone = !empty($type['params'][1]) ? new \DateTimeZone($type['params'][1]) : $this->defaultTimezone;
234
+        $timezone = ! empty($type['params'][1]) ? new \DateTimeZone($type['params'][1]) : $this->defaultTimezone;
235 235
         $formats = $this->getDeserializationFormats($type);
236 236
 
237 237
         $formatTried = [];
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
         throw new RuntimeException(sprintf(
257 257
             'Invalid datetime "%s", expected one of the format %s.',
258 258
             $data,
259
-            '"' . implode('", "', $formatTried) . '"',
259
+            '"'.implode('", "', $formatTried).'"',
260 260
         ));
261 261
     }
262 262
 
@@ -304,15 +304,15 @@  discard block
 block discarded – undo
304 304
         $format = 'P';
305 305
 
306 306
         if (0 < $dateInterval->y) {
307
-            $format .= $dateInterval->y . 'Y';
307
+            $format .= $dateInterval->y.'Y';
308 308
         }
309 309
 
310 310
         if (0 < $dateInterval->m) {
311
-            $format .= $dateInterval->m . 'M';
311
+            $format .= $dateInterval->m.'M';
312 312
         }
313 313
 
314 314
         if (0 < $dateInterval->d) {
315
-            $format .= $dateInterval->d . 'D';
315
+            $format .= $dateInterval->d.'D';
316 316
         }
317 317
 
318 318
         if (0 < $dateInterval->h || 0 < $dateInterval->i || 0 < $dateInterval->s) {
@@ -320,15 +320,15 @@  discard block
 block discarded – undo
320 320
         }
321 321
 
322 322
         if (0 < $dateInterval->h) {
323
-            $format .= $dateInterval->h . 'H';
323
+            $format .= $dateInterval->h.'H';
324 324
         }
325 325
 
326 326
         if (0 < $dateInterval->i) {
327
-            $format .= $dateInterval->i . 'M';
327
+            $format .= $dateInterval->i.'M';
328 328
         }
329 329
 
330 330
         if (0 < $dateInterval->s) {
331
-            $format .= $dateInterval->s . 'S';
331
+            $format .= $dateInterval->s.'S';
332 332
         }
333 333
 
334 334
         if ('P' === $format) {
Please login to merge, or discard this patch.
src/Annotation/Type.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
 
22 22
     public function __construct($values = [], $name = null)
23 23
     {
24
-        if ((null !== $name) && !is_string($name) && !(is_object($name) && method_exists($name, '__toString'))) {
24
+        if ((null !== $name) && ! is_string($name) && ! (is_object($name) && method_exists($name, '__toString'))) {
25 25
             throw new \RuntimeException(
26 26
                 'Type must be either string, null or object implements __toString() method.',
27 27
             );
Please login to merge, or discard this patch.
src/Metadata/Driver/AttributeDriver/AttributeReader.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -72,7 +72,7 @@
 block discarded – undo
72 72
      */
73 73
     private function buildAnnotation(array $attributes): ?SerializerAttribute
74 74
     {
75
-        if (!isset($attributes[0])) {
75
+        if ( ! isset($attributes[0])) {
76 76
             return null;
77 77
         }
78 78
 
Please login to merge, or discard this patch.
src/Metadata/Driver/AnnotationOrAttributeDriver.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
         $configured = false;
82 82
 
83 83
         $classMetadata = new ClassMetadata($name = $class->name);
84
-        $fileResource =  $class->getFilename();
84
+        $fileResource = $class->getFilename();
85 85
 
86 86
         if (false !== $fileResource) {
87 87
             $classMetadata->fileResources[] = $fileResource;
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
             }
168 168
         }
169 169
 
170
-        if (!$excludeAll) {
170
+        if ( ! $excludeAll) {
171 171
             foreach ($class->getProperties() as $property) {
172 172
                 if ($property->class !== $name || (isset($property->info) && $property->info['class'] !== $name)) {
173 173
                     continue;
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
                     } elseif ($annot instanceof Expose) {
202 202
                         $isExpose = true;
203 203
                         if (null !== $annot->if) {
204
-                            $propertyMetadata->excludeIf = $this->parseExpression('!(' . $annot->if . ')');
204
+                            $propertyMetadata->excludeIf = $this->parseExpression('!('.$annot->if.')');
205 205
                         }
206 206
                     } elseif ($annot instanceof Exclude) {
207 207
                         if (null !== $annot->if) {
@@ -248,7 +248,7 @@  discard block
 block discarded – undo
248 248
                                 throw new InvalidMetadataException(sprintf(
249 249
                                     'Invalid group name "%s" on "%s", did you mean to create multiple groups?',
250 250
                                     implode(', ', $propertyMetadata->groups),
251
-                                    $propertyMetadata->class . '->' . $propertyMetadata->name,
251
+                                    $propertyMetadata->class.'->'.$propertyMetadata->name,
252 252
                                 ));
253 253
                             }
254 254
                         }
@@ -270,7 +270,7 @@  discard block
 block discarded – undo
270 270
                     }
271 271
                 }
272 272
 
273
-                if (!$propertyMetadata->serializedName) {
273
+                if ( ! $propertyMetadata->serializedName) {
274 274
                     $propertyMetadata->serializedName = $this->namingStrategy->translateName($propertyMetadata);
275 275
                 }
276 276
 
@@ -281,7 +281,7 @@  discard block
 block discarded – undo
281 281
                 }
282 282
 
283 283
                 if (
284
-                    (ExclusionPolicy::NONE === $exclusionPolicy && !$isExclude)
284
+                    (ExclusionPolicy::NONE === $exclusionPolicy && ! $isExclude)
285 285
                     || (ExclusionPolicy::ALL === $exclusionPolicy && $isExpose)
286 286
                 ) {
287 287
                     $propertyMetadata->setAccessor($accessType, $accessor[0], $accessor[1]);
Please login to merge, or discard this patch.
src/Accessor/DefaultAccessorStrategy.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -132,7 +132,7 @@
 block discarded – undo
132 132
 
133 133
         $accessor = $this->writeAccessors[$metadata->class] ?? null;
134 134
         if (null === $accessor) {
135
-            $accessor = \Closure::bind(static function ($o, $name, $value): void {
135
+            $accessor = \Closure::bind(static function($o, $name, $value): void {
136 136
                 $o->$name = $value;
137 137
             }, null, $metadata->class);
138 138
             $this->writeAccessors[$metadata->class] = $accessor;
Please login to merge, or discard this patch.