Use DRUPAL_AUTHENTICATED_RID instead of hard-coded '2'
Dave Reid - January 17, 2009 - 03:02
| Project: | Admin Role |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
Great module so far! Little patch that simplifies things the Drupal way.
| Attachment | Size |
|---|---|
| adminrole-role-const-D6.patch | 810 bytes |
#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
#3
Yeah this looks good.
#4
Also would be covered by #615336: Admin Role - Permissions and Sync module with D7-implementation for upgrading ease..
#5
Fixed as a part of #615336: Admin Role - Permissions and Sync module with D7-implementation for upgrading ease..
#6
Automatically closed -- issue fixed for 2 weeks with no activity.