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.8] Use coalesce operator #28287

Merged
merged 1 commit into from Apr 22, 2019

Conversation

Projects
None yet
3 participants
@rentalhost
Copy link
Contributor

commented Apr 21, 2019

Following #28280.

Applying null coalesce operator when possible in 7 files and 8 cases.

There is another case on file src/Illuminate/View/Concerns/ManagesLayouts.php, but it was not too much accepted because it is hard to read. 😄

Thanks to @kalessil Php Inspections ​(EA Ultimate) plugin.

@rentalhost rentalhost force-pushed the rentalhost:null-coalesce-operator branch from 1fb6dab to f7bb1ca Apr 21, 2019

@rentalhost rentalhost force-pushed the rentalhost:null-coalesce-operator branch from f7bb1ca to 128ab45 Apr 22, 2019

@taylorotwell taylorotwell merged commit 230a35d into laravel:5.8 Apr 22, 2019

2 checks passed

continuous-integration/styleci/pr The analysis has passed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.