Download & Extend

Decouple administer permissions from system module "access administration pages" permissions

Project:Addresses
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

<?php
/**
* Implementation of hook_perm().
*/
function addresses_perm() {
  return array(
'administer addresses');
}
?>

should be added and changes made to addresses_menu().

Comments

#1

I created a patch for the above issue.

AttachmentSize
addresses-488072.patch 1.32 KB

#2

Status:active» needs review

#3

Not to clutter things, but I've written a patch for this as well (see attached).

AttachmentSize
addresses_permissions.patch 1.88 KB

#4

+1

I prefer monotaga's patch in #1. I think defining a constant as in #3 is unnecessary.

#5

Version:6.x-1.05» 6.x-1.x-dev
Status:needs review» fixed

Monotaga, thank you for submitting this idea and then submitting it as a patch (which always makes things easier).

keystr0k, the constant approach was not used as it does not adhere to the practices in core, but I do very much appreciate the effort and willingness to provide a patch - thank you.

#6

Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.