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.

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

Filter add method handler #11581

Closed
lockz opened this issue Mar 23, 2016 · 1 comment
Closed

Filter add method handler #11581

lockz opened this issue Mar 23, 2016 · 1 comment
Assignees
@sergeyklay
Labels
bug A bug report status: low Low
Milestone
3.0.0

Comments

@lockz
Copy link

lockz commented Mar 23, 2016

I think there should be "OR" instead "AND".
I have a callable handler array($this, 'function')
if typeof handler != "object" && !is_callable(handler) {

in file https://github.com/phalcon/cphalcon/blob/master/phalcon/filter.zep
@sergeyklay sergeyklay added this to the 2.1.0 milestone Mar 23, 2016
@sergeyklay sergeyklay self-assigned this Apr 15, 2016
sergeyklay added a commit that referenced this issue Apr 15, 2016
@sergeyklay
Merge pull request #11675 from sergeyklay/filter/add
1d8ebc0 
Fixed issue #11581
@sergeyklay
Copy link
Contributor

Fixed in 2.1.x branch

@niden niden added bug A bug report status: low Low and removed Bug - Low labels Dec 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sergeyklay sergeyklay

Labels
bug A bug report status: low Low
Projects
None yet
Milestone
3.0.0
Development

No branches or pull requests
3 participants
@niden @sergeyklay @lockz