From 686e84172e7eda2dea4b187089d77524a69ac4e8 Mon Sep 17 00:00:00 2001
From: Silvio Rhatto <rhatto@riseup.net>
Date: Tue, 18 Dec 2012 18:02:58 -0200
Subject: Refactoring reminder_parse_subscribers()

---
 reminder.module | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/reminder.module b/reminder.module
index b3a550a..1954a21 100644
--- a/reminder.module
+++ b/reminder.module
@@ -453,11 +453,18 @@ function reminder_send($nid) {
 /**
  * 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) {
-  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;
 }
 
 /**
-- 
cgit v1.2.3