During testing of current CVS code I noted that trying to delete a user has no effect. Whether it is a good idea to delete users is another question (see http://drupal.org/node/19055)...
in modules/user.module:user_edit(), when the admin clicks on the "Delete" button at the end of a user's page, this happens:
drupal_goto("user/$account->uid/delete");
Unfortunately, at least in my situation, the GET variable "destination" was set to "admin/user", so what drupal_goto() does is use this address rather than the parameter it receives. The net effect is that when you click Delete you end up on the user admin page again.
The obvious fix seems to unset() the destination, see enclosed patch.
Comments
Comment #1
Tobias Maier commentedsaw the same problem
please fix it
Thanks
Comment #2
Uwe Hermann commentedI can confirm this bug. It's critical IMHO. The patch seems broken, so here's a rerolled one for HEAD which works for me. Please test.
Comment #3
Steve Dondley commentedCan't reproduce. This problem seems to have been resolved.
Comment #4
chx commentedSteve, fixed is fixed, please do not close by hand.
Comment #5
(not verified) commented