Attachment 52142 Details for Bug 15900 – Bug 15900 - Remove use of recordpayment in ProcessOfflinePayment

[patch] Bug 15900 - Remove use of recordpayment in ProcessOfflinePayment

Bug-15900---Remove-use-of-recordpayment-in-Process.patch (text/plain), 1.25 KB, created by Kyle M Hall (khall) on 2016-06-07 15:11:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2016-06-07 15:11:16 UTC

Size: 1.25 KB

patch

obsolete

>From b52e9d86a736d4003edef1d76893b73cfadfdaa1 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Wed, 24 Feb 2016 15:11:53 +0000
>Subject: [PATCH] Bug 15900 - Remove use of recordpayment in
> ProcessOfflinePayment
>
>Test plan:
>1) Apply this patch
>2) prove t/db_dependent/Circulation.t
>---
> C4/Circulation.pm |    5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index 50e6c10..b3ddd95 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -45,6 +45,7 @@ use C4::RotatingCollections qw(GetCollectionItemBranches);
> use Algorithm::CheckDigits;
> 
> use Data::Dumper;
>+use Koha::Account;
> use Koha::DateUtils;
> use Koha::Calendar;
> use Koha::Items;
>@@ -3908,10 +3909,10 @@ sub ProcessOfflineIssue {
> sub ProcessOfflinePayment {
>     my $operation = shift;
> 
>-    my $borrower = C4::Members::GetMemberDetails( undef, $operation->{cardnumber} ); # Get borrower from operation cardnumber
>+    my $patron = Koha::Borrowers->find( { cardnumber => $operation->{cardnumber} });
>     my $amount = $operation->{amount};
> 
>-    recordpayment( $borrower->{borrowernumber}, $amount );
>+    Koha::Account->new( { patron_id => $patron->id } )->pay( { amount => $amount } );
> 
>     return "Success."
> }
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 15900: 48337 | 52142 | 54742 | 55261 | 55872 | 55873