diff options
-rw-r--r-- | perllib/FixMyStreet/App/Controller/Report.pm | 4 | ||||
-rw-r--r-- | t/app/controller/report_updates.t | 9 |
2 files changed, 11 insertions, 2 deletions
diff --git a/perllib/FixMyStreet/App/Controller/Report.pm b/perllib/FixMyStreet/App/Controller/Report.pm index decda1b6c..1e1c6c974 100644 --- a/perllib/FixMyStreet/App/Controller/Report.pm +++ b/perllib/FixMyStreet/App/Controller/Report.pm @@ -155,7 +155,7 @@ sub load_problem_or_display_error : Private { $c->stash->{problem} = $problem; my $updates = $c->model('DB::Comment')->search( - { problem_id => $problem->id }, #, state => 'confirmed' }, + { problem_id => $problem->id, state => 'confirmed' }, { select => [ 'id', 'name', 'text', @@ -170,7 +170,7 @@ sub load_problem_or_display_error : Private { -as => 'confirmed', } ], - order_by => 'confirmed' + order_by => 'confirmed', } ); diff --git a/t/app/controller/report_updates.t b/t/app/controller/report_updates.t index 13908fbff..55b181178 100644 --- a/t/app/controller/report_updates.t +++ b/t/app/controller/report_updates.t @@ -134,6 +134,15 @@ for my $test ( }; } +subtest "unconfirmed updates not displayed" => sub { + $comment->state( 'unconfirmed' ); + $comment->update; + $mech->get_ok("/report/$report_id"); + + my $meta = $mech->extract_update_metas; + is scalar @$meta, 0, 'update not displayed'; +}; + ok $comment->delete, 'deleted comment'; $mech->delete_user('commenter@example.com'); $mech->delete_user('test@example.com'); |