Commitf1daa3f

Amrouche Hamza

committed

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

1 parent1df45e4 commitf1daa3fCopy full SHA for f1daa3f

File tree

+29

-4

lines changed

src/Symfony/Component/Serializer
- Encoder
  - XmlEncoder.php
- Tests/Encoder
  - XmlEncoderTest.php

+29

-4

lines changed

Lines changed: 2 additions & 4 deletions

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`	`publicfunction__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 ($valueas$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`	`}`

Lines changed: 27 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -515,6 +515,33 @@ public function testDecodeIgnoreWhiteSpace()`
`515`	`515`	`$this->assertEquals($expected,$this->encoder->decode($source,'xml'));`
`516`	`516`	`}`
`517`	`517`
	`518`	`+publicfunctiontestDecodeAlwaysAsCollection()`
	`519`	`+ {`
	`520`	`+$this->encoder =newXmlEncoder('response',null);`
	`521`	`+$serializer =newSerializer(array(newCustomNormalizer()),array('xml' =>newXmlEncoder()));`
	`522`	`+$this->encoder->setSerializer($serializer);`
	`523`	`+`
	`524`	`+$source = <<<'XML'`
	`525`	`+<?xml version="1.0"?>`
	`526`	`+<order_rows nodeType="order_row" virtualEntity="true">`
	`527`	`+ <order_row>`
	`528`	`+ <id><![CDATA[16]]></id>`
	`529`	`+ <test><![CDATA[16]]></test>`
	`530`	`+ </order_row>`
	`531`	`+</order_rows>`
	`532`	`+XML;`
	`533`	`+$expected =array(`
	`534`	`+'@nodeType' =>'order_row',`
	`535`	`+'@virtualEntity' =>'true',`
	`536`	`+'order_row' =>array(array(`
	`537`	`+'id' =>16,`
	`538`	`+'test' =>16,`
	`539`	`+ )),`
	`540`	`+ );`
	`541`	`+`
	`542`	`+$this->assertEquals($expected,$this->encoder->decode($source,'xml',array('forceCollection' =>true)));`
	`543`	`+ }`
	`544`	`+`
`518`	`545`	`publicfunctiontestDecodeWithoutItemHash()`
`519`	`546`	`{`
`520`	`547`	`$obj =newScalarDummy();`

Comments

(0)