Skip to content
This repository has been archived by the owner on Nov 26, 2021. It is now read-only.

Implement better permissions model #22

Open
Pyroka opened this issue Apr 27, 2013 · 1 comment
Open

Implement better permissions model #22

Pyroka opened this issue Apr 27, 2013 · 1 comment

Comments

@Pyroka
Copy link
Contributor

Pyroka commented Apr 27, 2013

CakePHP supports using Acl/Aco to build a database of allowed and disallowed actions, look in to using this for permissions instead of checking everywhere for the groups.

@ghost ghost assigned Pyroka Jul 6, 2013
@Pyroka
Copy link
Contributor Author

Pyroka commented Jul 7, 2013

The default CakePHP implementation of Acl/Aco doesn't support members being in more than one group, in addition to this, it also doesn't support the fine-grained permissions we require (MembersController::edit is allowed dependant on the value of the parameter passed to the function, or even worse, the MembersController::view method hides certain data based on permissions),

Suggest we stick to the current method that uses the Controller::isAuthorized function, but instead of that checking the groups directly it checks against permissions that are set at a group level elsewhere. This will at-least allow us to specify the permissions a member/group has without having that hard-coded.

Removing the 'Pre 04' tag from this issue, the fix is non-trivial, and not vital at this moment.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants