[Patches] [PATCH] Bug 6273: Add support for recording SIP2 Fee Paid Payment Type
koha-patchbot at kohaaloha.com
koha-patchbot at kohaaloha.com
Sat Dec 3 14:16:24 NZDT 2011
From: Ian Walls <ian.walls at bywatersolutions.com>
Date: Fri, 2 Dec 2011 10:49:07 -0500
Subject: [PATCH] Bug 6273: Add support for recording SIP2 Fee Paid Payment Type
Adds support for recording in Koha accountlines a different accounttype
depending on how the fee was paid at the SIP2 station (cash, credit, etc)
Adds a new param to recordpayment(), $sip_paytype, which is appended to the
'Pay' type if present. The payment descriptin is also appended with " (via SIP2)"
if this is present.
In order for other scripts to keep working as expected, "eq 'Pay'" needed to
be replaced with a regex comparison "=~ /^Pay/", so that 'Pay' and 'Pay-##' would continue
to group together.
To test:
1. Make a payment over a SIP2 connection
2. Check the patron record in the staff client; you should see the modified description
3. Attempt to print a invoice or a reciept for the borrower; the payment should show up
where expected
---
C4/Accounts.pm | 18 ++++++++++++------
C4/SIP/ILS.pm | 2 +-
C4/SIP/ILS/Transaction/FeePayment.pm | 3 ++-
members/boraccount.pl | 2 +-
members/printfeercpt.pl | 2 +-
members/printinvoice.pl | 2 +-
6 files changed, 18 insertions(+), 11 deletions(-)
diff --git a/C4/Accounts.pm b/C4/Accounts.pm
index 860e187..1d9d712 100644
--- a/C4/Accounts.pm
+++ b/C4/Accounts.pm
@@ -60,11 +60,13 @@ patron.
=head2 recordpayment
- &recordpayment($borrowernumber, $payment);
+ &recordpayment($borrowernumber, $payment, $sip_paytype);
Record payment by a patron. C<$borrowernumber> is the patron's
borrower number. C<$payment> is a floating-point number, giving the
-amount that was paid.
+amount that was paid. C<$sip_paytype> is an optional flag to indicate this
+payment was made over a SIP2 interface, rather than the staff client. The
+value passed is the SIP2 payment type value (message 37, characters 21-22)
Amounts owed are paid off oldest first. That is, if the patron has a
$1 fine from Feb. 1, another $1 fine from Mar. 1, and makes a payment
@@ -77,7 +79,7 @@ will be credited to the next one.
sub recordpayment {
#here we update the account lines
- my ( $borrowernumber, $data ) = @_;
+ my ( $borrowernumber, $data, $sip_paytype ) = @_;
my $dbh = C4::Context->dbh;
my $newamtos = 0;
my $accdata = "";
@@ -126,9 +128,13 @@ sub recordpayment {
my $usth = $dbh->prepare(
"INSERT INTO accountlines
(borrowernumber, accountno,date,amount,description,accounttype,amountoutstanding)
- VALUES (?,?,now(),?,'Payment,thanks','Pay',?)"
+ VALUES (?,?,now(),?,?,?,?)"
);
- $usth->execute( $borrowernumber, $nextaccntno, 0 - $data, 0 - $amountleft );
+ my $payment_description = "Payment, thanks";
+ $payment_description .= " (via SIP2)" if defined $sip_paytype;
+ my $paytype = "Pay";
+ $paytype .= "-$sip_paytype" if defined $sip_paytype;
+ $usth->execute( $borrowernumber, $nextaccntno, 0 - $data, $payment_description, $paytype, 0 - $amountleft );
$usth->finish;
UpdateStats( $branch, 'payment', $data, '', '', '', $borrowernumber, $nextaccntno );
$sth->finish;
@@ -602,7 +608,7 @@ sub getcredits {
my $dbh = C4::Context->dbh;
my $sth = $dbh->prepare(
"SELECT * FROM accountlines,borrowers
- WHERE amount < 0 AND accounttype <> 'Pay' AND accountlines.borrowernumber = borrowers.borrowernumber
+ WHERE amount < 0 AND accounttype not like 'Pay%' AND accountlines.borrowernumber = borrowers.borrowernumber
AND timestamp >=TIMESTAMP(?) AND timestamp < TIMESTAMP(?)"
);
diff --git a/C4/SIP/ILS.pm b/C4/SIP/ILS.pm
index 432e4f1..be9ae4a 100644
--- a/C4/SIP/ILS.pm
+++ b/C4/SIP/ILS.pm
@@ -229,7 +229,7 @@ sub pay_fee {
$trans->screen_msg('Invalid patron barcode.');
return $trans;
}
- $trans->pay($patron->{borrowernumber},$fee_amt);
+ $trans->pay($patron->{borrowernumber},$fee_amt, $pay_type);
$trans->ok(1);
return $trans;
diff --git a/C4/SIP/ILS/Transaction/FeePayment.pm b/C4/SIP/ILS/Transaction/FeePayment.pm
index 3026296..168a343 100644
--- a/C4/SIP/ILS/Transaction/FeePayment.pm
+++ b/C4/SIP/ILS/Transaction/FeePayment.pm
@@ -47,8 +47,9 @@ sub pay {
my $self = shift;
my $borrowernumber = shift;
my $amt = shift;
+ my $type = shift;
warn("RECORD:$borrowernumber::$amt");
- recordpayment( $borrowernumber, $amt );
+ recordpayment( $borrowernumber, $amt,$type );
}
#sub DESTROY {
diff --git a/members/boraccount.pl b/members/boraccount.pl
index 2805f27..1366ec4 100755
--- a/members/boraccount.pl
+++ b/members/boraccount.pl
@@ -84,7 +84,7 @@ foreach my $accountline ( @{$accts}) {
$accountline->{date} = format_date($accountline->{date});
$accountline->{amount} = sprintf '%.2f', $accountline->{amount};
$accountline->{amountoutstanding} = sprintf '%.2f', $accountline->{amountoutstanding};
- if ($accountline->{accounttype} eq 'Pay') {
+ if ($accountline->{accounttype} =~ /^Pay/) {
$accountline->{payment} = 1;
$reverse_col = 1;
}
diff --git a/members/printfeercpt.pl b/members/printfeercpt.pl
index 59d5cb3..de8cb7c 100755
--- a/members/printfeercpt.pl
+++ b/members/printfeercpt.pl
@@ -99,7 +99,7 @@ for (my $i=0;$i<$numaccts;$i++){
'amount' => sprintf("%.2f",$accts->[$i]{'amount'}),
'amountoutstanding' => sprintf("%.2f",$accts->[$i]{'amountoutstanding'}),
'accountno' => $accts->[$i]{'accountno'},
- 'payment' => ( $accts->[$i]{'accounttype'} eq 'Pay' ),
+ 'payment' => ( $accts->[$i]{'accounttype'} =~ /^Pay/ ),
);
diff --git a/members/printinvoice.pl b/members/printinvoice.pl
index 039132f..4c5b188 100755
--- a/members/printinvoice.pl
+++ b/members/printinvoice.pl
@@ -98,7 +98,7 @@ for (my $i=0;$i<$numaccts;$i++){
'amount' => sprintf("%.2f",$accts->[$i]{'amount'}),
'amountoutstanding' => sprintf("%.2f",$accts->[$i]{'amountoutstanding'}),
'accountno' => $accts->[$i]{'accountno'},
- 'payment' => ( $accts->[$i]{'accounttype'} eq 'Pay' ),
+ 'payment' => ( $accts->[$i]{'accounttype'} =~ /^Pay/ ),
);
--
1.7.4.1
More information about the Patches
mailing list