From 188168f7e4039b68099680a3b9c1358fe5a654d9 Mon Sep 17 00:00:00 2001 From: Johanna Raisa Date: Tue, 26 Jul 2022 12:01:56 +0300 Subject: [PATCH] Bug 23202: Fix screen freeze when ordereing several items This patch fixes adding multiple items on order acquisitions The error is: Uncaught TypeError: $(...).parent(...).attr(...) is undefined Test plan: 1) Set dateaccessioned.pl plugin to ACQ framework's 952$d field. 2) Create basket to a vendor and add order from a new empty record. 3) Add date acquired value from calendar. 4) Add multiple items and see that the view freezes. 5) Apply the patch 6) Reload the page and repeat the steps 3 and 4. 7) See that the items are added. Signed-off-by: Katrin Fischer JD Amended patch: adjust commit title Signed-off-by: Jonathan Druart --- koha-tmpl/intranet-tmpl/prog/js/additem.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/koha-tmpl/intranet-tmpl/prog/js/additem.js b/koha-tmpl/intranet-tmpl/prog/js/additem.js index a83310f29d4..fac6b2648cf 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/additem.js +++ b/koha-tmpl/intranet-tmpl/prog/js/additem.js @@ -192,7 +192,7 @@ function cloneItemBlock(index, unique_item_fields, callback) { $(original).find("input[name='field_value']").each(function(){ var kohafield = $(this).siblings("input[name='kohafield']").val(); if($(this).val() && $.inArray(kohafield,dont_copy_fields) == -1) { - $(this).parent("div").attr("id").match(/^(subfield.)/); + $(this).closest("li div").attr("id").match(/^(subfield.)/); var id = RegExp.$1; var value = $(this).val(); $(clone).find("div[id^='"+id+"'] input[name='field_value']").val(value); -- 2.25.1