From 387ea53db2514bd69bbf00b1189db27a237c008f Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Fri, 20 Jan 2023 16:39:32 +0000
Subject: [PATCH] Bug 32662: Use template wrapper for item circulation alerts
 page

This patch implements the template WRAPPER system (see Bug 32571)
for building tabs on the item circulation alerts page.

To test, apply the patch and go to Administration -> Item
circulation alerts. There should be two tabs, "Checkout" and
"Check-in" which work correctly.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
---
 .../modules/admin/item_circulation_alerts.tt  | 29 +++++++++----------
 1 file changed, 13 insertions(+), 16 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt
index e067c194f2..f97c98c41e 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt
@@ -113,16 +113,14 @@
                 <h2>Circulation alerts for [% Branches.GetName( branch ) || 'Default' | html %]</h2>
                 <p>Click on the grid to toggle the settings.</p>
 
-
-                <div id="alerttabs" class="toptabs">
-
-                    <ul class="nav nav-tabs" role="tablist">
-                        <li role="presentation" class="active"><a href="#checkout" aria-controls="checkout" role="tab" data-toggle="tab">Checkout</a></li>
-                        <li role="presentation"><a href="#checkin" aria-controls="checkin" role="tab" data-toggle="tab">Check-in</a></li>
-                    </ul>
-
-                    <div class="tab-content">
-                        <div role="tabpanel" class="tab-pane active" id="checkout">
+                [% WRAPPER tabs id= "alerttabs" %]
+                    [% WRAPPER tabs_nav %]
+                        [% WRAPPER tab_item tabname= "checkout" bt_active= 1 %] Checkout [% END %]
+                        [% WRAPPER tab_item tabname= "checkin" %] Check-in [% END %]
+                    [% END %]
+
+                    [% WRAPPER tab_panels %]
+                        [% WRAPPER tab_panel tabname="checkout" bt_active= 1 %]
                             <h3>Checkout</h3>
                             <table class="grid active" width="100%">
                                 <thead>
@@ -144,9 +142,9 @@
                                     [% END %]
                                 </tbody>
                             </table> <!-- /.grid.active -->
-                        </div> <!-- /#checkout -->
+                        [% END #/ #checkout %]
 
-                        <div role="tabpanel" class="tab-pane" id="checkin">
+                        [% WRAPPER tab_panel tabname="checkin" %]
                             <h3>Check-in</h3>
                             <table class="grid active" width="100%">
                                 <thead>
@@ -168,11 +166,10 @@
                                     [% END %]
                                 </tbody>
                             </table>
-                        </div>
-                    </div> <!-- /.tab-content -->
-
-                </div> <!-- /#alerttabs -->
+                        [% END # /#checkin %]
+                    [% END # /.tab_panels %]
 
+                [% END # //#alerttabs %]
 
             </main>
         </div> <!-- /.col-sm-10.col-sm-push-2 -->
-- 
2.39.5