schmittjoh / serializer

src/Metadata/Driver/XmlDriver.php

Spacing +17 added lines, -17 removed lines

@@ -54,14 +54,14 @@ 
         }
 
         $metadata = new ClassMetadata($name = $class->name);
-        if (!$elems = $elem->xpath("./class[@name = '" . $name . "']")) {
+        if ( ! $elems = $elem->xpath("./class[@name = '".$name."']")) {
             throw new InvalidMetadataException(sprintf('Could not find class %s inside XML element.', $name));
         }
 
         $elem = reset($elems);
 
         $metadata->fileResources[] = $path;
-        $fileResource =  $class->getFilename();
+        $fileResource = $class->getFilename();
         if (false !== $fileResource) {
             $metadata->fileResources[] = $fileResource;
         }
@@ -100,7 +100,7 @@ 
         $discriminatorFieldName = (string) $elem->attributes()->{'discriminator-field-name'};
         $discriminatorMap = [];
         foreach ($elem->xpath('./discriminator-class') as $entry) {
-            if (!isset($entry->attributes()->value)) {
+            if ( ! isset($entry->attributes()->value)) {
                 throw new InvalidMetadataException('Each discriminator-class element must have a "value" attribute.');
             }
 
@@ -109,7 +109,7 @@ 
 
         if ('true' === (string) $elem->attributes()->{'discriminator-disabled'}) {
             $metadata->discriminatorDisabled = true;
-        } elseif (!empty($discriminatorFieldName) || !empty($discriminatorMap)) {
+        } elseif ( ! empty($discriminatorFieldName) || ! empty($discriminatorMap)) {
             $discriminatorGroups = [];
             foreach ($elem->xpath('./discriminator-groups/group') as $entry) {
                 $discriminatorGroups[] = (string) $entry;
@@ -119,7 +119,7 @@ 
         }
 
         foreach ($elem->xpath('./xml-namespace') as $xmlNamespace) {
-            if (!isset($xmlNamespace->attributes()->uri)) {
+            if ( ! isset($xmlNamespace->attributes()->uri)) {
                 throw new InvalidMetadataException('The prefix attribute must be set for all xml-namespace elements.');
             }
 
@@ -154,7 +154,7 @@ 
                     $this->parseExpression((string) $method->attributes()->expression)
                 );
             } else {
-                if (!isset($method->attributes()->method)) {
+                if ( ! isset($method->attributes()->method)) {
                     throw new InvalidMetadataException('The method attribute must be set for all virtual-property elements.');
                 }
 
@@ -165,7 +165,7 @@ 
             $propertiesNodes[] = $method;
         }
 
-        if (!$excludeAll) {
+        if ( ! $excludeAll) {
             foreach ($class->getProperties() as $property) {
                 if ($property->class !== $name || (isset($property->info) && $property->info['class'] !== $name)) {
                     continue;
@@ -174,7 +174,7 @@ 
                 $pName = $property->getName();
                 $propertiesMetadata[] = new PropertyMetadata($name, $pName);
 
-                $pElems = $elem->xpath("./property[@name = '" . $pName . "']");
+                $pElems = $elem->xpath("./property[@name = '".$pName."']");
                 $propertiesNodes[] = $pElems ? reset($pElems) : null;
             }
 
@@ -185,7 +185,7 @@ 
                     || isset($propertiesNodes[$propertyKey]);
 
                 $pElem = $propertiesNodes[$propertyKey];
-                if (!empty($pElem)) {
+                if ( ! empty($pElem)) {
                     if (null !== $exclude = $pElem->attributes()->exclude) {
                         $isExclude = 'true' === strtolower((string) $exclude);
                     }
@@ -207,7 +207,7 @@ 
                     }
 
                     if (null !== $excludeIf = $pElem->attributes()->{'expose-if'}) {
-                        $pMetadata->excludeIf = $this->parseExpression('!(' . (string) $excludeIf . ')');
+                        $pMetadata->excludeIf = $this->parseExpression('!('.(string) $excludeIf.')');
                         $isExpose = true;
                     }
 
@@ -335,16 +335,16 @@ 
                     $metadata->isMap = $metadata->isMap || PropertyMetadata::isCollectionMap($pMetadata->type);
                 }
 
-                if (!$pMetadata->serializedName) {
+                if ( ! $pMetadata->serializedName) {
                     $pMetadata->serializedName = $this->namingStrategy->translateName($pMetadata);
                 }
 
-                if (!empty($pElem) && null !== $name = $pElem->attributes()->name) {
+                if ( ! empty($pElem) && null !== $name = $pElem->attributes()->name) {
                     $pMetadata->name = (string) $name;
                 }
 
                 if (
-                    (ExclusionPolicy::NONE === (string) $exclusionPolicy && !$isExclude)
+                    (ExclusionPolicy::NONE === (string) $exclusionPolicy && ! $isExclude)
                     || (ExclusionPolicy::ALL === (string) $exclusionPolicy && $isExpose)
                 ) {
                     $metadata->addPropertyMetadata($pMetadata);
@@ -353,11 +353,11 @@ 
         }
 
         foreach ($elem->xpath('./callback-method') as $method) {
-            if (!isset($method->attributes()->type)) {
+            if ( ! isset($method->attributes()->type)) {
                 throw new InvalidMetadataException('The type attribute must be set for all callback-method elements.');
             }
 
-            if (!isset($method->attributes()->name)) {
+            if ( ! isset($method->attributes()->name)) {
                 throw new InvalidMetadataException('The name attribute must be set for all callback-method elements.');
             }
 
@@ -375,11 +375,11 @@ 
                     break;
 
                 case 'handler':
-                    if (!isset($method->attributes()->format)) {
+                    if ( ! isset($method->attributes()->format)) {
                         throw new InvalidMetadataException('The format attribute must be set for "handler" callback methods.');
                     }
 
-                    if (!isset($method->attributes()->direction)) {
+                    if ( ! isset($method->attributes()->direction)) {
                         throw new InvalidMetadataException('The direction attribute must be set for "handler" callback methods.');
                     }
 

src/Metadata/Driver/AbstractDoctrineTypeDriver.php

Spacing +3 added lines, -3 removed lines

@@ -83,7 +83,7 @@ 
         \assert($classMetadata instanceof ClassMetadata);
 
         // Abort if the given class is not a mapped entity
-        if (!$doctrineMetadata = $this->tryLoadingDoctrineMetadata($class->name)) {
+        if ( ! $doctrineMetadata = $this->tryLoadingDoctrineMetadata($class->name)) {
             return $classMetadata;
         }
 
@@ -129,7 +129,7 @@ 
 
     protected function tryLoadingDoctrineMetadata(string $className): ?DoctrineClassMetadata
     {
-        if (!$manager = $this->registry->getManagerForClass($className)) {
+        if ( ! $manager = $this->registry->getManagerForClass($className)) {
             return null;
         }
 
@@ -142,7 +142,7 @@ 
 
     protected function normalizeFieldType(string $type): ?string
     {
-        if (!isset($this->fieldMapping[$type])) {
+        if ( ! isset($this->fieldMapping[$type])) {
             return null;
         }
 

src/SerializationContext.php

Spacing +5 added lines, -5 removed lines

@@ -64,7 +64,7 @@ 
      */
     public function startVisiting($object): void
     {
-        if (!\is_object($object)) {
+        if ( ! \is_object($object)) {
             return;
         }
 
@@ -77,7 +77,7 @@ 
      */
     public function stopVisiting($object): void
     {
-        if (!\is_object($object)) {
+        if ( ! \is_object($object)) {
             return;
         }
 
@@ -94,7 +94,7 @@ 
      */
     public function isVisiting($object): bool
     {
-        if (!\is_object($object)) {
+        if ( ! \is_object($object)) {
             return false;
         }
 
@@ -108,7 +108,7 @@ 
             $path[] = \get_class($obj);
         }
 
-        if (!$path) {
+        if ( ! $path) {
             return null;
         }
 
@@ -127,7 +127,7 @@ 
 
     public function getObject(): ?object
     {
-        return !$this->visitingStack->isEmpty() ? $this->visitingStack->top() : null;
+        return ! $this->visitingStack->isEmpty() ? $this->visitingStack->top() : null;
     }
 
     public function getVisitingStack(): \SplStack

src/Naming/CamelCaseNamingStrategy.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 
     public function translateName(PropertyMetadata $property): string
     {
-        $name = preg_replace('/[A-Z]+/', $this->separator . '\\0', $property->name);
+        $name = preg_replace('/[A-Z]+/', $this->separator.'\\0', $property->name);
 
         if ($this->lowerCase) {
             return strtolower($name);

src/Accessor/DefaultAccessorStrategy.php

Spacing +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
 
         $accessor = $this->readAccessors[$metadata->class] ?? null;
         if (null === $accessor) {
-            $accessor = \Closure::bind(static function ($o, $name) {
+            $accessor = \Closure::bind(static function($o, $name) {
                 return $o->$name;
             }, null, $metadata->class);
             $this->readAccessors[$metadata->class] = $accessor;
@@ -124,7 +124,7 @@ 
 
         $accessor = $this->writeAccessors[$metadata->class] ?? null;
         if (null === $accessor) {
-            $accessor = \Closure::bind(static function ($o, $name, $value): void {
+            $accessor = \Closure::bind(static function($o, $name, $value): void {
                 $o->$name = $value;
             }, null, $metadata->class);
             $this->writeAccessors[$metadata->class] = $accessor;

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