schmittjoh / serializer

src/EventDispatcher/EventDispatcher.php

Spacing +9 added lines, -9 removed lines

@@ -30,7 +30,7 @@ 
 
     public static function getDefaultMethodName(string $eventName): string
     {
-        return 'on' . str_replace(['_', '.'], '', $eventName);
+        return 'on'.str_replace(['_', '.'], '', $eventName);
     }
 
     /**
@@ -56,7 +56,7 @@ 
     public function addSubscriber(EventSubscriberInterface $subscriber): void
     {
         foreach ($subscriber->getSubscribedEvents() as $eventData) {
-            if (!isset($eventData['event'])) {
+            if ( ! isset($eventData['event'])) {
                 throw new InvalidArgumentException(sprintf('Each event must have a "event" key.'));
             }
 
@@ -71,33 +71,33 @@ 
 
     public function hasListeners(string $eventName, string $class, string $format): bool
     {
-        if (!isset($this->listeners[$eventName])) {
+        if ( ! isset($this->listeners[$eventName])) {
             return false;
         }
 
-        if (!isset($this->classListeners[$eventName][$class][$format])) {
+        if ( ! isset($this->classListeners[$eventName][$class][$format])) {
             $this->classListeners[$eventName][$class][$format] = $this->initializeListeners($eventName, $class, $format);
         }
 
-        return !!$this->classListeners[$eventName][$class][$format];
+        return ! ! $this->classListeners[$eventName][$class][$format];
     }
 
     public function dispatch(string $eventName, string $class, string $format, Event $event): void
     {
-        if (!isset($this->listeners[$eventName])) {
+        if ( ! isset($this->listeners[$eventName])) {
             return;
         }
 
         $object = $event instanceof ObjectEvent ? $event->getObject() : null;
         $realClass = is_object($object) ? get_class($object) : '';
-        $objectClass = $realClass !== $class ? $realClass . $class : $class;
+        $objectClass = $realClass !== $class ? $realClass.$class : $class;
 
-        if (!isset($this->classListeners[$eventName][$objectClass][$format])) {
+        if ( ! isset($this->classListeners[$eventName][$objectClass][$format])) {
             $this->classListeners[$eventName][$objectClass][$format] = $this->initializeListeners($eventName, $class, $format);
         }
 
         foreach ($this->classListeners[$eventName][$objectClass][$format] as $listener) {
-            if (!empty($listener[3]) && !($object instanceof $listener[3])) {
+            if ( ! empty($listener[3]) && ! ($object instanceof $listener[3])) {
                 continue;
             }
 

src/EventDispatcher/Subscriber/DoctrineProxySubscriber.php

Spacing +6 added lines, -6 removed lines

@@ -40,14 +40,14 @@ 
         // If the set type name is not an actual class, but a faked type for which a custom handler exists, we do not
         // modify it with this subscriber. Also, we forgo autoloading here as an instance of this type is already created,
         // so it must be loaded if its a real class.
-        $virtualType = !class_exists($type['name'], false);
+        $virtualType = ! class_exists($type['name'], false);
 
         if (
             $object instanceof PersistentCollection
             || $object instanceof MongoDBPersistentCollection
             || $object instanceof PHPCRPersistentCollection
         ) {
-            if (!$virtualType) {
+            if ( ! $virtualType) {
                 $event->setType('ArrayCollection');
             }
 
@@ -56,13 +56,13 @@ 
 
         if (
             ($this->skipVirtualTypeInit && $virtualType) ||
-            (!$object instanceof Proxy && !$object instanceof LazyLoadingInterface)
+            ( ! $object instanceof Proxy && ! $object instanceof LazyLoadingInterface)
         ) {
             return;
         }
 
         // do not initialize the proxy if is going to be excluded by-class by some exclusion strategy
-        if (false === $this->initializeExcluded && !$virtualType) {
+        if (false === $this->initializeExcluded && ! $virtualType) {
             $context = $event->getContext();
             $exclusionStrategy = $context->getExclusionStrategy();
             $metadata = $context->getMetadataFactory()->getMetadataForClass(get_parent_class($object));
@@ -77,7 +77,7 @@ 
             $object->__load();
         }
 
-        if (!$virtualType) {
+        if ( ! $virtualType) {
             $event->setType(get_parent_class($object), $type['params']);
         }
     }
@@ -86,7 +86,7 @@ 
     {
         $type = $event->getType();
         // is a virtual type? then there is no need to change the event name
-        if (!class_exists($type['name'], false)) {
+        if ( ! class_exists($type['name'], false)) {
             return;
         }
 

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

src/Metadata/Driver/DocBlockDriver/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
@@ -125,7 +125,7 @@ 
             return [];
         }
 
-        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;
             }
@@ -143,7 +143,7 @@ 
      */
     private function filterNullFromTypes(array $types): array
     {
-        return array_values(array_filter(array_map(function (TypeNode $node) {
+        return array_values(array_filter(array_map(function(TypeNode $node) {
             return $this->isNullType($node) ? null : $node;
         }, $types)));
     }
@@ -186,7 +186,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()));
         }
 
@@ -199,7 +199,7 @@ 
             return $typeHint;
         }
 
