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

[bug] For default.inc.php in the translation line $ _lang ['remove_this_confirm'] does not work the parameter "[[+type]]" #14256

Open
Ruslan-Aleev opened this Issue Jan 11, 2019 · 2 comments

Comments

Projects
None yet
3 participants
@Ruslan-Aleev
Copy link
Contributor

Ruslan-Aleev commented Jan 11, 2019

Bug report

Summary

The string $_lang['remove_this_confirm'] in English looks like this:

Are you sure you want to remove the [[+type]]: "[[+name]]"?

However, the "[[+type]]" parameter does not work, tested in different languages, see the picture below.

lang_bug_1

Environment

MODX 2.7.0

@Ruslan-Aleev Ruslan-Aleev changed the title [bug] For default.inc.php in the translation line $ _lang ['remove_this_confirm'] does not work the parameter "type" [bug] For default.inc.php in the translation line $ _lang ['remove_this_confirm'] does not work the parameter "[[+type]]" Jan 11, 2019

@Alroniks

This comment has been minimized.

Copy link
Collaborator

Alroniks commented Jan 11, 2019

I see that it works fine. Chunk is a placeholder, it shouldn't be translated in this case. Also, lexicons do not support nested lexicon tags, only simple placeholders. So it should be fixed, but my comment as a clarification.

@Ruslan-Aleev

This comment has been minimized.

Copy link
Contributor

Ruslan-Aleev commented Jan 11, 2019

Placeholders are translated, in other places they are displayed correctly, but not in this pop-up message.

lang_bug_2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment