From e1fa520018de20af200826d49088f50b32793d22 Mon Sep 17 00:00:00 2001 From: Emmi Takkinen Date: Mon, 19 Feb 2024 12:55:26 +0200 Subject: [PATCH] Bug 36129: Make check on "Hide all columns" persist on item patch modification/deletion On item patch modification/deletion tool, if one checks "Hide all columns" checkbox and then reloads the page, checkbox is no longer selected. Columns are hidden as they should. This patch adds line to batchMod.js which sets "checked" attribute and class "selected" to checkbox. To test: 1. Find items to modify/delete and modify/delete them with corresponding tool. 2. Check checkbox "Hide all columns". 3. Refresh the page. => Note that columns are still hidden, but checkbox is now unselected. 4. Apply this patch. 5. Check checkbox again and refresh page. => Checkbox should be still checked. Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind Signed-off-by: Kyle M Hall Signed-off-by: Martin Renvoize (cherry picked from commit 1a154f1f0e1fa4d5c7295ba181fc27eaff99fd09) Signed-off-by: Lucas Gass (cherry picked from commit 909987cd4db25088e68b5ca75404f25dab6b3fba) Signed-off-by: Fridolin Somers --- koha-tmpl/intranet-tmpl/prog/js/pages/batchMod.js | 1 + 1 file changed, 1 insertion(+) diff --git a/koha-tmpl/intranet-tmpl/prog/js/pages/batchMod.js b/koha-tmpl/intranet-tmpl/prog/js/pages/batchMod.js index 128c37909f..df428f831b 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/pages/batchMod.js +++ b/koha-tmpl/intranet-tmpl/prog/js/pages/batchMod.js @@ -20,6 +20,7 @@ function hideColumns() { if (valCookie) { valCookie = valCookie.split("/"); $("#showall").prop("checked", false).parent().removeClass("selected"); + $("#hideall").prop("checked", true).parent().addClass("selected"); for ( var i = 0; i < valCookie.length; i++ ) { if (valCookie[i] !== '') { var index = valCookie[i] - nb_cols; -- 2.39.5