diff options
author | Dave Whiteland <dave@mysociety.org> | 2013-08-23 08:32:49 +0100 |
---|---|---|
committer | Dave Whiteland <dave@mysociety.org> | 2013-09-05 17:31:08 +0100 |
commit | 367fdaba0686150e841e879445ad2378f2f4d897 (patch) | |
tree | 7071106c72dbe43dd6061af0f177c765cfb21dc1 /perllib/FixMyStreet/App/Controller/Admin.pm | |
parent | 007f413b4db284ca74bc6b40b82e3424e48482da (diff) |
clarify presentation on lagged users page
now shows which flagged users are in abuse table, which seems important
(separate issue #523 to make managing and indeed viewing abuse table)
Diffstat (limited to 'perllib/FixMyStreet/App/Controller/Admin.pm')
-rw-r--r-- | perllib/FixMyStreet/App/Controller/Admin.pm | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/perllib/FixMyStreet/App/Controller/Admin.pm b/perllib/FixMyStreet/App/Controller/Admin.pm index 5e98c5d04..ba8f22a05 100644 --- a/perllib/FixMyStreet/App/Controller/Admin.pm +++ b/perllib/FixMyStreet/App/Controller/Admin.pm @@ -1027,8 +1027,20 @@ sub flagged : Path('flagged') : Args(0) { $c->stash->{problems} = [ $problems->all ]; my $users = $c->model('DB::User')->search( { flagged => 1 } ); + my @users = $users->all; + my %email2user = map { $_->email => $_ } @users; + $c->stash->{users} = [ @users ]; - $c->stash->{users} = $users; + my @abuser_emails = $c->model('DB::Abuse')->all(); + + foreach my $email (@abuser_emails) { + # 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 }; + } + } return 1; } |