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.

Sign up for GitHub

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

Jump to bottom

replace method_exists per is_callable #57

Closed

sneznaovca opened this issue Oct 27, 2014 · 5 comments

Contributor

sneznaovca commented Oct 27, 2014

What do you think about replace method_exists per is_callable?
https://github.com/Kdyby/Events/blob/master/src/Kdyby/Events/EventManager.php#L194

http://cz1.php.net/manual/en/function.method-exists.php see notes
is_callable allows use __call

Member

fprochazka commented Oct 27, 2014

@sneznaovca you want to create dynamic listeners? :)

Contributor Author

sneznaovca commented Oct 27, 2014

@fprochazka yes

Contributor

matej21 commented Oct 27, 2014

👍

Member

fprochazka commented Oct 27, 2014

@sneznaovca I don't see a reason to prohibit that. Can you send a pullrequest, ideally with a testcase please?

hranicka mentioned this issue

EventManager: Alow callable as listeners [Closes #57] #82

Closed

Member

fprochazka commented May 1, 2016

Solved with 69806d1...b6fc86b

fprochazka closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment