fix subscribed comment notifications for empty mails

diff --git a/docs/NEWS b/docs/NEWS
index 628bfdcaeb4f46726e7592b897894009b6da226d..9af973144e6c75db71bfc8e41f3ae36cf486bb41 100644 (file)
--- a/docs/NEWS
+++ b/docs/NEWS
@@ -3,8 +3,12 @@
 Version 1.0.1 ()
 ------------------------------------------------------------------------
    
+   * 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 (garvinhicking)
+     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)
 

diff --git a/include/functions_comments.inc.php b/include/functions_comments.inc.php
index 8aab6835d069449c0d1c970fb2f4b50bdd104705..ea1bb8fc6c0f61dd71599cc01e36f6c9a171f715 100644 (file)
--- a/include/functions_comments.inc.php
+++ b/include/functions_comments.inc.php
@@ -516,6 +516,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);