Attachment 119195 Details for Bug 24300 – Bug 24300: Add payout_amount method to Koha::Account

[patch] Bug 24300: Add payout_amount method to Koha::Account

Bug-24300-Add-payoutamount-method-to-KohaAccount.patch (text/plain), 4.02 KB, created by Martin Renvoize (ashimema) on 2021-04-06 11:06:34 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2021-04-06 11:06:34 UTC

Size: 4.02 KB

patch

obsolete

>From 4b2401276a03b511996712635ee670195eac6368 Mon Sep 17 00:00:00 2001
>From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>Date: Fri, 5 Feb 2021 14:28:32 +0000
>Subject: [PATCH] Bug 24300: Add payout_amount method to Koha::Account
>
>This patch adds a 'payout_amount' method to the Koha::Account class to
>allow payout of an amount against a list of credits (or any outstanding
>credits, oldest to newest).
>
>Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
>---
> Koha/Account.pm | 95 +++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 95 insertions(+)
>
>diff --git a/Koha/Account.pm b/Koha/Account.pm
>index 55ee594ff8..3ff37bc674 100644
>--- a/Koha/Account.pm
>+++ b/Koha/Account.pm
>@@ -627,6 +627,101 @@ sub add_debit {
>     return $line;
> }
> 
>+=head3 payout_amount
>+
>+    my $debit = $account->payout_amount(
>+        {
>+            payout_type => $payout_type,
>+            register_id => $register_id,
>+            staff_id    => $staff_id,
>+            interface   => 'intranet',
>+            amount      => $amount,
>+            credits     => $credit_lines
>+        }
>+    );
>+
>+This method allows an amount to be paid out from a patrons account against outstanding credits.
>+
>+$payout_type can be any of the defined payment_types:
>+
>+=cut
>+
>+sub payout_amount {
>+    my ( $self, $params ) = @_;
>+
>+    # Check for mandatory parameters
>+    my @mandatory =
>+      ( 'interface', 'staff_id', 'branch', 'payout_type', 'amount' );
>+    for my $param (@mandatory) {
>+        unless ( defined( $params->{$param} ) ) {
>+            Koha::Exceptions::MissingParameter->throw(
>+                error => "The $param parameter is mandatory" );
>+        }
>+    }
>+
>+    # Check for mandatory register
>+    Koha::Exceptions::Account::RegisterRequired->throw()
>+      if ( C4::Context->preference("UseCashRegisters")
>+        && ( $params->{payout_type} eq 'CASH' )
>+        && !defined($params->{cash_register}) );
>+
>+    # Amount should always be passed as a positive value
>+    my $amount = $params->{amount};
>+    unless ( $amount > 0 ) {
>+        Koha::Exceptions::Account::AmountNotPositive->throw(
>+            error => 'Debit amount passed is not positive' );
>+    }
>+
>+    # Amount should always be less than or equal to outstanding credit
>+    my $outstanding = 0;
>+    my $outstanding_credits =
>+      exists( $params->{credits} )
>+      ? $params->{credits}
>+      : $self->outstanding_credits->as_list;
>+    for my $credit ( @{$outstanding_credits} ) {
>+        $outstanding += $credit->amountoutstanding;
>+    }
>+    $outstanding = $outstanding * -1;
>+    Koha::Exceptions::ParameterTooHigh->throw( error =>
>+"Amount to payout ($amount) is higher than amountoutstanding ($outstanding)"
>+    ) unless ( $outstanding >= $amount );
>+
>+    my $payout;
>+    my $schema = Koha::Database->new->schema;
>+    $schema->txn_do(
>+        sub {
>+
>+            # A 'payout' is a 'debit'
>+            $payout = Koha::Account::Line->new(
>+                {
>+                    date              => \'NOW()',
>+                    amount            => $amount,
>+                    debit_type_code   => 'PAYOUT',
>+                    payment_type      => $params->{payout_type},
>+                    amountoutstanding => $amount,
>+                    manager_id        => $params->{staff_id},
>+                    borrowernumber    => $self->{patron_id},
>+                    interface         => $params->{interface},
>+                    branchcode        => $params->{branch},
>+                    register_id       => $params->{cash_register}
>+                }
>+            )->store();
>+
>+            # Offset against credits
>+            for my $credit ( @{$outstanding_credits} ) {
>+                $credit->apply(
>+                    { debits => [$payout], offset_type => 'PAYOUT' } );
>+                $payout->discard_changes;
>+            }
>+
>+            # Set payout as paid
>+            $payout->status('PAID')->store;
>+        }
>+    );
>+
>+    return $payout;
>+}
>+
> =head3 balance
> 
> my $balance = $self->balance
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 24300: 116395 | 116412 | 116413 | 116837 | 116838 | 117917 | 117918 | 117919 | 118335 | 118392 | 118393 | 118394 | 118395 | 118396 | 118464 | 118503 | 118601 | 118602 | 118603 | 118604 | 118605 | 118606 | 118612 | 119195 | 119196 | 119197 | 119198 | 119199 | 119200 | 120016 | 120017 | 120018 | 120019 | 120020 | 120021