[FrameworkBundle][Translations] Fix translation commands

kevin-verschaeve · kevin-verschaeve · commit b81a384b8acb · 2017-10-17T15:40:32.000+02:00
Search for translations also in the new translations dir at the project root
Search for .yaml file as well as .yml
diff --git a/src/Symfony/Bundle/FrameworkBundle/Command/TranslationDebugCommand.php b/src/Symfony/Bundle/FrameworkBundle/Command/TranslationDebugCommand.php
@@ -113,7 +113,10 @@ protected function execute(InputInterface $input, OutputInterface $output)
         $kernel = $this->getContainer()->get('kernel');
 
         // Define Root Path to App folder
-        $transPaths = array($kernel->getRootDir().'/Resources/');
+        $transPaths = array(
+            $kernel->getRootDir().'/Resources/',
+            $kernel->getProjectDir().'/',
+        );
 
         // Override with provided Bundle info
         if (null !== $input->getArgument('bundle')) {
diff --git a/src/Symfony/Bundle/FrameworkBundle/Command/TranslationUpdateCommand.php b/src/Symfony/Bundle/FrameworkBundle/Command/TranslationUpdateCommand.php
@@ -105,7 +105,10 @@ protected function execute(InputInterface $input, OutputInterface $output)
         $kernel = $this->getContainer()->get('kernel');
 
         // Define Root Path to App folder
-        $transPaths = array($kernel->getRootDir().'/Resources/');
+        $transPaths = array(
+            $kernel->getRootDir().'/Resources/',
+            $kernel->getProjectDir(),
+        );
         $currentName = 'app folder';
 
         // Override with provided Bundle info
diff --git a/src/Symfony/Bundle/FrameworkBundle/Resources/config/translation.xml b/src/Symfony/Bundle/FrameworkBundle/Resources/config/translation.xml
@@ -33,7 +33,7 @@
         </service>
 
         <service id="translation.loader.yml" class="Symfony\Component\Translation\Loader\YamlFileLoader" public="true">
-            <tag name="translation.loader" alias="yml" />
+            <tag name="translation.loader" alias="yaml" legacy-alias="yml" />
         </service>
 
         <service id="translation.loader.xliff" class="Symfony\Component\Translation\Loader\XliffFileLoader" public="true">
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/Command/TranslationDebugCommandTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/Command/TranslationDebugCommandTest.php
@@ -65,7 +65,10 @@ public function testDebugDefaultDirectory()
 
     public function testDebugCustomDirectory()
     {
-        $kernel = $this->getMockBuilder('Symfony\Component\HttpKernel\KernelInterface')->getMock();
+        $kernel = $this->getMockBuilder('Symfony\Component\HttpKernel\Kernel')
+            ->disableOriginalConstructor()
+            ->getMock();
+
         $kernel->expects($this->once())
             ->method('getBundle')
             ->with($this->equalTo($this->translationDir))
@@ -83,7 +86,10 @@ public function testDebugCustomDirectory()
      */
     public function testDebugInvalidDirectory()
     {
-        $kernel = $this->getMockBuilder('Symfony\Component\HttpKernel\KernelInterface')->getMock();
+        $kernel = $this->getMockBuilder('Symfony\Component\HttpKernel\Kernel')
+            ->disableOriginalConstructor()
+            ->getMock();
+
         $kernel->expects($this->once())
             ->method('getBundle')
             ->with($this->equalTo('dir'))
@@ -152,7 +158,10 @@ private function getContainer($extractedMessages = array(), $loadedMessages = ar
             );
 
         if (null === $kernel) {
-            $kernel = $this->getMockBuilder('Symfony\Component\HttpKernel\KernelInterface')->getMock();
+            $kernel = $this->getMockBuilder('Symfony\Component\HttpKernel\Kernel')
+                ->disableOriginalConstructor()
+                ->getMock();
+
             $kernel
                 ->expects($this->any())
                 ->method('getBundle')
@@ -167,6 +176,10 @@ private function getContainer($extractedMessages = array(), $loadedMessages = ar
             ->method('getRootDir')
             ->will($this->returnValue($this->translationDir));
 
+        $kernel
+            ->method('getProjectDir')
+            ->will($this->returnValue($this->translationDir));
+
         $container = $this->getMockBuilder('Symfony\Component\DependencyInjection\ContainerInterface')->getMock();
         $container
             ->expects($this->any())
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/Command/TranslationUpdateCommandTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/Command/TranslationUpdateCommandTest.php
@@ -132,7 +132,9 @@ private function getContainer($extractedMessages = array(), $loadedMessages = ar
             );
 
         if (null === $kernel) {
-            $kernel = $this->getMockBuilder('Symfony\Component\HttpKernel\KernelInterface')->getMock();
+            $kernel = $this->getMockBuilder('Symfony\Component\HttpKernel\Kernel')
+                ->disableOriginalConstructor()
+                ->getMock();
             $kernel
                 ->expects($this->any())
                 ->method('getBundle')
@@ -147,6 +149,10 @@ private function getContainer($extractedMessages = array(), $loadedMessages = ar
             ->method('getRootDir')
             ->will($this->returnValue($this->translationDir));
 
+        $kernel
+            ->method('getProjectDir')
+            ->will($this->returnValue($this->translationDir));
+
         $container = $this->getMockBuilder('Symfony\Component\DependencyInjection\ContainerInterface')->getMock();
         $container
             ->expects($this->any())
