ISSUE-278:

- Updated naming strategy. Channel 'test' become '$testLogger' and etc.

Author: adrenalinkin

DependencyInjection/Compiler/LoggerChannelPass.php

@@ -152,7 +152,9 @@ protected function createLogger($channel, $loggerId, ContainerBuilder $container
 
         // Allows only for Symfony 4.2+
         if (\method_exists($container, 'registerAliasForArgument')) {
-            $container->registerAliasForArgument($loggerId, LoggerInterface::class);
+            $parameterName = $channel . 'Logger';
+
+            $container->registerAliasForArgument($loggerId, LoggerInterface::class, $parameterName);
         }
     }
 

Tests/DependencyInjection/Compiler/LoggerChannelPassTest.php

@@ -53,6 +53,7 @@ public function testProcess()
 
         $this->assertNotNull($container->getDefinition('monolog.logger.additional'));
 
+        // test below acceptable only for Symfony 4.2+
         if (!\method_exists(ContainerBuilder::class, 'registerAliasForArgument')) {
             return;
         }
@@ -61,7 +62,7 @@ public function testProcess()
         $expectedChannels[] = 'additional';
 
         foreach ($expectedChannels as $channelName) {
-            $aliasName = LoggerInterface::class.' $monologLogger'.\ucfirst($channelName);
+            $aliasName = LoggerInterface::class.' $' .$channelName.'Logger';
             $this->assertTrue($container->hasAlias($aliasName), 'type-hinted alias should be exists for each logger channel');
         }
     }