Added more granular access controls for administering the front page
ac - August 18, 2008 - 16:33
| Project: | Front Page |
| Version: | 5.x-1.9 |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Description
Added the permission 'administer front page' to the access controls (It currently allows access if you have menu administration authorisation). Also changed the title in the Drupal menu system to 'Front page'.
| Attachment | Size |
|---|---|
| access_permissions.patch | 1.83 KB |
#1
#2
I've just written a patch that does this for 6.x - I didn't realize you'd already done this. It seems pretty clear to me that this should be rolled into the module: it doesn't make a lot of sense to have the ability to manage the front page configuration based on the 'administer menu' permission.
This patch won't work on the 6.x-1.2 version, as there's an implementation of hook_perm() already. I'm attaching my diff, which does not rename the administrative menu item (though 'Front page' is a much more sensible name than 'Advanced front page').
#3
Michaek,
Thanks for the patch. No one has looked at/commited this and they will never find your 6.x patch in a 5.x issue. I would suggest creating a new issue with the appropriate version and including a link to this issue so both can be commited.
Cheers
#4
#5