bug #13047 [FrameworkBundle][Logging Translator] skip if param "translator.logging" doesn't exist. (aitboudad)

fabpot · fabpot · commit 13495c21a542 · 2014-12-20T16:31:58.000+01:00
This PR was merged into the 2.6 branch. Discussion ---------- [FrameworkBundle][Logging Translator] skip if param "translator.logging" doesn't exist. | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Fixed tickets | #12989 | Tests pass? | yes | License | MIT Commits ------- 1008991 skip if param "translator.logging" doesn't exist.
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/LoggingTranslatorPass.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/LoggingTranslatorPass.php
@@ -30,7 +30,7 @@ public function process(ContainerBuilder $container)
             return;
         }
 
-        if ($container->getParameter('translator.logging')) {
+        if ($container->hasParameter('translator.logging') && $container->getParameter('translator.logging')) {
             $translatorAlias = $container->getAlias('translator');
             $definition = $container->getDefinition((string) $translatorAlias);
             $class = $container->getParameterBag()->resolveValue($definition->getClass());

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ public function process(ContainerBuilder $container)`
`30`	`30`	`return;`
`31`	`31`	`}`
`32`	`32`
`33`		`- if ($container->getParameter('translator.logging')) {`
	`33`	`+ if ($container->hasParameter('translator.logging') && $container->getParameter('translator.logging')) {`
`34`	`34`	`$translatorAlias = $container->getAlias('translator');`
`35`	`35`	`$definition = $container->getDefinition((string) $translatorAlias);`
`36`	`36`	`$class = $container->getParameterBag()->resolveValue($definition->getClass());`