-
Notifications
You must be signed in to change notification settings - Fork 11.2k
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
[6.x] Fixed deprecated "Doctrine/Common/Inflector/Inflector" class #32734
[6.x] Fixed deprecated "Doctrine/Common/Inflector/Inflector" class #32734
Conversation
@GrahamCampbell, |
will be this merged to the 7.x version? i'm having this error: |
It was merged into 7.x. |
This broke som code in a Laravel 6.x project for me after doing |
Please report the BC break to the upstream library for them to fix. |
Their pluralisation rules should be the same as in the 1.3.x series. |
I downgraded to php 7.2.30 and now it's working as expected. |
IMO changes from this PR will catch people out, as in our case a composer update resulted in Yes a solution is to manually require PHP 7.4.4 |
Those BC breaks are actually not intended. Please report any pluralisation mistakes to the upstream library to be corrected. v1.4 and v2.0 are meant to be identical apart from deprecated code removed. |
As I said, this won't work, because 1.4 is the same as 2.0. |
Please send fix PRs to the upstream library's 1.4 branch or open issues over there if they have broken something. There's nothing we can do here. |
New class is non-static Doctrine\Inflector\Inflector::class
I also changed the minimum version for
doctrine/inflector
package because the earlier1.1
supports php 5.6 and higher, while Laravel 6.x supports 7.2+.Package version
1.4
supports php 7.2+.Also see #32730