schmittjoh / serializer

src/Metadata/Driver/DocBlockTypeResolver.php

Spacing +13 added lines, -13 removed lines

@@ -53,7 +53,7 @@ 
      */
     public function getPropertyDocblockTypeHint(\ReflectionProperty $reflectionProperty): ?string
     {
-        if (!$reflectionProperty->getDocComment()) {
+        if ( ! $reflectionProperty->getDocComment()) {
             return null;
         }
 
@@ -73,7 +73,7 @@ 
 
         // The PhpDoc contains multiple non-null types which produces ambiguity when deserializing.
         if (count($typesWithoutNull) > 1) {
-            $typeHint = implode('|', array_map(static function (TypeNode $type) {
+            $typeHint = implode('|', array_map(static function(TypeNode $type) {
                 return (string) $type;
             }, $types));
 
@@ -92,20 +92,20 @@ 
         if ($type instanceof ArrayTypeNode) {
             $resolvedType = $this->resolveTypeFromTypeNode($type->type, $reflectionProperty);
 
-            return 'array<' . $resolvedType . '>';
+            return 'array<'.$resolvedType.'>';
         }
 
         // Generic array syntax: array<Product> | array<\Foo\Bar\Product> | array<int,Product>
         if ($type instanceof GenericTypeNode) {
-            if (!$this->isSimpleType($type->type, 'array')) {
+            if ( ! $this->isSimpleType($type->type, 'array')) {
                 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()));
             }
 
-            $resolvedTypes = array_map(function (TypeNode $node) use ($reflectionProperty) {
+            $resolvedTypes = array_map(function(TypeNode $node) use ($reflectionProperty) {
                 return $this->resolveTypeFromTypeNode($node, $reflectionProperty);
             }, $type->genericTypes);
 
-            return 'array<' . implode(',', $resolvedTypes) . '>';
+            return 'array<'.implode(',', $resolvedTypes).'>';
         }
 
         // Primitives and class names: Collection | \Foo\Bar\Product | string
@@ -121,7 +121,7 @@ 
      */
     private function flattenVarTagValueTypes(array $varTagValues): array
     {
-        return array_merge(...array_map(static function (VarTagValueNode $node) {
+        return array_merge(...array_map(static function(VarTagValueNode $node) {
             if ($node->type instanceof UnionTypeNode) {
                 return $node->type->types;
             }
@@ -139,7 +139,7 @@ 
      */
     private function filterNullFromTypes(array $types): array
     {
-        return array_filter(array_map(function (TypeNode $node) {
+        return array_filter(array_map(function(TypeNode $node) {
             return $this->isNullType($node) ? null : $node;
         }, $types));
     }
@@ -182,7 +182,7 @@ 
      */
     private function resolveTypeFromTypeNode(TypeNode $typeNode, \ReflectionProperty $reflectionProperty): 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, $reflectionProperty->getDeclaringClass()->getName(), $reflectionProperty->getName()));
         }
 
@@ -195,7 +195,7 @@ 
             return $typeHint;
         }
 
-        $expandedClassName = $declaringClass->getNamespaceName() . '\\' . $typeHint;
+        $expandedClassName = $declaringClass->getNamespaceName().'\\'.$typeHint;
         if ($this->isClassOrInterface($expandedClassName)) {
             return $expandedClassName;
         }
@@ -221,7 +221,7 @@ 
 
     private function endsWith(string $statementClassToCheck, string $typeHintToSearchFor): bool
     {
-        $typeHintToSearchFor = '\\' . $typeHintToSearchFor;
+        $typeHintToSearchFor = '\\'.$typeHintToSearchFor;
 
         return substr($statementClassToCheck, -strlen($typeHintToSearchFor)) === $typeHintToSearchFor;
     }
@@ -242,7 +242,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);
             }
         }
 
@@ -275,7 +275,7 @@ 
 
     private function resolveType(string $typeHint, \ReflectionProperty $reflectionProperty): string
     {
-        if (!$this->hasGlobalNamespacePrefix($typeHint) && !$this->isPrimitiveType($typeHint)) {
+        if ( ! $this->hasGlobalNamespacePrefix($typeHint) && ! $this->isPrimitiveType($typeHint)) {
             $typeHint = $this->expandClassNameUsingUseStatements($typeHint, $this->getDeclaringClassOrTrait($reflectionProperty), $reflectionProperty);
         }
 

src/XmlDeserializationVisitor.php

Spacing +14 added lines, -14 removed lines

@@ -84,7 +84,7 @@ 
 
         if (false !== stripos($data, '<!doctype')) {
             $internalSubset = $this->getDomDocumentTypeEntitySubset($data);
-            if (!in_array($internalSubset, $this->doctypeWhitelist, true)) {
+            if ( ! in_array($internalSubset, $this->doctypeWhitelist, true)) {
                 throw new InvalidArgumentException(sprintf(
                     'The document type "%s" is not allowed. If it is safe, you may add it to the whitelist configuration.',
                     $internalSubset
@@ -209,7 +209,7 @@ 
             $nodes = $data->xpath($entryName);
         }
 
-        if (null === $nodes || !\count($nodes)) {
+        if (null === $nodes || ! \count($nodes)) {
             return [];
         }
 
@@ -237,7 +237,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));
                     }
 
@@ -267,7 +267,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
@@ -297,7 +297,7 @@ 
         $name = $metadata->serializedName;
 
         if (true === $metadata->inline) {
-            if (!$metadata->type) {
+            if ( ! $metadata->type) {
                 throw RuntimeException::noMetadataForProperty($metadata->class, $metadata->name);
             }
 
@@ -307,7 +307,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);
                 }
 
@@ -318,7 +318,7 @@ 
         }
 
         if ($metadata->xmlValue) {
-            if (!$metadata->type) {
+            if ( ! $metadata->type) {
                 throw RuntimeException::noMetadataForProperty($metadata->class, $metadata->name);
             }
 
@@ -327,12 +327,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);
             }
 
@@ -344,13 +344,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();
             }
@@ -361,9 +361,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)) {
@@ -377,7 +377,7 @@ 
             $this->setCurrentMetadata($metadata);
         }
 
-        if (!$metadata->type) {
+        if ( ! $metadata->type) {
             throw RuntimeException::noMetadataForProperty($metadata->class, $metadata->name);
         }
 

src/Exclusion/GroupsExclusionStrategy.php

Spacing +7 added lines, -7 removed lines

@@ -54,14 +54,14 @@ 
         if ($this->nestedGroups) {
             $groups = $this->getGroupsFor($navigatorContext);
 
-            if (!$property->groups) {
-                return !in_array(self::DEFAULT_GROUP, $groups);
+            if ( ! $property->groups) {
+                return ! in_array(self::DEFAULT_GROUP, $groups);
             }
 
             return $this->shouldSkipUsingGroups($property, $groups);
         } else {
-            if (!$property->groups) {
-                return !isset($this->groups[self::DEFAULT_GROUP]);
+            if ( ! $property->groups) {
+                return ! isset($this->groups[self::DEFAULT_GROUP]);
             }
 
             foreach ($property->groups as $group) {
@@ -87,14 +87,14 @@ 
 
     public function getGroupsFor(Context $navigatorContext): array
     {
-        if (!$this->nestedGroups) {
+        if ( ! $this->nestedGroups) {
             return array_keys($this->groups);
         }
 
         $paths = $navigatorContext->getCurrentPath();
         $groups = $this->groups;
         foreach ($paths as $index => $path) {
-            if (!array_key_exists($path, $groups)) {
+            if ( ! array_key_exists($path, $groups)) {
                 if ($index > 0) {
                     $groups = [self::DEFAULT_GROUP];
                 } else {
@@ -105,7 +105,7 @@ 
             }
 
             $groups = $groups[$path];
-            if (!array_filter($groups, 'is_string')) {
+            if ( ! array_filter($groups, 'is_string')) {
                 $groups += [self::DEFAULT_GROUP];
             }
         }