From 7e5a6f2222918080f3c45f98c312c59b9af115ae Mon Sep 17 00:00:00 2001 From: Jonathan Druart 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 Signed-off-by: Brendan Lawlor Signed-off-by: Matt Blenkinsop --- .../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 83559a159e0..417f17b9f9a 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.39.5 (Apple Git-154)