From 7c62e5da36f84b156dbbce331da7ca3f7961f27f Mon Sep 17 00:00:00 2001
From: Matthew Somerville <matthew-github@dracos.co.uk>
Date: Wed, 15 Mar 2017 13:57:05 +0000
Subject: Do upload_dir check on start up, not each report.

If we're chugging through a list of reports, this save on a lot of
pointless statting of the upload directory which we know is there.
---
 perllib/FixMyStreet/App/Model/PhotoSet.pm | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

(limited to 'perllib/FixMyStreet/App/Model')

diff --git a/perllib/FixMyStreet/App/Model/PhotoSet.pm b/perllib/FixMyStreet/App/Model/PhotoSet.pm
index 04521b5ce..8fcc1700e 100644
--- a/perllib/FixMyStreet/App/Model/PhotoSet.pm
+++ b/perllib/FixMyStreet/App/Model/PhotoSet.pm
@@ -67,14 +67,7 @@ has upload_dir => (
     is => 'ro',
     lazy => 1,
     default => sub {
-        my $self = shift;
-        my $cache_dir = path(FixMyStreet->config('UPLOAD_DIR'))->absolute(FixMyStreet->path_to());
-        $cache_dir->mkpath;
-        unless ( -d $cache_dir && -w $cache_dir ) {
-            warn "Can't find/write to photo cache directory '$cache_dir'";
-            return;
-        }
-        $cache_dir;
+        path(FixMyStreet->config('UPLOAD_DIR'))->absolute(FixMyStreet->path_to());
     },
 );
 
-- 
cgit v1.2.3