[DIC] Add a explode env var processor

Orkin · Orkin · commit 44461c13a098 · 2019-06-05T13:07:10.000+02:00
diff --git a/src/Symfony/Component/DependencyInjection/CHANGELOG.md b/src/Symfony/Component/DependencyInjection/CHANGELOG.md
@@ -11,6 +11,7 @@ CHANGELOG
 -----
 
  * deprecated support for short factories and short configurators in Yaml
+ * added `%env(explode:...)%` processor to `explode` a string into an array
 
 4.3.0
 -----
diff --git a/src/Symfony/Component/DependencyInjection/EnvVarProcessor.php b/src/Symfony/Component/DependencyInjection/EnvVarProcessor.php
@@ -48,6 +48,7 @@ public static function getProvidedTypes()
             'string' => 'string',
             'trim' => 'string',
             'require' => 'bool|int|float|string|array',
+            'explode' => 'array',
         ];
     }
 
@@ -243,6 +244,10 @@ public function getEnv($prefix, $name, \Closure $getEnv)
             return trim($env);
         }
 
+        if ('explode' === $prefix) {
+            return explode(',', $env);
+        }
+
         throw new RuntimeException(sprintf('Unsupported env var prefix "%s".', $prefix));
     }
 }
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Compiler/RegisterEnvVarProcessorsPassTest.php b/src/Symfony/Component/DependencyInjection/Tests/Compiler/RegisterEnvVarProcessorsPassTest.php
@@ -46,6 +46,7 @@ public function testSimpleProcessor()
             'string' => ['string'],
             'trim' => ['string'],
             'require' => ['bool', 'int', 'float', 'string', 'array'],
+            'explode' => ['array'],
         ];
 
         $this->assertSame($expected, $container->getParameterBag()->getProvidedTypes());
diff --git a/src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php b/src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php
@@ -486,4 +486,17 @@ public function testRequireFile()
 
         $this->assertEquals('foo', $result);
     }
+
+    public function testGetEnvExplode()
+    {
+        $processor = new EnvVarProcessor(new Container());
+
+        $result = $processor->getEnv('explode', 'foo', function ($name) {
+            $this->assertSame('foo', $name);
+
+            return 'one,two,three';
+        });
+
+        $this->assertSame(['one', 'two', 'three'], $result);
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -48,6 +48,7 @@ public static function getProvidedTypes()`
`48`	`48`	`'string' => 'string',`
`49`	`49`	`'trim' => 'string',`
`50`	`50`	`'require' => 'bool\|int\|float\|string\|array',`
	`51`	`+ 'explode' => 'array',`
`51`	`52`	`];`
`52`	`53`	`}`
`53`	`54`
`@@ -243,6 +244,10 @@ public function getEnv($prefix, $name, \Closure $getEnv)`
`243`	`244`	`return trim($env);`
`244`	`245`	`}`
`245`	`246`
	`247`	`+ if ('explode' === $prefix) {`
	`248`	`+ return explode(',', $env);`
	`249`	`+ }`
	`250`	`+`
`246`	`251`	`throw new RuntimeException(sprintf('Unsupported env var prefix "%s".', $prefix));`
`247`	`252`	`}`
`248`	`253`	`}`