Conversation
Allowing IDEs to autocomplete when foreach'ing findMany result
|
Huh, any ideas why the tests fail for 7.0 ? |
Looks like a good correction to the code. It doesn't warrant a release in its own right in my opinion so I'll wait to bundle it with any other patches in the future.
My guess is because a newer version of PHPUnit is run on the PHP 7 image at Travis CI. One that isn't compatible with the test files in Paris. |
|
Thanks @treffynnon . At least with PHPStorm the development experience using Paris is not good without this fix. |
|
@treffynnon I have a question: Using Paris and PHPStorm how can we enable the autocomplete for model fields? Adding PHPDoc block didn't help, and we can't describe them by creating |
|
@stratoss, What do you mean adding PHPDoc block didn't help? (What did you try?). This SO answer suggests a specific docblock that might work. |
|
Yep, that looks like the comments you would need. Here's my previous answer
on the topic: http://stackoverflow.com/a/42428882/461813
|
|
@treffynnon thank you! Seems that I need to put the PHPDoc block right before the class definition whilst I was creating it in the beginning of the file. |
|
I've merged this pull request and fixed the CI failures. Thanks! |
|
Thank you! |
Allowing IDEs to autocomplete when foreach'ing findMany result. Fixes #127