From 36baa63f97a9f5e69bab4baf4caf080aa74634d4 Mon Sep 17 00:00:00 2001
From: Chris Nighswonger <cnighswonger@foundations.edu>
Date: Thu, 6 Mar 2008 09:05:45 +1300
Subject: [PATCH] First installment on adding patronimage upload form to member
 details view.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
---
 .../prog/en/modules/members/moremember.tmpl   | 31 ++++++++++++++-----
 members/moremember.pl                         |  9 +++---
 2 files changed, 29 insertions(+), 11 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tmpl
index c64ee7d24a..0fe0596e7f 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tmpl
@@ -194,6 +194,30 @@ for (var i=0;i<ItemsNodes.length;i++){
 
 
 </div>
+
+<!-- Begin Upload Patron Image Section -->
+
+<div>
+<!-- TMPL_IF NAME="patronimages" -->
+	<!-- TMPL_IF NAME="CAN_user_tools" --><!-- TMPL_UNLESS NAME="picture" -->
+	        <form method="post" action="/cgi-bin/koha/tools/picture-upload.pl" enctype="multipart/form-data">
+                <fieldset class="brief"><legend>Upload Patron Image</legend>
+                    <!-- TMPL_VAR NAME="firstname" --> <!-- TMPL_VAR NAME="surname" --> does not currently have an image available. To import an image for <!-- TMPL_VAR NAME="title" --> <!-- TMPL_VAR NAME="surname" -->, enter the name of an image file to upload.
+		    <input type="hidden" id="image" name="filetype" value="image" />
+		    <input type="hidden" id="patron" name="cardnumber" value="<!-- TMPL_VAR NAME="cardnumber" -->" />
+	             <ol>
+                        <li>
+                           <label for="uploadfile">Select the file to upload: </label><input type="file" id="uploadfile" name="uploadfile" />
+                         </li>
+    	             </ol>
+                    <fieldset class="action"><input type="submit" value="Upload" class="submit" />
+                </fieldset>
+                </form>
+<!-- /TMPL_IF -->
+</div>
+
+<!-- End Upload Patron Image Section -->
+
 </div>
  <div class="yui-u"> 
  <div id="patron-library-details">
@@ -450,11 +474,4 @@ for (var i=0;i<ItemsNodes.length;i++){
 <!-- TMPL_INCLUDE NAME="circ-menu.inc" -->
 </div>
 </div>
-<!-- TMPL_IF NAME="patronimages" --><script type="text/JavaScript" language="JavaScript">
-//<![CDATA[
-	<!-- TMPL_IF NAME="CAN_user_tools" -->$("#patronimage").after("<a style=\"display: block; font-size:80%; margin:0  0 1em 1em;\" href=\"/cgi-bin/koha/tools/picture-upload.pl?cardnumber=<!-- TMPL_VAR NAME="cardnumber" -->&filetype=image\">+ Click to Add Image</a>").css({cursor: "pointer"}).click(function(){
-		location.href="/cgi-bin/koha/tools/picture-upload.pl?cardnumber=<!-- TMPL_VAR NAME="cardnumber" -->&filetype=image";
-	});<!-- /TMPL_IF -->
-//]]>
-</script><!-- /TMPL_IF -->
 <!-- TMPL_INCLUDE NAME="intranet-bottom.inc" -->
diff --git a/members/moremember.pl b/members/moremember.pl
index 4ca67a83d3..995ecd1fa6 100755
--- a/members/moremember.pl
+++ b/members/moremember.pl
@@ -326,10 +326,11 @@ foreach (@$alerts) {
     $_->{relatedto} = findrelatedto( $_->{type}, $_->{externalid} );
 }
 
-# grab patron's image if available
-my $picture = GetPatronImage($data->{'cardnumber'});
-$template->param( picture => $picture );
-
+# check to see if patron's image exists in the database
+# basically this gives us a template var to condition the display of
+# patronimage related interface on
+my ($picture, $dberror) = GetPatronImage($data->{'cardnumber'});
+$template->param( picture => 1 ) if $picture;
 
 my $branch=C4::Context->userenv->{'branch'};
 
-- 
2.39.5