Now there is garbage left in {masquarade_users} if one of users with uid uid_from or user_to deleted. You should add the following code to masquerade_user.

    case 'delete':
      db_query("DELETE FROM {masquerade_users} WHERE uid_from = %d OR uid_to = %d", $edit_user->uid, $edit_user->uid);
      break;

Comments

dpantele’s picture

Status: Needs work » Active
deviantintegral’s picture

Category: feature » bug
StatusFileSize
new597 bytes
new597 bytes

Thanks for the report. Here are two untested patches.

deviantintegral’s picture

Status: Active » Needs review
deekayen’s picture

Status: Needs review » Fixed

Committed to DRUPAL-5, DRUPAL-6--1, and HEAD.

Status: Fixed » Closed (fixed)

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

  • Commit 316ad66 on master, 8.x-2.x, 8.x-2.x-admin-menu, 8.x-1.x-1836516 by deekayen:
    #897318 by Dmitry.Panteleev, deviantintegral: cleanup masquerade_users...

  • Commit 316ad66 on master, 8.x-2.x, 8.x-2.x-admin-menu, 8.x-1.x-1836516 by deekayen:
    #897318 by Dmitry.Panteleev, deviantintegral: cleanup masquerade_users...