symfony
diff --git a/‎src/Symfony/Component/DependencyInjection/EnvVarProcessor.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/EnvVarProcessor.php
+5-11Lines changed: 5 additions & 11 deletions b/‎src/Symfony/Component/DependencyInjection/EnvVarProcessor.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/EnvVarProcessor.php
+5-11Lines changed: 5 additions & 11 deletions
diff --git a/‎src/Symfony/Component/DependencyInjection/Tests/Compiler/RegisterEnvVarProcessorsPassTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/Compiler/RegisterEnvVarProcessorsPassTest.php
+1-1Lines changed: 1 addition & 1 deletion b/‎src/Symfony/Component/DependencyInjection/Tests/Compiler/RegisterEnvVarProcessorsPassTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/Compiler/RegisterEnvVarProcessorsPassTest.php
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php
+16-29Lines changed: 16 additions & 29 deletions b/‎src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php
+16-29Lines changed: 16 additions & 29 deletions
@@ -41,6 +41,7 @@ public static function getProvidedTypes()
         return [
             'base64' => 'string',
             'bool' => 'bool',
+            'not' => 'bool',
             'const' => 'bool|int|float|string|array',
             'csv' => 'array',
             'file' => 'string',
@@ -55,7 +56,6 @@ public static function getProvidedTypes()
             'string' => 'string',
             'trim' => 'string',
             'require' => 'bool|int|float|string|array',
-            'not' => 'bool',
         ];
     }
 
@@ -192,8 +192,10 @@ public function getEnv(string $prefix, string $name, \Closure $getEnv)
             return (string) $env;
         }
 
-        if ('bool' === $prefix) {
-            return (bool) (filter_var($env, \FILTER_VALIDATE_BOOLEAN) ?: filter_var($env, \FILTER_VALIDATE_INT) ?: filter_var($env, \FILTER_VALIDATE_FLOAT));
+        if (in_array($prefix, ['bool', 'not'], true)) {
+            $env = (bool) (filter_var($env, \FILTER_VALIDATE_BOOLEAN) ?: filter_var($env, \FILTER_VALIDATE_INT) ?: filter_var($env, \FILTER_VALIDATE_FLOAT));
+
+            return 'not' === $prefix ? !$env : $env;
         }
 
         if ('int' === $prefix) {
@@ -291,14 +293,6 @@ public function getEnv(string $prefix, string $name, \Closure $getEnv)
             return trim($env);
         }
 
-        if ('not' === $prefix) {
-            if (false === is_bool($env)) {
-                throw new RuntimeException(sprintf('Env var `%s` has not been resolved to a boolean type. Please prefix with `bool:` first.', $name));
-            }
-
-            return !$env;
-        }
-
         throw new RuntimeException(sprintf('Unsupported env var prefix "%s" for env name "%s".', $prefix, $name));
     }
 }
@@ -33,6 +33,7 @@ public function testSimpleProcessor()
             'foo' => ['string'],
             'base64' => ['string'],
             'bool' => ['bool'],
+            'not' => ['bool'],
             'const' => ['bool', 'int', 'float', 'string', 'array'],
             'csv' => ['array'],
             'file' => ['string'],
@@ -47,7 +48,6 @@ public function testSimpleProcessor()
             'string' => ['string'],
             'trim' => ['string'],
             'require' => ['bool', 'int', 'float', 'string', 'array'],
-            'not' => ['bool'],
         ];
 
         $this->assertSame($expected, $container->getParameterBag()->getProvidedTypes());
 
@@ -64,6 +64,22 @@ public function testGetEnvBool($value, $processed)
         $this->assertSame($processed, $result);
     }
 
+    /**
+     * @dataProvider validBools
+     */
+    public function testGetEnvNot($value, $processed)
+    {
+        $processor = new EnvVarProcessor(new Container());
+
+        $result = $processor->getEnv('not', 'foo', function ($name) use ($value) {
+            $this->assertSame('foo', $name);
+
+            return $value;
+        });
+
+        $this->assertSame(!$processed, $result);
+    }
+
     public function validBools()
     {
         return [
@@ -610,33 +626,4 @@ public function testGetEnvInvalidPrefixWithDefault()
             return null;
         });
     }
-
-    public function testGetEnvNot()
-    {
-        $processor = new EnvVarProcessor(new Container());
-
-        $result = $processor->getEnv('not', 'foo', function ($name) {
-            $this->assertSame('foo', $name);
-
-            return true;
-        });
-
-        $this->assertFalse($result);
-
-        $result = $processor->getEnv('not', 'foo', function () { return false; });
-        $this->assertTrue($result);
-    }
-
-    public function testGetEnvNotWithInvalidType()
-    {
-        $processor = new EnvVarProcessor(new Container());
-
-        $this->expectException(RuntimeException::class);
-        $this->expectExceptionMessage('Env var `foo` has not been resolved to a boolean type. Please prefix with `bool:` first.');
-        $processor->getEnv('not', 'foo', function ($name) {
-            $this->assertSame('foo', $name);
-
-            return 'bar';
-        });
-    }
 }