diff options
author | Matthew Somerville <matthew@mysociety.org> | 2011-06-23 13:33:02 +0100 |
---|---|---|
committer | Matthew Somerville <matthew@mysociety.org> | 2011-06-23 13:33:02 +0100 |
commit | ead78ff31d49cfeff2503f9faf793e0e53110cf6 (patch) | |
tree | 8b844d48a0d8d11353c17857975ffbf8dc3067e1 /perllib/FixMyStreet/App/Controller/Report.pm | |
parent | 98e23f576f1114d42f4b847e8bb57d090b19d633 (diff) |
Remove unneeded defaults checking and simplify.
Diffstat (limited to 'perllib/FixMyStreet/App/Controller/Report.pm')
-rw-r--r-- | perllib/FixMyStreet/App/Controller/Report.pm | 19 |
1 files changed, 2 insertions, 17 deletions
diff --git a/perllib/FixMyStreet/App/Controller/Report.pm b/perllib/FixMyStreet/App/Controller/Report.pm index b50325052..a15ee993f 100644 --- a/perllib/FixMyStreet/App/Controller/Report.pm +++ b/perllib/FixMyStreet/App/Controller/Report.pm @@ -110,23 +110,8 @@ sub format_problem_for_display : Private { $c->stash->{report_name} = $c->req->param('name'); - if ( $c->req->param('submit_update') ) { - # we may have munged these previously in /report/update - # so only set if they're not already in the stash - $c->stash->{form_name} ||= $c->req->param('name'); - $c->stash->{update_text} ||= $c->req->param('update'); - $c->stash->{email} ||= $c->req->param('rznvy'); - $c->stash->{fixed} ||= $c->req->param('fixed') ? ' checked' : ''; - $c->stash->{add_alert_checked} ||= - ( $c->req->param('add_alert') ? ' checked' : '' ); - } - else { - if ( $c->user ) { - $c->stash->{form_name} = $c->user->name; - $c->stash->{email} = $c->user->email; - } - $c->stash->{may_show_name} = ' checked'; - $c->stash->{add_alert_checked} = ' checked'; + unless ( $c->req->param('submit_update') ) { + $c->stash->{add_alert} = 1; } $c->forward('generate_map_tags'); |