schmittjoh / serializer

src/JMS/Serializer/EventDispatcher/EventDispatcher.php

Spacing +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
             $this->classListeners[$eventName][$loweredClass][$format] = $this->initializeListeners($eventName, $loweredClass, $format);
         }
 
-        return !!$this->classListeners[$eventName][$loweredClass][$format];
+        return ! ! $this->classListeners[$eventName][$loweredClass][$format];
     }
 
     public function dispatch($eventName, $class, $format, Event $event)

src/JMS/Serializer/EventDispatcher/Events.php

Braces +3 added lines, -1 removed lines

@@ -25,5 +25,7 @@
     const PRE_DESERIALIZE = 'serializer.pre_deserialize';
     const POST_DESERIALIZE = 'serializer.post_deserialize';
 
-    final private function __construct() { }
+    final private function __construct()
+    {
+}
 }

src/JMS/Serializer/Metadata/Driver/YamlDriver.php

Indentation +1 added lines, -1 removed lines

@@ -167,7 +167,7 @@
 
                     //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;
                     }

Spacing +5 added lines, -5 removed lines

@@ -53,7 +53,7 @@ 
         if (array_key_exists('virtual_properties', $config)) {
             foreach ($config['virtual_properties'] as $methodName => $propertySettings) {
                 if (isset($propertySettings['exp'])) {
-                    $virtualPropertyMetadata = new ExpressionPropertyMetadata( $name, $methodName, $propertySettings['exp']);
+                    $virtualPropertyMetadata = new ExpressionPropertyMetadata($name, $methodName, $propertySettings['exp']);
                     unset($propertySettings['exp']);
 
                 } else {
@@ -107,7 +107,7 @@ 
                     }
 
                     if (isset($pConfig['expose_if'])) {
-                        $pMetadata->excludeIf = "!(" . $pConfig['expose_if'].")";
+                        $pMetadata->excludeIf = "!(".$pConfig['expose_if'].")";
                     }
 
                     if (isset($pConfig['serialized_name'])) {
@@ -127,15 +127,15 @@ 
 
                         $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;
                         }

src/JMS/Serializer/TypeParser.php

Indentation +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
                 |(.)
             /x',
             array(self::T_NAME => 'T_NAME', self::T_STRING => 'T_STRING', self::T_OPEN_BRACKET => 'T_OPEN_BRACKET',
-                  self::T_CLOSE_BRACKET => 'T_CLOSE_BRACKET', self::T_COMMA => 'T_COMMA', self::T_NONE => 'T_NONE'),
+                    self::T_CLOSE_BRACKET => 'T_CLOSE_BRACKET', self::T_COMMA => 'T_COMMA', self::T_NONE => 'T_NONE'),
             function($value) {
                 switch ($value[0]) {
                     case '"':

src/JMS/Serializer/Util/Writer.php

Spacing +3 added lines, -3 removed lines

@@ -83,16 +83,16 @@
         $addition = '';
 
         $lines = explode("\n", $content);
-        for ($i=0,$c=count($lines); $i<$c; $i++) {
+        for ($i = 0, $c = count($lines); $i < $c; $i++) {
             if ($this->indentationLevel > 0
-                && !empty($lines[$i])
+                && ! empty($lines[$i])
                 && ((empty($addition) && "\n" === substr($this->content, -1)) || "\n" === substr($addition, -1))) {
                 $addition .= str_repeat(' ', $this->indentationLevel * $this->indentationSpaces);
             }
 
             $addition .= $lines[$i];
 
-            if ($i+1 < $c) {
+            if ($i + 1 < $c) {
                 $addition .= "\n";
             }
         }

src/JMS/Serializer/YamlSerializationVisitor.php

Indentation +2 added lines, -2 removed lines

@@ -168,8 +168,8 @@
 
         if ( ! $metadata->inline) {
             $this->writer
-                 ->writeln(Inline::dump($name).':')
-                 ->indent();
+                    ->writeln(Inline::dump($name).':')
+                    ->indent();
         }
 
         $this->setCurrentMetadata($metadata);

Spacing +1 added lines, -1 removed lines

@@ -91,7 +91,7 @@
                 continue;
             }
 
-            if ($isList && !$isHash) {
+            if ($isList && ! $isHash) {
                 $this->writer->writeln('-');
             } else {
                 $this->writer->writeln(Inline::dump($k).':');

src/JMS/Serializer/XmlDeserializationVisitor.php

Indentation +1 added lines, -1 removed lines

@@ -226,7 +226,7 @@
             throw new RuntimeException(sprintf('You must define a type for %s::$%s.', $metadata->reflection->class, $metadata->name));
         }
 
-       if ($metadata->xmlAttribute) {
+        if ($metadata->xmlAttribute) {
 
             $attributes = $data->attributes($metadata->xmlNamespace);
             if (isset($attributes[$name])) {

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 +7 added lines, -7 removed lines

@@ -63,7 +63,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
@@ -147,7 +147,7 @@ 
 
         if ($namespace === null && $this->objectMetadataStack->count()) {
             $classMetadata = $this->objectMetadataStack->top();
-            $namespace = isset($classMetadata->xmlNamespaces[''])?$classMetadata->xmlNamespaces['']:$namespace;
+            $namespace = isset($classMetadata->xmlNamespaces['']) ? $classMetadata->xmlNamespaces[''] : $namespace;
         }
 
         if (null !== $namespace) {
@@ -158,7 +158,7 @@ 
             $nodes = $data->xpath($entryName);
         }
 
-        if (!count($nodes)) {
+        if ( ! count($nodes)) {
             if (null === $this->result) {
                 return $this->result = array();
             }
@@ -249,7 +249,7 @@ 
 
         if ($metadata->xmlCollection) {
             $enclosingElem = $data;
-            if (!$metadata->xmlCollectionInline) {
+            if ( ! $metadata->xmlCollectionInline) {
                 $enclosingElem = $data->children($metadata->xmlNamespace)->$name;
             }
 
@@ -263,7 +263,7 @@ 
 
         if ($metadata->xmlNamespace) {
             $node = $data->children($metadata->xmlNamespace)->$name;
-            if (!$node->count()) {
+            if ( ! $node->count()) {
                 return;
             }
         } else {
@@ -273,9 +273,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;

src/JMS/Serializer/Handler/DateHandler.php

Braces +2 added lines, -4 removed lines

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

src/JMS/Serializer/SerializationContext.php

Unused Use Statements -1 removed lines

@@ -18,7 +18,6 @@
 
 namespace JMS\Serializer;
 
-use JMS\Serializer\Exception\LogicException;
 use JMS\Serializer\Exception\RuntimeException;
 use Metadata\MetadataFactoryInterface;