Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deduplicate the langcode to avoid a message repetition. #2715

Merged
merged 1 commit into from
Apr 20, 2017

Conversation

Artusamak
Copy link

When drush locale-update is run without any new translation to import, the following output may occur. This example is based on a 8.2.x core with a site installed in french and running with drush 8.1.10:

Traduction de mollom vérifiée.                              [ok]
Traduction de mollom vérifiée.                              [ok]
... x10
Traduction de paragraphs vérifiée.                          [ok]
Traduction de paragraphs vérifiée.                          [ok]
Traduction de paragraphs vérifiée.                          [ok]
Traduction de paragraphs vérifiée.                          [ok]
Traduction de paragraphs vérifiée.                          [ok]
... x10                                                                                                                                                                 [ok]
Traduction de pathauto vérifiée.                            [ok]
Traduction de pathauto vérifiée.                            [ok]
...

The command output may be totaly empty instead of showing the quoted output.

The strings are repeated as many times as locale_translation_get_status() returns projects.

After applying the fix from the PR, here is the new output (no more duplication):

Traduction de devel vérifiée.                          [ok]
Traduction de drupal vérifiée.                          [ok]
Traduction de embed vérifiée.                          [ok]
Traduction de entity vérifiée.                          [ok]
Traduction de entity_browser vérifiée.                          [ok]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants