diff options
| author | Struan Donald <struan@exo.org.uk> | 2012-10-01 12:31:51 +0100 |
|---|---|---|
| committer | Struan Donald <struan@exo.org.uk> | 2012-10-01 17:38:26 +0100 |
| commit | 741abb421c66bdf91a60a988d7250feb59c6ca45 (patch) | |
| tree | c0b9bdf04380158dd9375471e583eabb7eb3ea6b /perllib/FixMyStreet/App/Controller/Admin.pm | |
| parent | be97fe44d9954ce3c84d98030f2ebc6ae84a5a61 (diff) | |
Don't have the list of states harcoded in more than one place
Also sort out some warnings that were being issues during tests
Diffstat (limited to 'perllib/FixMyStreet/App/Controller/Admin.pm')
| -rw-r--r-- | perllib/FixMyStreet/App/Controller/Admin.pm | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/perllib/FixMyStreet/App/Controller/Admin.pm b/perllib/FixMyStreet/App/Controller/Admin.pm index 298c75352..1efa2827b 100644 --- a/perllib/FixMyStreet/App/Controller/Admin.pm +++ b/perllib/FixMyStreet/App/Controller/Admin.pm @@ -59,11 +59,9 @@ sub index : Path : Args(0) { %prob_counts = map { $_ => $prob_counts{$_} || 0 } - ('confirmed', 'investigating', 'in progress', 'closed', 'fixed - council', - 'fixed - user', 'fixed', 'unconfirmed', 'hidden', - 'partial', 'planned'); + ( FixMyStreet::DB::Result::Problem->all_states() ); $c->stash->{problems} = \%prob_counts; - $c->stash->{total_problems_live} += $prob_counts{$_} + $c->stash->{total_problems_live} += $prob_counts{$_} ? $prob_counts{$_} : 0 for ( FixMyStreet::DB::Result::Problem->visible_states() ); $c->stash->{total_problems_users} = $c->cobrand->problems->unique_users; |