akaunting 3.0 (the last dance)

app/Console/Commands/InvoiceReminder.php

@@ -8,6 +8,7 @@ use App\Models\Document\Document;
 use App\Notifications\Sale\Invoice as Notification;
 use App\Utilities\Date;
 use Illuminate\Console\Command;
+use Illuminate\Database\Eloquent\Builder;
 
 class InvoiceReminder extends Command
 {
@@ -35,22 +36,29 @@ class InvoiceReminder extends Command
         // Disable model cache
         config(['laravel-model-caching.enabled' => false]);
 
+        $today = Date::today();
+
+        $start_date = $today->copy()->subMonth()->toDateString() . ' 00:00:00';
+        $end_date = $today->copy()->addWeek()->toDateString() . ' 23:59:59';
+
         // Get all companies
-        $companies = Company::enabled()->with('invoices')->cursor();
+        $companies = Company::whereHas('invoices', function (Builder $query) use ($start_date, $end_date) {
+                                $query->allCompanies();
+                                $query->whereBetween('due_at', [$start_date, $end_date]);
+                                $query->accrued();
+                                $query->notPaid();
+                            })
+                            ->enabled()
+                            ->cursor();
 
         foreach ($companies as $company) {
-            // Has company invoices
-            if (!$company->invoices->count()) {
-                continue;
-            }
-
             $this->info('Sending invoice reminders for ' . $company->name . ' company.');
 
             // Set company
             $company->makeCurrent();
 
             // Don't send reminders if disabled
-            if (!setting('schedule.send_invoice_reminder')) {
+            if (! setting('schedule.send_invoice_reminder')) {
                 $this->info('Invoice reminders disabled by ' . $company->name . '.');
 
                 continue;
@@ -74,9 +82,11 @@ class InvoiceReminder extends Command
         $date = Date::today()->subDays($day)->toDateString();
 
         // Get upcoming invoices
-        $invoices = Document::invoice()->with('contact')->accrued()->notPaid()->due($date)->cursor();
+        $invoices = Document::with('contact')->invoice()->accrued()->notPaid()->due($date)->cursor();
 
         foreach ($invoices as $invoice) {
+            $this->info($invoice->document_number . ' invoice reminded.');
+
             try {
                 event(new DocumentReminded($invoice, Notification::class));
             } catch (\Throwable $e) {