Double check for valid email address.
authorSteve Sutton <steve@gaslightmedia.com>
Mon, 5 Dec 2016 20:57:52 +0000 (15:57 -0500)
committerSteve Sutton <steve@gaslightmedia.com>
Mon, 5 Dec 2016 20:57:52 +0000 (15:57 -0500)
For admin email there maybe more than one. Explode the to emails and
check each one.

classes/helper/notification.php

index 33bbf05..1ee49fd 100644 (file)
@@ -93,6 +93,10 @@ class GlmMembersAdmin_coupon_notification extends GlmDataMembers
         if ( !$notify_to ) {
             return false;
         }
+        $validEmail = filter_var( $notify_to, FILTER_VALIDATE_EMAIL );
+        if ( !$validEmail ) {
+            return false;
+        }
 
         $memberInfoData = new GlmDataMemberInfo( $this->wpdb, $this->config );
         $memberInfo     = $memberInfoData->getActiveInfoForMember( $memberId );
@@ -155,6 +159,14 @@ class GlmMembersAdmin_coupon_notification extends GlmDataMembers
         if ( !$notify_to ) {
             return false;
         }
+        // Double check the email for valid address
+        $emails = explode( ',', $notify_to );
+        foreach ( $emails as $email ) {
+            $validEmail = filter_var( $email, FILTER_VALIDATE_EMAIL );
+            if ( !$validEmail ) {
+                return false;
+            }
+        }
 
         // Setup the Smarty Engine
         $smarty   = new smartyTemplateSupport();