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

Treat callables as callables no matter their origin #368

Closed
wants to merge 2 commits into from

Conversation

AnrDaemon
Copy link
Contributor

Should fix #273

@AnrDaemon
Copy link
Contributor Author

Cases 4, 5 and 6 will need similar treatment, but I'm unable to setup testing because composer.json missing any dev dependencies whatsoever and I can't figure out correct test cases to trigger the problem.

Also try to fix broken PHP 5.3 test.
uwetews added a commit that referenced this pull request May 27, 2017
@uwetews
Copy link
Contributor

uwetews commented May 27, 2017

Case 4 and 5 need no othér treatment as there only functions will be called.
Case 6 the default plugin handler need some more thoughts as the default handler code needs to be called at runtime instead at compile time

uwetews added a commit that referenced this pull request May 27, 2017
uwetews added a commit that referenced this pull request May 27, 2017
@AnrDaemon
Copy link
Contributor Author

Thanks for taking over this. I got the test suite running, but it erroring too much and I yet to get to the errors of interest.

@AnrDaemon AnrDaemon closed this Jun 13, 2017
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.

Using closures as registered plugins
2 participants