Fixed count

Signed-off-by: snipe <snipe@snipe.net>
This commit is contained in:
snipe 2025-02-27 14:22:40 +00:00
parent b8799f8038
commit cd1d1b2d3e

View file

@ -70,23 +70,26 @@ class SendAcceptanceReminder extends Command
// The [0] is weird, but it allows for the item_count to work and grabs the appropriate info for each user. // The [0] is weird, but it allows for the item_count to work and grabs the appropriate info for each user.
// Collapsing and flattening the collection doesn't work above. // Collapsing and flattening the collection doesn't work above.
$acceptance = $unacceptedAssetGroup[0]['acceptance']; $acceptance = $unacceptedAssetGroup[0]['acceptance'];
$locale = $acceptance->assignedTo?->locale; $locale = $acceptance->assignedTo?->locale;
$email = $acceptance->assignedTo?->email; $email = $acceptance->assignedTo?->email;
if(!$email){ if(!$email){
$no_email_list[] = [ $no_email_list[] = [
'id' => $acceptance->assignedTo->id, 'id' => $acceptance->assignedTo?->id,
'name' => $acceptance->assignedTo->present()->fullName(), 'name' => $acceptance->assignedTo?->present()->fullName(),
]; ];
} else {
$count++;
} }
$item_count = $unacceptedAssetGroup->count(); $item_count = $unacceptedAssetGroup->count();
if ($locale && $email) { if ($locale && $email) {
Mail::to($email)->send((new UnacceptedAssetReminderMail($acceptance, $item_count))->locale($locale)); Mail::to($email)->send((new UnacceptedAssetReminderMail($acceptance, $item_count))->locale($locale));
} elseif ($email) { } elseif ($email) {
Mail::to($email)->send((new UnacceptedAssetReminderMail($acceptance, $item_count))); Mail::to($email)->send((new UnacceptedAssetReminderMail($acceptance, $item_count)));
} }
$count++;
} }
$this->info($count.' users notified.'); $this->info($count.' users notified.');