[DependencyInjection] Don't add the .container.known_envs parameter when empty

GromNaN · GromNaN · commit c3fa63bfbbaf · 2025-11-28T22:20:28.000+01:00
Other loaders does not create this parameter when there is no when@&lt;env&gt; key
diff --git a/src/Symfony/Component/DependencyInjection/Loader/XmlFileLoader.php b/src/Symfony/Component/DependencyInjection/Loader/XmlFileLoader.php
@@ -63,11 +63,10 @@ public function load(mixed $resource, ?string $type = null): mixed
                 $this->loadXml($xml, $path, $root);
             }
         }
-        $knownEnvs = $this->container->hasParameter('.container.known_envs') ? array_flip($this->container->getParameter('.container.known_envs')) : [];
         foreach ($xpath->query('//container:when') ?: [] as $root) {
-            $knownEnvs[$root->getAttribute('env')] = true;
+            $knownEnvs = $this->container->hasParameter('.container.known_envs') ? array_flip($this->container->getParameter('.container.known_envs')) : [];
+            $this->container->setParameter('.container.known_envs', array_keys($knownEnvs + [$root->getAttribute('env') => true]));
         }
-        $this->container->setParameter('.container.known_envs', array_keys($knownEnvs));
 
         return null;
     }
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Loader/XmlFileLoaderTest.php b/src/Symfony/Component/DependencyInjection/Tests/Loader/XmlFileLoaderTest.php
@@ -154,7 +154,6 @@ public function testLoadParameters()
             ],
             'mixedcase' => ['MixedCaseKey' => 'value'],
             'constant' => \PHP_EOL,
-            '.container.known_envs' => [],
         ];
 
         $this->assertEquals($expected, $actual, '->load() converts XML values to PHP ones');
@@ -191,7 +190,6 @@ public function testLoadImports()
             ],
             'mixedcase' => ['MixedCaseKey' => 'value'],
             'constant' => \PHP_EOL,
-            '.container.known_envs' => [],
             'bar' => '%foo%',
             'imported_from_ini' => true,
             'imported_from_yaml' => true,

Original file line number	Diff line number	Diff line change
`@@ -63,11 +63,10 @@ public function load(mixed $resource, ?string $type = null): mixed`
`63`	`63`	`$this->loadXml($xml, $path, $root);`
`64`	`64`	`}`
`65`	`65`	`}`
`66`		`- $knownEnvs = $this->container->hasParameter('.container.known_envs') ? array_flip($this->container->getParameter('.container.known_envs')) : [];`
`67`	`66`	`foreach ($xpath->query('//container:when') ?: [] as $root) {`
`68`		`- $knownEnvs[$root->getAttribute('env')] = true;`
	`67`	`+ $knownEnvs = $this->container->hasParameter('.container.known_envs') ? array_flip($this->container->getParameter('.container.known_envs')) : [];`
	`68`	`+ $this->container->setParameter('.container.known_envs', array_keys($knownEnvs + [$root->getAttribute('env') => true]));`
`69`	`69`	`}`
`70`		`- $this->container->setParameter('.container.known_envs', array_keys($knownEnvs));`
`71`	`70`
`72`	`71`	`return null;`
`73`	`72`	`}`