diff options
author | Dave Arter <davea@mysociety.org> | 2016-09-13 11:48:02 +0100 |
---|---|---|
committer | Dave Arter <davea@mysociety.org> | 2016-09-16 09:41:22 +0100 |
commit | e9a5d2fb6e1985a0d946ad3574234784f4f659c4 (patch) | |
tree | c0d5ac6ea3e6536d76f7e734449fe2dc64007544 /perllib/FixMyStreet/App/Controller/Admin.pm | |
parent | e8fb7318f94f78f768700f72b2bb8e3434f108a2 (diff) |
Fix admin user search for non-superusers
Diffstat (limited to 'perllib/FixMyStreet/App/Controller/Admin.pm')
-rw-r--r-- | perllib/FixMyStreet/App/Controller/Admin.pm | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/perllib/FixMyStreet/App/Controller/Admin.pm b/perllib/FixMyStreet/App/Controller/Admin.pm index a312a4399..0a7b8d049 100644 --- a/perllib/FixMyStreet/App/Controller/Admin.pm +++ b/perllib/FixMyStreet/App/Controller/Admin.pm @@ -1030,15 +1030,17 @@ sub users: Path('users') : Args(0) { my %email2user = map { $_->email => $_ } @users; $c->stash->{users} = [ @users ]; - my $emails = $c->model('DB::Abuse')->search( - { email => { ilike => $isearch } } - ) if $c->user->is_superuser; - foreach my $email ($emails->all) { - # Slight abuse of the boolean flagged value - if ($email2user{$email->email}) { - $email2user{$email->email}->flagged( 2 ); - } else { - push @{$c->stash->{users}}, { email => $email->email, flagged => 2 }; + if ( $c->user->is_superuser ) { + my $emails = $c->model('DB::Abuse')->search( + { email => { ilike => $isearch } } + ); + foreach my $email ($emails->all) { + # Slight abuse of the boolean flagged value + if ($email2user{$email->email}) { + $email2user{$email->email}->flagged( 2 ); + } else { + push @{$c->stash->{users}}, { email => $email->email, flagged => 2 }; + } } } |