diff options
author | Matthew Somerville <matthew@mysociety.org> | 2015-08-19 15:55:41 +0100 |
---|---|---|
committer | Matthew Somerville <matthew@mysociety.org> | 2015-08-19 15:55:41 +0100 |
commit | 8dfdbf84b57d8bbc179dcb895d6eb08c61fdef59 (patch) | |
tree | 4c463f153d69290e4850cbb71c016066e5ed5879 /perllib/FixMyStreet/App/Controller/Report/New.pm | |
parent | 0784bdae637e3d0ca20dc3d66ec50c5d68e489ec (diff) | |
parent | b812f8526525c0f38d72bdc469ddc08b376854d2 (diff) |
Merge branch 'bodies_str-factoring'
Diffstat (limited to 'perllib/FixMyStreet/App/Controller/Report/New.pm')
-rw-r--r-- | perllib/FixMyStreet/App/Controller/Report/New.pm | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/perllib/FixMyStreet/App/Controller/Report/New.pm b/perllib/FixMyStreet/App/Controller/Report/New.pm index b540a1961..e9332d852 100644 --- a/perllib/FixMyStreet/App/Controller/Report/New.pm +++ b/perllib/FixMyStreet/App/Controller/Report/New.pm @@ -884,12 +884,13 @@ sub process_report : Private { } else { # construct the bodies string: # 'x,x' - x are body IDs that have this category - # 'x,x|y' - x are body IDs that have this category, y body IDs with *no* contact my $body_string = join( ',', map { $_->body_id } @contacts ); - $body_string .= - '|' . join( ',', map { $_->id } @{ $c->stash->{missing_details_bodies} } ) - if $body_string && @{ $c->stash->{missing_details_bodies} }; $report->bodies_str($body_string); + # Record any body IDs which might have meant to match, but had no contact + if ($body_string && @{ $c->stash->{missing_details_bodies} }) { + my $missing = join( ',', map { $_->id } @{ $c->stash->{missing_details_bodies} } ); + $report->bodies_missing($missing); + } } my @extra; |