From 52c6a02e3c6bcc642f0ae29fdb396f6958ec7555 Mon Sep 17 00:00:00 2001
From: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Date: Thu, 25 Apr 2024 14:58:31 -0400
Subject: [PATCH] Bug 36701: Adjust confirmation to hide empty div

To test:
1. Place a hold on any bib record
2. Cancel the hold by clicking the trash can icon next to it
--> Note that there is an empty red page section in the modal
3. Apply patch and refresh the page
4. Repeat steps 1-2
--> No empty red page section
5. Confirm that the test plan from bug 33174 still works
---
 koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

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 11d6d34c81..c666a018e7 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
@@ -1280,7 +1280,7 @@
 
                 <div class="modal-body">
                     <p>Are you sure you want to cancel this hold?</p>
-                    <div id="cancel_hold_alert" class="alert alert-danger"></div>
+                    <div id="cancel_hold_alert" class="alert alert-danger" hidden="true"></div>
                     <fieldset class="action">
                         [% SET hold_cancellation = AuthorisedValues.GetAuthValueDropbox('HOLD_CANCELLATION') %]
                         [% IF hold_cancellation.count %]
@@ -1728,6 +1728,7 @@
                 $(this).prop('checked', !count);
                 $('.cancel_selected_holds').html(MSG_CANCEL_SELECTED.format($('.holds_table .select_hold:checked').length));
                 $('#cancel_hold_alert').html( MSG_CANCEL_ALERT.format($('.holds_table .select_hold:checked').length));
+                $('#cancel_hold_alert').show();
                 localStorage.selectedHolds = $('.holds_table .select_hold:checked').toArray().map(el => $(el).data('id'));
             });
 
@@ -1737,6 +1738,7 @@
                 $('.select_hold_all', table).prop('checked', !count);
                 $('.cancel_selected_holds').html(MSG_CANCEL_SELECTED.format($('.holds_table .select_hold:checked').length));
                 $('#cancel_hold_alert').html( MSG_CANCEL_ALERT.format($('.holds_table .select_hold:checked').length));
+                $('#cancel_hold_alert').show();
                 localStorage.selectedHolds = $('.holds_table .select_hold:checked').toArray().map(el => $(el).data('id'));
             });
 
-- 
2.34.1