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

Mistranslations are breaking the build #2998

Closed
gravitystorm opened this issue Dec 8, 2020 · 3 comments
Closed

Mistranslations are breaking the build #2998

gravitystorm opened this issue Dec 8, 2020 · 3 comments
Labels
bug Something is broken or not working as expected i18n Internationalisation - related to translation into different languages

Comments

@gravitystorm
Copy link
Collaborator

The latest commit from @translatewiki has broken the build, since it appears to have messed up some translations.

#2993 made, among other things, the following change:

 user_mailer:
   message_notification:
-    subject_header: "[OpenStreetMap] %{subject}"
+    subject: "[OpenStreetMap] %{message_title}"
     hi: "Hi %{to_user},"

However, f8f3271 has made a bunch of changes like this:

 user_mailer:
   message_notification:
+    subject: '[OpenStreetMap] %{user} het ''n nuwe boodskap aan u gestuur'
     hi: Hallo %{to_user},
 user_mailer:
   message_notification:
-    subject_header: '[OpenStreetMap] %{subject}'
+    subject: '[OpenStreetMap] %{user} je ći nowu powěsć pósłał'
     hi: Witaj %{to_user},

This is surprising, since they appear to be correct translations for "%{user} has sent you a new message", which is not a string we have in en.yml. It's also not a copy+paste from another translations that have subject as the key.

Digging into this, and going a long way back through the git history, I think I've found the cause. It appears that over 10 years ago, we deleted the key user_mailer.message_notification.subject, but it has lingered somewhere in the translatewiki database. And since #2993 has accidentally reused the key, the "zombie" translations appear to have been resurrected after all this time!

@translatewiki could you please delete all the user_mailer.message_notification.subject values which contain the %{user} variable? This can then let them be re-translated with the up to date variable and content, and will fix the build.

@gravitystorm gravitystorm added bug Something is broken or not working as expected i18n Internationalisation - related to translation into different languages labels Dec 8, 2020
@Nikerabbit
Copy link
Contributor

1985142 hopefully fixes this.

@tomhughes
Copy link
Member

Confirmed fixed.

@gravitystorm
Copy link
Collaborator Author

@Nikerabbit Thanks! And thanks for sorting it out so quickly, much appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is broken or not working as expected i18n Internationalisation - related to translation into different languages
Projects
None yet
Development

No branches or pull requests

3 participants