aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDave Arter <davea@mysociety.org>2018-09-25 15:36:08 +0100
committerDave Arter <davea@mysociety.org>2018-09-28 16:19:47 +0100
commit9b2b8675f6dfa22c45d609349f97730a33c975d3 (patch)
tree697d95b1614b75e0acbfb674fc676bf9f4148990
parent64cb4e23433b9fb7862f763e1819b6ac3318c3e6 (diff)
Handle failure of photo upload more gracefully
-rw-r--r--perllib/FixMyStreet/App/Controller/Photo.pm10
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);