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

German Email Translation Problem #246

Open
christianlersch opened this issue Nov 9, 2023 · 4 comments
Open

German Email Translation Problem #246

christianlersch opened this issue Nov 9, 2023 · 4 comments

Comments

@christianlersch
Copy link

At the moment I am working on a translation problem in the german email.
Bildschirmfoto 2023-11-09 um 17 03 34

I see that in the PO German Translation everything is correct translated. Also in the file 'bp-activity-subscription-functions.php' everything is correct with the sprintf Function...

I also copied the buddypress-group-email-subscription-de_DE.mo + po files to the wp-content/languages/plugins folder but this did not work.

I see that in Buddypress you need to "install" the Mails under Tools / Buddypress / Delete and reinstall Emails. But this also not Worked.

Under Emails (/wp-admin/edit.php?post_type=bp-email) I find some Mails ([{{{site.name}}}] {{{ges.subject}}}) that coming from you but not the Email in the Screenshot...

Earlier we use the "de_DE_formal" Language and I thought that the Problem comes from not completed Language Files because the Formal is not used so often. Now we switch to the normal de_DE language to get better and more translations. But it sesm not to work better...

Do you have an idea for me what I can do to check where is the Problem?

@boonebgorges
Copy link
Owner

It looks like the de_DE localization that ships with the plugin has some errors in it.

For example, the string "Your email setting for this group is" is represented in de_DE like this: https://github.com/boonebgorges/buddypress-group-email-subscription/blob/1063628f719619cf387eaf8015705d4eaab2b17d/languages/buddypress-group-email-subscription-de_DE.po#L633C5-L633C5 This says that the original string is Your email setting for this group is: . But the actual string in the codebase has a placeholder Your email setting for this group is: %s

$email_setting_string = __( 'Your email setting for this group is: %s', 'buddypress-group-email-subscription' );

Something similar is probably happening in the case of the other untranslated string.

It's likely that the translations were correct when they were first contributed, but that the code has since changed in ways that made the translations not match.

So I think the path forward is to update the translations. This could happen by someone downloading the latest .pot file https://github.com/boonebgorges/buddypress-group-email-subscription/blob/4.2.x/languages/buddypress-group-email-subscription.pot and translating using Poedit or something similar, then sending a pull request. Or you could translate at https://translate.wordpress.org/locale/de/default/wp-plugins/buddypress-group-email-subscription/, and we could stop shipping the translation with the plugin, inheriting it instead from wordpress.org.

@christianlersch
Copy link
Author

I would like to help with the german translation and would to it over Github if you like. :)

@christianlersch
Copy link
Author

Is this ok for you? Shout I commit the german translation?

@boonebgorges
Copy link
Owner

@christianlersch Yes, please feel free to submit corrections. Vielen Dank!

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

No branches or pull requests

2 participants