From 435c899bf66b49d405dfaf07d2d11ba13ad8bbaa Mon Sep 17 00:00:00 2001 From: Aleisha Amohia Date: Tue, 18 Jun 2024 03:29:32 +0000 Subject: [PATCH] Bug 37109: Don't initialise claims fields when duplicating acquisitions orders These fields don't exist in the aqorders table and don't need to be set/passed on. To test: 1. Go to Acquisitions 2. Create a vendor if you don't already have one 3. Create a basket if you don't already have one 4. Add an order to the basket and Save 5. Add another order to the basket, choose "From existing orders (copy)" 6. Continue through the process and confirm you're able to duplicate the order as normal 7. Confirm tests still pass t/db_dependent/Koha/Acquisition/Order.t Sponsored-by: Catalyst IT --- Koha/Acquisition/Order.pm | 3 --- 1 file changed, 3 deletions(-) diff --git a/Koha/Acquisition/Order.pm b/Koha/Acquisition/Order.pm index d869b9764da..c7c5e115178 100644 --- a/Koha/Acquisition/Order.pm +++ b/Koha/Acquisition/Order.pm @@ -455,15 +455,12 @@ sub duplicate_to { invoiceid datecancellationprinted cancellationreason - claims_count - claimed_date parent_ordernumber ) ) { undef $order_info->{$field}; } - $order_info->{placed_on} = dt_from_string; $order_info->{entrydate} = dt_from_string; $order_info->{orderstatus} = 'new'; $order_info->{quantityreceived} = 0; -- 2.39.2