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

monotaga - June 10, 2009 - 22:00
Project:Addresses
Version:6.x-1.05
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs review
Description

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

should be added and changes made to addresses_menu().

#1

monotaga - June 14, 2009 - 06:03

I created a patch for the above issue.

AttachmentSize
addresses-488072.patch 1.32 KB

#2

monotaga - June 14, 2009 - 06:10
Status:active» needs review

#3

keystr0k - July 28, 2009 - 16:05

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

AttachmentSize
addresses_permissions.patch 1.88 KB

#4

gapple - August 7, 2009 - 22:40

+1

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

 
 

Drupal is a registered trademark of Dries Buytaert.