From bb06060c4c3e0f4393f24096400ee4c8312a44ec Mon Sep 17 00:00:00 2001 From: Lucas Gass Date: Mon, 13 May 2019 18:05:38 +0000 Subject: [PATCH] Bug 22914: Add holds column to batch item delete to fix show/hide columns behaviour The changes on 19490 caused a bug, where the show/hide column feature hides the wrong column on the items table. The two templates share a js file, and the data passed is the same. Adding the column to the template brings things back into alignment, the data is already there, we just show it. TEST PLAN: 1. Go to Tools > Batch item deletion and enter at least one barcode. 2. Try to hide a column that is showing 3. Notice it hides the wrong column 4. apply patch 5. repeat steps 1 and 2. 6. Notice the correct column is being hidden 7. Verify the new column for number of holds on the item works correctly for item and record level holds Signed-off-by: Lisette Scheer Signed-off-by: Katrin Fischer Signed-off-by: Nick Clemens --- .../prog/en/modules/tools/batchMod-del.tt | 43 ++++++++++++++++++- 1 file changed, 41 insertions(+), 2 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batchMod-del.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batchMod-del.tt index fd7913b850..b938a16c87 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batchMod-del.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batchMod-del.tt @@ -97,14 +97,53 @@ [% IF ( show ) %] [% END %] Title + Holds [% FOREACH item_header_loo IN item_header_loop %] [% item_header_loo.header_value | html %] [% END %] - [% FOREACH item_loo IN item_loop %] [% IF ( show ) %][% IF ( item_loo.nomod ) %] Cannot edit[% ELSE %][% END %][% END %] - + [% FOREACH item_loo IN item_loop %] + + [% IF show %] + [% IF item_loo.nomod %] + Cannot edit + [% ELSE %] + + [% END %] + [% ELSE %] +   + [% END %] + + + + + [% IF item_loo.holds %] + [% IF item_loo.item_holds %] + + [% ELSE %] + + [% END %] + [% ELSE %] + [% IF item_loo.holds %] + + [% ELSE %] + + [% END %] + [% END %] + [% IF item_loo.holds %] + [% item_loo.item_holds | html %]/[% item_loo.holds | html %] + [% ELSE %] + [% item_loo.holds | html %] + [% END %] + + [% FOREACH item_valu IN item_loo.item_value %] [% item_valu.field | html %] [% END %] [% END %] -- 2.39.5