From 05ef470ecbd2eaccf1577f07ed9e5f750fcd931a 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
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

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>
(cherry picked from commit 0e4b703d239551207ca72c5b201475418c7cfd9c)
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
(cherry picked from commit 2903bc2909180d3f021f7caf2c1181b2a00c5b2f)
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
---
 .../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 80be359044..c909341907 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
@@ -227,8 +227,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>
@@ -246,7 +247,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 %]
@@ -295,7 +296,7 @@ function checkMultiHold() {
             <input type="hidden" name="biblionumbers" value="[% biblionumbers %]"/>
         [% END %]
     </form>
-  [% ELSE %]
+  [% ELSIF NOT noitems %]
 
 [% IF ( maxreserves || alreadyreserved || none_available || alreadypossession || ageRestricted ) %]
     <div class="dialog alert">
@@ -841,7 +842,6 @@ function checkMultiHold() {
 </form>
 [% END %]
 [% END %]
-[% END %]
 </div>
 </div>
 
-- 
2.39.5