Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix php 7.4 tagged model typed props #732

Merged
merged 7 commits into from
Dec 23, 2019
Merged

Fix php 7.4 tagged model typed props #732

merged 7 commits into from
Dec 23, 2019

Conversation

jpuck
Copy link
Contributor

@jpuck jpuck commented Dec 23, 2019

Fixes #730

php 7.4 introduced the new reflection method isInitialized to check for this exact case.

@jpuck
Copy link
Contributor Author

jpuck commented Dec 23, 2019

having issues with style ci and typed properties 😕 which didn't appear in a previous build

edit: so it looks like style ci doesn't support php 7.4 yet. amending the commit and pushing fixed it 🤷‍♂

@taylorotwell taylorotwell merged commit 265e01b into laravel:3.0 Dec 23, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PHP 7.4 property types break tag detection for models
2 participants