Merge pull request #6021 from turbo124/v5-develop

Fixes for reminders
This commit is contained in:
David Bomba 2021-06-15 08:20:11 +10:00 committed by GitHub
commit 35237849ff
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 14 additions and 6 deletions

View File

@ -53,7 +53,11 @@ class ReminderJob implements ShouldQueue
private function processReminders() private function processReminders()
{ {
Invoice::whereDate('next_send_date', '<=', now())->with('invitations')->cursor()->each(function ($invoice) { Invoice::whereDate('next_send_date', '<=', now())
->where('is_deleted', 0)
->whereIn('status_id', [Invoice::STATUS_SENT, Invoice::STATUS_PARTIAL])
->where('balance', '>', 0)
->with('invitations')->cursor()->each(function ($invoice) {
if ($invoice->isPayable()) { if ($invoice->isPayable()) {
$reminder_template = $invoice->calculateTemplate('invoice'); $reminder_template = $invoice->calculateTemplate('invoice');

View File

@ -58,7 +58,7 @@ class UpdatePaymentMethods
// } // }
private function updateMethods(Customer $customer, Client $client) public function updateMethods(Customer $customer, Client $client)
{ {
$card_methods = PaymentMethod::all([ $card_methods = PaymentMethod::all([
'customer' => $customer->id, 'customer' => $customer->id,
@ -145,7 +145,7 @@ class UpdatePaymentMethods
} }
private function buildPaymentMethodMeta(PaymentMethod $method, GatewayType $type_id) private function buildPaymentMethodMeta(PaymentMethod $method, $type_id)
{ {
switch ($type_id) { switch ($type_id) {

View File

@ -392,7 +392,8 @@ class InvoiceService
$this->invoice->reminder_last_sent = now()->format('Y-m-d'); $this->invoice->reminder_last_sent = now()->format('Y-m-d');
break; break;
default: default:
// code... $this->invoice->reminder1_sent = now()->format('Y-m-d');
$this->invoice->reminder_last_sent = now()->format('Y-m-d');
break; break;
} }

View File

@ -126,8 +126,11 @@ class UpdateReminder extends AbstractService
$date_collection->push($reminder_date); $date_collection->push($reminder_date);
} }
$this->invoice->next_send_date = $date_collection->sort()->first(); if($date_collection->count() >=1)
$this->invoice->next_send_date = $date_collection->sort()->first();
else
$this->invoice->next_send_date = null;
return $this->invoice; return $this->invoice;
} }
} }