Attachment 116153 Details for Bug 24190 – Bug 24190: (follow-up) Modify order of logging

[patch] Bug 24190: (follow-up) Modify order of logging

Bug-24190-follow-up-Modify-order-of-logging.patch (text/plain), 1.31 KB, created by Andrew Isherwood on 2021-02-01 12:05:59 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andrew Isherwood

Created: 2021-02-01 12:05:59 UTC

Size: 1.31 KB

patch

obsolete

>From 51389936d4a8bc4bb5b3e997aac029b783163650 Mon Sep 17 00:00:00 2001
>From: Andrew Isherwood <andrew.isherwood@ptfs-europe.com>
>Date: Mon, 26 Oct 2020 11:44:50 +0000
>Subject: [PATCH] Bug 24190: (follow-up) Modify order of logging
>
>Basket creation involves two steps in Koha, creating the basket then
>modifying its header. We were logging these two steps in the wrong
>order. This commit fixes that.
>---
> C4/Acquisition.pm | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
>diff --git a/C4/Acquisition.pm b/C4/Acquisition.pm
>index 5652f2b2f1..73bf85c6d6 100644
>--- a/C4/Acquisition.pm
>+++ b/C4/Acquisition.pm
>@@ -203,8 +203,6 @@ sub NewBasket {
>     $basketname           ||= q{}; # default to empty strings
>     $basketnote           ||= q{};
>     $basketbooksellernote ||= q{};
>-    ModBasketHeader( $basket, $basketname, $basketnote, $basketbooksellernote,
>-        $basketcontractnumber, $booksellerid, $deliveryplace, $billingplace, $is_standing, $create_items );
> 
>     # Log the basket creation
>     if (C4::Context->preference("AcqLog")) {
>@@ -217,6 +215,9 @@ sub NewBasket {
>         );
>     }
> 
>+    ModBasketHeader( $basket, $basketname, $basketnote, $basketbooksellernote,
>+        $basketcontractnumber, $booksellerid, $deliveryplace, $billingplace, $is_standing, $create_items );
>+
>     return $basket;
> }
> 
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 24190: 96364 | 96369 | 100243 | 105082 | 105083 | 106096 | 106097 | 106098 | 106099 | 111080 | 111081 | 111082 | 111083 | 111084 | 112509 | 112510 | 112511 | 112512 | 112513 | 112514 | 116148 | 116149 | 116150 | 116151 | 116152 | 116153 | 122040 | 122041 | 122042 | 122043 | 122044 | 122045 | 123385 | 123386 | 123387 | 123388 | 123389 | 123390 | 123391 | 125119