tim/invoiceninja
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fixes for incorrect fee calculation edge case

Browse source
This commit is contained in:
David Bomba 2021-09-22 11:57:03 +10:00
parent 0c89f998eb
commit a66fa1c678
1 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
app/Services/Invoice/AutoBillInvoice.php
View file

@ -70,7 +70,7 @@ class AutoBillInvoice extends AbstractService
/* Determine $amount */
if ($this->invoice->partial > 0) {
$is_partial = true;
$invoice_total = $this->invoice->amount;
$invoice_total = $this->invoice->balance;
$amount = $this->invoice->partial;
} elseif ($this->invoice->balance > 0) {
$amount = $this->invoice->balance;
@ -94,10 +94,14 @@ class AutoBillInvoice extends AbstractService
/* $gateway fee */
$this->invoice = $this->invoice->service()->addGatewayFee($gateway_token->gateway, $gateway_token->gateway_type_id, $amount)->save();
//change from $this->invoice->amount to $this->invoice->balance
if($is_partial)
$fee = $this->invoice->amount - $invoice_total;
$fee = $this->invoice->balance - $invoice_total;
else
$fee = $this->invoice->amount - $amount;
$fee = $this->invoice->balance - $amount;
if($fee > $amount)
$fee = 0;
/* Build payment hash */
$payment_hash = PaymentHash::create([