In the permissions settings of masquerade, there's a setting "Masquerade as admin" with description "Masquerade as the site admin (UID 1).". The description is misleading, because the meaning is according to my test rather "Masquerade as administrator user (as defined in the Masquerade module configuration).".

CommentFileSizeAuthor
#3 masquerade-description-1977024-3.patch551 byteshgoto
#1 masquerade-description-1977024-1.patch548 bytesmgifford
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tanius’s picture

tanius CreditAttribution: tanius commented
Issue summary: View changes

Typo correction.

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented
Issue summary: View changes
Status: Active » Needs review
FileSize
masquerade-description-1977024-1.patch548 bytes

It's definitely more wordy, but... More accurate.

Status: Needs review » Needs work

The last submitted patch, 1: masquerade-description-1977024-1.patch, failed testing.

hgoto’s picture

hgoto CreditAttribution: hgoto as a volunteer commented
Status: Needs work » Reviewed & tested by the community
FileSize
masquerade-description-1977024-3.patch551 bytes

It's so late but I'd like to proceed on this issue... The permission "Masquerade as admin" surely means one with that users can masquerade as admin roles which are defined in Masquerade module settings page. And UID 1 is also included in that "admin" group.

The patch cannot be applied to the latest dev and I created a new one. I revised the text a little considering the above point.

(The test failure is not related with this change because the changed line contains just a permission description.)

hgoto’s picture

hgoto CreditAttribution: hgoto as a volunteer commented
Status: Reviewed & tested by the community » Needs review

Oh, I'm sorry. The status should be NR.