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

Comments

Bevan’s picture

Bevan commented

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

jaydub’s picture

jaydub commented
StatusFileSize
new 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
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
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
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.