Download & Extend

Use DRUPAL_AUTHENTICATED_RID instead of hard-coded '2'

Project:Admin role
Version:6.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Great module so far! Little patch that simplifies things the Drupal way.

AttachmentSizeStatusTest resultOperations
adminrole-role-const-D6.patch810 bytesIgnored: Check issue status.NoneNone

Comments

#1

Patch code looks good. Needs to be tested that it applies and doesn't break anything.

#2

Here's a drupal 5 version of this patch which in passing also addresses #367977: Role Mismatch if there is a gap in the Role-IDs

AttachmentSizeStatusTest resultOperations
adminrole.module.360091.patch980 bytesIgnored: Check issue status.NoneNone

#3

Status:needs review» reviewed & tested by the community

Yeah this looks good.

#4

#5

Status:reviewed & tested by the community» fixed

Fixed as a part of #615336: Admin Role - Permissions and Sync module with D7-implementation for upgrading ease..

#6

Status:fixed» closed (fixed)

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