Attachment 102618 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), 5.46 KB, created by Owen Leonard on 2020-04-09 13:20:27 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2020-04-09 13:20:27 UTC

Size: 5.46 KB

patch

obsolete

>From 50553f6e860865db898761bcd8a05be286806f76 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Mon, 16 Mar 2020 18:53:10 +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).
>---
> .../en/modules/reports/guided_reports_start.tt     | 77 +++++++++++++---------
> 1 file changed, 46 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 21738b687e..dde6bc0aca 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
>@@ -1376,6 +1376,9 @@
>         }
> 
>         $(document).ready(function(){
>+
>+            var activeTab = Cookies.get("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?") ){
>@@ -1563,39 +1566,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() {
>@@ -1808,6 +1789,40 @@
>             });
>         });
> 
>+        function tabsInit( ui, rtable ){
>+            console.log( "tabsInit" );
>+            $("#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");
>         }
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

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