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

Fix security groups language files (PHP 5.5 compatibility) #226

Merged

Conversation

JanSiero
Copy link
Contributor

@JanSiero JanSiero commented May 1, 2015

When using SuiteCRM in the spanish language, the JavaScript language files are not compiled correctly, causing the jscalendar to show english titles. (english month names instead of the spanish ones)

This is caused by stray characters in Latin1 encoding in UTF8 language files. the JSON encoder stumbled on these characters, because it only accepts UTF8 as of PHP version 5.5.

These Latin1 characters are present in the Security Groups language files of es, fr, it and pt.

The diffs in Github don't show the changes, because apparently github adapts to the character encoding of each line. However, when you inspect the previous version of the file (showed as latin1 in Github) and the new file (showed as UTF8), the difference is shown.

@JanSiero JanSiero changed the title Fix security groups language files Fix security groups language files (PHP 5.5 compatibility) May 5, 2015
@mattlorimer mattlorimer added the Status:Fix Proposed A issue that has a PR related to it that provides a possible resolution label May 20, 2015
@willrennie willrennie merged commit f6ca385 into salesagility:master May 20, 2015
@JanSiero
Copy link
Contributor Author

JanSiero commented Aug 8, 2015

fixes #78

@Dillon-Brown Dillon-Brown removed the Status:Fix Proposed A issue that has a PR related to it that provides a possible resolution label Sep 6, 2017
@JanSiero JanSiero deleted the fix_SecurityGroups_language_files branch December 4, 2023 17:47
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 this pull request may close these issues.

None yet

4 participants