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

[5.2] Adds is_null check to Eloquent Builder callScope #22927

Closed
wants to merge 1 commit into from
Closed

[5.2] Adds is_null check to Eloquent Builder callScope #22927

wants to merge 1 commit into from

Conversation

alihamze
Copy link
Contributor

Issue #19381 also exists on Laravel 5.2 with PHP 7.2. This brings the fix to the 5.2 branch.

@GrahamCampbell
Copy link
Member

Sorry, 5.2 is closed for changes.

@peyobr
Copy link

peyobr commented Jan 26, 2018

Now i'm serving 200k users at https://a3.unipa.it/, upgrading to L>=5.3 is too dangerous at the moment.

Yesterday i was forced to upgrade server to PHP7.2 and now some production functionalities are broken.

Please reconsider backport.

Thank you in advance.

@jmarcher
Copy link
Contributor

@peyobr If that is the case then fork the framework fix those issues on your 5.2 branch and your problem is solved.

@peyobr
Copy link

peyobr commented Jan 26, 2018

Actually it would be great to enjoy of an official back-port. If no other option exists, you are right: i'll fork and fix on my own.

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.

4 participants