Disable DebugHandler when the profiler is disabled, fixes #37

symfony · Apr 22, 2013 · 1fc0864 · 1fc0864
1 parent d401cab
commit 1fc0864
Showing 1 changed file with 14 additions and 0 deletions.
diff --git a/DependencyInjection/Compiler/DebugHandlerPass.php b/DependencyInjection/Compiler/DebugHandlerPass.php
@@ -21,6 +21,7 @@
  * Adds the DebugHandler when the profiler is enabled.
  *
  * @author Christophe Coevoet <stof@notk.org>
+ * @author Jordi Boggiano <j.boggiano@seld.be>
  */
 class DebugHandlerPass implements CompilerPassInterface
 {
@@ -37,6 +38,19 @@ public function process(ContainerBuilder $container)
             return;
         }
 
+        // detect if the profiler is present but will be disabled
+        $enabled = true;
+        foreach ($container->getDefinition('profiler')->getMethodCalls() as $call) {
+            if ($call[0] === 'disable') {
+                $enabled = false;
+            } elseif ($call[0] === 'enable') {
+                $enabled = true;
+            }
+        }
+        if (!$enabled) {
+            return;
+        }
+
         $debugHandler = new Definition('%monolog.handler.debug.class%', array(Logger::DEBUG, true));
         $container->setDefinition('monolog.handler.debug', $debugHandler);