View Issue Details

IDProjectCategoryView StatusLast Update
0015616phpList 3 applicationAdmin Managementpublic09-06-11 14:53
Reporterh2b2 Assigned To 
Status resolvedResolutionfixed 
Product Version2.10.14 
Target Version2.10.XFixed in Version2.10.15 
Summary0015616: 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 InformationRelated forum thread:
TagsNo tags attached.



23-05-11 07:41

reporter   ~0051357


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.


23-05-11 07:51

reporter   ~0051358

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.


09-06-11 14:44

administrator   ~0051362

the difference is that with the sprintf the value is sanitised as a number, to avoid possible Sql injections


09-06-11 14:52

administrator   ~0051363