Attachment 175296 Details for Bug 38632 – Bug 38632: Do not show all columns when displaying the filters

[patch] Bug 38632: Do not show all columns when displaying the filters

Bug-38632-Do-not-show-all-columns-when-displaying-.patch (text/plain), 2.14 KB, created by Jonathan Druart on 2024-12-09 11:26:42 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2024-12-09 11:26:42 UTC

Size: 2.14 KB

patch

obsolete

>From 48e601d7130a634595a173488f5310b41c031306 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Mon, 9 Dec 2024 12:19:45 +0100
>Subject: [PATCH] Bug 38632: Do not show all columns when displaying the
> filters
>
>Yet another visibility bug fix for this specific table.
>When the filters are displayed the table is destroyed and recreated.
>We should not save the visibility changes made by DT and consider them
>changes made by the user.
>A temporary variable is used to store the user settings duringt this
>action, then we restore them once it's done.
>
>Test plan:
>Go to the detail page of bibliographic record with items
>Click on "show filters"
>=> Only columns displayed should still be displayed
>Retry but show/hide some columns before you display the filters
>=> Only columns displayed should still be displayed
>---
> .../en/includes/html_helpers/tables/items/catalogue_detail.inc | 3 +++
> 1 file changed, 3 insertions(+)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers/tables/items/catalogue_detail.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers/tables/items/catalogue_detail.inc
>index f6ce0667c3c..f70f9b716d2 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers/tables/items/catalogue_detail.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers/tables/items/catalogue_detail.inc
>@@ -275,12 +275,15 @@
>     function build_items_table (tab_id, add_filters, dt_options, drawcallback) {
> 
>         if ( dt_options && dt_options.hasOwnProperty('destroy') ) {
>+            // Keep a copy of the user settings, the destroy is going to trigger the column-visibility.dt event for all columns
>+            let user_colvis_bak= Object.assign({}, user_colvis[tab_id]);
>             let table_id = "#"+tab_id+"_table";
>             if( $.fn.dataTable.isDataTable(table_id) ) {
>                 $(table_id).DataTable().destroy();
>             }
>             $(table_id).replaceWith(table_nodes[tab_id]);
>             dt_options['destroy'] = null;
>+            user_colvis[tab_id] = user_colvis_bak;
>         }
>         let default_filters = {};
>         [% IF Koha.Preference('SeparateHoldings') %]
>-- 
>2.34.1

Actions: View | Diff | Splinter Review

Attachments on bug 38632: 175296 | 175297 | 175380 | 175381