From 58e960179ccc09d1a8291c145cc2508439943c49 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Mon, 20 Mar 2023 15:43:39 +0000
Subject: [PATCH] Bug 33278: Correct JS for activating default tab on various
 pages

The conversion of the header search tabs to Bootstrap resulted in a bug
on several staff interface pages: The first tab isn't selected by
default. The jQuery selector isn't specific enough.

The patch also corrects a template error which was introduced to
merge-record.inc by Bug 32683.

To test, apply the patch and test the following pages to confirm that
the first tab on each page is selected by default:

- Catalog -> Advanced search
- Catalog -> MARC details
- Catalog -> Search results -> Merge
- Administration -> Authorized values -> Edit
- Administratoin -> Library transfer limits
- Administration -> Item types -> Edit
- Authorities -> Authority details
- Authorities -> Search -> Merge

Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>
---
 koha-tmpl/intranet-tmpl/prog/en/includes/merge-record.inc       | 1 -
 .../intranet-tmpl/prog/en/modules/admin/authorised_values.tt    | 2 +-
 .../prog/en/modules/admin/branch_transfer_limits.tt             | 2 +-
 koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt      | 2 +-
 koha-tmpl/intranet-tmpl/prog/en/modules/authorities/detail.tt   | 2 +-
 koha-tmpl/intranet-tmpl/prog/en/modules/authorities/merge.tt    | 2 +-
 koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/MARCdetail.tt | 2 +-
 koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/advsearch.tt  | 2 +-
 koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt    | 2 +-
 9 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/merge-record.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/merge-record.inc
index c7da8bf74b..260d328fd1 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/merge-record.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/merge-record.inc
@@ -48,7 +48,6 @@
     <div id="tabs" class="toptabs">
         <h2>Source records</h2>
         <ul class="nav nav-tabs" role="tablist">
-    <div class="tab-content">
             [% FOREACH record IN sourcerecords %]
                 <li role="presentation">
                     <a href="#tabrecord[% record.recordid | uri %]" aria-controls="tabrecord[% record.recordid | uri %]" role="tab" data-toggle="tab">
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/authorised_values.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/authorised_values.tt
index 1e06bbcef3..9a26c45a94 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/authorised_values.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/authorised_values.tt
@@ -464,7 +464,7 @@
                 return confirm(_("Are you sure you want to delete this authorized value category?"));
             });
 
-            if( $(".tab-pane.active").length < 1 ){
+            if( $("#icons .tab-pane.active").length < 1 ){
                 $("#icons a:first").tab("show");
             }
         });
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branch_transfer_limits.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branch_transfer_limits.tt
index 478452fe46..aaa459d599 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branch_transfer_limits.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branch_transfer_limits.tt
@@ -136,7 +136,7 @@
                 });
             });
 
-            if( $(".tab-pane.active").length < 1 ){
+            if( $("#transferlimit_tabs .tab-pane.active").length < 1 ){
                   $("#transferlimit_tabs a:first").tab("show");
             }
 
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt
index ee95cd7f1e..a91de392aa 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt
@@ -643,7 +643,7 @@ Item types &rsaquo; Administration &rsaquo; Koha
                 toUC(this);
             });
             $(".library_limitation").tooltip();
-            if( $(".tab-pane.active").length < 1 ){
+            if( $("#icons .tab-pane.active").length < 1 ){
                 $("#icons a:first").tab("show");
             }
          });
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/detail.tt
index 5664c534a5..83a1e0623e 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/detail.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/detail.tt
@@ -136,7 +136,7 @@
     [% END %]
     <script>
         $(document).ready(function() {
-            if( $(".tab-pane.active").length < 1 ){
+            if( $("#authoritiestabs .tab-pane.active").length < 1 ){
                 $("#authoritiestabs a:first").tab("show");
             }
 
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/merge.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/merge.tt
index 7feeb2b80a..1389e676c8 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/merge.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/merge.tt
@@ -149,7 +149,7 @@ div#result { margin-top: 1em; }
             ev.preventDefault();
             newin=window.open($(this).attr('href'),'merge_reference', 'width=1000,height=600,toolbar=false,scrollbars=yes');
         });
-        if( $(".tab-pane.active").length < 1 ){
+        if( $("#tabs .tab-pane.active").length < 1 ){
             $("#tabs a:first").tab("show");
         }
     });
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/MARCdetail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/MARCdetail.tt
index 5628c2019b..0996fa9293 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/MARCdetail.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/MARCdetail.tt
@@ -232,7 +232,7 @@
             $("#Frameworks").on("change",function(){
                 Changefwk(this);
             });
-            if( $(".tab-pane.active").length < 1 ){
+            if( $("#bibliotabs .tab-pane.active").length < 1 ){
                 $("#bibliotabs a:first").tab("show");
             }
         });
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/advsearch.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/advsearch.tt
index f966029aa6..84931e7d73 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/advsearch.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/advsearch.tt
@@ -419,7 +419,7 @@
         var Sticky;
         $(document).ready(function() {
             $("input[name=q]:eq(0)").focus();
-            if( $(".tab-pane.active").length < 1 ){
+            if( $("#advsearches .tab-pane.active").length < 1 ){
                 $("#advsearches a:first").tab("show");
             }
             Sticky = $("#toolbar");
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt
index b04a327674..9e3176616e 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt
@@ -293,7 +293,7 @@ div#result { margin-top: 1em; }
               rebuild_target($("#tabs"), $("#resultul"));
             });
 
-            if( $(".tab-pane.active").length < 1 ){
+            if( $("#tabs .tab-pane.active").length < 1 ){
                 $("#tabs a:first").tab("show");
             }
 
-- 
2.30.2