diff options
| author | Dave Arter <davea@mysociety.org> | 2018-01-18 12:02:00 +0000 |
|---|---|---|
| committer | Dave Arter <davea@mysociety.org> | 2018-01-23 15:46:09 +0000 |
| commit | b3d26a80a0728e49d857c0fe5cbc2966512561ff (patch) | |
| tree | d8a58f095fbd0268c5b8d328ed2cc16b564fee74 /bin | |
| parent | 6e415f3c6d9355312b9fb84e4ecdb68f96cff618 (diff) | |
Hide 'updates not sent' message on reports sent via Open311
The 'updates are not sent to the council' message was incorrectly being
shown on reports where updates would in fact be sent. This was affecting
non-Bromley/Stevenage reports on fms.com and all cobrands using Open311.
This commit moves the logic out of the template and adds the
Problem->updates_sent_to_body method which inspects the receiving body's
Open311 configuration to determine whether updates will be sent.
The duplication of the Lewisham/Oxfordshire logic between Problem.pm and
send-comments isn't ideal but hopefully there won't be any new Open311
bodies that only send and don't receive updates. If there are we'll have
to look at refactoring that list.
Diffstat (limited to 'bin')
| -rwxr-xr-x | bin/send-comments | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/bin/send-comments b/bin/send-comments index 7363d7a0d..aecedcb08 100755 --- a/bin/send-comments +++ b/bin/send-comments @@ -49,7 +49,7 @@ my $bodies = FixMyStreet::DB->resultset('Body')->search( { while ( my $body = $bodies->next ) { - # XXX Cobrand specific + # XXX Cobrand specific - see also list in Problem->updates_sent_to_body if ($site eq 'fixmystreet.com') { # Oxfordshire (OCC) is special: # we do *receive* service_request_updates (aka comments) for OCC, but we never *send* them, so skip this pass |