[Serializer] Take unnamed variadic parameters into account when denormalizing

thijsBreker · thijsBreker · commit ed92235d790f · 2024-01-03T09:34:18.000+01:00
We shouldn't break when a constructor has variadic parameters without
named keys in the array.
diff --git a/src/Symfony/Component/Serializer/Normalizer/AbstractNormalizer.php b/src/Symfony/Component/Serializer/Normalizer/AbstractNormalizer.php
@@ -372,7 +372,7 @@ protected function instantiateObject(array &$data, string $class, array &$contex
                             $variadicParameters[$parameterKey] = $this->denormalizeParameter($reflectionClass, $constructorParameter, $paramName, $parameterData, $context, $format);
                         }
 
-                        $params = array_merge($params, $variadicParameters);
+                        $params = array_merge(array_values($params), $variadicParameters);
                         $unsetKeys[] = $key;
                     }
                 } elseif ($allowed && !$ignored && (isset($data[$key]) || \array_key_exists($key, $data))) {
diff --git a/src/Symfony/Component/Serializer/Tests/Fixtures/DummyWithWithVariadicParameterConstructor.php b/src/Symfony/Component/Serializer/Tests/Fixtures/DummyWithWithVariadicParameterConstructor.php
@@ -0,0 +1,46 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\Serializer\Tests\Fixtures;
+
+class DummyWithWithVariadicParameterConstructor
+{
+    private $foo;
+
+    private $bar;
+
+    private $baz;
+
+    public function __construct(string $foo, int $bar = 1, Dummy ...$baz)
+    {
+        $this->foo = $foo;
+        $this->bar = $bar;
+        $this->baz = $baz;
+    }
+
+    /** @return string */
+    public function getFoo()
+    {
+        return $this->foo;
+    }
+
+    /** @return int */
+    public function getBar()
+    {
+        return $this->bar;
+    }
+
+    /** @return Dummy[] */
+    public function getBaz()
+    {
+        return $this->baz;
+    }
+}
diff --git a/src/Symfony/Component/Serializer/Tests/Normalizer/AbstractNormalizerTest.php b/src/Symfony/Component/Serializer/Tests/Normalizer/AbstractNormalizerTest.php
@@ -29,6 +29,7 @@
 use Symfony\Component\Serializer\Tests\Fixtures\AbstractNormalizerDummy;
 use Symfony\Component\Serializer\Tests\Fixtures\Annotations\IgnoreDummy;
 use Symfony\Component\Serializer\Tests\Fixtures\Dummy;
+use Symfony\Component\Serializer\Tests\Fixtures\DummyWithWithVariadicParameterConstructor;
 use Symfony\Component\Serializer\Tests\Fixtures\NullableConstructorArgumentDummy;
 use Symfony\Component\Serializer\Tests\Fixtures\NullableOptionalConstructorArgumentDummy;
 use Symfony\Component\Serializer\Tests\Fixtures\StaticConstructorDummy;
@@ -247,6 +248,19 @@ public static function getNormalizer()
         yield [new ObjectNormalizer(null, null, null, $extractor)];
     }
 
+    public function testVariadicConstructorDenormalization()
+    {
+        $data = '{"foo":"woo","baz":[{"foo":null,"bar":null,"baz":null,"qux":null},{"foo":null,"bar":null,"baz":null,"qux":null}]}';
+        $expected = new DummyWithWithVariadicParameterConstructor('woo', 1, new Dummy(), new Dummy());
+
+        $normalizer = new ObjectNormalizer();
+        $serializer = new Serializer([$normalizer], [new JsonEncoder()]);
+        $normalizer->setSerializer($serializer);
+
+        $actual = $normalizer->denormalize(json_decode($data, true), DummyWithWithVariadicParameterConstructor::class);
+        $this->assertEquals($expected, $actual);
+    }
+
     public static function getNormalizerWithCustomNameConverter()
     {
         $extractor = new PhpDocExtractor();

Original file line number	Diff line number	Diff line change
`@@ -372,7 +372,7 @@ protected function instantiateObject(array &$data, string $class, array &$contex`
`372`	`372`	`$variadicParameters[$parameterKey] = $this->denormalizeParameter($reflectionClass, $constructorParameter, $paramName, $parameterData, $context, $format);`
`373`	`373`	`}`
`374`	`374`
`375`		`- $params = array_merge($params, $variadicParameters);`
	`375`	`+ $params = array_merge(array_values($params), $variadicParameters);`
`376`	`376`	`$unsetKeys[] = $key;`
`377`	`377`	`}`
`378`	`378`	`} elseif ($allowed && !$ignored && (isset($data[$key]) \|\| \array_key_exists($key, $data))) {`