From dcda5a4dc52eebe621ccfd491eddbb162c476ebb Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Wed, 30 Aug 2017 10:35:03 -0400 Subject: [PATCH] Bug 19208 - Pay select option doesn't pay the selected fine MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit The "Pay selected" option on the Fines tab in the borrower account page doesn't work as intended. The fine on top of the list gets the amount deducted, even if another fine is choosen from the list. Test Plan: 1) Create two or three fines, using the Create manual invoice function. 2) Choose one of the fines (not the one on the top) and click Pay selected 3) Pay a partial amount 4) Go back to the Pay fines tab an notice that the fine you selected has not changed. Instead, either the top fine or the total (see attachment) has ben affected. 5) Apply this patch 6) Repeat steps 1-3 7) Note the correct fine is paid Signed-off-by: Andreas Hedström Mace Signed-off-by: Julian Maurice Signed-off-by: Jonathan Druart (cherry picked from commit 831d75d09c276f25f919cbdc16c8e4105fe8ddc9) Signed-off-by: Fridolin Somers --- .../intranet-tmpl/prog/en/modules/members/paycollect.tt | 2 -- members/paycollect.pl | 5 +---- 2 files changed, 1 insertion(+), 6 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/paycollect.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/paycollect.tt index 1e317bc816..a1458b58f1 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/paycollect.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/paycollect.tt @@ -110,7 +110,6 @@ function moneyFormat(textObj) { - @@ -167,7 +166,6 @@ function moneyFormat(textObj) { - diff --git a/members/paycollect.pl b/members/paycollect.pl index 26236490a9..bc54099283 100755 --- a/members/paycollect.pl +++ b/members/paycollect.pl @@ -61,7 +61,6 @@ my $writeoff = $input->param('writeoff_individual'); my $select_lines = $input->param('selected'); my $select = $input->param('selected_accts'); my $payment_note = uri_unescape $input->param('payment_note'); -my $accountno; my $accountlines_id; if ( $individual || $writeoff ) { @@ -74,7 +73,6 @@ if ( $individual || $writeoff ) { $accountlines_id = $input->param('accountlines_id'); my $amount = $input->param('amount'); my $amountoutstanding = $input->param('amountoutstanding'); - $accountno = $input->param('accountno'); my $itemnumber = $input->param('itemnumber'); my $description = $input->param('description'); my $title = $input->param('title'); @@ -84,7 +82,6 @@ if ( $individual || $writeoff ) { $template->param( accounttype => $accounttype, accountlines_id => $accountlines_id, - accountno => $accountno, amount => $amount, amountoutstanding => $amountoutstanding, title => $title, @@ -134,7 +131,7 @@ if ( $total_paid and $total_paid ne '0.00' ) { { borrowernumber => $borrowernumber, amountoutstanding => { '<>' => 0 }, - accountno => { 'IN' => \@acc }, + accountlines_id => { 'IN' => \@acc }, }, { order_by => 'date' } ); -- 2.39.5