[DependencyInjection] Allow injecting the current env into php config closures

Author: HypeMC

CHANGELOG.md

@@ -13,6 +13,7 @@ CHANGELOG
  * Deprecate calling `ContainerAwareTrait::setContainer()` without arguments
  * Deprecate using numeric parameter names
  * Add support for tagged iterators/locators `exclude` option to the xml and yaml loaders/dumpers
+ * Allow injecting `string $env` into php config closures
 
 6.1
 ---

Loader/PhpFileLoader.php

@@ -123,6 +123,12 @@ private function executeCallback(callable $callback, ContainerConfigurator $cont
                 case self::class:
                     $arguments[] = $this;
                     break;
+                case 'string':
+                    if (null !== $this->env && 'env' === $parameter->getName()) {
+                        $arguments[] = $this->env;
+                        break;
+                    }
+                    // no break
                 default:
                     try {
                         $configBuilder = $this->configBuilder($type);
@@ -163,7 +169,7 @@ private function configBuilder(string $namespace): ConfigBuilderInterface
             return new $namespace();
         }
 
-        // If it does not start with Symfony\Config\ we dont know how to handle this
+        // If it does not start with Symfony\Config\ we don't know how to handle this
         if (!str_starts_with($namespace, 'Symfony\\Config\\')) {
             throw new InvalidArgumentException(sprintf('Could not find or generate class "%s".', $namespace));
         }

Tests/Fixtures/config/env_param.expected.yml

@@ -0,0 +1,8 @@
+parameters:
+    acme.configs: [{ color: blue }]
+
+services:
+    service_container:
+        class: Symfony\Component\DependencyInjection\ContainerInterface
+        public: true
+        synthetic: true

Tests/Fixtures/config/env_param.php

@@ -0,0 +1,11 @@
+<?php
+
+use Symfony\Component\DependencyInjection\Tests\Fixtures\AcmeConfig;
+
+return function (AcmeConfig $config, string $env) {
+    if ('prod' === $env) {
+        $config->color('blue');
+    } else {
+        $config->color('red');
+    }
+};

Tests/Loader/PhpFileLoaderTest.php

@@ -99,6 +99,7 @@ public function provideConfig()
         yield ['config_builder'];
         yield ['expression_factory'];
         yield ['closure'];
+        yield ['env_param'];
     }
 
     public function testAutoConfigureAndChildDefinition()