Merge pull request zendframework/zendframework#5375 from volftomas/pa…

…tch-1

Fixes default type == string

src/AbstractValue.php

@@ -445,6 +445,9 @@ protected static function _extractTypeAndValue(\SimpleXMLElement $xml, &$type, &
         // If no type was specified, the default is string
         if (!$type) {
             $type = self::XMLRPC_TYPE_STRING;
+            if (empty($value) and preg_match('#^<value>.*</value>$#', $xml->asXML())) {
+                $value = str_replace(array('<value>', '</value>'), '', $xml->asXML());
+            }
         }
     }
 

test/ValueTest.php

@@ -210,6 +210,21 @@ public function testMarshalStringFromNative()
         $this->assertSame($native, $val->getValue());
     }
 
+    public function testFactoryAutodetectsStringAndSetsValueInArray()
+    {
+        $val = AbstractValue::getXmlRpcValue('<value><array><data>'.
+            '<value><i4>8</i4></value>'.
+            '<value>a</value>'.
+            '<value>false</value>'.
+            '</data></array></value>', AbstractValue::XML_STRING
+        );
+        $this->assertXmlRpcType('array', $val);
+        $a = $val->getValue();
+        $this->assertSame(8, $a[0]);
+        $this->assertSame('a', $a[1]);
+        $this->assertSame('false', $a[2]);
+    }
+
     /**
      * @dataProvider ZendTest\XmlRpc\TestProvider::provideGenerators
      */