Refactoring reminder_parse_subscribers()

diff --git a/reminder.module b/reminder.module
index b3a550a01fe4f2264db3bc3502bc8f987a267c91..1954a2151aadf424ae0a36936fcdf23db7320ac2 100644 (file)
--- a/reminder.module
+++ b/reminder.module
@@ -453,11 +453,18 @@ function reminder_send($nid) {
 /**
  * Helper function to parse subscribers.
  *
 /**
  * Helper function to parse subscribers.
  *

- * @todo
- *   Ignore blank lines.
+ * @see http://stackoverflow.com/questions/1028553/how-to-get-email-address-from-a-long-string

  */
 function reminder_parse_subscribers($subscribers) {
  */
 function reminder_parse_subscribers($subscribers) {

-  return explode("\n", $subscribers);
+  foreach(preg_split('/ |,|;|\n/', $subscribers) as $token) {
+    $email = filter_var(filter_var($token, FILTER_SANITIZE_EMAIL), FILTER_VALIDATE_EMAIL);
+
+    if ($email !== false) {
+      $emails[] = $email;
+    }
+  }
+
+  return $emails;

 }
 
 /**
 }
 
 /**