diff --git a/app/Console/Commands/SendDigest.php b/app/Console/Commands/SendDigest.php index 056b4cf..b08d434 100644 --- a/app/Console/Commands/SendDigest.php +++ b/app/Console/Commands/SendDigest.php @@ -41,34 +41,38 @@ class SendDigest extends Command $test_address = $this->option('test') ? [config('app.test_address')] : null; - // These are seperated because I may want to send multiple types - // of digest in a single commend. - if ($this->option('daily')) { - foreach (($test_address ?? config('app.daily_digest_recipients')) as $recipient) { - Log::debug("Daily digest email sent to '{$recipient}'."); - Mail::to($recipient)->send(new Digest('daily', config('app.current_trip_id'))); + if (config('app.current_trip_id')) { + // These are seperated because I may want to send multiple types + // of digest in a single command. + if ($this->option('daily')) { + foreach (($test_address ?? config('app.daily_digest_recipients')) as $recipient) { + Log::debug("Daily digest email sent to '{$recipient}'."); + Mail::to($recipient)->send(new Digest('daily', config('app.current_trip_id'))); + } } - } - if ($this->option('weekly')) { - foreach (($test_address ?? config('app.weekly_digest_recipients')) as $recipient) { - Log::debug("Weekly digest email sent to '{$recipient}'."); - Mail::to($recipient)->send(new Digest('weekly', config('app.current_trip_id'))); + if ($this->option('weekly')) { + foreach (($test_address ?? config('app.weekly_digest_recipients')) as $recipient) { + Log::debug("Weekly digest email sent to '{$recipient}'."); + Mail::to($recipient)->send(new Digest('weekly', config('app.current_trip_id'))); + } } - } - if ($this->option('fortnightly')) { - foreach (($test_address ?? config('app.fortnightly_digest_recipients')) as $recipient) { - Log::debug("Fortnightly digest email sent to '{$recipient}'."); - Mail::to($recipient)->send(new Digest('fortnightly', config('app.current_trip_id'))); + if ($this->option('fortnightly')) { + foreach (($test_address ?? config('app.fortnightly_digest_recipients')) as $recipient) { + Log::debug("Fortnightly digest email sent to '{$recipient}'."); + Mail::to($recipient)->send(new Digest('fortnightly', config('app.current_trip_id'))); + } } - } - if ($this->option('monthly')) { - foreach (($test_address ?? config('app.monthly_digest_recipients')) as $recipient) { - Log::debug("Monthly digest email sent to '{$recipient}'."); - Mail::to($recipient)->send(new Digest('monthly', config('app.current_trip_id'))); + if ($this->option('monthly')) { + foreach (($test_address ?? config('app.monthly_digest_recipients')) as $recipient) { + Log::debug("Monthly digest email sent to '{$recipient}'."); + Mail::to($recipient)->send(new Digest('monthly', config('app.current_trip_id'))); + } } + } else { + Log::debug("No trip currently running, no digests sent."); } } }