schmittjoh / serializer

src/Builder/CallbackDriverFactory.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     public function createDriver(array $metadataDirs, ?Reader $reader = null): DriverInterface
     {
         $driver = \call_user_func($this->callback, $metadataDirs, $reader);
-        if (!$driver instanceof DriverInterface) {
+        if ( ! $driver instanceof DriverInterface) {
             throw new LogicException('The callback must return an instance of DriverInterface.');
         }
 

src/Context.php

Spacing +5 added lines, -5 removed lines

@@ -78,7 +78,7 @@ 
             $this->addExclusionStrategy(new VersionExclusionStrategy($this->attributes['version']));
         }
 
-        if (!empty($this->attributes['max_depth_checks'])) {
+        if ( ! empty($this->attributes['max_depth_checks'])) {
             $this->addExclusionStrategy(new DepthExclusionStrategy());
         }
 
@@ -133,7 +133,7 @@ 
 
     final protected function assertMutable(): void
     {
-        if (!$this->initialized) {
+        if ( ! $this->initialized) {
             return;
         }
 
@@ -222,7 +222,7 @@ 
     {
         $metadata = $this->metadataStack->pop();
 
-        if (!$metadata instanceof PropertyMetadata) {
+        if ( ! $metadata instanceof PropertyMetadata) {
             throw new RuntimeException('Context metadataStack not working well');
         }
     }
@@ -231,7 +231,7 @@ 
     {
         $metadata = $this->metadataStack->pop();
 
-        if (!$metadata instanceof ClassMetadata) {
+        if ( ! $metadata instanceof ClassMetadata) {
             throw new RuntimeException('Context metadataStack not working well');
         }
     }
@@ -243,7 +243,7 @@ 
 
     public function getCurrentPath(): array
     {
-        if (!$this->metadataStack) {
+        if ( ! $this->metadataStack) {
             return [];
         }
 

src/Metadata/Driver/TypedPropertiesDriver.php

Spacing +3 added lines, -3 removed lines

@@ -59,7 +59,7 @@ 
      */
     private function reorderTypes(array $types): array
     {
-        uasort($types, static function ($a, $b) {
+        uasort($types, static function($a, $b) {
             $order = ['null' => 0, 'true' => 1, 'false' => 2, 'bool' => 3, 'int' => 4, 'float' => 5, 'array' => 6, 'string' => 7];
 
             return ($order[$a['name']] ?? 8) <=> ($order[$b['name']] ?? 8);
@@ -159,7 +159,7 @@ 
      */
     private function shouldTypeHint(?ReflectionType $reflectionType): bool
     {
-        if (!$reflectionType instanceof ReflectionNamedType) {
+        if ( ! $reflectionType instanceof ReflectionNamedType) {
             return false;
         }
 
@@ -176,7 +176,7 @@ 
      */
     private function shouldTypeHintUnion(?ReflectionType $reflectionType)
     {
-        if (!$reflectionType instanceof \ReflectionUnionType) {
+        if ( ! $reflectionType instanceof \ReflectionUnionType) {
             return false;
         }
 

src/Handler/UnionHandler.php

Spacing +2 added lines, -2 removed lines

@@ -144,7 +144,7 @@ 
 
             case 'bool':
             case 'boolean':
-                return !is_array($data) && (string) (bool) $data === (string) $data;
+                return ! is_array($data) && (string) (bool) $data === (string) $data;
 
             case 'true':
                 return true === $data;
@@ -153,7 +153,7 @@ 
                 return false === $data;
 
             case 'string':
-                return !is_array($data) && !is_object($data);
+                return ! is_array($data) && ! is_object($data);
         }
 
         return false;

src/Handler/ConstraintViolationHandler.php

Spacing +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
                     'direction' => GraphNavigatorInterface::DIRECTION_SERIALIZATION,
                     'type' => $type,
                     'format' => $format,
-                    'method' => $method . 'To' . $format,
+                    'method' => $method.'To'.$format,
                 ];
             }
         }
@@ -43,7 +43,7 @@ 
     public function serializeListToXml(XmlSerializationVisitor $visitor, ConstraintViolationList $list): void
     {
         $currentNode = $visitor->getCurrentNode();
-        if (!$currentNode) {
+        if ( ! $currentNode) {
             $visitor->createRoot();
         }
 
@@ -67,7 +67,7 @@ 
         $violationNode = $visitor->getDocument()->createElement('violation');
 
         $parent = $visitor->getCurrentNode();
-        if (!$parent) {
+        if ( ! $parent) {
             $visitor->setCurrentAndRootNode($violationNode);
         } else {
             $parent->appendChild($violationNode);

src/Metadata/PropertyMetadata.php

Spacing +12 added lines, -12 removed lines

@@ -176,22 +176,22 @@ 
             $class = $this->getReflection()->getDeclaringClass();
 
             if (empty($getter)) {
-                if ($class->hasMethod('get' . $this->name) && $class->getMethod('get' . $this->name)->isPublic()) {
-                    $getter = 'get' . $this->name;
-                } elseif ($class->hasMethod('is' . $this->name) && $class->getMethod('is' . $this->name)->isPublic()) {
-                    $getter = 'is' . $this->name;
-                } elseif ($class->hasMethod('has' . $this->name) && $class->getMethod('has' . $this->name)->isPublic()) {
-                    $getter = 'has' . $this->name;
+                if ($class->hasMethod('get'.$this->name) && $class->getMethod('get'.$this->name)->isPublic()) {
+                    $getter = 'get'.$this->name;
+                } elseif ($class->hasMethod('is'.$this->name) && $class->getMethod('is'.$this->name)->isPublic()) {
+                    $getter = 'is'.$this->name;
+                } elseif ($class->hasMethod('has'.$this->name) && $class->getMethod('has'.$this->name)->isPublic()) {
+                    $getter = 'has'.$this->name;
                 } else {
-                    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));
+                    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));
                 }
             }
 
-            if (empty($setter) && !$this->readOnly) {
-                if ($class->hasMethod('set' . $this->name) && $class->getMethod('set' . $this->name)->isPublic()) {
-                    $setter = 'set' . $this->name;
+            if (empty($setter) && ! $this->readOnly) {
+                if ($class->hasMethod('set'.$this->name) && $class->getMethod('set'.$this->name)->isPublic()) {
+                    $setter = 'set'.$this->name;
                 } else {
-                    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));
+                    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));
                 }
             }
         }
@@ -216,7 +216,7 @@ 
         return is_array($type)
             && 'array' === $type['name']
             && isset($type['params'][0])
-            && !isset($type['params'][1]);
+            && ! isset($type['params'][1]);
     }
 
     /**

src/Metadata/Driver/DocBlockDriver/DocBlockTypeResolver.php

Spacing +13 added lines, -13 removed lines

@@ -126,7 +126,7 @@ 
         if ($type instanceof ArrayTypeNode) {
             $resolvedType = $this->resolveTypeFromTypeNode($type->type, $reflector);
 
-            return 'array<' . $resolvedType . '>';
+            return 'array<'.$resolvedType.'>';
         }
 
         // Generic array syntax: array<Product> | array<\Foo\Bar\Product> | array<int,Product>
@@ -134,13 +134,13 @@ 
             if ($this->isSimpleType($type->type, 'array')) {
                 $resolvedTypes = array_map(fn (TypeNode $node) => $this->resolveTypeFromTypeNode($node, $reflector), $type->genericTypes);
 
-                return 'array<' . implode(',', $resolvedTypes) . '>';
+                return 'array<'.implode(',', $resolvedTypes).'>';
             }
 
             if ($this->isSimpleType($type->type, 'list')) {
                 $resolvedTypes = array_map(fn (TypeNode $node) => $this->resolveTypeFromTypeNode($node, $reflector), $type->genericTypes);
 
-                return 'list<' . implode(',', $resolvedTypes) . '>';
+                return 'list<'.implode(',', $resolvedTypes).'>';
             }
 
             throw new \InvalidArgumentException(sprintf("Can't use non-array generic type %s for collection in %s:%s", (string) $type->type, $reflector->getDeclaringClass()->getName(), $reflector->getName()));
@@ -163,7 +163,7 @@ 
             return [];
         }
 
-        return array_merge(...array_map(static function ($node) {
+        return array_merge(...array_map(static function($node) {
             if ($node->type instanceof UnionTypeNode) {
                 return $node->type->types;
             }
@@ -248,7 +248,7 @@ 
      */
     private function resolveTypeFromTypeNode(TypeNode $typeNode, $reflector): string
     {
-        if (!($typeNode instanceof IdentifierTypeNode)) {
+        if ( ! ($typeNode instanceof IdentifierTypeNode)) {
             throw new \InvalidArgumentException(sprintf("Can't use unsupported type %s for collection in %s:%s", (string) $typeNode, $reflector->getDeclaringClass()->getName(), $reflector->getName()));
         }
 
@@ -260,7 +260,7 @@ 
      */
     private function expandClassNameUsingUseStatements(string $typeHint, \ReflectionClass $declaringClass, $reflector): string
     {
-        $expandedClassName = $declaringClass->getNamespaceName() . '\\' . $typeHint;
+        $expandedClassName = $declaringClass->getNamespaceName().'\\'.$typeHint;
         if ($this->isClassOrInterface($expandedClassName)) {
             return $expandedClassName;
         }
@@ -298,7 +298,7 @@ 
 
     private function endsWith(string $statementClassToCheck, string $typeHintToSearchFor): bool
     {
-        $typeHintToSearchFor = '\\' . $typeHintToSearchFor;
+        $typeHintToSearchFor = '\\'.$typeHintToSearchFor;
 
         return substr($statementClassToCheck, -strlen($typeHintToSearchFor)) === $typeHintToSearchFor;
     }
@@ -319,7 +319,7 @@ 
         preg_match_all(self::GROUP_USE_STATEMENTS_REGEX, $classContents, $foundGroupUseStatements);
         for ($useStatementIndex = 0; $useStatementIndex < count($foundGroupUseStatements[0]); $useStatementIndex++) {
             foreach (explode(',', $foundGroupUseStatements[2][$useStatementIndex]) as $singleUseStatement) {
-                $foundUseStatements[] = trim($foundGroupUseStatements[1][$useStatementIndex]) . trim($singleUseStatement);
+                $foundUseStatements[] = trim($foundGroupUseStatements[1][$useStatementIndex]).trim($singleUseStatement);
             }
         }
 
@@ -358,7 +358,7 @@ 
      */
     private function resolveType(string $typeHint, $reflector): string
     {
-        if (!$this->hasGlobalNamespacePrefix($typeHint) && !$this->isPrimitiveType($typeHint)) {
+        if ( ! $this->hasGlobalNamespacePrefix($typeHint) && ! $this->isPrimitiveType($typeHint)) {
             $typeHint = $this->expandClassNameUsingUseStatements($typeHint, $this->getDeclaringClassOrTrait($reflector), $reflector);
         }
 
@@ -376,15 +376,15 @@ 
     private function resolveTypeFromDocblock($reflector): array
     {
         $docComment = $reflector->getDocComment();
-        if (!$docComment && PHP_VERSION_ID >= 80000 && $reflector instanceof \ReflectionProperty && $reflector->isPromoted()) {
+        if ( ! $docComment && PHP_VERSION_ID >= 80000 && $reflector instanceof \ReflectionProperty && $reflector->isPromoted()) {
             $constructor = $reflector->getDeclaringClass()->getConstructor();
-            if (!$constructor) {
+            if ( ! $constructor) {
                 return [];
             }
 
             $docComment = $constructor->getDocComment();
 
-            if (!$docComment) {
+            if ( ! $docComment) {
                 return [];
             }
 
@@ -394,7 +394,7 @@ 
             return $this->flattenParamTagValueTypes($reflector->getName(), $phpDocNode->getParamTagValues());
         }
 
-        if (!$docComment) {
+        if ( ! $docComment) {
             return [];
         }
 

src/Metadata/Driver/DoctrineTypeDriver.php

Spacing +4 added lines, -4 removed lines

@@ -21,8 +21,8 @@ 
     {
         assert($doctrineMetadata instanceof ORMClassMetadata || $doctrineMetadata instanceof ODMClassMetadata);
         if (
-            empty($classMetadata->discriminatorMap) && !$classMetadata->discriminatorDisabled
-            && !empty($doctrineMetadata->discriminatorMap) && $doctrineMetadata->isRootEntity()
+            empty($classMetadata->discriminatorMap) && ! $classMetadata->discriminatorDisabled
+            && ! empty($doctrineMetadata->discriminatorMap) && $doctrineMetadata->isRootEntity()
         ) {
             if ($doctrineMetadata->discriminatorColumn instanceof DiscriminatorColumnMapping) {
                 // Doctrine 3.1+
@@ -59,11 +59,11 @@ 
             // For inheritance schemes, we cannot add any type as we would only add the super-type of the hierarchy.
             // On serialization, this would lead to only the supertype being serialized, and properties of subtypes
             // being ignored.
-            if ($targetMetadata instanceof ODMClassMetadata && !$targetMetadata->isInheritanceTypeNone()) {
+            if ($targetMetadata instanceof ODMClassMetadata && ! $targetMetadata->isInheritanceTypeNone()) {
                 return;
             }
 
-            if (!$doctrineMetadata->isSingleValuedAssociation($propertyName)) {
+            if ( ! $doctrineMetadata->isSingleValuedAssociation($propertyName)) {
                 $targetEntity = sprintf('ArrayCollection<%s>', $targetEntity);
             }
 

src/XmlDeserializationVisitor.php

Spacing +19 added lines, -19 removed lines

@@ -85,7 +85,7 @@ 
 
         if (false !== stripos($data, '<!doctype')) {
             $internalSubset = $this->getDomDocumentTypeEntitySubset($data);
-            if (!in_array($internalSubset, $this->doctypeAllowList, true)) {
+            if ( ! in_array($internalSubset, $this->doctypeAllowList, true)) {
                 throw new InvalidArgumentException(sprintf(
                     'The document type "%s" is not allowed. If it is safe, you may add it to the allowlist configuration.',
                     $internalSubset,
@@ -219,7 +219,7 @@ 
             $nodes = $data->xpath($entryName);
         }
 
-        if (null === $nodes || !\count($nodes)) {
+        if (null === $nodes || ! \count($nodes)) {
             return [];
         }
 
@@ -247,7 +247,7 @@ 
                 $nodes = $data->children($namespace)->$entryName;
                 foreach ($nodes as $v) {
                     $attrs = $v->attributes();
-                    if (!isset($attrs[$this->currentMetadata->xmlKeyAttribute])) {
+                    if ( ! isset($attrs[$this->currentMetadata->xmlKeyAttribute])) {
                         throw new RuntimeException(sprintf('The key attribute "%s" must be set for each entry of the map.', $this->currentMetadata->xmlKeyAttribute));
                     }
 
@@ -277,7 +277,7 @@ 
                 return (string) $data->attributes($metadata->xmlDiscriminatorNamespace)->{$metadata->discriminatorFieldName};
 
             // Check XML element with namespace for discriminatorFieldName
-            case !$metadata->xmlDiscriminatorAttribute && null !== $metadata->xmlDiscriminatorNamespace && isset($data->children($metadata->xmlDiscriminatorNamespace)->{$metadata->discriminatorFieldName}):
+            case ! $metadata->xmlDiscriminatorAttribute && null !== $metadata->xmlDiscriminatorNamespace && isset($data->children($metadata->xmlDiscriminatorNamespace)->{$metadata->discriminatorFieldName}):
                 return (string) $data->children($metadata->xmlDiscriminatorNamespace)->{$metadata->discriminatorFieldName};
 
             // Check XML element for discriminatorFieldName
@@ -307,7 +307,7 @@ 
         $name = $metadata->serializedName;
 
         if (true === $metadata->inline) {
-            if (!$metadata->type) {
+            if ( ! $metadata->type) {
                 throw RuntimeException::noMetadataForProperty($metadata->class, $metadata->name);
             }
 
@@ -317,7 +317,7 @@ 
         if ($metadata->xmlAttribute) {
             $attributes = $data->attributes($metadata->xmlNamespace);
             if (isset($attributes[$name])) {
-                if (!$metadata->type) {
+                if ( ! $metadata->type) {
                     throw RuntimeException::noMetadataForProperty($metadata->class, $metadata->name);
                 }
 
@@ -332,7 +332,7 @@ 
             $attributes = $data->attributes($metadata->xmlNamespace);
 
             if (isset($attributes[$attributeName])) {
-                if (!$metadata->type) {
+                if ( ! $metadata->type) {
                     throw RuntimeException::noMetadataForProperty($metadata->class, $metadata->name);
                 }
 
@@ -348,9 +348,9 @@ 
             $childDataNode = null;
             if ('' === $metadata->xmlNamespace) {
                 // Element explicitly in NO namespace
-                $xpathQuery = "./*[local-name()='" . $elementName . "' and (namespace-uri()='' or not(namespace-uri()))]";
+                $xpathQuery = "./*[local-name()='".$elementName."' and (namespace-uri()='' or not(namespace-uri()))]";
                 $matchingNodes = $data->xpath($xpathQuery);
-                if (!empty($matchingNodes)) {
+                if ( ! empty($matchingNodes)) {
                     $childDataNode = $matchingNodes[0];
                 }
             } elseif ($metadata->xmlNamespace) {
@@ -367,7 +367,7 @@ 
                 }
             }
 
-            if (!$childDataNode || !$childDataNode->getName()) {
+            if ( ! $childDataNode || ! $childDataNode->getName()) {
                 if (null === $metadata->xmlNamespace) {
                     $ns = '[default/none]';
                 } else {
@@ -381,7 +381,7 @@ 
             $attributes = $childDataNode->attributes($attributeTargetNs);
 
             if (isset($attributes[$attributeName])) {
-                if (!$metadata->type) {
+                if ( ! $metadata->type) {
                     throw RuntimeException::noMetadataForProperty($metadata->class, $metadata->name);
                 }
 
@@ -392,7 +392,7 @@ 
         }
 
         if ($metadata->xmlValue) {
-            if (!$metadata->type) {
+            if ( ! $metadata->type) {
                 throw RuntimeException::noMetadataForProperty($metadata->class, $metadata->name);
             }
 
@@ -401,12 +401,12 @@ 
 
         if ($metadata->xmlCollection) {
             $enclosingElem = $data;
-            if (!$metadata->xmlCollectionInline) {
+            if ( ! $metadata->xmlCollectionInline) {
                 $enclosingElem = $data->children($metadata->xmlNamespace)->$name;
             }
 
             $this->setCurrentMetadata($metadata);
-            if (!$metadata->type) {
+            if ( ! $metadata->type) {
                 throw RuntimeException::noMetadataForProperty($metadata->class, $metadata->name);
             }
 
@@ -418,13 +418,13 @@ 
 
         if ($metadata->xmlNamespace) {
             $node = $data->children($metadata->xmlNamespace)->$name;
-            if (!$node->count()) {
+            if ( ! $node->count()) {
                 throw new NotAcceptableException();
             }
         } elseif ('' === $metadata->xmlNamespace) {
             // See #1087 - element must be like: <element xmlns="" /> - https://www.w3.org/TR/REC-xml-names/#iri-use
             // Use of an empty string in a namespace declaration turns it into an "undeclaration".
-            $nodes = $data->xpath('./' . $name);
+            $nodes = $data->xpath('./'.$name);
             if (empty($nodes)) {
                 throw new NotAcceptableException();
             }
@@ -435,9 +435,9 @@ 
             if (isset($namespaces[''])) {
                 $prefix = uniqid('ns-');
                 $data->registerXPathNamespace($prefix, $namespaces['']);
-                $nodes = $data->xpath('./' . $prefix . ':' . $name);
+                $nodes = $data->xpath('./'.$prefix.':'.$name);
             } else {
-                $nodes = $data->xpath('./' . $name);
+                $nodes = $data->xpath('./'.$name);
             }
 
             if (empty($nodes)) {
@@ -451,7 +451,7 @@ 
             $this->setCurrentMetadata($metadata);
         }
 
-        if (!$metadata->type) {
+        if ( ! $metadata->type) {
             throw RuntimeException::noMetadataForProperty($metadata->class, $metadata->name);
         }