From 22249e4ceda925792a62c203b16bac3c4e35a1af Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Mon, 11 Jul 2016 15:36:54 +0000 Subject: [PATCH] Bug 16899: Add ability to disallow overpayments Some SIP services ( such as Comprise ) require that an attempt at over-paying a patron's account via SIP2 should fail, rather than create a credit on the account. We should make this a configurable option on a per-login basis in the SIP2 config file. Test Plan: 1) Apply this patch 2) Enable the new parameter disallow_overpayment="1" for the login to be used in this test. 3) Restart your SIP server 4) Create or find a patron with fines 5) Attempt to send a payment via SIP for more than what the patron's balance is 6) Note the response indicates a payment failure 7) Attempt to send a payment via SIP for the account balance or less 8) Note the response indicates the payment has succeeded 9) Verify in Koha that the payment was processed Signed-off-by: Rhonda Kuiper Signed-off-by: Marcel de Rooy Signed-off-by: Jonathan Druart --- C4/SIP/ILS.pm | 4 ++-- C4/SIP/ILS/Transaction/FeePayment.pm | 17 +++++++++++------ C4/SIP/Sip/MsgType.pm | 4 +++- etc/SIPconfig.xml | 2 +- 4 files changed, 17 insertions(+), 10 deletions(-) diff --git a/C4/SIP/ILS.pm b/C4/SIP/ILS.pm index 18b886b237..e0f5300fe1 100644 --- a/C4/SIP/ILS.pm +++ b/C4/SIP/ILS.pm @@ -250,7 +250,7 @@ sub end_patron_session { } sub pay_fee { - my ($self, $patron_id, $patron_pwd, $fee_amt, $fee_type, $pay_type, $fee_id, $trans_id, $currency, $is_writeoff) = @_; + my ($self, $patron_id, $patron_pwd, $fee_amt, $fee_type, $pay_type, $fee_id, $trans_id, $currency, $is_writeoff, $disallow_overpayment ) = @_; my $trans = C4::SIP::ILS::Transaction::FeePayment->new(); @@ -261,7 +261,7 @@ sub pay_fee { $trans->screen_msg('Invalid patron barcode.'); return $trans; } - my $ok = $trans->pay( $patron->{borrowernumber}, $fee_amt, $pay_type, $fee_id, $is_writeoff ); + my $ok = $trans->pay( $patron->{borrowernumber}, $fee_amt, $pay_type, $fee_id, $is_writeoff, $disallow_overpayment ); $trans->ok($ok); return $trans; diff --git a/C4/SIP/ILS/Transaction/FeePayment.pm b/C4/SIP/ILS/Transaction/FeePayment.pm index fe491979fd..41ed5c7ccb 100644 --- a/C4/SIP/ILS/Transaction/FeePayment.pm +++ b/C4/SIP/ILS/Transaction/FeePayment.pm @@ -42,12 +42,13 @@ sub new { } sub pay { - my $self = shift; - my $borrowernumber = shift; - my $amt = shift; - my $sip_type = shift; - my $fee_id = shift; - my $is_writeoff = shift; + my $self = shift; + my $borrowernumber = shift; + my $amt = shift; + my $sip_type = shift; + my $fee_id = shift; + my $is_writeoff = shift; + my $disallow_overpayment = shift; my $type = $is_writeoff ? 'writeoff' : undef; @@ -55,6 +56,10 @@ sub pay { my $account = Koha::Account->new( { patron_id => $borrowernumber } ); + if ($disallow_overpayment) { + return 0 if $account->balance < $amt; + } + if ($fee_id) { my $fee = Koha::Account::Lines->find($fee_id); if ( $fee ) { diff --git a/C4/SIP/Sip/MsgType.pm b/C4/SIP/Sip/MsgType.pm index dded51e22c..db451224a8 100644 --- a/C4/SIP/Sip/MsgType.pm +++ b/C4/SIP/Sip/MsgType.pm @@ -1063,7 +1063,9 @@ sub handle_fee_paid { my $status; my $resp = FEE_PAID_RESP; + my $disallow_overpayment = $server->{account}->{disallow_overpayment}; my $payment_type_writeoff = $server->{account}->{payment_type_writeoff} || q{}; + my $is_writeoff = $pay_type eq $payment_type_writeoff; $fee_amt = $fields->{ (FID_FEE_AMT) }; @@ -1075,7 +1077,7 @@ sub handle_fee_paid { $ils->check_inst_id( $inst_id, "handle_fee_paid" ); - $status = $ils->pay_fee( $patron_id, $patron_pwd, $fee_amt, $fee_type, $pay_type, $fee_id, $trans_id, $currency, $is_writeoff ); + $status = $ils->pay_fee( $patron_id, $patron_pwd, $fee_amt, $fee_type, $pay_type, $fee_id, $trans_id, $currency, $is_writeoff, $disallow_overpayment ); $resp .= ( $status->ok ? 'Y' : 'N' ) . timestamp; $resp .= add_field( FID_INST_ID, $inst_id ); diff --git a/etc/SIPconfig.xml b/etc/SIPconfig.xml index f51f0938b5..01334f50bb 100644 --- a/etc/SIPconfig.xml +++ b/etc/SIPconfig.xml @@ -44,7 +44,7 @@ - + -- 2.39.5