Issue 435 - Customer note notification : Dont send notification email…

… if customer has no email (#436) * [IM] - Customer note notification : Dont send notification email if customer has no email - fixes #435 * [IM] - Customer note notification : Dont send notification email if customer has no email - fixes #435 * [IM] - Customer note notification : Dont send notification email if customer has no email - fixes #435
inex · Jul 13, 2018 · 5183753 · 5183753
1 parent ead1e81
commit 5183753
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/app/Listeners/Customer/Note/EmailOnChange.php b/app/Listeners/Customer/Note/EmailOnChange.php
@@ -116,6 +116,10 @@ public function handle( $e )
                     continue;
                 }
 
+                if( !$user->getContact()->getEmail() || filter_var( $user->getContact()->getEmail() , FILTER_VALIDATE_EMAIL ) === false ) {
+                    continue;
+                }
+
                 if( !$user->getPreference( "customer-notes.notify" ) || $user->getPreference( "customer-notes.notify" ) == "default" || $user->getPreference( "customer-notes.notify" ) == "all" ) {
                     $to[] = [ 'name' => $user->getContact()->getName(), 'email' => $user->getContact()->getEmail() ];
                     continue;