Attachment 95153 Details for Bug 23806 – Bug 23806: FinePaymentAutoPopup does not trigger pop-up for writeoff by "Write off" button

[patch] Bug 23806: FinePaymentAutoPopup does not trigger pop-up for writeoff by "Write off" button

Bug-23806-FinePaymentAutoPopup-does-not-trigger-po.patch (text/plain), 1.24 KB, created by Jonathan Druart on 2019-11-07 12:53:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2019-11-07 12:53:08 UTC

Size: 1.24 KB

patch

obsolete

>From 1956ac65cdb3bfdd214ea504f39178ee294d6547 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Fri, 11 Oct 2019 09:07:14 -0700
>Subject: [PATCH] Bug 23806: FinePaymentAutoPopup does not trigger pop-up for
> writeoff by "Write off" button
>
>Test Plan:
>1) Enable reciept popups for payments and writeoffs
>2) Test the "Write off" button, no popup
>3) Apply this patch
>4) Restart all the things!
>5) Test again, you should get the popup!
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>Signed-off-by: Sean McGarvey <seanm@pascolibraries.org>
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> members/pay.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/members/pay.pl b/members/pay.pl
>index 32c8c36dec..5094a5f581 100755
>--- a/members/pay.pl
>+++ b/members/pay.pl
>@@ -112,7 +112,7 @@ elsif ( $input->param('confirm_writeoff') ) {
>               . "&error_over=1" );
> 
>     } else {
>-        Koha::Account->new( { patron_id => $borrowernumber } )->pay(
>+        $payment_id = Koha::Account->new( { patron_id => $borrowernumber } )->pay(
>             {
>                 amount     => $amount,
>                 lines      => [ scalar Koha::Account::Lines->find($accountlines_id) ],
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 23806: 94025 | 94273 | 95153