Attachment 167822 Details for Bug 37109 – Bug 37109: Don't initialise claims fields when duplicating acquisitions orders

[patch] Bug 37109: Don't initialise claims fields when duplicating acquisitions orders

Bug-37109-Dont-initialise-claims-fields-when-dupli.patch (text/plain), 1.95 KB, created by Owen Leonard on 2024-06-18 11:50:42 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2024-06-18 11:50:42 UTC

Size: 1.95 KB

patch

obsolete

>From f121221ccc24bf8f17652ffa127e2646725baab9 Mon Sep 17 00:00:00 2001
>From: Aleisha Amohia <aleishaamohia@hotmail.com>
>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.
>
>* claims_count (dropped in Bug 24161)
>* claimed_date (dropped in Bug 24161)
>* placed_on (no evidence of this field existing in aqorders)
>* received_on (no evidence of this field existing in aqorders)
>
>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
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>---
> Koha/Acquisition/Order.pm | 4 ----
> 1 file changed, 4 deletions(-)
>
>diff --git a/Koha/Acquisition/Order.pm b/Koha/Acquisition/Order.pm
>index d869b9764d..b8399e3524 100644
>--- a/Koha/Acquisition/Order.pm
>+++ b/Koha/Acquisition/Order.pm
>@@ -450,20 +450,16 @@ sub duplicate_to {
>             for my $field (
>                 qw(
>                 ordernumber
>-                received_on
>                 datereceived
>                 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

Actions: View | Diff | Splinter Review

Attachments on bug 37109: 167816 | 167817 | 167822 | 172493