schmittjoh / serializer

src/XmlDeserializationVisitor.php

Doc Comments +3 added lines, -1 removed lines

@@ -85,6 +85,9 @@ 
         return $doc;
     }
 
+    /**
+     * @return string
+     */
     private function emptyStringToSpaceCharacter($data)
     {
         return $data === '' ? ' ' : $data;
@@ -359,7 +362,6 @@ 
     /**
      * Retrieves internalSubset even in bugfixed php versions
      *
-     * @param \DOMDocumentType $child
      * @param string $data
      * @return string
      */

Spacing +13 added lines, -13 removed lines

@@ -60,7 +60,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
@@ -92,12 +92,12 @@ 
 
     public function visitString($data, array $type, Context $context)
     {
-        return (string)$data;
+        return (string) $data;
     }
 
     public function visitBoolean($data, array $type, Context $context)
     {
-        $data = (string)$data;
+        $data = (string) $data;
 
         if ('true' === $data || '1' === $data) {
             $data = true;
@@ -112,12 +112,12 @@ 
 
     public function visitInteger($data, array $type, Context $context)
     {
-        return (integer)$data;
+        return (integer) $data;
     }
 
     public function visitDouble($data, array $type, Context $context)
     {
-        return (double)$data;
+        return (double) $data;
     }
 
     public function visitArray($data, array $type, Context $context)
@@ -138,7 +138,7 @@ 
             $nodes = $data->xpath($entryName);
         }
 
-        if (!count($nodes)) {
+        if ( ! count($nodes)) {
             return array();
         }
 
@@ -165,7 +165,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));
                     }
 
