Attachment 54172 Details for Bug 15906 – [SIGNED-OFF] Bug 15906 - Remove use of makepayment in paycollect.pl

[patch] [SIGNED-OFF] Bug 15906 - Remove use of makepayment in paycollect.pl

SIGNED-OFF-Bug-15906---Remove-use-of-makepayment-i.patch (text/plain), 1.54 KB, created by Owen Leonard on 2016-08-08 14:54:29 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2016-08-08 14:54:29 UTC

Size: 1.54 KB

patch

obsolete

>From ec28b40888abd6117ed68ba79e2461f8e2a3a758 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Wed, 24 Feb 2016 15:51:07 +0000
>Subject: [PATCH] [SIGNED-OFF] Bug 15906 - Remove use of makepayment in
> paycollect.pl
>Content-Type: text/plain; charset="utf-8"
>
>Test plan:
>1) Apply this patch
>2) Make a payment in full using the "Pay" button
>3) Note payment succeeds
>
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>---
> members/paycollect.pl | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>
>diff --git a/members/paycollect.pl b/members/paycollect.pl
>index 5dd565c..79c9bd9 100755
>--- a/members/paycollect.pl
>+++ b/members/paycollect.pl
>@@ -110,8 +110,15 @@ if ( $total_paid and $total_paid ne '0.00' ) {
>     } else {
>         if ($individual) {
>             if ( $total_paid == $total_due ) {
>-                makepayment( $accountlines_id, $borrowernumber, $accountno, $total_paid, $user,
>-                    $branch, $payment_note );
>+                my $line = Koha::Account::Lines->find($accountlines_id);
>+                Koha::Account->new( { patron_id => $borrowernumber } )->pay(
>+                    {
>+                        lines      => [$line],
>+                        amount     => $total_paid,
>+                        library_id => $branch,
>+                        note       => $payment_note
>+                    }
>+                );
>             } else {
>                 makepartialpayment( $accountlines_id, $borrowernumber, $accountno, $total_paid,
>                     $user, $branch, $payment_note );
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 15906: 48347 | 52135 | 52148 | 54172 | 57168 | 57169