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
Fix #344 - lazy loading value holder magic method support #345
Fix #344 - lazy loading value holder magic method support #345
Conversation
…g proxies (virtual proxies)
… triggering directly a notice
…re-existing method, where applicable
…sting method, where applicable
… pre-existing method, where applicable
…xisting method, where applicable
if ($hasParent) { | ||
$parameter = $originalClass->getMethod('__get')->getParameters()[0]; | ||
|
||
$parameterType = $parameter->hasType() ? $parameter->getType() : null; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a fix for #346, no? Can you please revert it, and we move it to something else? We'd have to check it for all scenarios anyway...
* @author Jefersson Nathan <malukenho@phpse.net> | ||
* @license MIT | ||
*/ | ||
class ClassWithMagicMethodsWeirdlyTypeHinted |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not weirdly :D Also - possibly to be delayed until #346
@malukenho you said that you can't push 2.0.x or master? Did you already merge this change into those branches? |
@malukenho I un-protected |
Fixes #344
TODO:
3.1.0
in PHP 7.1 support #327__get
__set
__unset
__isset
__get
__set
__unset
__isset
__get
__set
__unset
__isset
__get
__set
__unset
__isset