From 961c622e706145bd77ae187349a6d59f39e8ea44 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Mon, 28 Jul 2014 10:18:59 -0500 Subject: [PATCH] Bug 12432 - Saved reports tabs not working In release 3.14.05.000 the tabs on the Saved Reports page worked correctly but after upgrading to 3.16.00.000 the tabs stop working. Visually the tabs change but the table of reports is not filtered. There are no errors reported in the browser console. Test Plan: 1) Attempt to filter saved reports by group tabs 2) Note no matter the tab you select, all reports appear 3) Apply this patch 4) Repeat step 1 5) Note the reports are now filtered correctly Signed-off-by: Christopher Brannon Signed-off-by: Katrin Fischer Passes all tests and QA script, works as described with the second patch applied as well. Signed-off-by: Tomas Cohen Arazi Signed-off-by: Galen Charlton (cherry picked from commit 273e86c045ad7e34a3eedeee0c5d3328af8a7605) Signed-off-by: Fridolin Somers --- .../prog/en/modules/reports/guided_reports_start.tt | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt index 63608f7a77..aa0f8e6727 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt @@ -76,7 +76,7 @@ $(document).ready(function(){ })); var rtabs = $("#tabs").tabs(); - rtabs.bind("tabsshow", function(e, ui) { + rtabs.on("tabsactivate", function(e, ui) { $("#subgroup_filter option").each(function() { if($(this).val().length > 0) { $(this).remove(); @@ -87,8 +87,9 @@ $(document).ready(function(){ rtable.fnSetColumnVis(4, true); rtable.fnSetColumnVis(5, true); - var g_id = $(ui.tab).attr('id') - var g_name = $(ui.tab).text(); + var g_id = $(ui.newTab).children().attr('id'); + var g_name = $(ui.newTab).text(); + if (g_id && g_id.length > 0) { rtable.fnFilter('^' + g_name + '$', 4, true, true, true, false); rtable.fnSetColumnVis(4, false); -- 2.39.5