Merge branch '5.4' into 6.2

* 5.4:
  [Translation] Decouple TranslatorPathsPass from "debug." convention
  [VarDumper] Add a bit of test coverage
  [TwigBridge] Fix flagged malicious url
  [HttpClient] Fix encoding "+" in URLs
  [DependencyInjection] Fix dumping array of enums parameters
  Bump Symfony version to 5.4.22
  Update VERSION for 5.4.21
  Update CONTRIBUTORS for 5.4.21
  Update CHANGELOG for 5.4.21
  Removed @internal tag on TraceableAuthenticator::getAuthenticator()

Author: nicolas-grekas

Dumper/PhpDumper.php

@@ -88,6 +88,7 @@ class PhpDumper extends Dumper
     private array $locatedIds = [];
     private string $serviceLocatorTag;
     private array $exportedVariables = [];
+    private array $dynamicParameters = [];
     private string $baseClass;
     private string $class;
     private DumperInterface $proxyDumper;
@@ -131,6 +132,7 @@ public function dump(array $options = []): string|array
         $this->targetDirRegex = null;
         $this->inlinedRequires = [];
         $this->exportedVariables = [];
+        $this->dynamicParameters = [];
         $options = array_merge([
             'class' => 'ProjectServiceContainer',
             'base_class' => 'Container',
@@ -206,11 +208,12 @@ public function dump(array $options = []): string|array
             $this->preload = array_combine($options['preload_classes'], $options['preload_classes']);
         }
 
+        $code = $this->addDefaultParametersMethod();
         $code =
             $this->startClass($options['class'], $baseClass, $this->inlineFactories && $proxyClasses).
             $this->addServices($services).
             $this->addDeprecatedAliases().
-            $this->addDefaultParametersMethod()
+            $code
         ;
 
         $proxyClasses ??= $this->generateProxyClasses();
@@ -374,6 +377,7 @@ class %s extends {$options['class']}
         $this->circularReferences = [];
         $this->locatedIds = [];
         $this->exportedVariables = [];
+        $this->dynamicParameters = [];
         $this->preload = [];
 
         $unusedEnvs = [];
@@ -1519,6 +1523,7 @@ private function addDefaultParametersMethod(): string
 
             if ($hasEnum || preg_match("/\\\$this->(?:getEnv\('(?:[-.\w\\\\]*+:)*+\w++'\)|targetDir\.'')/", $export[1])) {
                 $dynamicPhp[$key] = sprintf('%s%s => %s,', $export[0], $this->export($key), $export[1]);
+                $this->dynamicParameters[$key] = true;
             } else {
                 $php[] = sprintf('%s%s => %s,', $export[0], $this->export($key), $export[1]);
             }
@@ -1917,20 +1922,18 @@ private function dumpLiteralClass(string $class): string
 
     private function dumpParameter(string $name): string
     {
-        if ($this->container->hasParameter($name)) {
-            $value = $this->container->getParameter($name);
-            $dumpedValue = $this->dumpValue($value, false);
+        if (!$this->container->hasParameter($name) || ($this->dynamicParameters[$name] ?? false)) {
+            return sprintf('$this->getParameter(%s)', $this->doExport($name));
+        }
 
-            if (!$value || !\is_array($value)) {
-                return $dumpedValue;
-            }
+        $value = $this->container->getParameter($name);
+        $dumpedValue = $this->dumpValue($value, false);
 
-            if (!preg_match("/\\\$this->(?:getEnv\('(?:[-.\w\\\\]*+:)*+\w++'\)|targetDir\.'')/", $dumpedValue)) {
-                return sprintf('$this->parameters[%s]', $this->doExport($name));
-            }
+        if (!$value || !\is_array($value)) {
+            return $dumpedValue;
         }
 
-        return sprintf('$this->getParameter(%s)', $this->doExport($name));
+        return sprintf('$this->parameters[%s]', $this->doExport($name));
     }
 
     private function getServiceCall(string $id, Reference $reference = null): string

Tests/Dumper/PhpDumperTest.php

@@ -1228,6 +1228,11 @@ public function testDumpHandlesEnumeration()
             ->register('foo', FooClassWithEnumAttribute::class)
             ->setPublic(true)
             ->addArgument(FooUnitEnum::BAR);
+        $container
+            ->register('bar', \stdClass::class)
+            ->setPublic(true)
+            ->addArgument('%unit_enum%')
+            ->addArgument('%enum_array%');
 
         $container->setParameter('unit_enum', FooUnitEnum::BAR);
         $container->setParameter('enum_array', [FooUnitEnum::BAR, FooUnitEnum::FOO]);
@@ -1245,6 +1250,11 @@ public function testDumpHandlesEnumeration()
         $this->assertSame(FooUnitEnum::BAR, $container->getParameter('unit_enum'));
         $this->assertSame([FooUnitEnum::BAR, FooUnitEnum::FOO], $container->getParameter('enum_array'));
         $this->assertStringMatchesFormat(<<<'PHP'
+%A
+    protected function getBarService()
+    {
+        return $this->services['bar'] = new \stdClass(\Symfony\Component\DependencyInjection\Tests\Fixtures\FooUnitEnum::BAR, $this->getParameter('enum_array'));
+    }
 %A
     private function getDynamicParameter(string $name)
     {