From 31a020503e105f074abada767667013e895c9244 Mon Sep 17 00:00:00 2001 From: Emmi Takkinen Date: Thu, 23 Nov 2023 13:15:57 +0200 Subject: [PATCH] Bug 32477: Launch function hideColumns after changing datatables settings Hiding columns in batch item modification breaks if page is loaded again. If one hides e.g. column "Source of classification or shelving scheme" and then loads the page column "Holds" disappears. Also behaviour of checbox changes for "Source of classification..." column. This happens because we launch function "hideColumns" before changing datatables settings and "hidden" column with class "sorting_1" is still present. To test: 1. Find items to modify and modify them with batch item modification tool. 2. In modification page, hide column "Source of classification or shelving scheme". Confirm correct column is hidden. 3. Reload the page or modify items again. => Note that column "Holds" is now hidden and checking checkbox for column "Source of classification..." behaves incorrectly (unchecked checkbox shows column, checked hides it) 4. Apply this patch. 5. Repeat steps 1, 2 and 3. => Confirm correct column is now hidden when page is loaded again and checkbox works correctly. Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind --- koha-tmpl/intranet-tmpl/prog/js/pages/batchMod.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/koha-tmpl/intranet-tmpl/prog/js/pages/batchMod.js b/koha-tmpl/intranet-tmpl/prog/js/pages/batchMod.js index 4118c6b013..fd1ae93cd4 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/pages/batchMod.js +++ b/koha-tmpl/intranet-tmpl/prog/js/pages/batchMod.js @@ -117,7 +117,6 @@ function hideAllColumns() { } $(document).ready(function () { - hideColumns(); var items_table = KohaTable("itemst", { "aoColumnDefs": [ { "aTargets": [0, 1], "bSortable": false, "bSearchable": true }, @@ -126,6 +125,9 @@ $(document).ready(function () { ], "bPaginate": false, }); + + hideColumns(); + // Highlight in yellow item rows that cannot be deleted $(".error").parents('tr').find('td').css('background-color', '#ffff99'); -- 2.30.2