[Serializer] add a context key to return always as collection for XmlEncoder

Amrouche Hamza · Amrouche Hamza · commit f1daa3fbe5b7 · 2018-02-02T12:08:04.000+01:00
diff --git a/src/Symfony/Component/Serializer/Encoder/XmlEncoder.php b/src/Symfony/Component/Serializer/Encoder/XmlEncoder.php
@@ -41,8 +41,7 @@ class XmlEncoder implements EncoderInterface, DecoderInterface, NormalizationAwa
     /**
      * Construct new XmlEncoder and allow to change the root node element name.
      *
-     * @param string   $rootNodeName
-     * @param int|null $loadOptions  A bit field of LIBXML_* constants
+     * @param int|null $loadOptions A bit field of LIBXML_* constants
      */
     public function __construct(string $rootNodeName = 'response', int $loadOptions = null)
     {
@@ -333,9 +332,8 @@ private function parseXmlValue(\DOMNode $node, array $context = array())
                 $value[$subnode->nodeName][] = $val;
             }
         }
-
         foreach ($value as $key => $val) {
-            if (is_array($val) && 1 === count($val)) {
+            if (is_array($val) && 1 === count($val) && (!(isset($context['forceCollection']) && $context['forceCollection']) || !is_array($val[0]))) {
                 $value[$key] = current($val);
             }
         }
diff --git a/src/Symfony/Component/Serializer/Tests/Encoder/XmlEncoderTest.php b/src/Symfony/Component/Serializer/Tests/Encoder/XmlEncoderTest.php
@@ -515,6 +515,33 @@ public function testDecodeIgnoreWhiteSpace()
         $this->assertEquals($expected, $this->encoder->decode($source, 'xml'));
     }
 
+    public function testDecodeAlwaysAsCollection()
+    {
+        $this->encoder = new XmlEncoder('response', null);
+        $serializer = new Serializer(array(new CustomNormalizer()), array('xml' => new XmlEncoder()));
+        $this->encoder->setSerializer($serializer);
+
+        $source = <<<'XML'
+<?xml version="1.0"?>
+<order_rows nodeType="order_row" virtualEntity="true">
+    <order_row>
+        <id><![CDATA[16]]></id>
+        <test><![CDATA[16]]></test>
+    </order_row>
+</order_rows>
+XML;
+        $expected = array(
+            '@nodeType' => 'order_row',
+            '@virtualEntity' => 'true',
+            'order_row' => array(array(
+                'id' => 16,
+                'test' => 16,
+            )),
+        );
+
+        $this->assertEquals($expected, $this->encoder->decode($source, 'xml', array('forceCollection' => true)));
+    }
+
     public function testDecodeWithoutItemHash()
     {
         $obj = new ScalarDummy();

Original file line number	Diff line number	Diff line change
`@@ -41,8 +41,7 @@ class XmlEncoder implements EncoderInterface, DecoderInterface, NormalizationAwa`
`41`	`41`	`/**`
`42`	`42`	`* Construct new XmlEncoder and allow to change the root node element name.`
`43`	`43`	`*`
`44`		`- * @param string $rootNodeName`
`45`		`- * @param int\|null $loadOptions A bit field of LIBXML_* constants`
	`44`	`+ * @param int\|null $loadOptions A bit field of LIBXML_* constants`
`46`	`45`	`*/`
`47`	`46`	`public function __construct(string $rootNodeName = 'response', int $loadOptions = null)`
`48`	`47`	`{`
`@@ -333,9 +332,8 @@ private function parseXmlValue(\DOMNode $node, array $context = array())`
`333`	`332`	`$value[$subnode->nodeName][] = $val;`
`334`	`333`	`}`
`335`	`334`	`}`
`336`		`-`
`337`	`335`	`foreach ($value as $key => $val) {`
`338`		`- if (is_array($val) && 1 === count($val)) {`
	`336`	`+ if (is_array($val) && 1 === count($val) && (!(isset($context['forceCollection']) && $context['forceCollection']) \|\| !is_array($val[0]))) {`
`339`	`337`	`$value[$key] = current($val);`
`340`	`338`	`}`
`341`	`339`	`}`