diff options
author | Dave Arter <davea@mysociety.org> | 2018-09-25 15:36:08 +0100 |
---|---|---|
committer | Dave Arter <davea@mysociety.org> | 2018-09-28 16:19:47 +0100 |
commit | 9b2b8675f6dfa22c45d609349f97730a33c975d3 (patch) | |
tree | 697d95b1614b75e0acbfb674fc676bf9f4148990 | |
parent | 64cb4e23433b9fb7862f763e1819b6ac3318c3e6 (diff) |
Handle failure of photo upload more gracefully
-rw-r--r-- | perllib/FixMyStreet/App/Controller/Photo.pm | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/perllib/FixMyStreet/App/Controller/Photo.pm b/perllib/FixMyStreet/App/Controller/Photo.pm index f41702dcf..1884b9f26 100644 --- a/perllib/FixMyStreet/App/Controller/Photo.pm +++ b/perllib/FixMyStreet/App/Controller/Photo.pm @@ -7,6 +7,7 @@ BEGIN {extends 'Catalyst::Controller'; } use JSON::MaybeXS; use File::Path; use File::Slurp; +use Try::Tiny; use FixMyStreet::App::Model::PhotoSet; =head1 NAME @@ -101,8 +102,13 @@ sub upload : Local { c => $c, data_items => \@items, }); - - my $fileid = $photoset->data; + my $fileid = try { + $photoset->data; + } catch { + $c->log->debug("Photo upload failed."); + $c->stash->{photo_error} = _("Photo upload failed."); + return undef; + }; my $out; if ($c->stash->{photo_error} || !$fileid) { $c->res->status(500); |