schmittjoh / serializer

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);
         }
 

src/XmlSerializationVisitor.php

Spacing +19 added lines, -19 removed lines

@@ -105,7 +105,7 @@ 
 
     public function createRoot(?ClassMetadata $metadata = null, ?string $rootName = null, ?string $rootNamespace = null, ?string $rootPrefix = null): \DOMElement
     {
-        if (null !== $metadata && !empty($metadata->xmlRootName)) {
+        if (null !== $metadata && ! empty($metadata->xmlRootName)) {
             $rootPrefix = $metadata->xmlRootPrefix;
             $rootName = $metadata->xmlRootName;
             $rootNamespace = $metadata->xmlRootNamespace ?: $this->getClassDefaultNamespace($metadata);
@@ -117,7 +117,7 @@ 
 
         $document = $this->getDocument();
         if ($rootNamespace) {
-            $rootNode = $document->createElementNS($rootNamespace, (null !== $rootPrefix ? $rootPrefix . ':' : '') . $rootName);
+            $rootNode = $document->createElementNS($rootNamespace, (null !== $rootPrefix ? $rootPrefix.':' : '').$rootName);
         } else {
             $rootNode = $document->createElement($rootName);
         }
@@ -191,7 +191,7 @@ 
         $decimalsNumbers = $type['params'][2] ?? null;
         if (null === $decimalsNumbers) {
             $parts = explode('.', (string) $dataResult);
-            if (count($parts) < 2 || !$parts[1]) {
+            if (count($parts) < 2 || ! $parts[1]) {
                 $decimalsNumbers = 1;
             }
         }
@@ -266,7 +266,7 @@ 
             $node = $this->navigator->accept($v, $metadata->type);
             $this->revertCurrentMetadata();
 
-            if (!$node instanceof \DOMCharacterData) {
+            if ( ! $node instanceof \DOMCharacterData) {
                 throw new RuntimeException(sprintf('Unsupported value for XML attribute for %s. Expected character data, but got %s.', $metadata->name, json_encode($v)));
             }
 
@@ -277,7 +277,7 @@ 
 
         if (
             ($metadata->xmlValue && $this->currentNode->childNodes->length > 0)
-            || (!$metadata->xmlValue && $this->hasValue)
+            || ( ! $metadata->xmlValue && $this->hasValue)
         ) {
             throw new RuntimeException(sprintf('If you make use of @XmlValue, all other properties in the class must have the @XmlAttribute annotation. Invalid usage detected in class %s.', $metadata->class));
         }
@@ -289,7 +289,7 @@ 
             $node = $this->navigator->accept($v, $metadata->type);
             $this->revertCurrentMetadata();
 
-            if (!$node instanceof \DOMCharacterData) {
+            if ( ! $node instanceof \DOMCharacterData) {
                 throw new RuntimeException(sprintf('Unsupported value for property %s::$%s. Expected character data, but got %s.', $metadata->class, $metadata->name, \is_object($node) ? \get_class($node) : \gettype($node)));
             }
 
@@ -299,7 +299,7 @@ 
         }
 
         if ($metadata->xmlAttributeMap) {
-            if (!\is_array($v)) {
+            if ( ! \is_array($v)) {
                 throw new RuntimeException(sprintf('Unsupported value type for XML attribute map. Expected array but got %s.', \gettype($v)));
             }
 
@@ -308,7 +308,7 @@ 
                 $node = $this->navigator->accept($value, null);
                 $this->revertCurrentMetadata();
 
-                if (!$node instanceof \DOMCharacterData) {
+                if ( ! $node instanceof \DOMCharacterData) {
                     throw new RuntimeException(sprintf('Unsupported value for a XML attribute map value. Expected character data, but got %s.', json_encode($v)));
                 }
 
@@ -340,7 +340,7 @@ 
             return;
         }
 
-        if ($addEnclosingElement = !$this->isInLineCollection($metadata) && !$metadata->inline) {
+        if ($addEnclosingElement = ! $this->isInLineCollection($metadata) && ! $metadata->inline) {
             $namespace = $metadata->xmlNamespace ?? $this->getClassDefaultNamespace($this->objectMetadataStack->top());
 
             $element = $this->createElement($metadata->serializedName, $namespace);
@@ -402,7 +402,7 @@ 
             }
         }
 
-        if (!$targetElement) {
+        if ( ! $targetElement) {
             return false;
         }
 
@@ -452,7 +452,7 @@ 
 
     private function isSkippableEmptyObject(?\DOMElement $node, PropertyMetadata $metadata): bool
     {
-        return null === $node && !$metadata->xmlCollection && $metadata->skipWhenEmpty;
+        return null === $node && ! $metadata->xmlCollection && $metadata->skipWhenEmpty;
     }
 
     private function isSkippableCollection(PropertyMetadata $metadata): bool
@@ -462,7 +462,7 @@ 
 
     private function isElementEmpty(\DOMElement $element): bool
     {
-        return !$element->hasChildNodes() && !$element->hasAttributes();
+        return ! $element->hasChildNodes() && ! $element->hasAttributes();
     }
 
     public function endVisitingObject(ClassMetadata $metadata, object $data, array $type): void
@@ -567,7 +567,7 @@ 
         foreach ($metadata->xmlNamespaces as $prefix => $uri) {
             $attribute = 'xmlns';
             if ('' !== $prefix) {
-                $attribute .= ':' . $prefix;
+                $attribute .= ':'.$prefix;
             } elseif ($element->namespaceURI === $uri) {
                 continue;
             }
@@ -597,21 +597,21 @@ 
             return $this->document->createElementNS($namespace, $tagName);
         }
 
-        if (!($prefix = $this->currentNode->lookupPrefix($namespace)) && !($prefix = $this->document->lookupPrefix($namespace))) {
-            $prefix = 'ns-' . substr(sha1($namespace), 0, 8);
+        if ( ! ($prefix = $this->currentNode->lookupPrefix($namespace)) && ! ($prefix = $this->document->lookupPrefix($namespace))) {
+            $prefix = 'ns-'.substr(sha1($namespace), 0, 8);
         }
 
-        return $this->document->createElementNS($namespace, $prefix . ':' . $tagName);
+        return $this->document->createElementNS($namespace, $prefix.':'.$tagName);
     }
 
     private function setAttributeOnNode(\DOMElement $node, string $name, string $value, ?string $namespace = null): void
     {
         if (null !== $namespace) {
-            if (!$prefix = $node->lookupPrefix($namespace)) {
-                $prefix = 'ns-' . substr(sha1($namespace), 0, 8);
+            if ( ! $prefix = $node->lookupPrefix($namespace)) {
+                $prefix = 'ns-'.substr(sha1($namespace), 0, 8);
             }
 
-            $node->setAttributeNS($namespace, $prefix . ':' . $name, $value);
+            $node->setAttributeNS($namespace, $prefix.':'.$name, $value);
         } else {
             $node->setAttribute($name, $value);
         }