From 9110547de098264e2574e27f35859674f1ed18a2 Mon Sep 17 00:00:00 2001 From: Slava Shishkin Date: Fri, 6 Oct 2023 19:25:30 +0300 Subject: [PATCH] Bug 35004: Set row.quantity_received to 1 if row.quantity_received is empty and we're in 'cataloguing' mode Before the code changes only field value was set to 1, but "row.quantity_received" still had an undefined value. After correction row.quantity_received set to 1 if it is empty, but only when effective_create_items == 'cataloguing', and after that its value assigned to the form field. To test: 1. Add a new basket. 2. Create an order line 3. Receive shipment and create an invoice 4. Start receiving your order: Verify: - Quantity ordered: X - Quantity received: 1 (pre-filled, don't touch or change it) 5. Click "Confirm" 6. Observe the Receive error popup: "Order ZZZ: No quantity to receive set. No items will be created." 7. Verify that "Order receive" was not happen (received orders remains empty) 8. Apply the patch and make sure your order is received correctly after step 5. Signed-off-by: Nick Clemens Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi --- .../intranet-tmpl/prog/en/modules/acqui/orderreceive.tt | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt index eac0b64c35..a598ee2ec6 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt @@ -1180,7 +1180,10 @@ $("#creator").html(_("Account has been deleted")); } $("#quantity_to_receive").val(row.quantity).prop('readonly', !row.subscription_id); - $("#quantity").val( effective_create_items == 'cataloguing' ? row.quantity_received || 1 : row.quantity_received ) + if (effective_create_items == 'cataloguing' && ! row.quantity_received) { + row.quantity_received = 1; + } + $("#quantity").val( row.quantity_received ) .prop('readonly', !row.subscription_id && effective_create_items == 'receiving'); [% IF only_one_order %] $(".modal-save").prop('disabled', $("#quantity").val() == 0); -- 2.39.5