diff options
| author | Struan Donald <struan@exo.org.uk> | 2011-08-26 11:29:33 +0100 |
|---|---|---|
| committer | Struan Donald <struan@exo.org.uk> | 2011-08-26 11:29:33 +0100 |
| commit | 43069a7218e3b092cdcb214a0a6a56d13b38f39a (patch) | |
| tree | f487e9f5abb1c47a3ecbf3beda1a6a0d8cacc5af | |
| parent | 2f1eccb9d70c2e763761e3909235e803f80873c9 (diff) | |
test for removing flag and make it work
| -rw-r--r-- | perllib/FixMyStreet/App/Controller/Admin.pm | 2 | ||||
| -rw-r--r-- | t/app/controller/admin.t | 14 |
2 files changed, 15 insertions, 1 deletions
diff --git a/perllib/FixMyStreet/App/Controller/Admin.pm b/perllib/FixMyStreet/App/Controller/Admin.pm index 9cc046e13..467f29d26 100644 --- a/perllib/FixMyStreet/App/Controller/Admin.pm +++ b/perllib/FixMyStreet/App/Controller/Admin.pm @@ -801,7 +801,7 @@ sub user_edit : Path('user_edit') : Args(1) { $user->name( $c->req->param('name') ); $user->email( $c->req->param('email') ); $user->from_council( $c->req->param('council') || undef ); - $user->flagged( $c->req->param('flagged') ); + $user->flagged( $c->req->param('flagged') || 0 ); $user->update; if ($edited) { diff --git a/t/app/controller/admin.t b/t/app/controller/admin.t index 324d13890..4bfdd7821 100644 --- a/t/app/controller/admin.t +++ b/t/app/controller/admin.t @@ -1054,6 +1054,20 @@ for my $test ( log_count => 4, log_entries => [qw/edit edit edit edit/], }, + { + desc => 'edit user remove flagged', + fields => { + name => 'Changed User', + email => 'changed@example.com', + council => 2607, + flagged => 'on', + }, + changes => { + flagged => undef, + }, + log_count => 4, + log_entries => [qw/edit edit edit edit/], + }, ) { subtest $test->{desc} => sub { $mech->get_ok( '/admin/user_edit/' . $user->id ); |