Exclude certain permissions

gordon - April 29, 2009 - 00:55
Project:Admin Role
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs review
Description

I need to be able to exclude certain permissions from the admin role such as execute php and the devel permissions.

This would be a static list that you would set up and would be excluded from the admin roles, even when new modules are loaded.

#1

gcassie - June 23, 2009 - 19:46

A seconding of this and a concrete example. Ubercart defines an "act as seller" permission which, when enabled on a site running Paypal payment gateway, requires the user with that permission to provide a Paypal email address on their account.

#2

jim0203 - September 18, 2009 - 14:58

Thirded. There are often permissions I want to exclude from my administrators. Perhaps the most flexible feature would be for admin role to provide an inverse of the standard Drupal permissions system, whereby multiple roles would be admin roles, and each one could have specific permissions switched off.

#3

Steve Dondley - September 23, 2009 - 02:32

fourthed?

#4

gcassie - September 23, 2009 - 21:22
Status:active» needs review

this patch creates a textarea in the adminrole settings page. permissions listed there, one per line, are ignored when adminrole does its updates.

AttachmentSize
adminrole.exclude_perms.patch 1.15 KB

#5

Robin Monks - October 6, 2009 - 11:15

This is an a new patch for this feature; providing a friendlier UI and currently in production over at Prometheus Labor. Sponsored by Prometheus Labor.

AttachmentSize
adminrole_exceptions.diff 4.9 KB

#6

Steve Dondley - October 6, 2009 - 13:53
Status:needs review» reviewed & tested by the community

+1 Reviewed and test. Works great!

#7

acrollet - October 30, 2009 - 16:33

I've tested this as well, works fine for me. +1 for the functionality.

I'm attaching a slightly cleaned up version of the patch in #5 that should apply cleanly with drush_make.

AttachmentSize
adminrole_exceptions.patch 4.94 KB

#8

jim0203 - October 30, 2009 - 19:10
Status:reviewed & tested by the community» needs review

New patch needs new review. Change status.

#9

acrollet - October 30, 2009 - 19:29
Status:needs review» reviewed & tested by the community

@#8: fwiw, the patch is the same code as #5 - only the format of the diff is changed, so I believe it's justified to change the status to 'reviewed and tested', as it's the same code that #6 and I have both tested.

#10

Dave Reid - November 25, 2009 - 20:44
Status:reviewed & tested by the community» needs review

I'm not sure this is the right approach. I'd like to get the original maintainer's (Jacob Signh) thoughts on this first.

 
 

Drupal is a registered trademark of Dries Buytaert.