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

Subject of mail when sharing files via link does not get translated to german #37321

Closed
weberdigital opened this issue Apr 29, 2020 · 6 comments · Fixed by #37322
Closed

Subject of mail when sharing files via link does not get translated to german #37321

weberdigital opened this issue Apr 29, 2020 · 6 comments · Fixed by #37322

Comments

@weberdigital
Copy link

Steps to reproduce

  1. Set default_language to de_DE in the config.php.
  2. Click on the sharing button of a file or folder.
  3. Create a public link.
  4. Share it with someone over mail with the designated button.

Expected behaviour

The subject "X has shared Y with you" should be in german (default_language).

Actual behaviour

The subject "X has shared Y with you" is not getting translated to german.

This is because in lib/private/Share/MailNotifications.php:257, the string that should get translated is "%s shared »%s« with you" - which seems correct.

However, in apps/files_sharing/l10n/de_DE.json there is only the string "%2$s shared %1$s with you" (https://github.com/owncloud/core/blob/master/apps/files_sharing/l10n/de_DE.json#L64) - but not the correct one:
(https://github.com/owncloud/core/blob/master/lib/private/Share/MailNotifications.php#L257).

To fix this, maybe add the string "%s shared »%s« with you" to the de_DE.json file?

@DeepDiver1975
Copy link
Member

Which version are you running? We recently fixed an issue in this area? THX

@DeepDiver1975
Copy link
Member

Actually this issue - #37040 - but this is at a different code location. I'll have a look ....

@weberdigital
Copy link
Author

Sorry, forgot to mention the version we are running. We are running version 10.4.1.
8382368 fixxed it for the "sendInternalShareMail", but not for "sendLinkShareMail".

@DeepDiver1975
Copy link
Member

@weber-ebusiness feel free to test the changes in #37322

@weberdigital
Copy link
Author

Yes, that fixed the issue. Thank you very much for the fast response!

@DeepDiver1975
Copy link
Member

Thank you very much for the fast response!

You are welcome!

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 a pull request may close this issue.

2 participants