fix: skip sending message when null returned

laravel-notification-channels · Nov 12, 2019 · c1289b5 · c1289b5
1 parent f9feffd
commit c1289b5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/SmsBroadcastChannel.php b/src/SmsBroadcastChannel.php
@@ -27,6 +27,10 @@ public function send($notifiable, Notification $notification): void
             $message = new SmsBroadcastMessage($message);
         }
 
+        if (! $message instanceof SmsBroadcastMessage) {
+            return;
+        }
+
         $this->smsBroadcastClient->send(
             $to,
             $message->content,

diff --git a/src/SmsBroadcastServiceProvider.php b/src/SmsBroadcastServiceProvider.php
@@ -3,8 +3,8 @@
 namespace NotificationChannels\SmsBroadcast;
 
 use Atymic\SmsBroadcast\Api\Client;
-use Illuminate\Support\ServiceProvider;
 use Atymic\SmsBroadcast\Factory\ClientFactory;
+use Illuminate\Support\ServiceProvider;
 
 class SmsBroadcastServiceProvider extends ServiceProvider
 {