-
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
[5.4] Adds is_null check to Eloquent Builder callScope #19381
Conversation
is this fix possible to be port over lower versions of laravel? *currently encountering this error on version |
Maybe better just change |
It would be fixed in L5.2.* as well? Yesterday upgraded to PHP7.2 and now production service is broken. Please let me know. Thank you. |
Yes thats only one problem which i had after upgrading to 7.2 |
How far back to go? I wouldn't expect unsupported versions of a framework to continue working on each iteration of PHP. However, if it's a simple fix, maybe a PR would be accepted. |
fix null bug; laravel/framework#19381
@taylorotwell Can we commit this to 5.2 and 5.3 branches also? It will open those up to PHP 7.2 as well. |
This fixes #19380 by adding an
is_null
check before callingcount()
and defaulting to 0 if it isnull
.