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

pukku’s picture

Assigned: Unassigned » pukku

Hm. I think that there was a reason I was using the 'edit' array data. I will look into this and get back to you...

pukku’s picture

So, I have replicated that this is a bug. I will look at committing [a|your] fix to it tomorrow.

Ricky

pukku’s picture

Status: Needs review » Fixed

Hi! 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.

scafmac’s picture

Yeah I noticed a little earlier today that I missed the same problem when changing the display options. Glad you found it.

pukku’s picture

Status: Fixed » Closed (fixed)