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

should be added and changes made to addresses_menu().

CommentFileSizeAuthor
#3 addresses_permissions.patch1.88 KBagileadam
#1 addresses-488072.patch1.32 KBmonotaga

Comments

monotaga’s picture

monotaga commented
StatusFileSize
new addresses-488072.patch1.32 KB

I created a patch for the above issue.

monotaga’s picture

monotaga commented
Status: Active » Needs review
agileadam’s picture

agileadam
Location Maine, USA
commented
StatusFileSize
new addresses_permissions.patch1.88 KB

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

gapple’s picture

gapple
he/they
Primary language English
commented

+1

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

codycraven’s picture

codycraven commented
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.

Status: Fixed » Closed (fixed)

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