From 1214ee738f66e32a5cd10d3ab2af80a9ee5455c3 Mon Sep 17 00:00:00 2001
From: Aleisha Amohia <aleishaamohia@hotmail.com>
Date: Wed, 21 Dec 2016 02:27:46 +0000
Subject: [PATCH] Bug 17794: Menu items in Tools menu and Admin menu not
 showing bold when active but not on linked page

To reproduce:
admin-menu
1) Go to a page like admin/categories.pl
2) Click New Category
3) Notice Patron categories in the sidebar menu is no longer bold
tools-menu
4) Go to a page like reviews/reviewswaiting.pl
5) Notice bold Comments in sidebar menu
6) Click Approved Comments tab
7) Notice no longer bold in sidebar menu
To test:
8) Apply patch and refresh page
9) Follow above steps to reproduce bug, confirm that active links are
always bold
10) Test with other links to confirm nothing has broken
Special cases
11) Go To admin -> MARC bibliographic frameworks -> Actions for any
framework -> MARC structure
12) Confirm sidebar menu link is still bold
13) Go to admin -> Authority types -> Actions for any auth type -> MARC
structure
14) Confirm sidebar menu link is still bold
15) Go to admin -> OAI sets config -> Actions for any set -> Define
mappings
16) Confirm sidebar menu link is still bold

Sponsored-by: Catalyst IT
---
 koha-tmpl/intranet-tmpl/prog/en/includes/admin-menu.inc | 13 ++++++++-----
 koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc |  7 +------
 2 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/admin-menu.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/admin-menu.inc
index e736272..afd64a1 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/admin-menu.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/admin-menu.inc
@@ -2,12 +2,15 @@
 <script type="text/javascript">//<![CDATA[
     $(document).ready(function() {
         var path = location.pathname.substring(1);
-        var url = window.location.toString();
-        var params = '';
-        if ( url.match(/\?(.+)$/) ) {
-            params = "?" + RegExp.$1;
+        if (path == "cgi-bin/koha/admin/marctagstructure.pl") {
+            $('#navmenulist a[href$="/cgi-bin/koha/admin/biblio_framework.pl"]').css('font-weight','bold');
+        } else if (path == "cgi-bin/koha/admin/auth_tag_structure.pl") {
+            $('#navmenulist a[href$="/cgi-bin/koha/admin/authtypes.pl"]').css('font-weight','bold');
+        } else if (path == "cgi-bin/koha/admin/oai_set_mappings.pl") {
+            $('#navmenulist a[href$="/cgi-bin/koha/admin/oai_sets.pl"]').css('font-weight','bold');
+        } else {
+            $('#navmenulist a[href$="/' + path + '"]').css('font-weight','bold');
         }
-        $('#navmenulist a[href$="/' + path + params + '"]').css('font-weight','bold');
     });
 //]]>
 </script>
diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc
index 63c318e..a4bf84a 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc
@@ -1,11 +1,6 @@
 <script type="text/javascript">//<![CDATA[
     $(document).ready(function() {
         var path = location.pathname.substring(1);
-        var url = window.location.toString();
-        var params = '';
-        if ( url.match(/\?(.+)$/) ) {
-            params = "?" + RegExp.$1;
-        }
         if (path.indexOf("labels") >= 0 && path.indexOf("spine") < 0 ) {
           $('#navmenulist a[href$="/cgi-bin/koha/labels/label-home.pl"]').css('font-weight','bold');
         } else if (path.indexOf("patroncards") >= 0 ) {
@@ -13,7 +8,7 @@
        } else if (path.indexOf("patron_lists") >= 0 ) {
           $('#navmenulist a[href$="/cgi-bin/koha/patron_lists/lists.pl"]').css('font-weight','bold');
         } else {
-          $('#navmenulist a[href$="/' + path + params + '"]').css('font-weight','bold');
+          $('#navmenulist a[href$="/' + path + '"]').css('font-weight','bold');
         }
     });
 //]]>
-- 
2.1.4