Passed
Pull Request — master (#1446)
by Marcin
07:41
created
src/Exclusion/DepthExclusionStrategy.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
         $relativeDepth = 0;
29 29
 
30 30
         foreach ($context->getMetadataStack() as $metadata) {
31
-            if (!$metadata instanceof PropertyMetadata) {
31
+            if ( ! $metadata instanceof PropertyMetadata) {
32 32
                 continue;
33 33
             }
34 34
 
Please login to merge, or discard this patch.
src/JsonDeserializationStrictVisitor.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
             return null;
50 50
         }
51 51
 
52
-        if (! is_string($data)) {
52
+        if ( ! is_string($data)) {
53 53
             throw NonVisitableTypeException::fromDataAndType($data, $type);
54 54
         }
55 55
 
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
             return null;
66 66
         }
67 67
 
68
-        if (! is_bool($data)) {
68
+        if ( ! is_bool($data)) {
69 69
             throw NonVisitableTypeException::fromDataAndType($data, $type);
70 70
         }
71 71
 
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
             return null;
82 82
         }
83 83
 
84
-        if (! is_int($data)) {
84
+        if ( ! is_int($data)) {
85 85
             throw NonVisitableTypeException::fromDataAndType($data, $type);
86 86
         }
87 87
 
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
             return null;
98 98
         }
99 99
 
100
-        if (! is_float($data)) {
100
+        if ( ! is_float($data)) {
101 101
             throw NonVisitableTypeException::fromDataAndType($data, $type);
102 102
         }
103 103
 
Please login to merge, or discard this patch.
src/Handler/SymfonyUidHandler.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@
 block discarded – undo
70 70
                     'direction' => GraphNavigatorInterface::DIRECTION_DESERIALIZATION,
71 71
                     'format'    => $format,
72 72
                     'type'      => $class,
73
-                    'method'    => 'deserializeUidFrom' . ucfirst($format),
73
+                    'method'    => 'deserializeUidFrom'.ucfirst($format),
74 74
                 ];
75 75
             }
76 76
         }
Please login to merge, or discard this patch.
src/Handler/FormErrorHandler.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
                 'direction' => GraphNavigatorInterface::DIRECTION_SERIALIZATION,
44 44
                 'type' => FormInterface::class,
45 45
                 'format' => $format,
46
-                'method' => 'serializeFormTo' . ucfirst($format),
46
+                'method' => 'serializeFormTo'.ucfirst($format),
47 47
             ];
