diff options
| author | Dave Arter <davea@mysociety.org> | 2020-04-23 15:47:59 +0100 |
|---|---|---|
| committer | Dave Arter <davea@mysociety.org> | 2020-07-02 14:38:29 +0100 |
| commit | d157073eb872a98f9c697a0e3e959f498a922e12 (patch) | |
| tree | 16c658d06a55ff20f825004a735c13d21ec26ef4 | |
| parent | 603cc8c79564cada62a4dfa8d5c12bdd489f02f0 (diff) | |
Don’t show `sent_to` on moderation diff
| -rw-r--r-- | perllib/FixMyStreet/DB/Result/ModerationOriginalData.pm | 2 | ||||
| -rw-r--r-- | t/app/controller/moderate.t | 3 |
2 files changed, 3 insertions, 2 deletions
diff --git a/perllib/FixMyStreet/DB/Result/ModerationOriginalData.pm b/perllib/FixMyStreet/DB/Result/ModerationOriginalData.pm index 1805e1fd2..6d14e6a9f 100644 --- a/perllib/FixMyStreet/DB/Result/ModerationOriginalData.pm +++ b/perllib/FixMyStreet/DB/Result/ModerationOriginalData.pm @@ -163,7 +163,7 @@ sub compare_extra { my $new = $other->get_extra_metadata; my $both = { %$old, %$new }; - my @all_keys = sort keys %$both; + my @all_keys = grep { $_ ne 'sent_to' } sort keys %$both; my @s; foreach (@all_keys) { if ($old->{$_} && $new->{$_}) { diff --git a/t/app/controller/moderate.t b/t/app/controller/moderate.t index 8e84bd392..43ae1c980 100644 --- a/t/app/controller/moderate.t +++ b/t/app/controller/moderate.t @@ -51,7 +51,7 @@ sub create_report { longitude => '0.007831', user_id => $user2->id, photo => '74e3362283b6ef0c48686fb0e161da4043bbcc97.jpeg', - extra => { moon => 'waxing full' }, + extra => { moon => 'waxing full', sent_to => [ 'authority@example.org' ] }, }); } my $report = create_report(); @@ -115,6 +115,7 @@ subtest 'Problem moderation' => sub { }}); $mech->base_like( qr{\Q$REPORT_URL\E} ); $mech->content_like(qr/Moderated by Bromley Council/); + $mech->content_lacks('sent_to = ARRAY(0x'); $report->discard_changes; is $report->title, 'Good good'; |