[DependencyInjection] More predictable EnvVarProcessor CSV empty string parsing

andreyserdjuk · nicolas-grekas · commit aea85f5bf7f6 · 2023-04-21T17:34:39.000+02:00
diff --git a/src/Symfony/Component/DependencyInjection/EnvVarProcessor.php b/src/Symfony/Component/DependencyInjection/EnvVarProcessor.php
@@ -313,7 +313,7 @@ public function getEnv(string $prefix, string $name, \Closure $getEnv): mixed
         }
 
         if ('csv' === $prefix) {
-            return str_getcsv($env, ',', '"', '');
+            return '' === $env ? [] : str_getcsv($env, ',', '"', '');
         }
 
         if ('trim' === $prefix) {
diff --git a/src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php b/src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php
@@ -714,7 +714,7 @@ public static function validCsv()
 CSV;
 
         return [
-            ['', [null]],
+            ['', []],
             [',', ['', '']],
             ['1', ['1']],
             ['1,2," 3 "', ['1', '2', ' 3 ']],

Original file line number	Diff line number	Diff line change
`@@ -313,7 +313,7 @@ public function getEnv(string $prefix, string $name, \Closure $getEnv): mixed`
`313`	`313`	`}`
`314`	`314`
`315`	`315`	`if ('csv' === $prefix) {`
`316`		`- return str_getcsv($env, ',', '"', '');`
	`316`	`+ return '' === $env ? [] : str_getcsv($env, ',', '"', '');`
`317`	`317`	`}`
`318`	`318`
`319`	`319`	`if ('trim' === $prefix) {`