From f9f270efd6f40afdd3be861d7bd77e29bff769ec Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 29 Jun 2018 17:30:39 -0300 Subject: [PATCH] Bug 20806: Do not rely on column position It is not the first time we have this splice call, it seems safer to rely on the column's name instead of the column's position Signed-off-by: Jonathan Druart Signed-off-by: Nick Clemens (cherry picked from commit 2a369663fd37c41fb3ac810a8f42d84d69fa6f02) Signed-off-by: Martin Renvoize --- koha-tmpl/intranet-tmpl/prog/en/modules/members/holdshistory.tt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/holdshistory.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/holdshistory.tt index ff19d79c36..8ba606f5dc 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/holdshistory.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/holdshistory.tt @@ -125,7 +125,7 @@ var columns_settings = [% ColumnsSettings.GetColumns('members', 'holdshistory', 'holdshistory-table', 'json') %]; [% UNLESS show_itemtype_column %] //Remove item type column settings - columns_settings.splice(8,1); + columns_settings = columns_settings.filter(function(c){return c['columnname'] != 'itemtype';}); [% END %] var table = KohaTable("table_holdshistory", { "sPaginationType": "four_button", -- 2.39.5