Has there been any planning around a version of Masquerade for Drupal 7 yet? I find this module incredibly useful and would love to see it join the #D7CX movement.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

deekayen’s picture

Category: feature » support
Status: Active » Fixed

I'm not going to tag it #D7CX to be ready on release date, but it will upgrade to D7 and patches will speed that up.

afreeman’s picture

Status: Fixed » Needs review
FileSize
17.08 KB

Patch w/tests include. Note: if a user attempts to masquerade from ?q=masquerade/unswitch they will be switched and then immediately switched back.

afreeman’s picture

FileSize
18.42 KB

Fixed typo & included tests this time. Yay -N

deekayen’s picture

Status: Needs review » Needs work

Patch appears to be against HEAD. There's a whole lot of stuff that has only been committed to DRUPAL-6--1 lately. HEAD and 6 are out of sync. Need to sync 6 with HEAD before rolling this patch.

afreeman’s picture

Status: Needs work » Needs review
FileSize
26.03 KB

Rerolled to include the missing code from DRUPAL-6--1 branch.

deekayen’s picture

Status: Needs review » Fixed

committed to HEAD

Status: Fixed » Closed (fixed)

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

  • Commit 109f144 on master, 8.x-2.x, 8.x-2.x-admin-menu, 8.x-1.x-1836516 by deekayen:
    #787188 by afreeman - Drupal 7 upgrade and sync with DRUPAL-6--1 branch
    
    
  • Commit 2e2ae82 on master, 8.x-2.x, 8.x-2.x-admin-menu, 8.x-1.x-1836516 by deekayen:
    #787188 by afreeman - Drupal 7 tests
    
    

  • Commit 109f144 on master, 8.x-2.x, 8.x-2.x-admin-menu, 8.x-1.x-1836516 by deekayen:
    #787188 by afreeman - Drupal 7 upgrade and sync with DRUPAL-6--1 branch
    
    
  • Commit 2e2ae82 on master, 8.x-2.x, 8.x-2.x-admin-menu, 8.x-1.x-1836516 by deekayen:
    #787188 by afreeman - Drupal 7 tests