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

Merge pull request #4489 from turbo124/v5-develop

Browse source 
Fix for doubling client balance erroneously on emailing"
This commit is contained in:
David Bomba 2020-12-13 11:05:16 +11:00 • committed by GitHub
commit 5a66069833
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/Http/Controllers/EmailController.php
View file

@ -132,8 +132,6 @@ class EmailController extends BaseController
}
});
$entity_obj = $entity_obj->service()->markSent()->save();
$entity_obj->last_sent_date = now();
$entity_obj->save();

7
app/Services/Invoice/MarkSent.php
View file

@ -37,6 +37,9 @@ class MarkSent extends AbstractService
return $this->invoice;
}
info("in mark sent");
info($this->invoice->balance);
$this->invoice->markInvitationsSent();
$this->invoice->setReminder();
@ -49,8 +52,12 @@ class MarkSent extends AbstractService
->updateBalance($this->invoice->amount)
->save();
info($this->invoice->balance);
$this->client->service()->updateBalance($this->invoice->balance)->save();
info($this->client->balance);
$this->invoice->ledger()->updateInvoiceBalance($this->invoice->balance);
event(new InvoiceWasUpdated($this->invoice, $this->invoice->company, Ninja::eventVars()));