From 79e0433a86eeb3c69df5c8bcd5e1ef5f1b5be95e Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Tue, 10 May 2022 16:43:19 +0000
Subject: [PATCH] Bug 30579: (follow-up) Expand range of controls which are
 disabled/enabled

This patch updates the process of switching between "hold next" and
"hold specific item" so that all relevant form fields are disabled.
---
 .../prog/en/modules/reserve/request.tt        | 22 +++++++++----------
 1 file changed, 11 insertions(+), 11 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 30d47f645c..690676dc47 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
@@ -535,11 +535,11 @@
                         <fieldset class="action">
                             [% IF ( patron.borrowernumber ) %]
                                 [% IF ( override_required ) %]
-                                    <button type="submit" class="btn btn-default warning"><i class="fa fa-exclamation-triangle "></i> Place hold</button>
+                                    <button type="submit" id="hold_any_btn" class="btn btn-default warning"><i class="fa fa-exclamation-triangle "></i> Place hold</button>
                                 [% ELSIF ( none_available ) %]
-                                    <button type="submit" disabled="disabled" class="btn btn-default btn-disabled">Place hold</button>
+                                    <button type="submit" id="hold_any_btn" disabled="disabled" class="btn btn-default btn-disabled">Place hold</button>
                                 [% ELSE %]
-                                    <button type="submit" class="btn btn-default">Place hold</button>
+                                    <button type="submit" id="hold_any_btn" class="btn btn-default">Place hold</button>
                                 [% END %]
                             [% END %]
                         </fieldset>
@@ -769,11 +769,11 @@
                             <fieldset class="action">
                                 [% IF ( patron AND patron.borrowernumber ) %]
                                     [% IF ( override_required ) %]
-                                        <button type="submit" class="btn btn-default warning"><i class="fa fa-exclamation-triangle "></i> Place holds</button>
+                                        <button type="submit" id="hold_item_btn" class="btn btn-default warning"><i class="fa fa-exclamation-triangle "></i> Place holds</button>
                                     [% ELSIF ( none_available ) %]
-                                        <button class="btn btn-default" type="submit">Place holds</button>
+                                        <button class="btn btn-default" id="hold_item_btn" type="submit">Place holds</button>
                                     [% ELSE %]
-                                        <button type="submit" class="btn btn-default">Place hold</button>
+                                        <button type="submit" id="hold_item_btn" class="btn btn-default">Place hold</button>
                                     [% END %]
                                 [% END # /IF patron %]
                             </fieldset> <!-- /.action -->
@@ -938,7 +938,7 @@
                             </table> <!-- /#requesttitles -->
 
 
-                        <fieldset class="action">
+                        <fieldset class="action">??
                             [% IF ( patron AND patron.borrowernumber ) %]
                                 [% IF ( override_required ) %]
                                     <button type="submit" class="btn btn-default warning"><i class="fa fa-exclamation-triangle "></i> Place holds</button>
@@ -1382,11 +1382,11 @@
 
         function ToggleHoldsToPlace() {
             if ( $("#requestany").prop('checked') ) {
-                $("#holds_to_place_count, #pickup, #itemtype").prop('disabled', false);
-                $(".requestspecific").prop("disabled", true );
+                $("#holds_to_place_count, #pickup, #itemtype, #hold_any_btn").prop('disabled', false);
+                $(".requestspecific, #hold_item_btn").prop("disabled", true );
             } else {
-                $("#holds_to_place_count, #pickup, #itemtype").prop('disabled', true);
-                $(".requestspecific").prop("disabled", false );
+                $("#holds_to_place_count, #pickup, #itemtype, #hold_any_btn").prop('disabled', true);
+                $(".requestspecific, #hold_item_btn").prop("disabled", false );
             }
         }
 
-- 
2.20.1