From c1bfaa1de489b51de89d4320c615b2c65bf81f17 Mon Sep 17 00:00:00 2001 From: Maxime Steinhausser Date: Mon, 8 Jul 2019 16:06:09 +0200 Subject: [PATCH] [Serializer] XmlEncoder: don't cast padded strings --- .../Component/Serializer/Encoder/XmlEncoder.php | 2 +- .../Serializer/Tests/Encoder/XmlEncoderTest.php | 11 +++++++++++ 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/src/Symfony/Component/Serializer/Encoder/XmlEncoder.php b/src/Symfony/Component/Serializer/Encoder/XmlEncoder.php index f0a4afb191b3..afbd63e4962a 100644 --- a/src/Symfony/Component/Serializer/Encoder/XmlEncoder.php +++ b/src/Symfony/Component/Serializer/Encoder/XmlEncoder.php @@ -303,7 +303,7 @@ private function parseXmlAttributes(\DOMNode $node, array $context = []) $typeCastAttributes = $this->resolveXmlTypeCastAttributes($context); foreach ($node->attributes as $attr) { - if (!is_numeric($attr->nodeValue) || !$typeCastAttributes) { + if (!is_numeric($attr->nodeValue) || !$typeCastAttributes || (isset($attr->nodeValue[1]) && '0' === $attr->nodeValue[0])) { $data['@'.$attr->nodeName] = $attr->nodeValue; continue; diff --git a/src/Symfony/Component/Serializer/Tests/Encoder/XmlEncoderTest.php b/src/Symfony/Component/Serializer/Tests/Encoder/XmlEncoderTest.php index 7a6a7d3fbca1..573605c1ab09 100644 --- a/src/Symfony/Component/Serializer/Tests/Encoder/XmlEncoderTest.php +++ b/src/Symfony/Component/Serializer/Tests/Encoder/XmlEncoderTest.php @@ -306,6 +306,17 @@ public function testNoTypeCastAttribute() $this->assertSame($expected, $data); } + public function testDoesNotTypeCastStringsStartingWith0() + { + $source = << + +XML; + + $data = $this->encoder->decode($source, 'xml'); + $this->assertSame('018', $data['@a']); + } + public function testEncode() { $source = $this->getXmlSource();