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

No translated lexicon for some languages #14672

Open
Ruslan-Aleev opened this issue Aug 13, 2019 · 8 comments

Comments

@Ruslan-Aleev
Copy link
Contributor

commented Aug 13, 2019

Bug report

Summary

Apparently, after a recent lexicon update with Crowdin, some lines for some languages disappeared (became in English), see screenshots.

Sections:
"Refresh URLs"
Items in "Reports"

Perhaps in other elements of the manager panel.

AR:
ar

RU:
ru

Most likely in other languages (in Belorussian for sure)

Environment

MODX 2.x

@JoshuaLuckers

This comment has been minimized.

Copy link
Collaborator

commented Aug 13, 2019

They have to be changed in Crowdin.

@Ruslan-Aleev

This comment has been minimized.

Copy link
Contributor Author

commented Aug 13, 2019

For some reason, in Crowdin, this file was completely cleared.

topmenu_inc_error

@Ruslan-Aleev

This comment has been minimized.

Copy link
Contributor Author

commented Aug 13, 2019

@opengeek There is some information and thoughts why the file topmenu.inc.php was cleared and how to restore it?

@opengeek

This comment has been minimized.

Copy link
Member

commented Aug 13, 2019

Not a clue. Nothing has been done to it other than the usual process which includes sources being uploaded from the English files (any changes there would clear related translations) and a build done in CrowdIn and pulled down.

@Ruslan-Aleev

This comment has been minimized.

Copy link
Contributor Author

commented Aug 13, 2019

@opengeek Can you restore the file?

@opengeek

This comment has been minimized.

Copy link
Member

commented Aug 13, 2019

Not until we know why it has been cleared in CrowdIn or the same thing will just happen again.

@Ruslan-Aleev

This comment has been minimized.

Copy link
Contributor Author

commented Aug 13, 2019

Some thoughts about Crowdin #14673

@opengeek

This comment has been minimized.

Copy link
Member

commented Aug 16, 2019

Thanks to @theboxer , we have determined why these files were losing translations. It appears translators have been escaping single-quotes in the translations on CrowdIn which makes the file invalid. So when it is synced with CrowdIn, it resets the entire file. I am manually searching and replacing any instances of \\ in the lexicons now and will re-upload those specific files to try and correct the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.