Closed (fixed)
Project:
Site User List
Version:
5.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
9 Apr 2007 at 19:38 UTC
Updated:
11 Apr 2007 at 14:31 UTC
In the latest release I just tried, the search form values are not used to filter the user list.
I didn't attach a patch, but here is the fix:
Old code site_user_list.module:
// now, get the search parameters
$search_restrictions = array();
if ($_REQUEST['edit']['search_form']) {
$search_restrictions = $_REQUEST['edit']['search_form'];
}
Replace with:
// now, get the search parameters
$search_restrictions = array();
if ($_REQUEST['search_form']) {
$search_restrictions = $_REQUEST['search_form'];
}
Comments
Comment #1
pukku commentedHm. I think that there was a reason I was using the 'edit' array data. I will look into this and get back to you...
Comment #2
pukku commentedSo, I have replicated that this is a bug. I will look at committing [a|your] fix to it tomorrow.
Ricky
Comment #3
pukku commentedHi! I've applied your change (you missed one place, actually). There should be a better way to do this; I'm now worried that there are namespace issues that are going to show up.
More research needs to be done, but at least with this commit, things work.
Comment #4
scafmac commentedYeah I noticed a little earlier today that I missed the same problem when changing the display options. Glad you found it.
Comment #5
pukku commented