Merge pull request #280 from mpdude/fix-app-channel

Support the "app" channel in the monolog:channels configuration option

Author: Seldaek

DependencyInjection/Compiler/LoggerChannelPass.php

@@ -81,6 +81,9 @@ public function process(ContainerBuilder $container)
 
         // create additional channels
         foreach ($container->getParameter('monolog.additional_channels') as $chan) {
+            if ($chan === 'app') {
+                continue;
+            }
             $loggerId = sprintf('monolog.logger.%s', $chan);
             $this->createLogger($chan, $loggerId, $container);
             $container->getDefinition($loggerId)->setPublic(true);

Tests/DependencyInjection/Compiler/LoggerChannelPassTest.php

@@ -134,6 +134,14 @@ public function testTagNotBreakingIfNoLogger()
         $this->assertEquals(array(), $dummyService->getArguments());
     }
 
+    public function testChannelsConfigurationOptionSupportsAppChannel()
+    {
+        $container = $this->getFunctionalContainer();
+
+        $container->setParameter('monolog.additional_channels', array('app'));
+        $container->compile();
+    }
+
     private function getContainer()
     {
         $container = new ContainerBuilder();
@@ -202,6 +210,9 @@ private function getContainerWithSetter()
         return $container;
     }
 
+    /**
+     * @return ContainerBuilder
+     */
     private function getFunctionalContainer()
     {
         $container = new ContainerBuilder();