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
Closure variants for hydration #9
Closure variants for hydration #9
Conversation
Maybe I should move the PHP7-specific implementation to a new package so that code coverage remains 100% |
Looks great! Sorry for the delay.
|
@haampie What's the status of this? Would you be able to finish this PR? |
Hey @haampie, I just released a new major version which only supports PHP ^7.0. Maybe that helps to get this PR ready? If you're not interested in pushing it, just let me know, that's okay too :) |
Sorry, it's been a while, I'll dig into it ;) |
aaec435
to
74d0024
Compare
74d0024
to
1392a12
Compare
1392a12
to
68b7076
Compare
68b7076
to
e9a270b
Compare
I think this is ready to be merged after rebasing a couple times :p |
This PR adds (among some minor details) two new Hydrate implementations, which exploit closure binding via
Closure::bindTo
andClosure::call
(PHP7).If support for PHP 5.5 must be provided, I'm of course willing to update this PR. The closure trick works on PHP 5.5 as well.