Skip to content

Commit

Permalink
bug #35170 [FrameworkBundle][TranslationUpdateCommand] Do not output …
Browse files Browse the repository at this point in the history
…positive feedback on stderr (fancyweb)

This PR was merged into the 3.4 branch.

Discussion
----------

[FrameworkBundle][TranslationUpdateCommand] Do not output positive feedback on stderr

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | #34865 (comment)
| License       | MIT
| Doc PR        | -

Outputing everything on stderr seems wrong. I'm going to check upper branches.

Commits
-------

87212e4 [FrameworkBundle][TranslationUpdateCommand] Do not output positive feedback on stderr
  • Loading branch information
fabpot committed Jan 7, 2020
2 parents 4b419f2 + 87212e4 commit 7fa87b4
Showing 1 changed file with 7 additions and 7 deletions.
Expand Up @@ -200,12 +200,12 @@ protected function execute(InputInterface $input, OutputInterface $output)
}
}

$errorIo->title('Translation Messages Extractor and Dumper');
$errorIo->comment(sprintf('Generating "<info>%s</info>" translation files for "<info>%s</info>"', $input->getArgument('locale'), $currentName));
$io->title('Translation Messages Extractor and Dumper');
$io->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'));
$errorIo->comment('Parsing templates...');
$io->comment('Parsing templates...');
$prefix = $input->getOption('prefix');
// @deprecated since version 3.4, to be removed in 4.0 along with the --no-prefix option
if ($input->getOption('no-prefix')) {
Expand All @@ -221,7 +221,7 @@ protected function execute(InputInterface $input, OutputInterface $output)

// load any existing messages from the translation files
$currentCatalogue = new MessageCatalogue($input->getArgument('locale'));
$errorIo->comment('Loading translation files...');
$io->comment('Loading translation files...');
foreach ($transPaths as $path) {
if (is_dir($path)) {
$this->reader->read($path, $currentCatalogue);
Expand Down Expand Up @@ -274,7 +274,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
}

if ('xlf' == $input->getOption('output-format')) {
$errorIo->comment('Xliff output version is <info>1.2</info>');
$io->comment('Xliff output version is <info>1.2</info>');
}

$resultMessage = sprintf('%d message%s successfully extracted', $extractedMessagesCount, $extractedMessagesCount > 1 ? 's were' : ' was');
Expand All @@ -286,7 +286,7 @@ protected function execute(InputInterface $input, OutputInterface $output)

// save the files
if (true === $input->getOption('force')) {
$errorIo->comment('Writing files...');
$io->comment('Writing files...');

$bundleTransPath = false;
foreach ($transPaths as $path) {
Expand All @@ -306,7 +306,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
}
}

$errorIo->success($resultMessage.'.');
$io->success($resultMessage.'.');

return null;
}
Expand Down

0 comments on commit 7fa87b4

Please sign in to comment.