From e2d05f3fdfde20977ca5525510e8f4c0daa964f1 Mon Sep 17 00:00:00 2001
From: Kyle M Hall <kyle@bywatersolutions.com>
Date: Thu, 26 Oct 2017 07:35:53 -0400
Subject: [PATCH] Bug 19533: Hold pulldown for itemtype is empty if hold
 placement needs override

Test Plan:
1) Enable AllowHoldPolicyOverride
2) Enable AllowHoldItemTypeSelection
3) Create a situation where adding a hold for a patron would trigger a tooManyReserves
   warning.
4) Note the itemtype pulldown is empty
5) Apply this patch
6) Reload the page
7) Itemtype pulldown should have values

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

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
---
 reserve/request.pl | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/reserve/request.pl b/reserve/request.pl
index f4cdc16fa7..919de1f002 100755
--- a/reserve/request.pl
+++ b/reserve/request.pl
@@ -485,6 +485,8 @@ foreach my $biblionumber (@biblionumbers) {
                     # If AllowHoldPolicyOverride is set, it should override EVERY restriction, not just branch item rules
                     $item->{override} = 1;
                     $num_override++;
+
+                    push( @available_itemtypes, $item->{itype} );
                 }
 
                 # If none of the conditions hold true, then neither override nor available is set and the item cannot be checked
-- 
2.39.5