Emails/notifications not sending #675 (#686)

* Add test call to test notification #675

* Fix queue driver with emails not sending; formatting #675

app/Console/Commands/DevCommands.php

@@ -7,6 +7,7 @@ use App\Models\Acars;
 use App\Models\Airline;
 use App\Models\Pirep;
 use App\Models\User;
+use App\Notifications\Messages\UserRegistered;
 use App\Repositories\AcarsRepository;
 use App\Services\AirportService;
 use App\Services\AwardService;
@@ -60,6 +61,7 @@ class DevCommands extends Command
             'metar'             => 'getMetar',
             'recalculate-stats' => 'recalculateStats',
             'reset-install'     => 'resetInstall',
+            'new-user-email'    => 'newUserEmail',
             'xml-to-yaml'       => 'xmlToYaml',
         ];
 
@@ -290,6 +292,16 @@ class DevCommands extends Command
         $this->info('Done!');
     }
 
+    /**
+     * Test sending a user a registered email
+     */
+    protected function newUserEmail()
+    {
+        $user_id = $this->argument('param');
+        $user = User::find($user_id);
+        $user->notify(new UserRegistered($user));
+    }
+
     public function liveFlights(): void
     {
         $acarsRepo = app(AcarsRepository::class);

app/Notifications/BaseNotification.php

@@ -20,7 +20,7 @@ class BaseNotification extends Notification implements ShouldQueue
         $klass = get_class($this);
         $notif_config = config('notifications.channels', []);
         if (!array_key_exists($klass, $notif_config)) {
-            Log::error('Notification type '.$klass.' missing from notifications config');
+            Log::error('Notification type '.$klass.' missing from notifications config, defaulting to mail');
             return;
         }
 

app/Notifications/Channels/MailChannel.php

@@ -6,9 +6,9 @@ use Illuminate\Notifications\Messages\MailMessage;
 
 trait MailChannel
 {
-    private $mailSubject;
-    private $mailTemplate;
-    private $mailTemplateArgs;
+    protected $mailSubject;
+    protected $mailTemplate;
+    protected $mailTemplateArgs;
 
     /**
      * Set the arguments for the toMail() method

app/Notifications/Messages/AdminUserRegistered.php

@@ -10,6 +10,8 @@ class AdminUserRegistered extends BaseNotification
 {
     use MailChannel;
 
+    public $channels = ['mail'];
+
     private $user;
 
     /**

app/Notifications/Messages/NewsAdded.php

@@ -10,6 +10,8 @@ class NewsAdded extends BaseNotification
 {
     use MailChannel;
 
+    public $channels = ['mail'];
+
     private $news;
 
     public function __construct(News $news)

app/Notifications/Messages/PirepAccepted.php

@@ -13,6 +13,8 @@ class PirepAccepted extends BaseNotification
 {
     use MailChannel;
 
+    public $channels = ['mail'];
+
     private $pirep;
 
     /**

app/Notifications/Messages/PirepRejected.php

@@ -10,6 +10,8 @@ class PirepRejected extends BaseNotification
 {
     use MailChannel;
 
+    public $channels = ['mail'];
+
     private $pirep;
 
     /**

app/Notifications/Messages/PirepSubmitted.php

@@ -10,6 +10,8 @@ class PirepSubmitted extends BaseNotification
 {
     use MailChannel;
 
+    public $channels = ['mail'];
+
     private $pirep;
 
     /**

app/Notifications/Messages/UserPending.php

@@ -10,6 +10,8 @@ class UserPending extends BaseNotification
 {
     use MailChannel;
 
+    public $channels = ['mail'];
+
     private $user;
 
     /**
@@ -17,6 +19,8 @@ class UserPending extends BaseNotification
      */
     public function __construct(User $user)
     {
+        parent::__construct();
+
         $this->user = $user;
 
         $this->setMailable(

app/Notifications/Messages/UserRegistered.php

@@ -10,6 +10,8 @@ class UserRegistered extends BaseNotification
 {
     use MailChannel;
 
+    public $channels = ['mail'];
+
     private $user;
 
     /**

app/Notifications/Messages/UserRejected.php

@@ -10,6 +10,8 @@ class UserRejected extends BaseNotification
 {
     use MailChannel;
 
+    public $channels = ['mail'];
+
     private $user;
 
     /**

config/queue.php

@@ -14,7 +14,7 @@ return [
     |
     */
 
-    'default' => env('QUEUE_DRIVER', 'database'),
+    'default' => env('QUEUE_DRIVER', 'sync'),
 
     /*
     |--------------------------------------------------------------------------

modules/Importer/Services/BaseImporter.php

@@ -5,14 +5,13 @@ namespace Modules\Importer\Services;
 use App\Services\Installer\LoggerTrait;
 use Carbon\Carbon;
 use Illuminate\Bus\Queueable;
-use Illuminate\Contracts\Queue\ShouldQueue;
 use Illuminate\Foundation\Bus\Dispatchable;
 use Illuminate\Queue\InteractsWithQueue;
 use Illuminate\Support\Facades\Log;
 use Modules\Importer\Utils\IdMapper;
 use Modules\Importer\Utils\ImporterDB;
 
-abstract class BaseImporter implements ShouldQueue
+abstract class BaseImporter
 {
     use LoggerTrait;
     use Dispatchable;

resources/stubs/modules/mail.stub

@@ -2,10 +2,8 @@
 
 namespace $NAMESPACE$;
 
-use Illuminate\Bus\Queueable;
 use Illuminate\Mail\Mailable;
 use Illuminate\Queue\SerializesModels;
-use Illuminate\Contracts\Queue\ShouldQueue;
 
 /**
  * Class $CLASS$
@@ -13,7 +11,7 @@ use Illuminate\Contracts\Queue\ShouldQueue;
  */
 class $CLASS$ extends Mailable
 {
-    use Queueable, SerializesModels;
+    use SerializesModels;
 
     /**
      * Create a new message instance.