From 9caea3fcfcea694b41f9cf2dc79cf78d3f3560e2 Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
Date: Wed, 25 Oct 2023 12:13:49 +0000
Subject: [PATCH] Bug 30579: (follow-up) Add required label and warning when
 hold levels required

Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
---
 .../prog/en/modules/reserve/request.tt              | 13 +++++++++++++
 1 file changed, 13 insertions(+)

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 7948fd4d73..05142728a0 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
@@ -621,12 +621,24 @@
                                     [% ELSIF force_hold_level == 'record' %]
                                         <input type="radio" id="requestany" checked="checked" value="Any" disabled="true"/>
                                         <input type="hidden" name="request" value="Any"/>
+                                        <span class="error"><i>(Required)</i></span>
                                     [% ELSE %]
                                         <input type="radio" id="requestany" name="request" checked="checked" value="Any" />
                                     [% END %]
                                 </legend>
                                 <input type="hidden" name="alreadyreserved" value="[% alreadyreserved | html %]" />
                                 <fieldset class="enable_request_any disable_request_group disable_request_specific">
+                                [% IF force_hold_level == 'item' # Patron has placed a item level hold previously for this record %]
+                                    <span class="error">
+                                        <i class="fa fa-times fa-lg" title="Cannot be put on hold"></i>
+                                        Hold must be item level
+                                    </span>
+                                [% ELSIF force_hold_level == 'item_group' # Patron has placed an item group level hold previously for this record %]
+                                    <span class="error">
+                                        <i class="fa fa-times fa-lg" title="Cannot be put on hold"></i>
+                                        Hold must be item group level
+                                    </span>
+                                [% ELSE %]
                                     <ol>
 
                                         <li>
@@ -659,6 +671,7 @@
                                             <input type="hidden" name="holds_to_place_count" value="1" />
                                         [% END %]
                                     </ol>
+                                [% END %]
 
 
                                 <fieldset class="action">
-- 
2.20.1