[Debug] make screaming configurable

nicolas-grekas · nicolas-grekas · commit c3d012c77526 · 2014-09-30T14:16:33.000+02:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php
@@ -127,13 +127,14 @@ public function load(array $configs, ContainerBuilder $container)
         $definition = $container->findDefinition('debug.debug_handlers_listener');
 
         if ($container->hasParameter('templating.helper.code.file_link_format')) {
-            $definition->replaceArgument(4, '%templating.helper.code.file_link_format%');
+            $definition->replaceArgument(5, '%templating.helper.code.file_link_format%');
         }
 
         if ($container->getParameter('kernel.debug')) {
             $loader->load('debug.xml');
 
             $definition->replaceArgument(0, array(new Reference('http_kernel', ContainerInterface::NULL_ON_INVALID_REFERENCE), 'terminateWithException'));
+            $definition->replaceArgument(3, E_ALL | E_STRICT);
 
             $definition = $container->findDefinition('http_kernel');
             $definition->replaceArgument(2, new Reference('debug.controller_resolver'));
diff --git a/src/Symfony/Bundle/FrameworkBundle/Resources/config/debug_prod.xml b/src/Symfony/Bundle/FrameworkBundle/Resources/config/debug_prod.xml
@@ -16,7 +16,8 @@
             <argument /><!-- Exception handler -->
             <argument type="service" id="logger" on-invalid="null" />
             <argument /><!-- Log levels map for enabled error levels -->
-            <argument>%kernel.debug%</argument>
+            <argument>null</argument>
+            <argument>true</argument>
             <argument>null</argument><!-- %templating.helper.code.file_link_format% -->
         </service>
 
diff --git a/src/Symfony/Component/HttpKernel/EventListener/DebugHandlersListener.php b/src/Symfony/Component/HttpKernel/EventListener/DebugHandlersListener.php
@@ -27,45 +27,52 @@ class DebugHandlersListener implements EventSubscriberInterface
     private $exceptionHandler;
     private $logger;
     private $levels;
-    private $debug;
+    private $throwAt;
+    private $scream;
+    private $fileLinkFormat;
 
     /**
-     * @param callable             $exceptionHandler A handler that will be called on Exception
+     * @param callable|null        $exceptionHandler A handler that will be called on Exception
      * @param LoggerInterface|null $logger           A PSR-3 logger
      * @param array|int            $levels           An array map of E_* to LogLevel::* or an integer bit field of E_* constants
-     * @param bool                 $debug            Enables/disables debug mode
+     * @param int|null             $throwAt          Thrown errors in a bit field of E_* constants, or null to keep the current value
+     * @param bool                 $scream           Enables/disables screaming mode, where even silenced errors are logged
      * @param string               $fileLinkFormat   The format for links to source files
      */
-    public function __construct($exceptionHandler, LoggerInterface $logger = null, $levels = null, $debug = true, $fileLinkFormat = null)
+    public function __construct($exceptionHandler, LoggerInterface $logger = null, $levels = null, $throwAt = -1, $scream = true, $fileLinkFormat = null)
     {
         $this->exceptionHandler = $exceptionHandler;
         $this->logger = $logger;
         $this->levels = $levels;
-        $this->debug = $debug;
+        $this->throwAt = is_numeric($throwAt) ? (int) $throwAt : (null === $throwAt ? null : ($throwAt ? -1 : null));
+        $this->scream = (bool) $scream;
         $this->fileLinkFormat = $fileLinkFormat ?: ini_get('xdebug.file_link_format') ?: get_cfg_var('xdebug.file_link_format');
     }
 
     public function configure()
     {
-        if ($this->logger) {
-            $handler = set_error_handler('var_dump', 0);
-            $handler = is_array($handler) ? $handler[0] : null;
-            restore_error_handler();
-            if ($handler instanceof ErrorHandler) {
-                if ($this->debug) {
-                    $handler->throwAt(-1);
-                }
+        $handler = set_error_handler('var_dump', 0);
+        $handler = is_array($handler) ? $handler[0] : null;
+        restore_error_handler();
+        if ($handler instanceof ErrorHandler) {
+            if ($this->logger) {
                 $handler->setDefaultLogger($this->logger, $this->levels);
                 if (is_array($this->levels)) {
-                    $scream = 0;
+                    $levels = 0;
                     foreach ($this->levels as $type => $log) {
-                        $scream |= $type;
+                        $levels |= $type;
                     }
-                    $this->levels = $scream;
+                } else {
+                    $levels = null === $this->levels ? E_ALL | E_STRICT : $this->levels;
                 }
-                $handler->screamAt($this->levels);
+                if ($this->scream) {
+                    $handler->screamAt($levels);
+                }
+                $this->logger = null;
+            }
+            if (null !== $this->throwAt) {
+                $handler->throwAt($this->throwAt, true);
             }
-            $this->logger = $this->levels = null;
         }
         if ($this->exceptionHandler) {
             $handler = set_exception_handler('var_dump');
