Download & Extend
Admin role » Issues

Use DRUPAL_AUTHENTICATED_RID instead of hard-coded '2'

Posted by Dave Reid on January 17, 2009 at 3:02am
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
Login or register to post comments

Comments

#1

Posted by Bevan on January 19, 2009 at 3:27am

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

#2

Posted by jaydub on August 14, 2009 at 6:03pm

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

Posted by Dave Reid on November 25, 2009 at 2:29am
Status:needs review» reviewed & tested by the community

Yeah this looks good.

#4

Posted by Dave Reid on November 25, 2009 at 3:52am

Also would be covered by #615336: Admin Role - Permissions and Sync module with D7-implementation for upgrading ease..

#5

Posted by Dave Reid on November 25, 2009 at 3:49pm
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

Posted by System Message on December 9, 2009 at 3:50pm
Status:fixed» closed (fixed)

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