-        $expandedClassName = $declaringClass->getNamespaceName() . '\\' . $typeHint;
+        $expandedClassName = $declaringClass->getNamespaceName().'\\'.$typeHint;
         if ($this->isClassOrInterface($expandedClassName)) {
             return $expandedClassName;
         }
@@ -225,7 +225,7 @@ 
 
     private function endsWith(string $statementClassToCheck, string $typeHintToSearchFor): bool
     {
-        $typeHintToSearchFor = '\\' . $typeHintToSearchFor;
+        $typeHintToSearchFor = '\\'.$typeHintToSearchFor;
 
         return substr($statementClassToCheck, -strlen($typeHintToSearchFor)) === $typeHintToSearchFor;
     }
@@ -246,7 +246,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);
             }
         }
 
@@ -279,7 +279,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/GraphNavigator/SerializationGraphNavigator.php

Spacing +4 added lines, -4 removed lines

@@ -139,7 +139,7 @@ 
 
         switch ($type['name']) {
             case 'NULL':
-                if (!$this->shouldSerializeNull && !$this->isRootNullAllowed()) {
+                if ( ! $this->shouldSerializeNull && ! $this->isRootNullAllowed()) {
                     throw new NotAcceptableException();
                 }
 
@@ -169,7 +169,7 @@ 
             case 'resource':
                 $msg = 'Resources are not supported in serialized data.';
                 if (null !== $path = $this->context->getPath()) {
-                    $msg .= ' Path: ' . $path;
+                    $msg .= ' Path: '.$path;
                 }
 
                 throw new RuntimeException($msg);
@@ -235,8 +235,8 @@ 
                     throw new ExcludedClassException();
                 }
 
-                if (!is_object($data)) {
-                    throw new InvalidArgumentException('Value at ' . $this->context->getPath() . ' is expected to be an object of class ' . $type['name'] . ' but is of type ' . gettype($data));
+                if ( ! is_object($data)) {
+                    throw new InvalidArgumentException('Value at '.$this->context->getPath().' is expected to be an object of class '.$type['name'].' but is of type '.gettype($data));
                 }
 
                 $this->context->pushClassMetadata($metadata);

src/SerializerBuilder.php

Spacing +15 added lines, -15 removed lines

@@ -216,7 +216,7 @@ 
 
     private function getAccessorStrategy(): AccessorStrategyInterface
     {
-        if (!$this->accessorStrategy) {
+        if ( ! $this->accessorStrategy) {
             $this->accessorStrategy = new DefaultAccessorStrategy($this->expressionEvaluator);
         }
 
@@ -260,11 +260,11 @@ 
 
     public function setCacheDir(string $dir): self
     {
-        if (!is_dir($dir)) {
+        if ( ! is_dir($dir)) {
             $this->createDir($dir);
         }
 
-        if (!is_writable($dir)) {
+        if ( ! is_writable($dir)) {
             throw new InvalidArgumentException(sprintf('The cache directory "%s" is not writable.', $dir));
         }
 
@@ -386,7 +386,7 @@ 
     public function setMetadataDirs(array $namespacePrefixToDirMap): self
     {
         foreach ($namespacePrefixToDirMap as $dir) {
-            if (!is_dir($dir)) {
+            if ( ! is_dir($dir)) {
                 throw new InvalidArgumentException(sprintf('The directory "%s" does not exist.', $dir));
             }
         }
@@ -424,7 +424,7 @@ 
      */
     public function addMetadataDir(string $dir, string $namespacePrefix = ''): self
     {
-        if (!is_dir($dir)) {
+        if ( ! is_dir($dir)) {
             throw new InvalidArgumentException(sprintf('The directory "%s" does not exist.', $dir));
         }
 
@@ -463,11 +463,11 @@ 
      */
     public function replaceMetadataDir(string $dir, string $namespacePrefix = ''): self
     {
-        if (!is_dir($dir)) {
+        if ( ! is_dir($dir)) {
             throw new InvalidArgumentException(sprintf('The directory "%s" does not exist.', $dir));
         }
 
-        if (!isset($this->metadataDirs[$namespacePrefix])) {
+        if ( ! isset($this->metadataDirs[$namespacePrefix])) {
             throw new InvalidArgumentException(sprintf('There is no directory configured for namespace prefix "%s". Please use addMetadataDir() for adding new directories.', $namespacePrefix));
         }
 
@@ -566,19 +566,19 @@ 
         if (null !== $this->metadataCache) {
             $metadataFactory->setCache($this->metadataCache);
         } elseif (null !== $this->cacheDir) {
-            $this->createDir($this->cacheDir . '/metadata');
-            $metadataFactory->setCache(new FileCache($this->cacheDir . '/metadata'));
+            $this->createDir($this->cacheDir.'/metadata');
+            $metadataFactory->setCache(new FileCache($this->cacheDir.'/metadata'));
         }
 
-        if (!$this->handlersConfigured) {
+        if ( ! $this->handlersConfigured) {
             $this->addDefaultHandlers();
         }
 
-        if (!$this->listenersConfigured) {
+        if ( ! $this->listenersConfigured) {
             $this->addDefaultListeners();
         }
 
-        if (!$this->visitorsAdded) {
+        if ( ! $this->visitorsAdded) {
             $this->addDefaultSerializationVisitors();
             $this->addDefaultDeserializationVisitors();
         }
@@ -645,12 +645,12 @@ 
     private function decorateAnnotationReader(Reader $annotationReader): Reader
     {
         if (null !== $this->cacheDir) {
-            $this->createDir($this->cacheDir . '/annotations');
+            $this->createDir($this->cacheDir.'/annotations');
             if (class_exists(FilesystemAdapter::class)) {
-                $annotationsCache = new FilesystemAdapter('', 0, $this->cacheDir . '/annotations');
+                $annotationsCache = new FilesystemAdapter('', 0, $this->cacheDir.'/annotations');
                 $annotationReader = new PsrCachedReader($annotationReader, $annotationsCache, $this->debug);
             } else {
-                $annotationsCache = new FilesystemCache($this->cacheDir . '/annotations');
+                $annotationsCache = new FilesystemCache($this->cacheDir.'/annotations');
                 $annotationReader = new CachedReader($annotationReader, $annotationsCache, $this->debug);
             }
         }

src/Metadata/Driver/AttributeDriver/AttributeReader.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
 
     private function buildAnnotation(array $attributes): ?object
     {
-        if (!isset($attributes[0])) {
+        if ( ! isset($attributes[0])) {
             return null;
         }
 

src/Annotation/AnnotationUtilsTrait.php

Spacing +3 added lines, -3 removed lines

@@ -10,9 +10,9 @@ 
 {
     private function loadAnnotationParameters(array $vars): void
     {
-        if (!array_key_exists('values', $vars)) {
+        if ( ! array_key_exists('values', $vars)) {
             $values = [];
-        } elseif (!is_array($vars['values'])) {
+        } elseif ( ! is_array($vars['values'])) {
             $values = ['value' => $vars['values']];
         } else {
             $values = $vars['values'];
@@ -30,7 +30,7 @@ 
         }
 
         foreach ($vars as $key => $value) {
-            if (!property_exists(static::class, $key)) {
+            if ( ! property_exists(static::class, $key)) {
                 throw new InvalidArgumentException(sprintf('Unknown property "%s" on annotation "%s".', $key, static::class));
             }