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

[FEATURE REQUEST] Removal of nodiacritic #600

Closed
simaotwx opened this issue Jan 27, 2023 · 3 comments
Closed

[FEATURE REQUEST] Removal of nodiacritic #600

simaotwx opened this issue Jan 27, 2023 · 3 comments

Comments

@simaotwx
Copy link

Is your feature request related to a problem? Please describe.
The nodiacritic library is licensed under GPL-2.0 (see https://github.com/vria/nodiacritic/blob/master/LICENSE). This means it is incompatible when used in GPL-3.0 projects. Also, it looks like using it in a MIT licensed project is also not correct, but this project is licensed under MIT.

This causes following issue: publishpress/PublishPress-Planner#1237 via https://github.com/publishpress/wp-browser which is a fork.

Describe the solution you'd like
The remove_accents() function has existed in WordPress since 1.2.1 (see https://developer.wordpress.org/reference/functions/remove_accents/#changelog). Hence, in my opinion, it is not necessary to include this polyfill anymore. Removing this library would solve the license problem.

Describe alternatives you've considered
The alternative would be for the author to relicense nodiacritic or to rewrite it with a new license.

Additional context

return NoDiacritic::filter($string);

@lucatume
Copy link
Owner

I'm updating the project to be compatible with Codeception 5 and PHP 8: that looks the right scenario to remove that dependency.
I'm ideally open to removing it if, in the final version, it turns out to not provide more value or be easily replaced or internalized, I will look into that in the current work.

@simaotwx
Copy link
Author

Thank you for your letting me know. I'm looking forward to it.

@lucatume
Copy link
Owner

Removed in version 4 of the project.

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

No branches or pull requests

2 participants