tsnoad / Irondata

Data warehouse report generator
4 stars 1 forks source link

ACL checked are hardcoded to URLs #37

Closed eleybourn closed 14 years ago

eleybourn commented 14 years ago

in module.php - this is bad. It means adding a new module required a change to module.php.

Module specific checks should be run by the module themselves.

eleybourn commented 14 years ago

I have now completely rewritten the permission check (backend not the awesome interface). There is a new hook - hook_permission_check in which a module will return true or false if a user has permission to access it. I have recreated the existing module permissions in this new system.

I have also created a manage database permission for the catalogue module.

Some additional testing is still required.