From 8f26477bdfdda1cf9d5d6975c6eeab49bc23cf9a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adolfo=20Rodr=C3=ADguez?= Date: Fri, 7 Mar 2025 12:39:49 +0100 Subject: [PATCH] Bug 38411: Check mandatory item fields when creating several items ordering or receiving. This patch adds a check of mandatory item fields when adding several items Test plan: 1 We need mandatory subfields for items in the ACQ framework and an open basket configured to create items on order or a closed basket configured to create items on receiving an order 2 In both cases, in the form to create items check that there are mandatory fields. 3 Click the Add multiple items button, write a number and then add. 4 Check that the items have been added. 5 Apply patch, restart services 6 Repeat step 3 Now it doesn't add the items and shows an alert regarding the empty mandatory fields --- koha-tmpl/intranet-tmpl/prog/js/additem.js | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/koha-tmpl/intranet-tmpl/prog/js/additem.js b/koha-tmpl/intranet-tmpl/prog/js/additem.js index 5ce8be0349..0903a0b96d 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/additem.js +++ b/koha-tmpl/intranet-tmpl/prog/js/additem.js @@ -59,7 +59,24 @@ function addItem(node, unique_item_fields) { } function addMulti(count, node, unique_item_fields) { + var item_form = $(node).closest("div"); var index = $(node).closest("div").attr("id"); + + //We need to verify the item form before saving + var empty_item_mandatory = CheckMandatorySubfields(item_form); + if (empty_item_mandatory > 0) { + var _alertString = + __("Form not submitted because of the following problem(s)") + "\n"; + + _alertString += + "-------------------------------------------------------------------\n\n"; + _alertString += + "\n- " + + __("%s item mandatory fields empty").format(empty_item_mandatory); + alert(_alertString); + return false; + } + var countItemsBefore = $("#items_list tbody tr").length; var current_qty = parseInt($("#quantity").val(), 10); $("#procModal").modal("show"); -- 2.30.2