diff options
author | Sunil Joshi <joshi_sunil@in.com> | 2013-09-04 09:57:17 +1000 |
---|---|---|
committer | Simon Green <sgreen@redhat.com> | 2013-09-04 09:57:17 +1000 |
commit | 41eda66a29eea7fffe23508ecff6857bd951fcad (patch) | |
tree | eed7bfc67657b4eb03102e77d8a0a7ca56747dbb /editusers.cgi | |
parent | a1911aa1dcf5a7e3fe6c56b61a03098f18e596c2 (diff) | |
download | bugs-41eda66a29eea7fffe23508ecff6857bd951fcad.tar bugs-41eda66a29eea7fffe23508ecff6857bd951fcad.tar.gz bugs-41eda66a29eea7fffe23508ecff6857bd951fcad.tar.bz2 bugs-41eda66a29eea7fffe23508ecff6857bd951fcad.tar.xz bugs-41eda66a29eea7fffe23508ecff6857bd951fcad.zip |
Bug 331529 - editusers list should let me search for disabled users
r=simon, a=simon
Diffstat (limited to 'editusers.cgi')
-rwxr-xr-x | editusers.cgi | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/editusers.cgi b/editusers.cgi index e12fc35b6..f4e3c0841 100755 --- a/editusers.cgi +++ b/editusers.cgi @@ -68,7 +68,7 @@ if ($action eq 'search') { my $matchstr = trim($cgi->param('matchstr')); my $matchtype = $cgi->param('matchtype'); my $grouprestrict = $cgi->param('grouprestrict') || '0'; - my $enabled_only = $cgi->param('enabled_only') || '0'; + my $is_enabled = scalar $cgi->param('is_enabled'); my $query = 'SELECT DISTINCT userid, login_name, realname, is_enabled, ' . $dbh->sql_date_format('last_seen_date', '%Y-%m-%d') . ' AS last_seen_date ' . 'FROM profiles'; @@ -160,11 +160,12 @@ if ($action eq 'search') { $query .= " $nextCondition ugm.group_id IN($grouplist) "; } - if ($enabled_only eq '1') { - $query .= " $nextCondition profiles.is_enabled = 1 "; + detaint_natural($is_enabled); + if ($is_enabled == 0 || $is_enabled == 1) { + $query .= " $nextCondition profiles.is_enabled = ?"; $nextCondition = 'AND'; + push(@bindValues, $is_enabled); } - $query .= ' ORDER BY profiles.login_name'; $vars->{'users'} = $dbh->selectall_arrayref($query, |