diff options
| author | Jonathan "Duke" Leto <jonathan@leto.net> | 2011-05-22 18:05:16 -0700 |
|---|---|---|
| committer | Jonathan "Duke" Leto <jonathan@leto.net> | 2011-05-22 18:05:16 -0700 |
| commit | fbab7044c8c4943fd761416415d80465d57850d8 (patch) | |
| tree | 6b7705e953a28c1214f632571b971a58723a8dbd /lib/jitterbug/Task.pm | |
| parent | Do not use a global temp directory in our tests (diff) | |
| download | jitterbug-fbab7044c8c4943fd761416415d80465d57850d8.tar.gz | |
Error out early if a task id is not given
Diffstat (limited to 'lib/jitterbug/Task.pm')
| -rw-r--r-- | lib/jitterbug/Task.pm | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/lib/jitterbug/Task.pm b/lib/jitterbug/Task.pm index dd7d1f6..bd949e3 100644 --- a/lib/jitterbug/Task.pm +++ b/lib/jitterbug/Task.pm @@ -5,10 +5,14 @@ use Dancer::Plugin::DBIC; use jitterbug::Plugin::Template; get '/:id' => sub { - my $task = schema->resultset('Task')->find( params->{id} ); + unless ( defined params->{id} ) { + send_error("task id missing!", 400); + return; + } + my $task = schema->resultset('Task')->find( params->{id} ); - if ( !defined $task ) { + unless ( defined $task ) { send_error("task does not exist!", 404); return; } |