diff options
author | Matthew Somerville <matthew@mysociety.org> | 2015-02-05 17:01:41 +0000 |
---|---|---|
committer | Matthew Somerville <matthew@mysociety.org> | 2015-02-05 18:12:49 +0000 |
commit | 6bc3566f0281f4aede0e11e8ab5e21476bc0b674 (patch) | |
tree | 6ec8384cf90bf64d719fb9ce77d69b5146c743a7 /perllib/FixMyStreet/App/Controller/Photo.pm | |
parent | 46128a0f1998b01b6d698a22c8811f4898b69b0f (diff) |
Have code fallback to Image::Size if no jhead.
Don't bother installing it on Travis.
Diffstat (limited to 'perllib/FixMyStreet/App/Controller/Photo.pm')
-rw-r--r-- | perllib/FixMyStreet/App/Controller/Photo.pm | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/perllib/FixMyStreet/App/Controller/Photo.pm b/perllib/FixMyStreet/App/Controller/Photo.pm index 09afabecf..d70be50e6 100644 --- a/perllib/FixMyStreet/App/Controller/Photo.pm +++ b/perllib/FixMyStreet/App/Controller/Photo.pm @@ -8,6 +8,7 @@ use DateTime::Format::HTTP; use Digest::SHA qw(sha1_hex); use File::Path; use File::Slurp; +use Image::Size; use Path::Class; use if !$ENV{TRAVIS}, 'Image::Magick'; @@ -181,7 +182,11 @@ sub process_photo_upload : Private { my $photo_blob = eval { my $filename = $upload->tempname; my $out = `jhead -se -autorot $filename 2>&1`; - die _("Please upload a JPEG image only"."\n") if $out =~ /Not JPEG:/; + unless (defined $out) { + my ($w, $h, $err) = Image::Size::imgsize($filename); + die _("Please upload a JPEG image only") . "\n" if !defined $w || $err ne 'JPG'; + } + die _("Please upload a JPEG image only") . "\n" if $out && $out =~ /Not JPEG:/; my $photo = $upload->slurp; return $photo; }; |