@@ -190,7 +190,7 @@ 
     {
         $name = $this->namingStrategy->translateName($metadata);
 
-        if (!$metadata->type) {
+        if ( ! $metadata->type) {
             throw new RuntimeException(sprintf('You must define a type for %s::$%s.', $metadata->reflection->class, $metadata->name));
         }
 
@@ -214,7 +214,7 @@ 
 
         if ($metadata->xmlCollection) {
             $enclosingElem = $data;
-            if (!$metadata->xmlCollectionInline) {
+            if ( ! $metadata->xmlCollectionInline) {
                 $enclosingElem = $data->children($metadata->xmlNamespace)->$name;
             }
 
@@ -228,7 +228,7 @@ 
 
         if ($metadata->xmlNamespace) {
             $node = $data->children($metadata->xmlNamespace)->$name;
-            if (!$node->count()) {
+            if ( ! $node->count()) {
                 return;
             }
         } else {
@@ -238,9 +238,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)) {
                 return;
@@ -296,7 +296,7 @@ 
 
     public function getResult()
     {
-        throw new \Exception(__METHOD__ . " has been deprecated for deserialization visitors");
+        throw new \Exception(__METHOD__." has been deprecated for deserialization visitors");
     }
 
     /**

src/Serializer.php

Braces +1 added lines, -2 removed lines

@@ -83,8 +83,7 @@
         EventDispatcherInterface $dispatcher = null,
         TypeParser $typeParser = null,
         ExpressionEvaluatorInterface $expressionEvaluator = null
-    )
-    {
+    ) {
         $this->factory = $factory;
         $this->handlerRegistry = $handlerRegistry;
         $this->objectConstructor = $objectConstructor;

Doc Comments +4 added lines, -1 removed lines

@@ -73,7 +73,7 @@ 
      * @param Construction\ObjectConstructorInterface $objectConstructor
      * @param \PhpCollection\MapInterface $serializationVisitors of VisitorInterface
      * @param \PhpCollection\MapInterface $deserializationVisitors of VisitorInterface
-     * @param EventDispatcher\EventDispatcherInterface $dispatcher
+     * @param EventDispatcherInterface $dispatcher
      * @param TypeParser $typeParser
      * @param ExpressionEvaluatorInterface|null $expressionEvaluator
      */
@@ -181,6 +181,9 @@ 
             ->get();
     }
 
+    /**
+     * @param string $format
+     */
     private function visit(GraphNavigatorInterface $navigator, $visitor, Context $context, $data, $format, array $type = null)
     {
         $context->initialize(

Spacing +5 added lines, -5 removed lines

@@ -110,7 +110,7 @@ 
         }
 
         return $this->serializationVisitors->get($format)
-            ->map(function (SerializationVisitorInterface $visitor) use ($context, $data, $format, $type) {
+            ->map(function(SerializationVisitorInterface $visitor) use ($context, $data, $format, $type) {
 
                 $type = $this->findInitialType($type, $context);
 
@@ -138,7 +138,7 @@ 
         }
 
         return $this->deserializationVisitors->get($format)
-            ->map(function (DeserializationVisitorInterface $visitor) use ($context, $data, $format, $type) {
+            ->map(function(DeserializationVisitorInterface $visitor) use ($context, $data, $format, $type) {
                 $preparedData = $visitor->prepare($data);
                 return $this->visit($this->deserializationNavigator, $visitor, $context, $preparedData, $format, $this->typeParser->parse($type));
             })
@@ -155,14 +155,14 @@ 
         }
 
         return $this->serializationVisitors->get('json')
-            ->map(function (JsonSerializationVisitor $visitor) use ($context, $data, $type) {
+            ->map(function(JsonSerializationVisitor $visitor) use ($context, $data, $type) {
 
                 $type = $this->findInitialType($type, $context);
 
                 $this->visit($this->serializationNavigator, $visitor, $context, $data, 'json', $type);
                 $result = $this->removeInternalArrayObjects($visitor->getRoot());
 
-                if (!is_array($result)) {
+                if ( ! is_array($result)) {
                     throw new RuntimeException(sprintf(
                         'The input data of type "%s" did not convert to an array, but got a result of type "%s".',
                         is_object($data) ? get_class($data) : gettype($data),
@@ -185,7 +185,7 @@ 
         }
 
         return $this->deserializationVisitors->get('json')
-            ->map(function (JsonDeserializationVisitor $visitor) use ($data, $type, $context) {
+            ->map(function(JsonDeserializationVisitor $visitor) use ($data, $type, $context) {
                 return $this->visit($this->deserializationNavigator, $visitor, $context, $data, 'json', $this->typeParser->parse($type));
             })
             ->get();

src/Annotation/VirtualProperty.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
         }
 
         foreach ($data as $key => $value) {
-            if (!property_exists(__CLASS__, $key)) {
+            if ( ! property_exists(__CLASS__, $key)) {
                 throw new \BadMethodCallException(sprintf('Unknown property "%s" on annotation "%s".', $key, __CLASS__));
             }
             $this->{$key} = $value;

src/Handler/DateHandler.php

Braces +2 added lines, -4 removed lines

@@ -73,8 +73,7 @@ 
         \DateTimeInterface $date,
         array $type,
         SerializationContext $context
-    )
-    {
+    ) {
         if ($visitor instanceof XmlSerializationVisitor && false === $this->xmlCData) {
             return $visitor->visitSimpleString($date->format($this->getFormat($type)), $type, $context);
         }
@@ -97,8 +96,7 @@ 
         \DateTimeImmutable $date,
         array $type,
         SerializationContext $context
-    )
-    {
+    ) {
         return $this->serializeDateTimeInterface($visitor, $date, $type, $context);
     }
 

Spacing +10 added lines, -10 removed lines

@@ -53,7 +53,7 @@ 
                     'type' => $type,
                     'format' => $format,
                     'direction' => GraphNavigator::DIRECTION_SERIALIZATION,
-                    'method' => 'serialize' . $type,
+                    'method' => 'serialize'.$type,
                 );
             }
         }
@@ -116,7 +116,7 @@ 
     private function isDataXmlNull($data)
     {
         $attributes = $data->attributes('xsi', true);
-        return isset($attributes['nil'][0]) && (string)$attributes['nil'][0] === 'true';
+        return isset($attributes['nil'][0]) && (string) $attributes['nil'][0] === 'true';
     }
 
     public function deserializeDateTimeFromXml(XmlDeserializationVisitor $visitor, $data, array $type)
@@ -179,9 +179,9 @@ 
         $format = $this->getFormat($type);
 
         if ($immutable) {
-            $datetime = \DateTimeImmutable::createFromFormat($format, (string)$data, $timezone);
+            $datetime = \DateTimeImmutable::createFromFormat($format, (string) $data, $timezone);
         } else {
-            $datetime = \DateTime::createFromFormat($format, (string)$data, $timezone);
+            $datetime = \DateTime::createFromFormat($format, (string) $data, $timezone);
         }
 
         if (false === $datetime) {
@@ -221,15 +221,15 @@ 
         $format = 'P';
 
         if (0 < $dateInterval->y) {
-            $format .= $dateInterval->y . 'Y';
+            $format .= $dateInterval->y.'Y';
         }
 
         if (0 < $dateInterval->m) {
-            $format .= $dateInterval->m . 'M';
+            $format .= $dateInterval->m.'M';
         }
 
         if (0 < $dateInterval->d) {
-            $format .= $dateInterval->d . 'D';
+            $format .= $dateInterval->d.'D';
         }
 
         if (0 < $dateInterval->h || 0 < $dateInterval->i || 0 < $dateInterval->s) {
@@ -237,15 +237,15 @@ 
         }
 
         if (0 < $dateInterval->h) {
-            $format .= $dateInterval->h . 'H';
+            $format .= $dateInterval->h.'H';
         }
 
         if (0 < $dateInterval->i) {
-            $format .= $dateInterval->i . 'M';
+            $format .= $dateInterval->i.'M';
         }
 
         if (0 < $dateInterval->s) {
-            $format .= $dateInterval->s . 'S';
+            $format .= $dateInterval->s.'S';
         }
 
         if ($format === 'P') {

src/Handler/StdClassHandler.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
         $classMetadata = $context->getMetadataFactory()->getMetadataForClass('stdClass');
         $visitor->startVisitingObject($classMetadata, $stdClass, array('name' => 'stdClass'), $context);
 
-        foreach ((array)$stdClass as $name => $value) {
+        foreach ((array) $stdClass as $name => $value) {
             $metadata = new StaticPropertyMetadata('stdClass', $name, $value);
             $visitor->visitProperty($metadata, $value, $context);
         }

src/Expression/ExpressionEvaluator.php

Spacing +2 added lines, -2 removed lines

@@ -64,13 +64,13 @@
      */
     public function evaluate($expression, array $data = array())
     {
-        if (!is_string($expression)) {
+        if ( ! is_string($expression)) {
             return $expression;
         }
 
         $context = $data + $this->context;
 
-        if (!array_key_exists($expression, $this->cache)) {
+        if ( ! array_key_exists($expression, $this->cache)) {
             $this->cache[$expression] = $this->expressionLanguage->parse($expression, array_keys($context));
         }
 

src/EventDispatcher/Subscriber/SymfonyValidatorValidatorSubscriber.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
             ? null
             : $context->attributes->get('validation_groups')->get();
 
-        if (!$groups) {
+        if ( ! $groups) {
             return;
         }
 

src/GraphNavigator.php

Braces +1 added lines, -2 removed lines

@@ -62,8 +62,7 @@
         MetadataFactoryInterface $metadataFactory,
         HandlerRegistryInterface $handlerRegistry,
         EventDispatcherInterface $dispatcher
-    )
-    {
+    ) {
         $this->dispatcher = $dispatcher;
         $this->metadataFactory = $metadataFactory;
         $this->handlerRegistry = $handlerRegistry;

src/SerializationGraphNavigator.php

Spacing +2 added lines, -2 removed lines

@@ -110,7 +110,7 @@ 
             case 'resource':
                 $msg = 'Resources are not supported in serialized data.';
                 if (null !== $path = $context->getPath()) {
-                    $msg .= ' Path: ' . $path;
+                    $msg .= ' Path: '.$path;
                 }
 
                 throw new RuntimeException($msg);
@@ -154,7 +154,7 @@ 
                 /** @var $metadata ClassMetadata */
                 $metadata = $this->metadataFactory->getMetadataForClass($type['name']);
 
-                if ($metadata->usingExpression && !$this->expressionExclusionStrategy) {
+                if ($metadata->usingExpression && ! $this->expressionExclusionStrategy) {
                     throw new ExpressionLanguageRequiredException("To use conditional exclude/expose in {$metadata->name} you must configure the expression language.");
                 }
 

Braces +1 added lines, -2 removed lines

@@ -49,8 +49,7 @@
         HandlerRegistryInterface $handlerRegistry,
         EventDispatcherInterface $dispatcher,
         ExpressionEvaluatorInterface $expressionEvaluator = null
-    )
-    {
+    ) {
         parent::__construct($metadataFactory, $handlerRegistry, $dispatcher);
         if ($expressionEvaluator) {
             $this->expressionExclusionStrategy = new ExpressionLanguageExclusionStrategy($expressionEvaluator);