From defb2eec5735d9df5f3394f5c651881019f47298 Mon Sep 17 00:00:00 2001
From: Joshua Ferraro <jmf@liblime.com>
Date: Sun, 25 Nov 2007 00:09:22 -0600
Subject: [PATCH] adding a pre-check for patronimages/ dir, which won't be
 there by

default

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
---
 .../prog/en/modules/tools/picture-upload.tmpl              | 7 +++++--
 tools/picture-upload.pl                                    | 7 +++++++
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/picture-upload.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/picture-upload.tmpl
index 3c55468c5f..539f2d4fb3 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/picture-upload.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/picture-upload.tmpl
@@ -31,9 +31,12 @@
     <!-- /TMPL_LOOP -->
 <!-- TMPL_ELSE -->
 
-    <!-- TMPL_IF name="errors" -->
-        <!-- TMPL_LOOP name="errors" -->
+    <!-- TMPL_IF name="ERRORS" -->
+        <!-- TMPL_LOOP name="ERRORS" -->
         <ul class="error">
+		<!-- TMPL_IF NAME="NODIR" -->
+			<li>The patronimages/ directory isn't present on your server, please ask your systems administrator to create it</li>
+		<!-- /TMPL_IF -->
         <!-- TMPL_IF name="NOTZIP" -->
             <li>The upload file does not appear to be a zip file.  The extention is not '.zip'.</li>
         <!-- /TMPL_IF -->
diff --git a/tools/picture-upload.pl b/tools/picture-upload.pl
index c1834cad65..445b071db3 100755
--- a/tools/picture-upload.pl
+++ b/tools/picture-upload.pl
@@ -22,6 +22,13 @@ my ($template, $loggedinuser, $cookie)
 					debug => 0,
 					});
 
+unless (-d $destdir) {
+	$errors{'NODIR'} = 1;
+	warn "patronimages directory not present";
+}
+if ( %errors ) {
+    $template->param( ERRORS => [ \%errors ] );
+}
 my $uploadfilename = $input->param( 'uploadfile' );
 my $uploadfile = $input->upload( 'uploadfile' );
 my ( $total, $handled, @counts );
-- 
2.39.5