View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0015616||phpList 3 application||Admin Management||public||22-05-11 19:03||09-06-11 14:53|
|Target Version||2.10.X||Fixed in Version||2.10.15|
|Summary||0015616: Deleting users from the 'List all users' page doesn't work|
|Description||'Del' doesn't remove user in the following situations:|
1. when used on the 'list all users page' (admin/?page=users).
2. when used in the search results generated from the 'list all users page'
It does indeed work on the user details page.
|Additional Information||Related forum thread: http://forums.phplist.com/viewtopic.php?f=17&t=36219#p85689|
|Tags||No tags attached.|
I ran into this issue as well today. I made an edit to commonlib/pages/users.php.
At line 22 place the following:
$delete = $_GET['delete'];
The $delete variable was not set anywhere within the file so this should remedy the issue. Confirmed on my 2.10.14 installation.
Minor edit to above note. I looked at the SVN and previous implementations used:
$delete = sprintf("%d",$_GET["delete"]);
I don't really see a difference between the two but hey, when in Rome.
||the difference is that with the sprintf the value is sanitised as a number, to avoid possible Sql injections|