schmittjoh / serializer

src/Metadata/PropertyMetadata.php

Spacing +11 added lines, -11 removed lines

@@ -69,22 +69,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 RuntimeException(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 RuntimeException(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 RuntimeException(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 RuntimeException(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));
                 }
             }
         }

src/JsonDeserializationVisitor.php

Spacing +10 added lines, -10 removed lines

@@ -49,32 +49,32 @@ 
 
     public function visitString($data, array $type): string
     {
-        return (string)$data;
+        return (string) $data;
     }
 
     public function visitBoolean($data, array $type): bool
     {
-        return (bool)$data;
+        return (bool) $data;
     }
 
     public function visitInteger($data, array $type): int
     {
-        return (int)$data;
+        return (int) $data;
     }
 
     public function visitDouble($data, array $type): float
     {
-        return (double)$data;
+        return (double) $data;
     }
 
     public function visitArray($data, array $type): array
     {
-        if (!\is_array($data)) {
+        if ( ! \is_array($data)) {
             throw new RuntimeException(sprintf('Expected array, but got %s: %s', \gettype($data), json_encode($data)));
         }
 
         // If no further parameters were given, keys/values are just passed as is.
-        if (!$type['params']) {
+        if ( ! $type['params']) {
             return $data;
         }
 
@@ -109,7 +109,7 @@ 
     public function visitDiscriminatorMapProperty($data, ClassMetadata $metadata): string
     {
         if (isset($data[$metadata->discriminatorFieldName])) {
-            return (string)$data[$metadata->discriminatorFieldName];
+            return (string) $data[$metadata->discriminatorFieldName];
         }
 
         throw new LogicException(sprintf(
@@ -132,15 +132,15 @@ 
             return;
         }
 
-        if (!\is_array($data)) {
+        if ( ! \is_array($data)) {
             throw new RuntimeException(sprintf('Invalid data "%s"(%s), expected "%s".', $data, $metadata->type['name'], $metadata->class));
         }
 
-        if (!array_key_exists($name, $data)) {
+        if ( ! array_key_exists($name, $data)) {
             return;
         }
 
-        if (!$metadata->type) {
+        if ( ! $metadata->type) {
             throw new RuntimeException(sprintf('You must define a type for %s::$%s.', $metadata->class, $metadata->name));
         }
 

src/Metadata/ClassMetadata.php

Spacing +8 added lines, -8 removed lines

@@ -100,12 +100,12 @@ 
      */
     public function setAccessorOrder(string $order, array $customOrder = []):void
     {
-        if (!in_array($order, [self::ACCESSOR_ORDER_UNDEFINED, self::ACCESSOR_ORDER_ALPHABETICAL, self::ACCESSOR_ORDER_CUSTOM], true)) {
+        if ( ! in_array($order, [self::ACCESSOR_ORDER_UNDEFINED, self::ACCESSOR_ORDER_ALPHABETICAL, self::ACCESSOR_ORDER_CUSTOM], true)) {
             throw new InvalidArgumentException(sprintf('The accessor order "%s" is invalid.', $order));
         }
 
         foreach ($customOrder as $name) {
-            if (!\is_string($name)) {
+            if ( ! \is_string($name)) {
                 throw new InvalidArgumentException(sprintf('$customOrder is expected to be a list of strings, but got element of value %s.', json_encode($name)));
             }
         }
@@ -141,7 +141,7 @@ 
 
     public function merge(MergeableInterface $object):void
     {
-        if (!$object instanceof ClassMetadata) {
+        if ( ! $object instanceof ClassMetadata) {
             throw new InvalidArgumentException('$object must be an instance of ClassMetadata.');
         }
         parent::merge($object);
@@ -165,7 +165,7 @@ 
                 $this->discriminatorBaseClass,
                 $this->discriminatorBaseClass
             ));
-        } elseif (!$this->discriminatorFieldName && $object->discriminatorFieldName) {
+        } elseif ( ! $this->discriminatorFieldName && $object->discriminatorFieldName) {
             $this->discriminatorFieldName = $object->discriminatorFieldName;
             $this->discriminatorMap = $object->discriminatorMap;
         }
@@ -180,7 +180,7 @@ 
             $this->discriminatorBaseClass = $object->discriminatorBaseClass;
         }
 
-        if ($this->discriminatorMap && !$this->getReflection()->isAbstract()) {
+        if ($this->discriminatorMap && ! $this->getReflection()->isAbstract()) {
             if (false === $typeValue = array_search($this->name, $this->discriminatorMap, true)) {
                 throw new LogicException(sprintf(
                     'The sub-class "%s" is not listed in the discriminator of the base class "%s".',
@@ -192,7 +192,7 @@ 
             $this->discriminatorValue = $typeValue;
 
             if (isset($this->propertyMetadata[$this->discriminatorFieldName])
-                && !$this->propertyMetadata[$this->discriminatorFieldName] instanceof StaticPropertyMetadata
+                && ! $this->propertyMetadata[$this->discriminatorFieldName] instanceof StaticPropertyMetadata
             ) {
                 throw new LogicException(sprintf(
                     'The discriminator field name "%s" of the base-class "%s" conflicts with a regular property of the sub-class "%s".',
@@ -220,12 +220,12 @@ 
 
     public function registerNamespace(string $uri, ?string $prefix = null):void
     {
-        if (!\is_string($uri)) {
+        if ( ! \is_string($uri)) {
             throw new InvalidArgumentException(sprintf('$uri is expected to be a strings, but got value %s.', json_encode($uri)));
         }
 
         if ($prefix !== null) {
-            if (!\is_string($prefix)) {
+            if ( ! \is_string($prefix)) {
                 throw new InvalidArgumentException(sprintf('$prefix is expected to be a strings, but got value %s.', json_encode($prefix)));
             }
         } else {

src/Metadata/Driver/DoctrineTypeDriver.php

Spacing +4 added lines, -4 removed lines

@@ -32,8 +32,8 @@ 
 {
     protected function setDiscriminator(DoctrineClassMetadata $doctrineMetadata, ClassMetadata $classMetadata):void
     {
-        if (empty($classMetadata->discriminatorMap) && !$classMetadata->discriminatorDisabled
-            && !empty($doctrineMetadata->discriminatorMap) && $doctrineMetadata->isRootEntity()
+        if (empty($classMetadata->discriminatorMap) && ! $classMetadata->discriminatorDisabled
+            && ! empty($doctrineMetadata->discriminatorMap) && $doctrineMetadata->isRootEntity()
         ) {
             $classMetadata->setDiscriminator(
                 $doctrineMetadata->discriminatorColumn['name'],
@@ -57,11 +57,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 DoctrineClassMetadata && !$targetMetadata->isInheritanceTypeNone()) {
+            if ($targetMetadata instanceof DoctrineClassMetadata && ! $targetMetadata->isInheritanceTypeNone()) {
                 return;
             }
 
-            if (!$doctrineMetadata->isSingleValuedAssociation($propertyName)) {
+            if ( ! $doctrineMetadata->isSingleValuedAssociation($propertyName)) {
                 $targetEntity = "ArrayCollection<{$targetEntity}>";
             }
 

src/Metadata/Driver/XmlDriver.php

Spacing +72 added lines, -72 removed lines

@@ -49,7 +49,7 @@ 
         $this->namingStrategy = $namingStrategy;
     }
 
-    protected function loadMetadataFromFile(\ReflectionClass $class, $path):?\Metadata\ClassMetadata
+    protected function loadMetadataFromFile(\ReflectionClass $class, $path): ?\Metadata\ClassMetadata
     {
         $previous = libxml_use_internal_errors(true);
         libxml_clear_errors();
@@ -62,97 +62,97 @@ 
         }
 
         $metadata = new ClassMetadata($name = $class->name);
-        if (!$elems = $elem->xpath("./class[@name = '" . $name . "']")) {
+        if ( ! $elems = $elem->xpath("./class[@name = '".$name."']")) {
             throw new RuntimeException(sprintf('Could not find class %s inside XML element.', $name));
         }
         $elem = reset($elems);
 
         $metadata->fileResources[] = $path;
         $metadata->fileResources[] = $class->getFileName();
-        $exclusionPolicy = strtoupper((string)$elem->attributes()->{'exclusion-policy'}) ?: 'NONE';
-        $excludeAll = null !== ($exclude = $elem->attributes()->exclude) ? 'true' === strtolower((string)$exclude) : false;
-        $classAccessType = (string)($elem->attributes()->{'access-type'} ?: PropertyMetadata::ACCESS_TYPE_PROPERTY);
+        $exclusionPolicy = strtoupper((string) $elem->attributes()->{'exclusion-policy'}) ?: 'NONE';
+        $excludeAll = null !== ($exclude = $elem->attributes()->exclude) ? 'true' === strtolower((string) $exclude) : false;
+        $classAccessType = (string) ($elem->attributes()->{'access-type'} ?: PropertyMetadata::ACCESS_TYPE_PROPERTY);
 
         $propertiesMetadata = [];
         $propertiesNodes = [];
 
         if (null !== $accessorOrder = $elem->attributes()->{'accessor-order'}) {
-            $metadata->setAccessorOrder((string)$accessorOrder, preg_split('/\s*,\s*/', (string)$elem->attributes()->{'custom-accessor-order'}));
+            $metadata->setAccessorOrder((string) $accessorOrder, preg_split('/\s*,\s*/', (string) $elem->attributes()->{'custom-accessor-order'}));
         }
 
         if (null !== $xmlRootName = $elem->attributes()->{'xml-root-name'}) {
-            $metadata->xmlRootName = (string)$xmlRootName;
+            $metadata->xmlRootName = (string) $xmlRootName;
         }
 
         if (null !== $xmlRootNamespace = $elem->attributes()->{'xml-root-namespace'}) {
-            $metadata->xmlRootNamespace = (string)$xmlRootNamespace;
+            $metadata->xmlRootNamespace = (string) $xmlRootNamespace;
         }
 
-        $readOnlyClass = 'true' === strtolower((string)$elem->attributes()->{'read-only'});
+        $readOnlyClass = 'true' === strtolower((string) $elem->attributes()->{'read-only'});
 
-        $discriminatorFieldName = (string)$elem->attributes()->{'discriminator-field-name'};
+        $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 RuntimeException('Each discriminator-class element must have a "value" attribute.');
             }
 
-            $discriminatorMap[(string)$entry->attributes()->value] = (string)$entry;
+            $discriminatorMap[(string) $entry->attributes()->value] = (string) $entry;
         }
 
-        if ('true' === (string)$elem->attributes()->{'discriminator-disabled'}) {
+        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;
+                $discriminatorGroups[] = (string) $entry;
             }
             $metadata->setDiscriminator($discriminatorFieldName, $discriminatorMap, $discriminatorGroups);
         }
 
         foreach ($elem->xpath('./xml-namespace') as $xmlNamespace) {
-            if (!isset($xmlNamespace->attributes()->uri)) {
+            if ( ! isset($xmlNamespace->attributes()->uri)) {
                 throw new RuntimeException('The prefix attribute must be set for all xml-namespace elements.');
             }
 
             if (isset($xmlNamespace->attributes()->prefix)) {
-                $prefix = (string)$xmlNamespace->attributes()->prefix;
+                $prefix = (string) $xmlNamespace->attributes()->prefix;
             } else {
                 $prefix = null;
             }
 
-            $metadata->registerNamespace((string)$xmlNamespace->attributes()->uri, $prefix);
+            $metadata->registerNamespace((string) $xmlNamespace->attributes()->uri, $prefix);
         }
 
         foreach ($elem->xpath('./xml-discriminator') as $xmlDiscriminator) {
             if (isset($xmlDiscriminator->attributes()->attribute)) {
-                $metadata->xmlDiscriminatorAttribute = (string)$xmlDiscriminator->attributes()->attribute === 'true';
+                $metadata->xmlDiscriminatorAttribute = (string) $xmlDiscriminator->attributes()->attribute === 'true';
             }
             if (isset($xmlDiscriminator->attributes()->cdata)) {
-                $metadata->xmlDiscriminatorCData = (string)$xmlDiscriminator->attributes()->cdata === 'true';
+                $metadata->xmlDiscriminatorCData = (string) $xmlDiscriminator->attributes()->cdata === 'true';
             }
             if (isset($xmlDiscriminator->attributes()->namespace)) {
-                $metadata->xmlDiscriminatorNamespace = (string)$xmlDiscriminator->attributes()->namespace;
+                $metadata->xmlDiscriminatorNamespace = (string) $xmlDiscriminator->attributes()->namespace;
             }
         }
 
         foreach ($elem->xpath('./virtual-property') as $method) {
 
             if (isset($method->attributes()->expression)) {
-                $virtualPropertyMetadata = new ExpressionPropertyMetadata($name, (string)$method->attributes()->name, (string)$method->attributes()->expression);
+                $virtualPropertyMetadata = new ExpressionPropertyMetadata($name, (string) $method->attributes()->name, (string) $method->attributes()->expression);
             } else {
-                if (!isset($method->attributes()->method)) {
+                if ( ! isset($method->attributes()->method)) {
                     throw new RuntimeException('The method attribute must be set for all virtual-property elements.');
                 }
-                $virtualPropertyMetadata = new VirtualPropertyMetadata($name, (string)$method->attributes()->method);
+                $virtualPropertyMetadata = new VirtualPropertyMetadata($name, (string) $method->attributes()->method);
             }
 
             $propertiesMetadata[] = $virtualPropertyMetadata;
             $propertiesNodes[] = $method;
         }
 
-        if (!$excludeAll) {
+        if ( ! $excludeAll) {
 
             foreach ($class->getProperties() as $property) {
                 if ($property->class !== $name || (isset($property->info) && $property->info['class'] !== $name)) {
@@ -160,7 +160,7 @@ 
                 }
 
                 $propertiesMetadata[] = new PropertyMetadata($name, $pName = $property->getName());
-                $pElems = $elem->xpath("./property[@name = '" . $pName . "']");
+                $pElems = $elem->xpath("./property[@name = '".$pName."']");
 
                 $propertiesNodes[] = $pElems ? reset($pElems) : null;
             }
@@ -172,10 +172,10 @@ 
                     || $pMetadata instanceof ExpressionPropertyMetadata;
 
                 $pElem = $propertiesNodes[$propertyKey];
-                if (!empty($pElem)) {
+                if ( ! empty($pElem)) {
 
                     if (null !== $exclude = $pElem->attributes()->exclude) {
-                        $isExclude = 'true' === strtolower((string)$exclude);
+                        $isExclude = 'true' === strtolower((string) $exclude);
                     }
 
                     if ($isExclude) {
@@ -183,44 +183,44 @@ 
                     }
 
                     if (null !== $expose = $pElem->attributes()->expose) {
-                        $isExpose = 'true' === strtolower((string)$expose);
+                        $isExpose = 'true' === strtolower((string) $expose);
                     }
 
                     if (null !== $excludeIf = $pElem->attributes()->{'exclude-if'}) {
-                        $pMetadata->excludeIf = (string)$excludeIf;
+                        $pMetadata->excludeIf = (string) $excludeIf;
                     }
 
                     if (null !== $skip = $pElem->attributes()->{'skip-when-empty'}) {
-                        $pMetadata->skipWhenEmpty = 'true' === strtolower((string)$skip);
+                        $pMetadata->skipWhenEmpty = 'true' === strtolower((string) $skip);
                     }
 
                     if (null !== $excludeIf = $pElem->attributes()->{'expose-if'}) {
-                        $pMetadata->excludeIf = "!(" . (string)$excludeIf . ")";
+                        $pMetadata->excludeIf = "!(".(string) $excludeIf.")";
                         $isExpose = true;
                     }
 
                     if (null !== $version = $pElem->attributes()->{'since-version'}) {
-                        $pMetadata->sinceVersion = (string)$version;
+                        $pMetadata->sinceVersion = (string) $version;
                     }
 
                     if (null !== $version = $pElem->attributes()->{'until-version'}) {
-                        $pMetadata->untilVersion = (string)$version;
+                        $pMetadata->untilVersion = (string) $version;
                     }
 
                     if (null !== $serializedName = $pElem->attributes()->{'serialized-name'}) {
-                        $pMetadata->serializedName = (string)$serializedName;
+                        $pMetadata->serializedName = (string) $serializedName;
                     }
 
                     if (null !== $type = $pElem->attributes()->type) {
-                        $pMetadata->setType($this->typeParser->parse((string)$type));
+                        $pMetadata->setType($this->typeParser->parse((string) $type));
                     } elseif (isset($pElem->type)) {
-                        $pMetadata->setType($this->typeParser->parse((string)$pElem->type));
+                        $pMetadata->setType($this->typeParser->parse((string) $pElem->type));
                     }
 
                     if (null !== $groups = $pElem->attributes()->groups) {
-                        $pMetadata->groups = preg_split('/\s*,\s*/', trim((string)$groups));
+                        $pMetadata->groups = preg_split('/\s*,\s*/', trim((string) $groups));
                     } elseif (isset($pElem->groups)) {
-                        $pMetadata->groups = (array)$pElem->groups->value;
+                        $pMetadata->groups = (array) $pElem->groups->value;
                     }
 
                     if (isset($pElem->{'xml-list'})) {
@@ -229,21 +229,21 @@ 
 
                         $colConfig = $pElem->{'xml-list'};
                         if (isset($colConfig->attributes()->inline)) {
-                            $pMetadata->xmlCollectionInline = 'true' === (string)$colConfig->attributes()->inline;
+                            $pMetadata->xmlCollectionInline = 'true' === (string) $colConfig->attributes()->inline;
                         }
 
                         if (isset($colConfig->attributes()->{'entry-name'})) {
-                            $pMetadata->xmlEntryName = (string)$colConfig->attributes()->{'entry-name'};
+                            $pMetadata->xmlEntryName = (string) $colConfig->attributes()->{'entry-name'};
                         }
 
                         if (isset($colConfig->attributes()->{'skip-when-empty'})) {
-                            $pMetadata->xmlCollectionSkipWhenEmpty = 'true' === (string)$colConfig->attributes()->{'skip-when-empty'};
+                            $pMetadata->xmlCollectionSkipWhenEmpty = 'true' === (string) $colConfig->attributes()->{'skip-when-empty'};
                         } else {
                             $pMetadata->xmlCollectionSkipWhenEmpty = true;
                         }
 
                         if (isset($colConfig->attributes()->namespace)) {
-                            $pMetadata->xmlEntryNamespace = (string)$colConfig->attributes()->namespace;
+                            $pMetadata->xmlEntryNamespace = (string) $colConfig->attributes()->namespace;
                         }
                     }
 
@@ -252,56 +252,56 @@ 
 
                         $colConfig = $pElem->{'xml-map'};
                         if (isset($colConfig->attributes()->inline)) {
-                            $pMetadata->xmlCollectionInline = 'true' === (string)$colConfig->attributes()->inline;
+                            $pMetadata->xmlCollectionInline = 'true' === (string) $colConfig->attributes()->inline;
                         }
 
                         if (isset($colConfig->attributes()->{'entry-name'})) {
-                            $pMetadata->xmlEntryName = (string)$colConfig->attributes()->{'entry-name'};
+                            $pMetadata->xmlEntryName = (string) $colConfig->attributes()->{'entry-name'};
                         }
 
                         if (isset($colConfig->attributes()->namespace)) {
-                            $pMetadata->xmlEntryNamespace = (string)$colConfig->attributes()->namespace;
+                            $pMetadata->xmlEntryNamespace = (string) $colConfig->attributes()->namespace;
                         }
 
                         if (isset($colConfig->attributes()->{'key-attribute-name'})) {
-                            $pMetadata->xmlKeyAttribute = (string)$colConfig->attributes()->{'key-attribute-name'};
+                            $pMetadata->xmlKeyAttribute = (string) $colConfig->attributes()->{'key-attribute-name'};
                         }
                     }
 
                     if (isset($pElem->{'xml-element'})) {
                         $colConfig = $pElem->{'xml-element'};
                         if (isset($colConfig->attributes()->cdata)) {
-                            $pMetadata->xmlElementCData = 'true' === (string)$colConfig->attributes()->cdata;
+                            $pMetadata->xmlElementCData = 'true' === (string) $colConfig->attributes()->cdata;
                         }
 
                         if (isset($colConfig->attributes()->namespace)) {
-                            $pMetadata->xmlNamespace = (string)$colConfig->attributes()->namespace;
+                            $pMetadata->xmlNamespace = (string) $colConfig->attributes()->namespace;
                         }
                     }
 
                     if (isset($pElem->attributes()->{'xml-attribute'})) {
-                        $pMetadata->xmlAttribute = 'true' === (string)$pElem->attributes()->{'xml-attribute'};
+                        $pMetadata->xmlAttribute = 'true' === (string) $pElem->attributes()->{'xml-attribute'};
                     }
 
                     if (isset($pElem->attributes()->{'xml-attribute-map'})) {
-                        $pMetadata->xmlAttributeMap = 'true' === (string)$pElem->attributes()->{'xml-attribute-map'};
+                        $pMetadata->xmlAttributeMap = 'true' === (string) $pElem->attributes()->{'xml-attribute-map'};
                     }
 
                     if (isset($pElem->attributes()->{'xml-value'})) {
-                        $pMetadata->xmlValue = 'true' === (string)$pElem->attributes()->{'xml-value'};
+                        $pMetadata->xmlValue = 'true' === (string) $pElem->attributes()->{'xml-value'};
                     }
 
                     if (isset($pElem->attributes()->{'xml-key-value-pairs'})) {
-                        $pMetadata->xmlKeyValuePairs = 'true' === (string)$pElem->attributes()->{'xml-key-value-pairs'};
+                        $pMetadata->xmlKeyValuePairs = 'true' === (string) $pElem->attributes()->{'xml-key-value-pairs'};
                     }
 
                     if (isset($pElem->attributes()->{'max-depth'})) {
-                        $pMetadata->maxDepth = (int)$pElem->attributes()->{'max-depth'};
+                        $pMetadata->maxDepth = (int) $pElem->attributes()->{'max-depth'};
                     }
 
                     //we need read-only before setter and getter set, because that method depends on flag being set
                     if (null !== $readOnly = $pElem->attributes()->{'read-only'}) {
-                        $pMetadata->readOnly = 'true' === strtolower((string)$readOnly);
+                        $pMetadata->readOnly = 'true' === strtolower((string) $readOnly);
                     } else {
                         $pMetadata->readOnly = $pMetadata->readOnly || $readOnlyClass;
                     }
@@ -309,26 +309,26 @@ 
                     $getter = $pElem->attributes()->{'accessor-getter'};
                     $setter = $pElem->attributes()->{'accessor-setter'};
                     $pMetadata->setAccessor(
-                        (string)($pElem->attributes()->{'access-type'} ?: $classAccessType),
-                        $getter ? (string)$getter : null,
-                        $setter ? (string)$setter : null
+                        (string) ($pElem->attributes()->{'access-type'} ?: $classAccessType),
+                        $getter ? (string) $getter : null,
+                        $setter ? (string) $setter : null
                     );
 
                     if (null !== $inline = $pElem->attributes()->inline) {
-                        $pMetadata->inline = 'true' === strtolower((string)$inline);
+                        $pMetadata->inline = 'true' === strtolower((string) $inline);
                     }
                 }
 
-                if (!$pMetadata->serializedName) {
+                if ( ! $pMetadata->serializedName) {
                     $pMetadata->serializedName = $this->namingStrategy->translateName($pMetadata);
                 }
 
-                if (!empty($pElem) && null !== $name = $pElem->attributes()->name) {
-                    $pMetadata->name = (string)$name;
+                if ( ! empty($pElem) && null !== $name = $pElem->attributes()->name) {
+                    $pMetadata->name = (string) $name;
                 }
 
-                if ((ExclusionPolicy::NONE === (string)$exclusionPolicy && !$isExclude)
-                    || (ExclusionPolicy::ALL === (string)$exclusionPolicy && $isExpose)
+                if ((ExclusionPolicy::NONE === (string) $exclusionPolicy && ! $isExclude)
+                    || (ExclusionPolicy::ALL === (string) $exclusionPolicy && $isExpose)
                 ) {
 
 
@@ -338,31 +338,31 @@ 
         }
 
         foreach ($elem->xpath('./callback-method') as $method) {
-            if (!isset($method->attributes()->type)) {
+            if ( ! isset($method->attributes()->type)) {
                 throw new RuntimeException('The type attribute must be set for all callback-method elements.');
             }
-            if (!isset($method->attributes()->name)) {
+            if ( ! isset($method->attributes()->name)) {
                 throw new RuntimeException('The name attribute must be set for all callback-method elements.');
             }
 
-            switch ((string)$method->attributes()->type) {
+            switch ((string) $method->attributes()->type) {
                 case 'pre-serialize':
-                    $metadata->addPreSerializeMethod(new MethodMetadata($name, (string)$method->attributes()->name));
+                    $metadata->addPreSerializeMethod(new MethodMetadata($name, (string) $method->attributes()->name));
                     break;
 
                 case 'post-serialize':
-                    $metadata->addPostSerializeMethod(new MethodMetadata($name, (string)$method->attributes()->name));
+                    $metadata->addPostSerializeMethod(new MethodMetadata($name, (string) $method->attributes()->name));
                     break;
 
                 case 'post-deserialize':
-                    $metadata->addPostDeserializeMethod(new MethodMetadata($name, (string)$method->attributes()->name));
+                    $metadata->addPostDeserializeMethod(new MethodMetadata($name, (string) $method->attributes()->name));
                     break;
 
                 case 'handler':
-                    if (!isset($method->attributes()->format)) {
+                    if ( ! isset($method->attributes()->format)) {
                         throw new RuntimeException('The format attribute must be set for "handler" callback methods.');
                     }
-                    if (!isset($method->attributes()->direction)) {
+                    if ( ! isset($method->attributes()->direction)) {
                         throw new RuntimeException('The direction attribute must be set for "handler" callback methods.');
                     }
 

src/Metadata/Driver/NullDriver.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 
 class NullDriver implements DriverInterface
 {
-    public function loadMetadataForClass(\ReflectionClass $class):?\Metadata\ClassMetadata
+    public function loadMetadataForClass(\ReflectionClass $class): ?\Metadata\ClassMetadata
     {
         $classMetadata = new ClassMetadata($name = $class->name);
         $classMetadata->fileResources[] = $class->getFilename();

src/Metadata/Driver/YamlDriver.php

Spacing +46 added lines, -46 removed lines

@@ -49,11 +49,11 @@ 
         $this->namingStrategy = $namingStrategy;
     }
 
-    protected function loadMetadataFromFile(\ReflectionClass $class, string $file):?\Metadata\ClassMetadata
+    protected function loadMetadataFromFile(\ReflectionClass $class, string $file): ?\Metadata\ClassMetadata
     {
         $config = Yaml::parse(file_get_contents($file));
 
-        if (!isset($config[$name = $class->name])) {
+        if ( ! isset($config[$name = $class->name])) {
             throw new RuntimeException(sprintf('Expected metadata for class %s to be defined in %s.', $class->name, $file));
         }
 
@@ -62,9 +62,9 @@ 
         $metadata->fileResources[] = $file;
         $metadata->fileResources[] = $class->getFileName();
         $exclusionPolicy = isset($config['exclusion_policy']) ? strtoupper($config['exclusion_policy']) : 'NONE';
-        $excludeAll = isset($config['exclude']) ? (Boolean)$config['exclude'] : false;
+        $excludeAll = isset($config['exclude']) ? (Boolean) $config['exclude'] : false;
         $classAccessType = isset($config['access_type']) ? $config['access_type'] : PropertyMetadata::ACCESS_TYPE_PROPERTY;
-        $readOnlyClass = isset($config['read_only']) ? (Boolean)$config['read_only'] : false;
+        $readOnlyClass = isset($config['read_only']) ? (Boolean) $config['read_only'] : false;
         $this->addClassProperties($metadata, $config);
 
         $propertiesMetadata = [];
@@ -75,20 +75,20 @@ 
                     unset($propertySettings['exp']);
                 } else {
 
-                    if (!$class->hasMethod($methodName)) {
-                        throw new RuntimeException('The method ' . $methodName . ' not found in class ' . $class->name);
+                    if ( ! $class->hasMethod($methodName)) {
+                        throw new RuntimeException('The method '.$methodName.' not found in class '.$class->name);
                     }
                     $virtualPropertyMetadata = new VirtualPropertyMetadata($name, $methodName);
                 }
 
-                $pName = !empty($propertySettings["name"]) ? $propertySettings["name"] : $virtualPropertyMetadata->name;
+                $pName = ! empty($propertySettings["name"]) ? $propertySettings["name"] : $virtualPropertyMetadata->name;
 
                 $propertiesMetadata[$pName] = $virtualPropertyMetadata;
                 $config['properties'][$pName] = $propertySettings;
             }
         }
 
-        if (!$excludeAll) {
+        if ( ! $excludeAll) {
             foreach ($class->getProperties() as $property) {
                 if ($property->class !== $name || (isset($property->info) && $property->info['class'] !== $name)) {
                     continue;
@@ -108,7 +108,7 @@ 
                     $pConfig = $config['properties'][$pName];
 
                     if (isset($pConfig['exclude'])) {
-                        $isExclude = (Boolean)$pConfig['exclude'];
+                        $isExclude = (Boolean) $pConfig['exclude'];
                     }
 
                     if ($isExclude) {
@@ -116,35 +116,35 @@ 
                     }
 
                     if (isset($pConfig['expose'])) {
-                        $isExpose = (Boolean)$pConfig['expose'];
+                        $isExpose = (Boolean) $pConfig['expose'];
                     }
 
                     if (isset($pConfig['skip_when_empty'])) {
-                        $pMetadata->skipWhenEmpty = (Boolean)$pConfig['skip_when_empty'];
+                        $pMetadata->skipWhenEmpty = (Boolean) $pConfig['skip_when_empty'];
                     }
 
                     if (isset($pConfig['since_version'])) {
-                        $pMetadata->sinceVersion = (string)$pConfig['since_version'];
+                        $pMetadata->sinceVersion = (string) $pConfig['since_version'];
                     }
 
                     if (isset($pConfig['until_version'])) {
-                        $pMetadata->untilVersion = (string)$pConfig['until_version'];
+                        $pMetadata->untilVersion = (string) $pConfig['until_version'];
                     }
 
                     if (isset($pConfig['exclude_if'])) {
-                        $pMetadata->excludeIf = (string)$pConfig['exclude_if'];
+                        $pMetadata->excludeIf = (string) $pConfig['exclude_if'];
                     }
 
                     if (isset($pConfig['expose_if'])) {
-                        $pMetadata->excludeIf = "!(" . $pConfig['expose_if'] . ")";
+                        $pMetadata->excludeIf = "!(".$pConfig['expose_if'].")";
                     }
 
                     if (isset($pConfig['serialized_name'])) {
-                        $pMetadata->serializedName = (string)$pConfig['serialized_name'];
+                        $pMetadata->serializedName = (string) $pConfig['serialized_name'];
                     }
 
                     if (isset($pConfig['type'])) {
-                        $pMetadata->setType($this->typeParser->parse((string)$pConfig['type']));
+                        $pMetadata->setType($this->typeParser->parse((string) $pConfig['type']));
                     }
 
                     if (isset($pConfig['groups'])) {
@@ -156,21 +156,21 @@ 
 
                         $colConfig = $pConfig['xml_list'];
                         if (isset($colConfig['inline'])) {
-                            $pMetadata->xmlCollectionInline = (Boolean)$colConfig['inline'];
+                            $pMetadata->xmlCollectionInline = (Boolean) $colConfig['inline'];
                         }
 
                         if (isset($colConfig['entry_name'])) {
-                            $pMetadata->xmlEntryName = (string)$colConfig['entry_name'];
+                            $pMetadata->xmlEntryName = (string) $colConfig['entry_name'];
                         }
 
                         if (isset($colConfig['skip_when_empty'])) {
-                            $pMetadata->xmlCollectionSkipWhenEmpty = (Boolean)$colConfig['skip_when_empty'];
+                            $pMetadata->xmlCollectionSkipWhenEmpty = (Boolean) $colConfig['skip_when_empty'];
                         } else {
                             $pMetadata->xmlCollectionSkipWhenEmpty = true;
                         }
 
                         if (isset($colConfig['namespace'])) {
-                            $pMetadata->xmlEntryNamespace = (string)$colConfig['namespace'];
+                            $pMetadata->xmlEntryNamespace = (string) $colConfig['namespace'];
                         }
                     }
 
@@ -179,15 +179,15 @@ 
 
                         $colConfig = $pConfig['xml_map'];
                         if (isset($colConfig['inline'])) {
-                            $pMetadata->xmlCollectionInline = (Boolean)$colConfig['inline'];
+                            $pMetadata->xmlCollectionInline = (Boolean) $colConfig['inline'];
                         }
 
                         if (isset($colConfig['entry_name'])) {
-                            $pMetadata->xmlEntryName = (string)$colConfig['entry_name'];
+                            $pMetadata->xmlEntryName = (string) $colConfig['entry_name'];
                         }
 
                         if (isset($colConfig['namespace'])) {
-                            $pMetadata->xmlEntryNamespace = (string)$colConfig['namespace'];
+                            $pMetadata->xmlEntryNamespace = (string) $colConfig['namespace'];
                         }
 
                         if (isset($colConfig['key_attribute_name'])) {
@@ -198,33 +198,33 @@ 
                     if (isset($pConfig['xml_element'])) {
                         $colConfig = $pConfig['xml_element'];
                         if (isset($colConfig['cdata'])) {
-                            $pMetadata->xmlElementCData = (Boolean)$colConfig['cdata'];
+                            $pMetadata->xmlElementCData = (Boolean) $colConfig['cdata'];
                         }
 
                         if (isset($colConfig['namespace'])) {
-                            $pMetadata->xmlNamespace = (string)$colConfig['namespace'];
+                            $pMetadata->xmlNamespace = (string) $colConfig['namespace'];
                         }
                     }
 
                     if (isset($pConfig['xml_attribute'])) {
-                        $pMetadata->xmlAttribute = (Boolean)$pConfig['xml_attribute'];
+                        $pMetadata->xmlAttribute = (Boolean) $pConfig['xml_attribute'];
                     }
 
                     if (isset($pConfig['xml_attribute_map'])) {
-                        $pMetadata->xmlAttributeMap = (Boolean)$pConfig['xml_attribute_map'];
+                        $pMetadata->xmlAttributeMap = (Boolean) $pConfig['xml_attribute_map'];
                     }
 
                     if (isset($pConfig['xml_value'])) {
-                        $pMetadata->xmlValue = (Boolean)$pConfig['xml_value'];
+                        $pMetadata->xmlValue = (Boolean) $pConfig['xml_value'];
                     }
 
                     if (isset($pConfig['xml_key_value_pairs'])) {
-                        $pMetadata->xmlKeyValuePairs = (Boolean)$pConfig['xml_key_value_pairs'];
+                        $pMetadata->xmlKeyValuePairs = (Boolean) $pConfig['xml_key_value_pairs'];
                     }
 
                     //we need read_only before setter and getter set, because that method depends on flag being set
                     if (isset($pConfig['read_only'])) {
-                        $pMetadata->readOnly = (Boolean)$pConfig['read_only'];
+                        $pMetadata->readOnly = (Boolean) $pConfig['read_only'];
                     } else {
                         $pMetadata->readOnly = $pMetadata->readOnly || $readOnlyClass;
                     }
@@ -236,15 +236,15 @@ 
                     );
 
                     if (isset($pConfig['inline'])) {
-                        $pMetadata->inline = (Boolean)$pConfig['inline'];
+                        $pMetadata->inline = (Boolean) $pConfig['inline'];
                     }
 
                     if (isset($pConfig['max_depth'])) {
-                        $pMetadata->maxDepth = (int)$pConfig['max_depth'];
+                        $pMetadata->maxDepth = (int) $pConfig['max_depth'];
                     }
                 }
 
-                if (!$pMetadata->serializedName) {
+                if ( ! $pMetadata->serializedName) {
                     $pMetadata->serializedName = $this->namingStrategy->translateName($pMetadata);
                 }
 
@@ -252,11 +252,11 @@ 
                     $pConfig = $config['properties'][$pName];
 
                     if (isset($pConfig['name'])) {
-                        $pMetadata->name = (string)$pConfig['name'];
+                        $pMetadata->name = (string) $pConfig['name'];
                     }
                 }
 
-                if ((ExclusionPolicy::NONE === $exclusionPolicy && !$isExclude)
+                if ((ExclusionPolicy::NONE === $exclusionPolicy && ! $isExclude)
                     || (ExclusionPolicy::ALL === $exclusionPolicy && $isExpose)
                 ) {
                     $metadata->addPropertyMetadata($pMetadata);
@@ -288,7 +288,7 @@ 
 
     private function addClassProperties(ClassMetadata $metadata, array $config)
     {
-        if (isset($config['custom_accessor_order']) && !isset($config['accessor_order'])) {
+        if (isset($config['custom_accessor_order']) && ! isset($config['accessor_order'])) {
             $config['accessor_order'] = 'custom';
         }
 
@@ -297,11 +297,11 @@ 
         }
 
         if (isset($config['xml_root_name'])) {
-            $metadata->xmlRootName = (string)$config['xml_root_name'];
+            $metadata->xmlRootName = (string) $config['xml_root_name'];
         }
 
         if (isset($config['xml_root_namespace'])) {
-            $metadata->xmlRootNamespace = (string)$config['xml_root_namespace'];
+            $metadata->xmlRootNamespace = (string) $config['xml_root_namespace'];
         }
 
         if (array_key_exists('xml_namespaces', $config)) {
@@ -315,25 +315,25 @@ 
             if (isset($config['discriminator']['disabled']) && true === $config['discriminator']['disabled']) {
                 $metadata->discriminatorDisabled = true;
             } else {
-                if (!isset($config['discriminator']['field_name'])) {
+                if ( ! isset($config['discriminator']['field_name'])) {
                     throw new RuntimeException('The "field_name" attribute must be set for discriminators.');
                 }
 
-                if (!isset($config['discriminator']['map']) || !\is_array($config['discriminator']['map'])) {
+                if ( ! isset($config['discriminator']['map']) || ! \is_array($config['discriminator']['map'])) {
                     throw new RuntimeException('The "map" attribute must be set, and be an array for discriminators.');
                 }
                 $groups = isset($config['discriminator']['groups']) ? $config['discriminator']['groups'] : [];
                 $metadata->setDiscriminator($config['discriminator']['field_name'], $config['discriminator']['map'], $groups);
 
                 if (isset($config['discriminator']['xml_attribute'])) {
-                    $metadata->xmlDiscriminatorAttribute = (bool)$config['discriminator']['xml_attribute'];
+                    $metadata->xmlDiscriminatorAttribute = (bool) $config['discriminator']['xml_attribute'];
                 }
                 if (isset($config['discriminator']['xml_element'])) {
                     if (isset($config['discriminator']['xml_element']['cdata'])) {
-                        $metadata->xmlDiscriminatorCData = (bool)$config['discriminator']['xml_element']['cdata'];
+                        $metadata->xmlDiscriminatorCData = (bool) $config['discriminator']['xml_element']['cdata'];
                     }
                     if (isset($config['discriminator']['xml_element']['namespace'])) {
-                        $metadata->xmlDiscriminatorNamespace = (string)$config['discriminator']['xml_element']['namespace'];
+                        $metadata->xmlDiscriminatorNamespace = (string) $config['discriminator']['xml_element']['namespace'];
                     }
                 }
             }
@@ -344,13 +344,13 @@ 
     {
         if (\is_string($config)) {
             $config = [$config];
-        } elseif (!\is_array($config)) {
+        } elseif ( ! \is_array($config)) {
             throw new RuntimeException(sprintf('callback methods expects a string, or an array of strings that represent method names, but got %s.', json_encode($config['pre_serialize'])));
         }
 
         $methods = [];
         foreach ($config as $name) {
-            if (!$class->hasMethod($name)) {
+            if ( ! $class->hasMethod($name)) {
                 throw new RuntimeException(sprintf('The method %s does not exist in class %s.', $name, $class->name));
             }
 

src/Metadata/Driver/AbstractDoctrineTypeDriver.php

Spacing +3 added lines, -3 removed lines

@@ -88,7 +88,7 @@ 
         $classMetadata = $this->delegate->loadMetadataForClass($class);
 
         // Abort if the given class is not a mapped entity
-        if (!$doctrineMetadata = $this->tryLoadingDoctrineMetadata($class->name)) {
+        if ( ! $doctrineMetadata = $this->tryLoadingDoctrineMetadata($class->name)) {
             return $classMetadata;
         }
 
@@ -153,7 +153,7 @@ 
      */
     protected function tryLoadingDoctrineMetadata(string $className): ?DoctrineClassMetadata
     {
-        if (!$manager = $this->registry->getManagerForClass($className)) {
+        if ( ! $manager = $this->registry->getManagerForClass($className)) {
             return null;
         }
 
@@ -166,7 +166,7 @@ 
 
     protected function normalizeFieldType($type): ?string
     {
-        if (!isset($this->fieldMapping[$type])) {
+        if ( ! isset($this->fieldMapping[$type])) {
             return null;
         }