Merge pull request #675 from schmittjoh/xml-array

Fixed xml arrays with namespaced entry triggers error

Author: goetas

src/JMS/Serializer/XmlDeserializationVisitor.php

@@ -150,7 +150,12 @@ public function visitArray($data, array $type, Context $context)
             $namespace = isset($classMetadata->xmlNamespaces[''])?$classMetadata->xmlNamespaces['']:$namespace;
         }
 
-        $hasNode = null !== $namespace ? isset($data->children($namespace)->$entryName) : isset($data->$entryName);
+        if (0 === $data->count()){
+            $hasNode = false;
+        } else {
+            $hasNode = null !== $namespace ? isset($data->children($namespace)->$entryName) : isset($data->$entryName);
+        }
+
         if (false === $hasNode) {
             if (null === $this->result) {
                 return $this->result = array();

tests/JMS/Serializer/Tests/Fixtures/ObjectWithAbsentXmlListNode.php

@@ -38,4 +38,10 @@ class ObjectWithAbsentXmlListNode
      * @Serializer\Type("array<string>")
      */
     public $skipDefault;
+
+    /**
+     * @Serializer\XmlList(inline=false, namespace="http://www.example.com")
+     * @Serializer\Type("array<string>")
+     */
+    public $absentAndNs;
 }

tests/JMS/Serializer/Tests/Serializer/XmlSerializationTest.php

@@ -184,6 +184,22 @@ public function testVirtualXmlMap()
             $this->serialize(new ObjectWithVirtualXmlProperties(), SerializationContext::create()->setGroups(array('map')))
         );
     }
+
+    public function testUnserializeMissingArray()
+    {
+        $xml = '<result></result>';
+        $object = $this->serializer->deserialize($xml, 'JMS\Serializer\Tests\Fixtures\ObjectWithAbsentXmlListNode', 'xml');
+        $this->assertEquals($object->absentAndNs, array());
+
+        $xml = '<result xmlns:x="http://www.example.com">
+                    <absent_and_ns>
+                        <x:entry>foo</x:entry>
+                    </absent_and_ns>
+                  </result>';
+        $object = $this->serializer->deserialize($xml, 'JMS\Serializer\Tests\Fixtures\ObjectWithAbsentXmlListNode', 'xml');
+        $this->assertEquals($object->absentAndNs, array("foo"));
+    }
+
     public function testObjectWithNamespacesAndList()
     {
         $object = new ObjectWithNamespacesAndList();