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
Fix case-insensitive match in ACL checking #147
Conversation
ACL checking of DokuWiki is currently always case-sensitive regardless of auth backend setting ($auth->isCaseSensitive). This commit enables case-insensitive match in the same way of auth_isMember().
protected $oldAuth; | ||
protected $oldAuthAcl; | ||
|
||
function setup() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should call the parent::setup() method setup DokuWikis test environment.
When doing this you can delete the $this->oldConfig stuff.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I copied this test file from _test/tests/inc/auth_aclcheck.test.php as a working example, so I didn't notice that the parent::setUp() is recommended.
By the way, almost all test files in _test/tests directory have no parent::setUp(). Do I really need to fix this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i would say it's not mandatory but a nice to have ;-)
anyway -> 👍 |
@dom-mel how about this: I merge this PR and you'll check our test cases for bad style setup() routines? |
Yes |
Fix case-insensitive match in ACL checking
Thanks for merging. |
WIP: Lookups aka Non-Page Tables
A modified version of #146.
@ALL
group