From 0e4b703d239551207ca72c5b201475418c7cfd9c Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Mon, 14 Dec 2015 09:32:50 +0000
Subject: [PATCH]  Bug 15357: Display existing holds even if no items exist

If all the items have been deleted for a record and holds exist, the
holds are not displayed.
You are not able to delete the items from the record detail page, but
you can from the items page.

Test plan:
1 - Place 1 or more title level holds on a record.
2 - Delete each item individually.
3 - Note that you see the number of holds on the record details page
With this patch, that holds are accessible via the holds tab.

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
---
 .../intranet-tmpl/prog/en/modules/reserve/request.tt   | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
index 3152f7f75f..85f687ab84 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
@@ -228,8 +228,9 @@ function checkMultiHold() {
     [% ELSE %]
         <strong>Cannot place hold:</strong> this record has no items attached.
     [% END %]
-    </ div>
-[% ELSE %]
+    </div>
+[% END %]
+
   [% IF ( messagetransfert ) %]
 		<div class="dialog message">
 				<h2>Hold found for ([% nextreservtitle %]), please transfer</h2>
@@ -247,7 +248,7 @@ function checkMultiHold() {
     <h1>Confirm holds</h1>
   [% END %]
 
-  [% UNLESS ( borrowernumber ) %]
+  [% UNLESS borrowernumber OR noitems %]
     [% IF ( messageborrower ) %]
       <div class="dialog alert"><h3>Patron not found</h3><p>No patron with this name, please, try another</p> </div>
     [% END %]
@@ -296,7 +297,7 @@ function checkMultiHold() {
             <input type="hidden" name="biblionumbers" value="[% biblionumbers %]"/>
         [% END %]
     </form>
-  [% ELSE %]
+  [% ELSIF NOT noitems %]
 
 [% IF ( exceeded_maxreserves || alreadyreserved || none_available || alreadypossession || ageRestricted ) %]
     <div class="dialog alert">
@@ -851,7 +852,6 @@ function checkMultiHold() {
 </form>
 [% END %]
 [% END %]
-[% END %]
 </div>
 </div>
 
-- 
2.39.5