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

Closure is not executed for wildcard acl #12333

Closed
Jurigag opened this issue Oct 18, 2016 · 0 comments
Closed

Closure is not executed for wildcard acl #12333

Jurigag opened this issue Oct 18, 2016 · 0 comments
Assignees
@Jurigag
Labels
bug A bug report status: medium Medium
Milestone
3.0.2

Comments

@Jurigag
Copy link
Contributor

Jurigag commented Oct 18, 2016

There are missing fetch funcAccess, funcList[accessKey]; after those lines:

https://github.com/phalcon/cphalcon/blob/master/phalcon/acl/adapter/memory.zep#L595
https://github.com/phalcon/cphalcon/blob/master/phalcon/acl/adapter/memory.zep#L625

Will fix this as soon as i can.
@sergeyklay sergeyklay added this to the 3.0.2 milestone Oct 18, 2016
Jurigag added a commit to Jurigag/cphalcon that referenced this issue Oct 19, 2016
@Jurigag
Fixes phalcon#12333
ca9be52
sergeyklay added a commit that referenced this issue Oct 21, 2016
@sergeyklay
Merge pull request #12338 from Jurigag/3.0.x-memory-acl-12333
cda3929 
Fixes #12333
@Jurigag Jurigag closed this as completed Oct 21, 2016
@niden niden added bug A bug report status: medium Medium and removed Bug - Medium labels Dec 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Jurigag Jurigag

Labels
bug A bug report status: medium Medium
Projects
None yet
Milestone
3.0.2
Development

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