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

Save & Close button showing & in translations #12653

Closed
1 task done
RCheesley opened this issue Aug 15, 2023 · 3 comments
Closed
1 task done

Save & Close button showing & in translations #12653

RCheesley opened this issue Aug 15, 2023 · 3 comments
Labels
bug Issues or PR's relating to bugs good first issue hacktoberfest Issues that would be great for Hacktoberfest participants to work on regression A bug that broke something in the last release T1 Low difficulty to fix (issue) or test (PR) translations Anything related to translations

Comments

@RCheesley
Copy link
Sponsor Member

Mautic Version

5.0.x series

PHP version

8

What browsers are you seeing the problem on?

Not relevant

What happened?

Noticed this when debugging something with translations:

screenshot-mautic ddev site-2023 08 15-16_56_11

And also in German:

screenshot-mautic ddev site-2023 08 15-16_59_48

How can we reproduce this issue?

Step 1: Switch to a translation
Step 2: Notice the save & close button is not correctly formatted.

Relevant log output

No response

Code of Conduct

  • I confirm that I have read and agree to follow this project's Code of Conduct
@RCheesley RCheesley added bug Issues or PR's relating to bugs needs-triage For new issues/PRs that need to be triaged T1 Low difficulty to fix (issue) or test (PR) translations Anything related to translations regression A bug that broke something in the last release good first issue and removed needs-triage For new issues/PRs that need to be triaged labels Aug 15, 2023
@RCheesley RCheesley added this to the 5.0-Release Candidate milestone Sep 27, 2023
@RCheesley RCheesley added the hacktoberfest Issues that would be great for Hacktoberfest participants to work on label Oct 3, 2023
@escopecz
Copy link
Sponsor Member

escopecz commented Oct 4, 2023

This seem to be an issue with the translation itself:

mautic.core.form.saveandclose="Speichern & Schließen"

You can see that it is using just the ampersand (not URL-encoded) in the source language: https://github.com/mautic/mautic/blob/5.x/app/bundles/CoreBundle/Translations/en_US/messages.ini#L238

In Transifex:
Screenshot 2023-10-04 at 16 54 21

I fixed it for the German translation in Transifex. If this issue is in other languages, please provide a list or fix it directly.

@escopecz escopecz closed this as completed Oct 4, 2023
@escopecz escopecz removed this from the 5.0-Release Candidate-1 milestone Oct 4, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 4, 2023

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If this issue is continuing with the lastest stable version of Mautic, please open a new issue that references this one.

@mautibot
Copy link

This issue has been mentioned on Mautic Forums. There might be relevant details there:

https://forum.mautic.org/t/mautic-5-0-3-characters-in-certain-button-labels-and-config-values-are-being-encoded/30907/3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issues or PR's relating to bugs good first issue hacktoberfest Issues that would be great for Hacktoberfest participants to work on regression A bug that broke something in the last release T1 Low difficulty to fix (issue) or test (PR) translations Anything related to translations
Projects
Archived in project
Development

No branches or pull requests

3 participants