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

fixed isset problem with parent objects, used magic method __isset() #15

Closed
wants to merge 2 commits into from

Conversation

nsbucky
Copy link

@nsbucky nsbucky commented Jul 31, 2013

When using the service provider to automagically create Presenter instances of objects, I found there was a problem with using Form::model(), as it uses isset() to check values on the bound model. Since Presenter is just a wrapper, the values are not set. By implementing the __isset() it keeps the Form::model() working as expected. Hopefully this is an acceptable addition.

@robclancy
Copy link
Owner

Needs a better comment, just something like "this is called when isset() is called".

Space after the comments like others.

You need to support arrays too. And needs tests.

Otherwise I'll PR it later and make the changes myself, just not much time with this lately.

@nsbucky
Copy link
Author

nsbucky commented Aug 1, 2013

ok thanks

@robclancy
Copy link
Owner

This has been merged into the develop branch.

@robclancy robclancy closed this Aug 13, 2013
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.

2 participants