48 48
             $methods[] = [
49 49
                 'direction' => GraphNavigatorInterface::DIRECTION_SERIALIZATION,
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
 
58 58
     public function __construct(?object $translator = null, string $translationDomain = 'validators')
59 59
     {
60
-        if (null !== $translator && (!$translator instanceof TranslatorInterface && !$translator instanceof TranslatorContract)) {
60
+        if (null !== $translator && ( ! $translator instanceof TranslatorInterface && ! $translator instanceof TranslatorContract)) {
61 61
             throw new \InvalidArgumentException(sprintf(
62 62
                 'The first argument passed to %s must be instance of %s or %s, %s given',
63 63
                 self::class,
Please login to merge, or discard this patch.
src/Metadata/Driver/DoctrineTypeDriver.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -22,8 +22,8 @@  discard block
 block discarded – undo
22 22
     protected function setDiscriminator(DoctrineClassMetadata $doctrineMetadata, ClassMetadata $classMetadata): void
23 23
     {
24 24
         if (
25
-            empty($classMetadata->discriminatorMap) && !$classMetadata->discriminatorDisabled
26
-            && !empty($doctrineMetadata->discriminatorMap) && $doctrineMetadata->isRootEntity()
25
+            empty($classMetadata->discriminatorMap) && ! $classMetadata->discriminatorDisabled
26
+            && ! empty($doctrineMetadata->discriminatorMap) && $doctrineMetadata->isRootEntity()
27 27
         ) {
28 28
             $classMetadata->setDiscriminator(
29 29
                 $doctrineMetadata->discriminatorColumn['name'],
@@ -51,11 +51,11 @@  discard block
 block discarded – undo
51 51
             // For inheritance schemes, we cannot add any type as we would only add the super-type of the hierarchy.
52 52
             // On serialization, this would lead to only the supertype being serialized, and properties of subtypes
53 53
             // being ignored.
54
-            if ($targetMetadata instanceof ClassMetadataInfo && !$targetMetadata->isInheritanceTypeNone()) {
54
+            if ($targetMetadata instanceof ClassMetadataInfo && ! $targetMetadata->isInheritanceTypeNone()) {
55 55
                 return;
56 56
             }
57 57
 
58
-            if (!$doctrineMetadata->isSingleValuedAssociation($propertyName)) {
58
+            if ( ! $doctrineMetadata->isSingleValuedAssociation($propertyName)) {
59 59
                 $targetEntity = sprintf('ArrayCollection<%s>', $targetEntity);
60 60
             }
61 61
 
Please login to merge, or discard this patch.
src/SerializerBuilder.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
 
215 215
     private function getAccessorStrategy(): AccessorStrategyInterface
216 216
     {
217
-        if (!$this->accessorStrategy) {
217
+        if ( ! $this->accessorStrategy) {
218 218
             $this->accessorStrategy = new DefaultAccessorStrategy($this->expressionEvaluator);
219 219
         }
220 220
 
@@ -251,11 +251,11 @@  discard block
 block discarded – undo
251 251
 
252 252
     public function setCacheDir(string $dir): self
253 253
     {
254
-        if (!is_dir($dir)) {
254
+        if ( ! is_dir($dir)) {
255 255
             $this->createDir($dir);
256 256
         }
257 257
 
258
-        if (!is_writable($dir)) {
258
+        if ( ! is_writable($dir)) {
259 259
             throw new InvalidArgumentException(sprintf('The cache directory "%s" is not writable.', $dir));
260 260
         }
261 261
 
@@ -377,7 +377,7 @@  discard block
 block discarded – undo
377 377
     public function setMetadataDirs(array $namespacePrefixToDirMap): self
378 378
     {
379 379
         foreach ($namespacePrefixToDirMap as $dir) {
380
-            if (!is_dir($dir)) {
380
+            if ( ! is_dir($dir)) {
381 381
                 throw new InvalidArgumentException(sprintf('The directory "%s" does not exist.', $dir));
382 382
             }
383 383
         }
@@ -415,7 +415,7 @@  discard block
 block discarded – undo
415 415
      */
416 416
     public function addMetadataDir(string $dir, string $namespacePrefix = ''): self
417 417
     {
418
-        if (!is_dir($dir)) {
418
+        if ( ! is_dir($dir)) {
419 419
             throw new InvalidArgumentException(sprintf('The directory "%s" does not exist.', $dir));
420 420
         }
421 421
 
@@ -454,11 +454,11 @@  discard block
 block discarded – undo
454 454
      */
455 455
     public function replaceMetadataDir(string $dir, string $namespacePrefix = ''): self
456 456
     {
457
-        if (!is_dir($dir)) {
457
+        if ( ! is_dir($dir)) {
458 458
             throw new InvalidArgumentException(sprintf('The directory "%s" does not exist.', $dir));
459 459
         }
460 460
 
461
-        if (!isset($this->metadataDirs[$namespacePrefix])) {
461
+        if ( ! isset($this->metadataDirs[$namespacePrefix])) {
462 462
             throw new InvalidArgumentException(sprintf('There is no directory configured for namespace prefix "%s". Please use addMetadataDir() for adding new directories.', $namespacePrefix));
463 463
         }
464 464
 
@@ -553,23 +553,23 @@  discard block
 block discarded – undo
553 553
         if (null !== $this->metadataCache) {
554 554
             $metadataFactory->setCache($this->metadataCache);
555 555
         } elseif (null !== $this->cacheDir) {
556
-            $this->createDir($this->cacheDir . '/metadata');
557
-            $metadataFactory->setCache(new FileCache($this->cacheDir . '/metadata'));
556
+            $this->createDir($this->cacheDir.'/metadata');
557
+            $metadataFactory->setCache(new FileCache($this->cacheDir.'/metadata'));
558 558
         }
559 559
 
560
-        if (!$this->handlersConfigured) {
560
+        if ( ! $this->handlersConfigured) {
561 561
             $this->addDefaultHandlers();
562 562
         }
563 563
 
564
-        if (!$this->listenersConfigured) {
564
+        if ( ! $this->listenersConfigured) {
565 565
             $this->addDefaultListeners();
566 566
         }
567 567
 
568
-        if (!$this->serializationVisitorsAdded) {
568
+        if ( ! $this->serializationVisitorsAdded) {
569 569
             $this->addDefaultSerializationVisitors();
570 570
         }
571 571
 
572
-        if (!$this->deserializationVisitorsAdded) {
572
+        if ( ! $this->deserializationVisitorsAdded) {
573 573
             $this->addDefaultDeserializationVisitors();
574 574
         }
575 575
 
@@ -635,12 +635,12 @@  discard block
 block discarded – undo
635 635
     private function decorateAnnotationReader(Reader $annotationReader): Reader
636 636
     {
637 637
         if (null !== $this->cacheDir) {
638
-            $this->createDir($this->cacheDir . '/annotations');
638
+            $this->createDir($this->cacheDir.'/annotations');
639 639
             if (class_exists(FilesystemAdapter::class)) {
640
-                $annotationsCache = new FilesystemAdapter('', 0, $this->cacheDir . '/annotations');
640
+                $annotationsCache = new FilesystemAdapter('', 0, $this->cacheDir.'/annotations');
641 641
                 $annotationReader = new PsrCachedReader($annotationReader, $annotationsCache, $this->debug);
642 642
             } elseif (class_exists(FilesystemCache::class) && class_exists(CachedReader::class)) {
643
-                $annotationsCache = new FilesystemCache($this->cacheDir . '/annotations');
643
+                $annotationsCache = new FilesystemCache($this->cacheDir.'/annotations');
644 644
                 $annotationReader = new CachedReader($annotationReader, $annotationsCache, $this->debug);
645 645
             }
646 646
         }
Please login to merge, or discard this patch.
src/Metadata/Driver/EnumPropertiesDriver.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@
 block discarded – undo
74 74
     {
75 75
         $reflectionType = $propertyReflection->getType();
76 76
 
77
-        if (!($reflectionType instanceof \ReflectionNamedType)) {
77
+        if ( ! ($reflectionType instanceof \ReflectionNamedType)) {
78 78
             return null;
79 79
         }
80 80
 
Please login to merge, or discard this patch.
src/Handler/EnumHandler.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
         SerializationContext $context
46 46
     ) {
47 47
         if (isset($type['params'][1]) && 'value' === $type['params'][1]) {
48
-            if (!$enum instanceof \BackedEnum) {
48
+            if ( ! $enum instanceof \BackedEnum) {
49 49
                 throw new InvalidMetadataException(sprintf('The type "%s" is not a backed enum, thus you can not use "value" as serialization mode for its value.', get_class($enum)));
50 50
             }
51 51
 
@@ -67,13 +67,13 @@  discard block
 block discarded – undo
67 67
         $caseValue = (string) $data;
68 68
 
69 69
         $ref = new \ReflectionEnum($enumType);
70
-        if (isset($type['params'][1]) && 'value' === $type['params'][1] || (!isset($type['params'][1]) && is_a($enumType, \BackedEnum::class, true))) {
71
-            if (!is_a($enumType, \BackedEnum::class, true)) {
70
+        if (isset($type['params'][1]) && 'value' === $type['params'][1] || ( ! isset($type['params'][1]) && is_a($enumType, \BackedEnum::class, true))) {
71
+            if ( ! is_a($enumType, \BackedEnum::class, true)) {
72 72
                 throw new InvalidMetadataException(sprintf('The type "%s" is not a backed enum, thus you can not use "value" as serialization mode for its value.', $enumType));
73 73
             }
74 74
 
75 75
             if ('int' === $ref->getBackingType()->getName()) {
76
-                if (!is_numeric($caseValue)) {
76
+                if ( ! is_numeric($caseValue)) {
77 77
                     throw new RuntimeException(sprintf('"%s" is not a valid backing value for enum "%s"', $caseValue, $enumType));
78 78
                 }
79 79
 
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 
83 83
             return $enumType::from($caseValue);
84 84
         } else {
85
-            if (!$ref->hasCase($caseValue)) {
85
+            if ( ! $ref->hasCase($caseValue)) {
86 86
                 throw new InvalidMetadataException(sprintf('The type "%s" does not have the case "%s"', $ref->getName(), $caseValue));
87 87
             }
88 88
 
Please login to merge, or discard this patch.
src/Metadata/Driver/DocBlockDriver/DocBlockTypeResolver.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -84,25 +84,25 @@  discard block
 block discarded – undo
84 84
         if ($type instanceof ArrayTypeNode) {
85 85
             $resolvedType = $this->resolveTypeFromTypeNode($type->type, $reflectionProperty);
86 86
 
87
-            return 'array<' . $resolvedType . '>';
87
+            return 'array<'.$resolvedType.'>';
88 88
         }
89 89
 
90 90
         // Generic array syntax: array<Product> | array<\Foo\Bar\Product> | array<int,Product>
91 91
         if ($type instanceof GenericTypeNode) {
92 92
             if ($this->isSimpleType($type->type, 'array')) {
93
-                $resolvedTypes = array_map(function (TypeNode $node) use ($reflectionProperty) {
93
+                $resolvedTypes = array_map(function(TypeNode $node) use ($reflectionProperty) {
94 94
                     return $this->resolveTypeFromTypeNode($node, $reflectionProperty);
95 95
                 }, $type->genericTypes);
96 96
 
97
-                return 'array<' . implode(',', $resolvedTypes) . '>';
97
+                return 'array<'.implode(',', $resolvedTypes).'>';
98 98
             }
99 99
 
100 100
             if ($this->isSimpleType($type->type, 'list')) {
101
-                $resolvedTypes = array_map(function (TypeNode $node) use ($reflectionProperty) {
101
+                $resolvedTypes = array_map(function(TypeNode $node) use ($reflectionProperty) {
102 102
                     return $this->resolveTypeFromTypeNode($node, $reflectionProperty);
103 103
                 }, $type->genericTypes);
104 104
 
105
-                return 'array<int, ' . implode(',', $resolvedTypes) . '>';
105
+                return 'array<int, '.implode(',', $resolvedTypes).'>';
106 106
             }
107 107
 
108 108
             throw new \InvalidArgumentException(sprintf("Can't use non-array generic type %s for collection in %s:%s", (string) $type->type, $reflectionProperty->getDeclaringClass()->getName(), $reflectionProperty->getName()));
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
             return [];
126 126
         }
127 127
 
128
-        return array_merge(...array_map(static function (VarTagValueNode $node) {
128
+        return array_merge(...array_map(static function(VarTagValueNode $node) {
129 129
             if ($node->type instanceof UnionTypeNode) {
130 130
                 return $node->type->types;
131 131
             }
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
      */
170 170
     private function filterNullFromTypes(array $types): array
171 171
     {
172
-        return array_values(array_filter(array_map(function (TypeNode $node) {
172
+        return array_values(array_filter(array_map(function(TypeNode $node) {
173 173
             return $this->isNullType($node) ? null : $node;
174 174
         }, $types)));
175 175
     }
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
      */
213 213
     private function resolveTypeFromTypeNode(TypeNode $typeNode, \ReflectionProperty $reflectionProperty): string
214 214
     {
215
-        if (!($typeNode instanceof IdentifierTypeNode)) {
215
+        if ( ! ($typeNode instanceof IdentifierTypeNode)) {
216 216
             throw new \InvalidArgumentException(sprintf("Can't use unsupported type %s for collection in %s:%s", (string) $typeNode, $reflectionProperty->getDeclaringClass()->getName(), $reflectionProperty->getName()));
217 217
         }
218 218
 
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
             return $typeHint;
226 226
         }
227 227
 
228
-        $expandedClassName = $declaringClass->getNamespaceName() . '\\' . $typeHint;
228
+        $expandedClassName = $declaringClass->getNamespaceName().'\\'.$typeHint;
229 229
         if ($this->isClassOrInterface($expandedClassName)) {
230 230
             return $expandedClassName;
231 231
         }
@@ -259,7 +259,7 @@  discard block
 block discarded – undo
259 259
 
260 260
     private function endsWith(string $statementClassToCheck, string $typeHintToSearchFor): bool
261 261
     {
262
-        $typeHintToSearchFor = '\\' . $typeHintToSearchFor;
262
+        $typeHintToSearchFor = '\\'.$typeHintToSearchFor;
263 263
 
264 264
         return substr($statementClassToCheck, -strlen($typeHintToSearchFor)) === $typeHintToSearchFor;
265 265
     }
@@ -280,7 +280,7 @@  discard block
 block discarded – undo
280 280
         preg_match_all(self::GROUP_USE_STATEMENTS_REGEX, $classContents, $foundGroupUseStatements);
281 281
         for ($useStatementIndex = 0; $useStatementIndex < count($foundGroupUseStatements[0]); $useStatementIndex++) {
282 282
             foreach (explode(',', $foundGroupUseStatements[2][$useStatementIndex]) as $singleUseStatement) {
283
-                $foundUseStatements[] = trim($foundGroupUseStatements[1][$useStatementIndex]) . trim($singleUseStatement);
283
+                $foundUseStatements[] = trim($foundGroupUseStatements[1][$useStatementIndex]).trim($singleUseStatement);
284 284
             }
285 285
         }
286 286
 
@@ -313,7 +313,7 @@  discard block
 block discarded – undo
313 313
 
314 314
     private function resolveType(string $typeHint, \ReflectionProperty $reflectionProperty): string
315 315
     {
316
-        if (!$this->hasGlobalNamespacePrefix($typeHint) && !$this->isPrimitiveType($typeHint)) {
316
+        if ( ! $this->hasGlobalNamespacePrefix($typeHint) && ! $this->isPrimitiveType($typeHint)) {
317 317
             $typeHint = $this->expandClassNameUsingUseStatements($typeHint, $this->getDeclaringClassOrTrait($reflectionProperty), $reflectionProperty);
318 318
         }
319 319
 
@@ -328,7 +328,7 @@  discard block
 block discarded – undo
328 328
     private function resolveTypeFromDocblock(\ReflectionProperty $reflectionProperty): ?array
329 329
     {
330 330
         $docComment = $reflectionProperty->getDocComment();
331
-        if (!$docComment && PHP_VERSION_ID >= 80000 && $reflectionProperty->isPromoted()) {
331
+        if ( ! $docComment && PHP_VERSION_ID >= 80000 && $reflectionProperty->isPromoted()) {
332 332
             $docComment = $reflectionProperty->getDeclaringClass()->getConstructor()->getDocComment();
333 333
 
334 334
             $tokens = $this->lexer->tokenize($docComment);
@@ -337,7 +337,7 @@  discard block
 block discarded – undo
337 337
             return $this->flattenParamTagValueTypes($reflectionProperty->getName(), $phpDocNode->getParamTagValues());
338 338
         }
339 339
 
340
-        if (!$docComment) {
340
+        if ( ! $docComment) {
341 341
             return null;
342 342
         }
343 343
 
@@ -372,7 +372,7 @@  discard block
 block discarded – undo
372 372
 
373 373
                     return sprintf('array<%s>', implode(
374 374
                         ',',
375
-                        array_map(static function (string $type) use ($reflectionProperty, $self) {
375
+                        array_map(static function(string $type) use ($reflectionProperty, $self) {
376 376
                             return $self->resolveType(trim($type), $reflectionProperty);
377 377
                         }, $types)
378 378
                     ));
Please login to merge, or discard this patch.