schmittjoh / serializer

src/JMS/Serializer/XmlSerializationVisitor.php

Doc Comments +9 added lines

@@ -44,6 +44,9 @@ 
     private $nullWasVisited;
     private $objectMetadataStack;
 
+    /**
+     * @param Naming\PropertyNamingStrategyInterface $namingStrategy
+     */
     public function __construct($namingStrategy)
     {
         parent::__construct($namingStrategy);
@@ -437,6 +440,9 @@ 
         }
     }
 
+    /**
+     * @return \DOMNode
+     */
     private function createElement($tagName, $namespace = null)
     {
         if (null !== $namespace) {
@@ -456,6 +462,9 @@ 
         }
     }
 
+    /**
+     * @param string $value
+     */
     private function setAttributeOnNode(\DOMElement $node, $name, $value, $namespace = null)
     {
         if (null !== $namespace) {

Spacing +7 added lines, -7 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;
@@ -444,10 +444,10 @@ 
             if ($this->currentNode->isDefaultNamespace($namespace)) {
                 return $this->document->createElementNS($namespace, $tagName);
             } else {
-                if (!$prefix = $this->currentNode->lookupPrefix($namespace)) {
-                    $prefix = 'ns-'.  substr(sha1($namespace), 0, 8);
+                if ( ! $prefix = $this->currentNode->lookupPrefix($namespace)) {
+                    $prefix = 'ns-'.substr(sha1($namespace), 0, 8);
                 }
-                return $this->document->createElementNS($namespace, $prefix . ':' . $tagName);
+                return $this->document->createElementNS($namespace, $prefix.':'.$tagName);
             }
 
 
@@ -459,8 +459,8 @@ 
     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);
     }
 
 }

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])) {

Spacing +6 added lines, -6 removed lines

@@ -151,10 +151,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();
             }
@@ -246,7 +246,7 @@ 
 
         if ($metadata->xmlCollection) {
             $enclosingElem = $data;
-            if (!$metadata->xmlCollectionInline) {
+            if ( ! $metadata->xmlCollectionInline) {
                 $enclosingElem = $data->children($metadata->xmlNamespace)->$name;
             }
 
@@ -260,7 +260,7 @@ 
 
         if ($metadata->xmlNamespace) {
             $node = $data->children($metadata->xmlNamespace)->$name;
-            if (!$node->count()) {
+            if ( ! $node->count()) {
                 return;
             }
         } else {
@@ -270,13 +270,13 @@ 
             if (isset($namespaces[''])) {
                 $prefix = uniqid('ns-');
                 $data->registerXPathNamespace($prefix, $namespaces['']);
-                $nodes = $data->xpath('./'.$prefix. ':'.$name );
+                $nodes = $data->xpath('./'.$prefix.':'.$name);
                 if (empty($nodes)) {
                     return;
                 }
                 $node = reset($nodes);
             } else {
-                if (!isset($data->$name)) {
+                if ( ! isset($data->$name)) {
                     return;
                 }
                 $node = $data->$name;

src/JMS/Serializer/Metadata/PropertyMetadata.php

Spacing +1 added lines, -1 removed lines

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

Braces +1 added lines, -1 removed lines

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