[Profiler][Translation] Logging false by default and desactivated when using the profiler

Amrouche Hamza · Amrouche Hamza · commit 1dd741e367f3 · 2017-11-01T19:40:32.000+01:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/LoggingTranslatorPass.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/LoggingTranslatorPass.php
@@ -28,6 +28,10 @@ public function process(ContainerBuilder $container)
             return;
         }
 
+        if ($container->has('profiler') && $container->hasParameter('translator.logging')) {
+            $container->setParameter('translator.logging', false);
+        }
+
         if ($container->hasParameter('translator.logging') && $container->getParameter('translator.logging')) {
             $translatorAlias = $container->getAlias('translator');
             $definition = $container->getDefinition((string) $translatorAlias);
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php
@@ -622,7 +622,7 @@ private function addTranslatorSection(ArrayNodeDefinition $rootNode)
                             ->prototype('scalar')->end()
                             ->defaultValue(array('en'))
                         ->end()
-                        ->booleanNode('logging')->defaultValue($this->debug)->end()
+                        ->booleanNode('logging')->defaultValue(false)->end()
                         ->scalarNode('formatter')->defaultValue('translator.formatter.default')->end()
                         ->arrayNode('paths')
                             ->prototype('scalar')->end()
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Compiler/LoggingTranslatorPassTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Compiler/LoggingTranslatorPassTest.php
@@ -38,7 +38,7 @@ public function testProcess()
             ->method('getDefinition')
             ->will($this->returnValue($definition));
 
-        $container->expects($this->once())
+        $container->expects($this->atLeastOnce())
             ->method('hasParameter')
             ->with('translator.logging')
             ->will($this->returnValue(true));
@@ -80,6 +80,37 @@ public function testProcess()
         $pass->process($container);
     }
 
+    public function testProcessWithProfiler()
+    {
+        $container = $this->getMockBuilder('Symfony\Component\DependencyInjection\ContainerBuilder')->getMock();
+
+        $container->expects($this->exactly(2))
+            ->method('hasAlias')
+            ->will($this->returnValue(true));
+
+        $container->expects($this->once())
+            ->method('getParameter')
+            ->will($this->returnValue(false));
+
+        $container->expects($this->atLeastOnce())
+            ->method('hasParameter')
+            ->with('translator.logging')
+            ->will($this->returnValue(true));
+
+        $container->expects($this->atLeastOnce())
+            ->method('setParameter')
+            ->with('translator.logging')
+            ->will($this->returnValue(false));
+
+        $container->expects($this->atLeastOnce())
+            ->method('has')
+            ->with('profiler')
+            ->will($this->returnValue(true));
+
+        $pass = new LoggingTranslatorPass();
+        $pass->process($container);
+    }
+
     public function testThatCompilerPassIsIgnoredIfThereIsNotLoggerDefinition()
     {
         $container = $this->getMockBuilder('Symfony\Component\DependencyInjection\ContainerBuilder')->getMock();
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/ConfigurationTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/ConfigurationTest.php
@@ -152,7 +152,7 @@ protected static function getBundleDefaultConfig()
             'translator' => array(
                 'enabled' => !class_exists(FullStack::class),
                 'fallbacks' => array('en'),
-                'logging' => true,
+                'logging' => false,
                 'formatter' => 'translator.formatter.default',
                 'paths' => array(),
             ),
