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

Check and recommend intl extension #333

Open
Zodiac1978 opened this issue May 2, 2019 · 6 comments

Comments

@Zodiac1978
Copy link
Contributor

commented May 2, 2019

Feature request/bug description

This extension should be recommended IMHO. It is necessary to use the Normalizer function which would solve many problems with internationalization issues for special characters:
https://www.php.net/manual/en/intl.requirements.php

@fierevere

This comment has been minimized.

Copy link

commented May 8, 2019

intl has hard dependency on icu system libraries, which will consume over 30 Mb diskspace on smaller (VPS/embedded) systems. Each loaded library claims some memory and initialization time overhead.

Bigger distributions can link libicu to libxml2 library and therefore it can be loaded by libxml2 linked extensions, but this dependency is optional

@Zodiac1978

This comment has been minimized.

Copy link
Contributor Author

commented May 9, 2019

@fierevere Thanks for the additional info! But what is your recommendation? I just want to know if the normalize_normalize function is available.
https://php.net/manual/de/normalizer.normalize.php

Thankfully I can easily add this myself (and have already a little plugin for this) but maybe this is something that is useful for all (in the future).

So do you recommend to not use intl, because it consumes too much space and memory?

@fierevere

This comment has been minimized.

Copy link

commented May 9, 2019

Its just a statement, not a recommendation to use or not to use.

WordPress can play with remove_accents() https://developer.wordpress.org/reference/functions/remove_accents/ built-in function if needed. I believe it does same thing.

@Zodiac1978

This comment has been minimized.

Copy link
Contributor Author

commented May 10, 2019

I believe it does same thing.

Unfortunately it does not help here. See this ticket: https://core.trac.wordpress.org/ticket/35951

@Zodiac1978

This comment has been minimized.

Copy link
Contributor Author

commented Jul 16, 2019

If someone wants to test for intl in Health Check, here is a small plugin solution:
https://gist.github.com/Zodiac1978/ce9db99becd3507ec8d8207c583e947b

@Zodiac1978

This comment has been minimized.

Copy link
Contributor Author

commented Jul 29, 2019

There is a ongoing discussion in the hosting team about which extension is recommended/required. intl is mentioned three times:
https://make.wordpress.org/hosting/2019/06/11/php-requirements-check-in/#comment-73
https://make.wordpress.org/hosting/2019/06/11/php-requirements-check-in/#comment-75
https://make.wordpress.org/hosting/2019/06/11/php-requirements-check-in/#comment-79

Next step is outlined here https://make.wordpress.org/hosting/2019/07/27/hosting-meeting-notes-july-22-2019/

@Zodiac1978 requested an update on the PHP Extension recommendations. This is at the point where feedback needs to be compiled and shared. If you’d like to help put a post together, please leave a comment on this one!

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