Attachment 105890 Details for Bug 24958 – Bug 24958: Remember last selected tab in SQL reports

[patch] Bug 24958: Remember last selected tab in SQL reports

Bug-24958-Remember-last-selected-tab-in-SQL-report.patch (text/plain), 6.22 KB, created by Owen Leonard on 2020-06-15 19:23:48 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2020-06-15 19:23:48 UTC

Size: 6.22 KB

patch

obsolete

>From 5f0af9e9e7a24904e3b6c2fe6659e4c743a66b43 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Tue, 14 Apr 2020 18:49:35 +0000
>Subject: [PATCH] Bug 24958: Remember last selected tab in SQL reports
>
>This patch modifies the saved reports page so that it remembers, for the
>duration of the browser session, the last active tab.
>
>To test you should have multiple reports in multiple report groups.
>Apply the patch and go to Reports -> Saved reports.
>
>- Select a tab to filter the table of saved reports to a particular
>  report group.
>- Navigate away from the page
>- Return to the save reports page. The tab you previously selected
>  should be selected again.
>- Restart your browser and return to the saved reports page. The tab
>  should no longer be preselected.
>
>Update: The tabs filtering JavaScript has been moved to a separate
>function so that the function can be triggered by both the "create"
>event (when the tabs are initialized) and the "activate" event (when a
>tab is selected).
>
>Update II: Persistence is now enable through localStorage instead of
>Cookies. The localStorage item is now cleared during the logOut
>function.
>---
> .../en/modules/reports/guided_reports_start.tt     | 80 +++++++++++++---------
> koha-tmpl/intranet-tmpl/prog/js/staff-global.js    |  1 +
> 2 files changed, 50 insertions(+), 31 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 38e0a42b3b..e3f1e70530 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
>@@ -1374,6 +1374,9 @@
>         }
> 
>         $(document).ready(function(){
>+
>+            var activeTab = localStorage.getItem("sql_reports_activetab");
>+
>             $("body").on('click',".fetch_chart_data",function(){
>                 if( [% unlimited_total || 0 | $raw %] > 1000 ){
>                     if( confirm("Fetching full chart data for reports with many rows can cause performance issues. Are you sure you with to chart this report?") ){
>@@ -1561,39 +1564,17 @@
>                     },
>                 }, columns_settings);
> 
>-                var rtabs = $("#tabs").tabs();
>-                rtabs.on("tabsactivate", function(e, ui) {
>-                    $("#subgroup_filter option").each(function() {
>-                        if($(this).val().length > 0) {
>-                            $(this).remove();
>-                        }
>-                    });
>-                    rtable.fnFilter('', 4);
>-                    rtable.fnFilter('', 5);
>-                    rtable.fnSetColumnVis(4, true);
>-                    rtable.fnSetColumnVis(5, true);
>-
>-                    var g_id = $(ui.newTab).children().attr('id');
>-                    var g_name = $(ui.newTab).text();
>-                    if ( g_name == _("All") ) {
>-                        g_id = "";
>-                        g_name = "";
>-                    }
>-
>-                    if (g_id && g_id.length > 0) {
>-                        rtable.fnFilter('^' + g_name + '$', 4, true, true, true, false);
>-                        rtable.fnSetColumnVis(4, false);
>-                        for(var i in group_subgroups[g_id]) {
>-                            $("#subgroup_filter").append(
>-                                '<option value="' + group_subgroups[g_id][i][0] + '">'
>-                                + group_subgroups[g_id][i][1] + '</option>'
>-                            );
>-                        }
>-                        $("#subgroup_filter_block").show();
>-                    } else {
>-                        $("#subgroup_filter_block").hide();
>+                var rtabs = $("#tabs").tabs({
>+                    create: function( e, ui ){
>+                        tabsInit( ui, rtable );
>+                    },
>+                    activate: function( e, ui ){
>+                        tabsInit( ui, rtable );
>                     }
>                 });
>+                if( activeTab ){
>+                    $("#tabs").tabs("option","active", activeTab );
>+                }
>                 $("#subgroup_filter_block").hide();
> 
>                 $("#subgroup_filter").change(function() {
>@@ -1806,6 +1787,43 @@
>             });
>         });
> 
>+        function tabsInit( ui, rtable ){
>+            var thisTab = ui.newTab ? ui.newTab : ui.tab;
>+            var activeTab = thisTab.index();
>+            localStorage.setItem("sql_reports_activetab", activeTab );
>+
>+            $("#subgroup_filter option").each(function() {
>+                if($(this).val().length > 0) {
>+                    $(this).remove();
>+                }
>+            });
>+            rtable.fnFilter('', 4);
>+            rtable.fnFilter('', 5);
>+            rtable.fnSetColumnVis(4, true);
>+            rtable.fnSetColumnVis(5, true);
>+
>+            var g_id = $(ui.newTab).children().attr('id');
>+            var g_name = $(ui.newTab).text();
>+            if ( g_name == _("All") ) {
>+                g_id = "";
>+                g_name = "";
>+            }
>+
>+            if (g_id && g_id.length > 0) {
>+                rtable.fnFilter('^' + g_name + '$', 4, true, true, true, false);
>+                rtable.fnSetColumnVis(4, false);
>+                for(var i in group_subgroups[g_id]) {
>+                    $("#subgroup_filter").append(
>+                        '<option value="' + group_subgroups[g_id][i][0] + '">'
>+                        + group_subgroups[g_id][i][1] + '</option>'
>+                    );
>+                }
>+                $("#subgroup_filter_block").show();
>+            } else {
>+                $("#subgroup_filter_block").hide();
>+            }
>+        }
>+
>         function addColumn() {
>             $("#availableColumns option:selected").clone().appendTo("#selectedColumns").attr("selected", "selected");
>         }
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/staff-global.js b/koha-tmpl/intranet-tmpl/prog/js/staff-global.js
>index 41db5cf96b..b54ab89063 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/staff-global.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/staff-global.js
>@@ -171,6 +171,7 @@ function logOut(){
>         delBasket('main', true);
>     }
>     clearHoldFor();
>+    localStorage.removeItem("sql_reports_activetab");
> }
> 
> function openHelp(){
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 24958: 102463 | 102618 | 102940 | 103252 | 105890 | 106168 | 109967 | 110741 | 110785