Attachment 122588 Details for Bug 28663 – Bug 28663: Prevent discounts against void lines

[patch] Bug 28663: Prevent discounts against void lines

Bug-28663-Prevent-discounts-against-void-lines.patch (text/plain), 2.39 KB, created by Martin Renvoize (ashimema) on 2021-07-05 18:40:27 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2021-07-05 18:40:27 UTC

Size: 2.39 KB

patch

obsolete

>From b7140a855c5f8f06593f4731b115287e453f4296 Mon Sep 17 00:00:00 2001
>From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>Date: Mon, 5 Jul 2021 19:37:57 +0100
>Subject: [PATCH] Bug 28663: Prevent discounts against void lines
>
>With the introduction of double entry accounting for VOID actions, we
>need to add an additional filter to the 'Apply discount' button appearance
>
>Test plan
>1/ Add a debt
>2/ Pay the debt
>3/ Void the payment
>4/ Confirm that with the patch applied the 'Apply discount' button does
>not appear on the 'Void' accountline.
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/members/boraccount.tt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/boraccount.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/boraccount.tt
>index fd4484aeaa..7df3ac933c 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/boraccount.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/boraccount.tt
>@@ -133,7 +133,7 @@
>         [% IF CAN_user_updatecharges_refund && account.is_debit && ( account.amountoutstanding != account.amount ) && !(account.status == 'REFUNDED') && !(account.debit_type_code == 'PAYOUT') %]
>           <button type="button"  class="btn btn-default btn-xs" data-toggle="modal" data-target="#issueRefundModal" data-item="[%- PROCESS account_type_description account=account -%]" data-accountline="[% account.accountlines_id | html %]" data-amount="[% account.amount | $Price %]" data-amountoutstanding="[% account.amountoutstanding | $Price %]" data-member="[% account.borrowernumber | html %]"><i class="fa fa-money"></i> Issue refund</button>
>         [% END %]
>-        [% IF CAN_user_updatecharges_discount && account.is_debit && ( account.amountoutstanding == account.amount ) && !(account.debit_type_code == 'PAYOUT') %]
>+        [% IF CAN_user_updatecharges_discount && account.is_debit && ( account.amountoutstanding == account.amount ) && !(account.debit_type_code == 'PAYOUT') && !(account.debit_type_code == 'VOID') %]
>           <button type="button" data-toggle="modal" data-target="#applyDiscountModal" data-item="[%- PROCESS account_type_description account=account -%]" data-accountline="[% account.accountlines_id | html %]" data-amount="[% account.amount | $Price %]" data-amountoutstanding="[% account.amountoutstanding | $Price %]" class="btn btn-default btn-xs">Apply discount</button>
>         [% END %]
>       </td>
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 28663: 122588 | 127519 | 127538 | 130772 | 130927