Attachment 143770 Details for Bug 16223 – Bug 16223: (QA follow-up) Move trigger to apply

[patch] Bug 16223: (QA follow-up) Move trigger to apply

Bug-16223-QA-follow-up-Move-trigger-to-apply.patch (text/plain), 1.74 KB, created by Martin Renvoize (ashimema) on 2022-11-11 10:47:12 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2022-11-11 10:47:12 UTC

Size: 1.74 KB

patch

obsolete

>From 3cb5f965c0332ef18efd088ea568551dc7ff4c48 Mon Sep 17 00:00:00 2001
>From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>Date: Fri, 11 Nov 2022 09:54:05 +0000
>Subject: [PATCH] Bug 16223: (QA follow-up) Move trigger to apply
>
>This patch moves the del_restrictions_after_payment from before the
>actual credit application call inside Koha::Account::pay to after the
>application of credits to debits in Koha::Account::Line::apply.
>---
> Koha/Account.pm      | 3 ---
> Koha/Account/Line.pm | 3 +++
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/Koha/Account.pm b/Koha/Account.pm
>index 43d8e65f00..e37e646596 100644
>--- a/Koha/Account.pm
>+++ b/Koha/Account.pm
>@@ -36,7 +36,6 @@ use Koha::Account::DebitTypes;
> use Koha::Exceptions;
> use Koha::Exceptions::Account;
> use Koha::Plugins;
>-use Koha::Patron::Debarments;
> 
> =head1 NAME
> 
>@@ -103,8 +102,6 @@ sub pay {
>         }
>     );
> 
>-    Koha::Patron::Debarments::del_restrictions_after_payment({ borrowernumber => $self->{patron_id} });
>-
>     # NOTE: Pay historically always applied as much credit as it could to all
>     # existing outstanding debits, whether passed specific debits or otherwise.
>     if ( $payment->amountoutstanding ) {
>diff --git a/Koha/Account/Line.pm b/Koha/Account/Line.pm
>index b940d6acef..d300cb7da2 100644
>--- a/Koha/Account/Line.pm
>+++ b/Koha/Account/Line.pm
>@@ -29,6 +29,7 @@ use Koha::Database;
> use Koha::DateUtils qw( dt_from_string );
> use Koha::Exceptions::Account;
> use Koha::Items;
>+use Koha::Patron::Debarments;
> 
> use base qw(Koha::Object Koha::Object::Mixin::AdditionalFields);
> 
>@@ -701,6 +702,8 @@ sub apply {
>         }
>     });
> 
>+    Koha::Patron::Debarments::del_restrictions_after_payment({ borrowernumber => $self->borrowernumber });
>+
>     return $self;
> }
> 
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 16223: 50078 | 50084 | 99697 | 99698 | 104399 | 104400 | 114272 | 114273 | 118526 | 118572 | 118574 | 118575 | 122768 | 122769 | 127252 | 128005 | 132284 | 132285 | 132286 | 132287 | 143388 | 143389 | 143390 | 143462 | 143670 | 143671 | 143672 | 143688 | 143689 | 143690 | 143765 | 143766 | 143767 | 143768 | 143769 | 143770 | 143936 | 146051 | 146052 | 146053 | 146054 | 146055 | 146056 | 146057 | 148307 | 148308 | 148309 | 148310 | 148311 | 148312 | 148313 | 150024 | 150025 | 150026 | 150027 | 150028 | 150029 | 153024 | 153025 | 153026 | 153027 | 153028 | 153029 | 155644 | 155645 | 155646 | 155647 | 155648 | 155649 | 155650 | 156841 | 156842 | 156843 | 156844 | 156845 | 156846 | 156847 | 156848 | 156849 | 156850