The debug handler should always be added last

Author: Seldaek

DependencyInjection/Compiler/DebugHandlerPass.php

@@ -24,14 +24,26 @@
  */
 class DebugHandlerPass implements CompilerPassInterface
 {
+    private $channelPass;
+
+    public function __construct(LoggerChannelPass $channelPass)
+    {
+        $this->channelPass = $channelPass;
+    }
+
     public function process(ContainerBuilder $container)
     {
-        if (!$container->hasDefinition('monolog.logger_prototype') || !$container->hasDefinition('profiler')) {
+        if (!$container->hasDefinition('profiler')) {
             return;
         }
 
         $debugHandler = new Definition('%monolog.handler.debug.class%', array(Logger::DEBUG, true));
         $container->setDefinition('monolog.handler.debug', $debugHandler);
-        $container->getDefinition('monolog.logger_prototype')->addMethodCall('pushHandler', array(new Reference('monolog.handler.debug')));
+
+        foreach ($this->channelPass->getChannels() as $channel) {
+            $container
+                ->getDefinition($channel === 'app' ? 'monolog.logger' : 'monolog.logger.'.$channel)
+                ->addMethodCall('pushHandler', array(new Reference('monolog.handler.debug')));
+        }
     }
 }

DependencyInjection/Compiler/LoggerChannelPass.php

@@ -62,6 +62,11 @@ public function process(ContainerBuilder $container)
         }
     }
 
+    public function getChannels()
+    {
+        return $this->channels;
+    }
+
     protected function processChannels($configuration)
     {
         if (null === $configuration) {

MonologBundle.php

@@ -28,8 +28,8 @@ public function build(ContainerBuilder $container)
     {
         parent::build($container);
 
-        $container->addCompilerPass(new LoggerChannelPass());
-        $container->addCompilerPass(new DebugHandlerPass());
+        $container->addCompilerPass($channelPass = new LoggerChannelPass());
+        $container->addCompilerPass(new DebugHandlerPass($channelPass));
         $container->addCompilerPass(new AddProcessorsPass());
     }
 }