Merge branch '5.4' into 6.2

* 5.4:
  [PhpUnitBridge] Kill the last concurrent process when it stales for more than 60s
  [Intl] fix test
  [Intl] Use VarExporter::export() in PhpBundleWriter
  Use triggering class to generate baseline for deprecation messages from DebugClassLoader
  [Security] Fix false-string handling in RememberMeAuthenticator
  [CssSelector] Tests on Xpath translator will always pass
  [Serializer] Fix Normalizer not utilizing converted name for index variadic param
  [DepdencyInjection] Fix costly logic when checking errored definitions
  fix children cond
  [DoctrineBridge] Load refreshed user proxy
  [DependencyInjection] Don't ignore attributes on the actual decorator
  [FrameworkBundle] Prevent `cache:clear` to lose files on subsequent runs

Author: nicolas-grekas

Normalizer/AbstractNormalizer.php

@@ -339,12 +339,12 @@ protected function instantiateObject(array &$data, string $class, array &$contex
                 $ignored = !$this->isAllowedAttribute($class, $paramName, $format, $context);
                 if ($constructorParameter->isVariadic()) {
                     if ($allowed && !$ignored && (isset($data[$key]) || \array_key_exists($key, $data))) {
-                        if (!\is_array($data[$paramName])) {
+                        if (!\is_array($data[$key])) {
                             throw new RuntimeException(sprintf('Cannot create an instance of "%s" from serialized data because the variadic parameter "%s" can only accept an array.', $class, $constructorParameter->name));
                         }
 
                         $variadicParameters = [];
-                        foreach ($data[$paramName] as $parameterKey => $parameterData) {
+                        foreach ($data[$key] as $parameterKey => $parameterData) {
                             $variadicParameters[$parameterKey] = $this->denormalizeParameter($reflectionClass, $constructorParameter, $paramName, $parameterData, $attributeContext, $format);
                         }
 

Tests/Normalizer/AbstractNormalizerTest.php

@@ -20,6 +20,7 @@
 use Symfony\Component\Serializer\Mapping\Factory\ClassMetadataFactory;
 use Symfony\Component\Serializer\Mapping\Factory\ClassMetadataFactoryInterface;
 use Symfony\Component\Serializer\Mapping\Loader\LoaderChain;
+use Symfony\Component\Serializer\NameConverter\NameConverterInterface;
 use Symfony\Component\Serializer\Normalizer\AbstractNormalizer;
 use Symfony\Component\Serializer\Normalizer\ObjectNormalizer;
 use Symfony\Component\Serializer\Normalizer\PropertyNormalizer;
@@ -168,6 +169,7 @@ public function testObjectWithNullableNonOptionalConstructorArgumentWithoutInput
 
     /**
      * @dataProvider getNormalizer
+     * @dataProvider getNormalizerWithCustomNameConverter
      */
     public function testObjectWithVariadicConstructorTypedArguments(AbstractNormalizer $normalizer)
     {
@@ -241,6 +243,25 @@ public static function getNormalizer()
         yield [new ObjectNormalizer(null, null, null, $extractor)];
     }
 
+    public static function getNormalizerWithCustomNameConverter()
+    {
+        $extractor = new PhpDocExtractor();
+        $nameConverter = new class() implements NameConverterInterface {
+            public function normalize(string $propertyName): string
+            {
+                return ucfirst($propertyName);
+            }
+
+            public function denormalize(string $propertyName): string
+            {
+                return lcfirst($propertyName);
+            }
+        };
+
+        yield [new PropertyNormalizer(null, $nameConverter, $extractor)];
+        yield [new ObjectNormalizer(null, $nameConverter, null, $extractor)];
+    }
+
     public function testIgnore()
     {
         $classMetadata = new ClassMetadata(IgnoreDummy::class);