Stopping the propagation of an event in case no one subscribed for event

reference: https://laravel.com/docs/5.5/events#defining-listeners
herpaderpaldent · Mar 24, 2019 · c75a255 · c75a255
1 parent 9342777
commit c75a255
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/src/Listeners/GroupApplicationNotification.php b/src/Listeners/GroupApplicationNotification.php
@@ -67,7 +67,7 @@ public function handle(GroupApplication $event)
 
             if($recipients->isEmpty()){
                 logger()->debug('No Receiver found for ' . AbstractSeatGroupApplicationNotification::getTitle() . ' Notification. This job is going to be deleted.');
-                $this->delete();
+                return false;
             }
 
             $recipients->groupBy('driver')

diff --git a/src/Listeners/GroupSyncFailedNotification.php b/src/Listeners/GroupSyncFailedNotification.php
@@ -55,7 +55,7 @@ public function handle(GroupSyncFailed $event)
 
             if($recipients->isEmpty()){
                 logger()->debug('No Receiver found for ' . AbstractSeatGroupErrorNotification::getTitle() . ' Notification. This job is going to be deleted.');
-                $this->delete();
+                return false;
             }
 
             $recipients->groupBy('driver')

diff --git a/src/Listeners/GroupSyncedNotification.php b/src/Listeners/GroupSyncedNotification.php
@@ -60,7 +60,7 @@ public function handle(GroupSynced $event)
 
             if($recipients->isEmpty()){
                 logger()->debug('No Receiver found for ' . AbstractSeatGroupSyncNotification::getTitle() . ' Notification. This job is going to be deleted.');
-                $this->delete();
+                return false;
             }
 
             $recipients->groupBy('driver')

diff --git a/src/Listeners/MissingRefreshTokenNotification.php b/src/Listeners/MissingRefreshTokenNotification.php
@@ -62,7 +62,7 @@ public function handle(MissingRefreshToken $event)
 
             if($recipients->isEmpty()){
                 logger()->debug('No Receiver found for ' . AbstractMissingRefreshTokenNotification::getTitle() . ' Notification. This job is going to be deleted.');
-                $this->delete();
+                return false;
             }
 
             $recipients->groupBy('driver')