Use stderr for some other commands

Author: ogizanagi

Command/ConfigDumpReferenceCommand.php

@@ -74,9 +74,9 @@ protected function configure()
     protected function execute(InputInterface $input, OutputInterface $output)
     {
         $io = new SymfonyStyle($input, $output);
+        $errorIo = $output instanceof ConsoleOutputInterface ? new SymfonyStyle($input, $output->getErrorOutput()) : $io;
 
         if (null === $name = $input->getArgument('name')) {
-            $errorIo = $output instanceof ConsoleOutputInterface ? new SymfonyStyle($input, $output->getErrorOutput()) : $io;
             $this->listBundles($errorIo);
             $errorIo->comment(array(
                 'Provide the name of a bundle as the first argument of this command to dump its default configuration. (e.g. <comment>config:dump-reference FrameworkBundle</comment>)',
@@ -96,7 +96,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         $path = $input->getArgument('path');
 
         if ($path !== null && 'yaml' !== $format) {
-            $io->error('The "path" option is only available for the "yaml" format.');
+            $errorIo->error('The "path" option is only available for the "yaml" format.');
 
             return 1;
         }

Command/TranslationDebugCommand.php

@@ -12,6 +12,7 @@
 namespace Symfony\Bundle\FrameworkBundle\Command;
 
 use Symfony\Bundle\FrameworkBundle\Translation\TranslationLoader;
+use Symfony\Component\Console\Output\ConsoleOutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputArgument;
@@ -159,7 +160,8 @@ protected function execute(InputInterface $input, OutputInterface $output)
                 $outputMessage .= sprintf(' and domain "%s"', $domain);
             }
 
-            $io->warning($outputMessage);
+            $errorIo = $output instanceof ConsoleOutputInterface ? new SymfonyStyle($input, $output->getErrorOutput()) : $io;
+            $errorIo->warning($outputMessage);
 
             return;
         }

Command/TranslationUpdateCommand.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\Bundle\FrameworkBundle\Command;
 
+use Symfony\Component\Console\Output\ConsoleOutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 use Symfony\Component\Translation\Catalogue\TargetOperation;
 use Symfony\Component\Translation\Catalogue\MergeOperation;
@@ -85,10 +86,11 @@ public function isEnabled()
     protected function execute(InputInterface $input, OutputInterface $output)
     {
         $io = new SymfonyStyle($input, $output);
+        $errorIo = $output instanceof ConsoleOutputInterface ? new SymfonyStyle($input, $output->getErrorOutput()) : $io;
 
         // check presence of force or dump-message
         if ($input->getOption('force') !== true && $input->getOption('dump-messages') !== true) {
-            $io->error('You must choose one of --force or --dump-messages');
+            $errorIo->error('You must choose one of --force or --dump-messages');
 
             return 1;
         }
@@ -97,7 +99,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         $writer = $this->getContainer()->get('translation.writer');
         $supportedFormats = $writer->getFormats();
         if (!in_array($input->getOption('output-format'), $supportedFormats)) {
-            $io->error(array('Wrong output format', 'Supported formats are: '.implode(', ', $supportedFormats).'.'));
+            $errorIo->error(array('Wrong output format', 'Supported formats are: '.implode(', ', $supportedFormats).'.'));
 
             return 1;
         }
@@ -127,12 +129,12 @@ protected function execute(InputInterface $input, OutputInterface $output)
             }
         }
 
-        $io->title('Translation Messages Extractor and Dumper');
-        $io->comment(sprintf('Generating "<info>%s</info>" translation files for "<info>%s</info>"', $input->getArgument('locale'), $currentName));
+        $errorIo->title('Translation Messages Extractor and Dumper');
+        $errorIo->comment(sprintf('Generating "<info>%s</info>" translation files for "<info>%s</info>"', $input->getArgument('locale'), $currentName));
 
         // load any messages from templates
         $extractedCatalogue = new MessageCatalogue($input->getArgument('locale'));
-        $io->comment('Parsing templates...');
+        $errorIo->comment('Parsing templates...');
         $extractor = $this->getContainer()->get('translation.extractor');
         $extractor->setPrefix($input->getOption('no-prefix') ? '' : $input->getOption('prefix'));
         foreach ($transPaths as $path) {
@@ -144,7 +146,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
 
         // load any existing messages from the translation files
         $currentCatalogue = new MessageCatalogue($input->getArgument('locale'));
-        $io->comment('Loading translation files...');
+        $errorIo->comment('Loading translation files...');
         $loader = $this->getContainer()->get('translation.loader');
         foreach ($transPaths as $path) {
             $path .= 'translations';
@@ -165,7 +167,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
 
         // Exit if no messages found.
         if (!count($operation->getDomains())) {
-            $io->warning('No translation messages were found.');
+            $errorIo->warning('No translation messages were found.');
 
             return;
         }
@@ -199,7 +201,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
             }
 
             if ($input->getOption('output-format') == 'xlf') {
-                $io->comment('Xliff output version is <info>1.2</info>');
+                $errorIo->comment('Xliff output version is <info>1.2</info>');
             }
 
             $resultMessage = sprintf('%d message%s successfully extracted', $extractedMessagesCount, $extractedMessagesCount > 1 ? 's were' : ' was');
@@ -211,7 +213,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
 
         // save the files
         if ($input->getOption('force') === true) {
-            $io->comment('Writing files...');
+            $errorIo->comment('Writing files...');
 
             $bundleTransPath = false;
             foreach ($transPaths as $path) {
@@ -232,7 +234,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
             }
         }
 
-        $io->success($resultMessage.'.');
+        $errorIo->success($resultMessage.'.');
     }
 
     private function filterCatalogue(MessageCatalogue $catalogue, $domain)