diff options
author | Matthew Somerville <matthew@mysociety.org> | 2012-06-21 14:04:24 +0100 |
---|---|---|
committer | Matthew Somerville <matthew@mysociety.org> | 2012-06-21 14:08:16 +0100 |
commit | b3b1beeefb113da5c196a5f0ed62756f4777b162 (patch) | |
tree | 1c6f1f2775f110c6c3f3039598761f91e3a1acd6 /perllib/FixMyStreet/App/Controller/Report/Update.pm | |
parent | 8bc12f19e115085083b68ed3bc73f80413048d15 (diff) |
Call process_extras() from updates too, so that error checking runs.
Diffstat (limited to 'perllib/FixMyStreet/App/Controller/Report/Update.pm')
-rw-r--r-- | perllib/FixMyStreet/App/Controller/Report/Update.pm | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/perllib/FixMyStreet/App/Controller/Report/Update.pm b/perllib/FixMyStreet/App/Controller/Report/Update.pm index e0433473d..779c74f03 100644 --- a/perllib/FixMyStreet/App/Controller/Report/Update.pm +++ b/perllib/FixMyStreet/App/Controller/Report/Update.pm @@ -201,13 +201,16 @@ sub process_update : Private { $update->problem_state( $params{state} ); } + my @extra; # Next function fills this, but we don't need it here. + # This is just so that the error checkign for these extra fields runs. + # TODO Use extra here as it is used on reports. + $c->cobrand->process_extras( $c, $update->problem->council, \@extra ); + if ( $c->req->param('fms_extra_title') ) { my %extras = (); $extras{title} = $c->req->param('fms_extra_title'); $extras{email_alerts_requested} = $c->req->param('add_alert'); $update->extra( \%extras ); - - $c->stash->{fms_extra_title} = $c->req->param('fms_extra_title'); } if ( $c->stash->{ first_name } && $c->stash->{ last_name } ) { |