From 9678cfc38dfe9394239a445df2d19e57657e9f79 Mon Sep 17 00:00:00 2001
From: Chris Cormack <crc@liblime.com>
Date: Mon, 7 Jan 2008 14:34:16 -0600
Subject: [PATCH] Fix for bug 1738, opac-reserve now blocks reserving not for
 loan items

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
---
 koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tmpl | 4 +++-
 opac/opac-reserve.pl                                  | 5 +++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tmpl b/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tmpl
index a9b4299d60..5cfc2762c3 100644
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tmpl
+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tmpl
@@ -80,14 +80,16 @@
                     <!-- /TMPL_LOOP -->
             </select>
             </p>
+			<!-- TMPL_IF NAME="forloan" -->
             <p> <label for="requestany">Place a hold on the next available copy </label>
             <input type="checkbox" id="requestany" name="request" checked="checked" value="Any" />
             </p>
+
             <input type="hidden" name="biblioitem" value="<!-- TMPL_VAR NAME="biblioitemnumber" -->" />
             <input type="submit" value="Place Hold" />
             <input type="hidden" name="all" value="1"/></p>
             <input type="hidden" name="place_reserve" value="1"/>
-
+			<!-- /TMPL_IF -->            
         <!-- TMPL_LOOP name="bibitemloop" -->    
 
             <!-- TMPL_UNLESS NAME="item-level_itypes" -->
diff --git a/opac/opac-reserve.pl b/opac/opac-reserve.pl
index 6eb8b9519b..7c1ad03f31 100755
--- a/opac/opac-reserve.pl
+++ b/opac/opac-reserve.pl
@@ -151,6 +151,7 @@ my %itemtypes;
 my @duedates;
 #die @items;
 my %itemhash;
+my $forloan;
 foreach my $itm (@items) {
     push @duedates, { date_due => format_date( $itm->{'date_due'} ) }
       if defined $itm->{'date_due'};
@@ -175,6 +176,9 @@ foreach my $itm (@items) {
         }
     }
 	$itemhash{$itm->{'itemnumber'}}=$itm;
+	if (!$itm->{'notforloan'} && !$itm->{'itemnotforloan'}){
+		$forloan=1;
+	}
 }
 
 $template->param( ITEMS => \@duedates );
@@ -424,6 +428,7 @@ foreach my $biblioitemnumber (@biblioitemnumbers) {
 
 # display infos
 $template->param(
+	forloan           => $forloan,
     bibitemloop       => \@bibitemloop,
 );
 output_html_with_http_headers $query, $cookie, $template->output;
-- 
2.39.5