fix subscribed comment notifications for empty mails

diff --git a/docs/NEWS b/docs/NEWS
index 0b377951237aa99642f17d760e0d0471ad275c42..c147f6925e8e47b34340407465d521c8d9198a60 100644 (file)
--- a/docs/NEWS
+++ b/docs/NEWS
@@ -214,8 +214,14 @@ Version 1.1-alpha5()
 Version 1.0.1 ()
 ------------------------------------------------------------------------
 
-    * Chorizo Security Checks: Fixed possible browser-dependant attribute 
-      injection in the templatechooser-Plugin (garvinhicking)
+   * Fix bug sending comment-notification mails to subscribed users 
+     without an email address (garvinhicking)
+
+   * Chorizo Security Checks: Fixed possible browser-dependant attribute 
+     injection in the templatechooser-Plugin [NOT REALLY SURE YET IF THIS
+     INDEED WAS A BUG. NEEDS CLARITY] (garvinhicking)
+
+   * Fixed icelandic language bug preventing upgrade (garvinhicking)
 
 Version 1.0 (June 15th, 2006)
 ------------------------------------------------------------------------

diff --git a/include/functions_comments.inc.php b/include/functions_comments.inc.php
index d461f1d45675659ed82303e9cd7a46ae56fcd1f6..86ee43a28813822f4e32fcfed0ed687aa1a3c570 100644 (file)
--- a/include/functions_comments.inc.php
+++ b/include/functions_comments.inc.php
@@ -538,6 +538,7 @@ function serendipity_mailSubscribers($entry_id, $poster, $posterMail, $title, $f
             FROM {$serendipity['dbPrefix']}comments
             WHERE entry_id = '". (int)$entry_id ."'
               AND email <> '" . serendipity_db_escape_string($posterMail) . "'
+              AND email <> ''
               AND subscribed = 'true' $mysql_insert";
     $subscribers = serendipity_db_query($sql);