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

CommentFileSizeAuthor
#2 adminrole.module.360091.patch980 bytesjaydub
adminrole-role-const-D6.patch810 bytesDave Reid
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Bevan’s picture

Bevan CreditAttribution: Bevan commented

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

jaydub’s picture

jaydub CreditAttribution: jaydub commented
FileSize
adminrole.module.360091.patch980 bytes

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

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs review » Reviewed & tested by the community

Yeah this looks good.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented

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

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
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..

Status: Fixed » Closed (fixed)

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