schmittjoh / serializer

src/JMS/Serializer/Serializer.php

Doc Comments +3 added lines

@@ -162,6 +162,9 @@
         ;
     }
 
+    /**
+     * @param string $format
+     */
     private function visit(VisitorInterface $visitor, Context $context, $data, $format, array $type = null)
     {
         $context->initialize(

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/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/GenericSerializationVisitor.php

Doc Comments +3 added lines, -1 removed lines

@@ -32,6 +32,9 @@ 
     private $dataStack;
     private $data;
 
+    /**
+     * @param Naming\PropertyNamingStrategyInterface $namingStrategy
+     */
     public function __construct($namingStrategy)
     {
         parent::__construct($namingStrategy);
@@ -194,7 +197,6 @@ 
     /**
      * Allows you to remove data from the current object based on name.
      *
-     * @param string $key
      * @throws Exception\InvalidArgumentException
      */
     public function removeDataPropertyName($propertyName)

Spacing +2 added lines, -2 removed lines

@@ -206,7 +206,7 @@ 
         $propMetadata->name = $propertyName;
         $key = $this->namingStrategy->translateName($propMetadata);
 
-        if (!array_key_exists($key, $this->data)) {
+        if ( ! array_key_exists($key, $this->data)) {
             throw new InvalidArgumentException(sprintf('There is no data for "%s".', $key));
         }
 
@@ -223,7 +223,7 @@ 
      */
     public function removeData($key)
     {
-        if (!array_key_exists($key, $this->data)) {
+        if ( ! array_key_exists($key, $this->data)) {
             throw new InvalidArgumentException(sprintf('There is no data for "%s".', $key));
         }
 

src/JMS/Serializer/XmlDeserializationVisitor.php

Doc Comments -1 removed lines

@@ -346,7 +346,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,10 +147,10 @@ 
 
         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 ( ! isset($data->$entryName) ) {
+        if ( ! isset($data->$entryName)) {
             if (null === $this->result) {
                 return $this->result = array();
             }
@@ -242,7 +242,7 @@ 
 
         if ($metadata->xmlCollection) {
             $enclosingElem = $data;
-            if (!$metadata->xmlCollectionInline) {
+            if ( ! $metadata->xmlCollectionInline) {
                 $enclosingElem = $data->children($metadata->xmlNamespace)->$name;
             }
 
@@ -256,7 +256,7 @@ 
 
         if ($metadata->xmlNamespace) {
             $node = $data->children($metadata->xmlNamespace)->$name;
-            if (!$node->count()) {
+            if ( ! $node->count()) {
                 return;
             }
         } else {
@@ -266,9 +266,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/Metadata/PropertyMetadata.php

Spacing +2 added lines, -2 removed lines

@@ -166,10 +166,10 @@
             $parentStr
         ) = $unserialized;
 
-        if (isset($unserialized['xmlEntryNamespace'])){
+        if (isset($unserialized['xmlEntryNamespace'])) {
             $this->xmlEntryNamespace = $unserialized['xmlEntryNamespace'];
         }
-        if (isset($unserialized['xmlCollectionSkipWhenEmpty'])){
+        if (isset($unserialized['xmlCollectionSkipWhenEmpty'])) {
             $this->xmlCollectionSkipWhenEmpty = $unserialized['xmlCollectionSkipWhenEmpty'];
         }
         

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

Spacing +3 added lines, -3 removed lines

@@ -113,15 +113,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/XmlSerializationVisitor.php

Spacing +9 added lines, -9 removed lines

@@ -197,7 +197,7 @@ 
             $this->document = $this->createDocument(null, null, false);
             if ($metadata->xmlRootName) {
                 $rootName = $metadata->xmlRootName;
-                $rootNamespace = $metadata->xmlRootNamespace?:$this->getClassDefaultNamespace($metadata);
+                $rootNamespace = $metadata->xmlRootNamespace ?: $this->getClassDefaultNamespace($metadata);
             } else {
                 $rootName = $this->defaultRootName;
                 $rootNamespace = $this->defaultRootNamespace;
@@ -303,7 +303,7 @@ 
         if ($addEnclosingElement) {
             $this->revertCurrentNode();
 
-            if ($this->nodeNotEmpty($element) || ((!$metadata->xmlCollection || !$metadata->xmlCollectionSkipWhenEmpty) && $node === null && $v !== null && !$context->isVisiting($v))) {
+            if ($this->nodeNotEmpty($element) || (( ! $metadata->xmlCollection || ! $metadata->xmlCollectionSkipWhenEmpty) && $node === null && $v !== null && ! $context->isVisiting($v))) {
                 $this->currentNode->appendChild($element);
             }
         }
@@ -449,18 +449,18 @@ 
         if ($this->currentNode->isDefaultNamespace($namespace)) {
             return $this->document->createElementNS($namespace, $tagName);
         }
-        if (!($prefix = $this->currentNode->lookupPrefix($namespace)) && !($prefix = $this->document->lookupPrefix($namespace))) {
-            $prefix = 'ns-'.  substr(sha1($namespace), 0, 8);
-            return $this->document->createElementNS($namespace, $prefix . ':' . $tagName);
+        if ( ! ($prefix = $this->currentNode->lookupPrefix($namespace)) && ! ($prefix = $this->document->lookupPrefix($namespace))) {
+            $prefix = 'ns-'.substr(sha1($namespace), 0, 8);
+            return $this->document->createElementNS($namespace, $prefix.':'.$tagName);
         }
-        return $this->document->createElement($prefix . ':' . $tagName);
+        return $this->document->createElement($prefix.':'.$tagName);
     }
 
     private function setAttributeOnNode(\DOMElement $node, $name, $value, $namespace = null)
     {
         if (null !== $namespace) {
-            if (!$prefix = $node->lookupPrefix($namespace)) {
-                $prefix = 'ns-'.  substr(sha1($namespace), 0, 8);
+            if ( ! $prefix = $node->lookupPrefix($namespace)) {
+                $prefix = 'ns-'.substr(sha1($namespace), 0, 8);
             }
             $node->setAttributeNS($namespace, $prefix.':'.$name, $value);
         } else {
@@ -470,7 +470,7 @@ 
 
     private function getClassDefaultNamespace(ClassMetadata $metadata)
     {
-        return (isset($metadata->xmlNamespaces[''])?$metadata->xmlNamespaces['']:null);
+        return (isset($metadata->xmlNamespaces['']) ? $metadata->xmlNamespaces[''] : null);
     }
 
 }

Doc Comments +15 added lines

@@ -48,12 +48,18 @@ 
     private $nullWasVisited;
     private $objectMetadataStack;
 
+    /**
+     * @param Naming\PropertyNamingStrategyInterface $namingStrategy
+     */
     public function __construct($namingStrategy)
     {
         parent::__construct($namingStrategy);
         $this->objectMetadataStack = new \SplStack;
     }
 
+    /**
+     * @param Naming\PropertyNamingStrategyInterface $namingStrategy
+     */
     public function __construct($namingStrategy)
     {
         parent::__construct($namingStrategy);
@@ -151,6 +157,9 @@ 
         return $doCData ? $this->document->createCDATASection($data) : $this->document->createTextNode((string) $data);
     }
 
+    /**
+     * @param string $data
+     */
     public function visitSimpleString($data, array $type, Context $context)
     {
         if (null === $this->document) {
@@ -468,6 +477,9 @@ 
         }
     }
 
+    /**
+     * @return \DOMNode
+     */
     private function createElement($tagName, $namespace = null)
     {
         if (null === $namespace) {
@@ -483,6 +495,9 @@ 
         return $this->document->createElement($prefix . ':' . $tagName);
     }
 
+    /**
+     * @param string $value
+     */
     private function setAttributeOnNode(\DOMElement $node, $name, $value, $namespace = null)
     {
         if (null !== $namespace) {