From 2eb61b08cb4e3eaf6e7a069c7fbee7a61eaed643 Mon Sep 17 00:00:00 2001 From: David Bomba Date: Sat, 3 Dec 2022 09:15:27 +1100 Subject: [PATCH] Do not send reminders to trashed contacts --- app/Jobs/Util/ReminderJob.php | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/app/Jobs/Util/ReminderJob.php b/app/Jobs/Util/ReminderJob.php index 2ff606b4ce21..e5a5d71bf274 100644 --- a/app/Jobs/Util/ReminderJob.php +++ b/app/Jobs/Util/ReminderJob.php @@ -148,8 +148,12 @@ class ReminderJob implements ShouldQueue (Ninja::isSelfHost() || $invoice->company->account->isPaidHostedClient())) { $invoice->invitations->each(function ($invitation) use ($invoice, $reminder_template) { - EmailEntity::dispatch($invitation, $invitation->company, $reminder_template)->delay(now()->addSeconds(3)); - nlog("Firing reminder email for invoice {$invoice->number} - {$reminder_template}"); + + if(!$invitation->contact->trashed() && $invitation->contact->email) { + EmailEntity::dispatch($invitation, $invitation->company, $reminder_template)->delay(now()->addSeconds(3)); + nlog("Firing reminder email for invoice {$invoice->number} - {$reminder_template}"); + } + }); if ($invoice->invitations->count() > 0) {