[FrameworkBundle] framework.php_errors.log now accept a log level

Amrouche Hamza · Amrouche Hamza · commit b02c7d7713bd · 2018-03-26T08:33:08.000+02:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php
@@ -833,10 +833,13 @@ private function addPhpErrorsSection(ArrayNodeDefinition $rootNode)
                     ->info('PHP errors handling configuration')
                     ->addDefaultsIfNotSet()
                     ->children()
-                        ->booleanNode('log')
+                        ->scalarNode('log')
                             ->info('Use the app logger instead of the PHP logger for logging PHP errors.')
                             ->defaultValue($this->debug)
                             ->treatNullLike($this->debug)
+                            ->validate()
+                            ->ifTrue(function ($v) { return !(\is_int($v) || \is_bool($v)); })
+                            ->thenInvalid('The "php_errors.log" parameter should be either a int or a boolean.')
                         ->end()
                         ->booleanNode('throw')
                             ->info('Throw PHP errors as \ErrorException instances.')
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php
@@ -605,6 +605,10 @@ private function registerDebugConfiguration(array $config, ContainerBuilder $con
             $definition->replaceArgument(1, null);
         }
 
+        if (\is_int($config['log']) && $config['log']) {
+            $definition->replaceArgument(4, $config['log']);
+        }
+
         if (!$config['throw']) {
             $container->setParameter('debug.error_handler.throw_at', 0);
         }

Original file line number	Diff line number	Diff line change
`@@ -605,6 +605,10 @@ private function registerDebugConfiguration(array $config, ContainerBuilder $con`
`605`	`605`	`$definition->replaceArgument(1, null);`
`606`	`606`	`}`
`607`	`607`
	`608`	`+ if (\is_int($config['log']) && $config['log']) {`
	`609`	`+ $definition->replaceArgument(4, $config['log']);`
	`610`	`+ }`
	`611`	`+`
`608`	`612`	`if (!$config['throw']) {`
`609`	`613`	`$container->setParameter('debug.error_handler.throw_at', 0);`
`610`	`614`	`}`