From: garvinhicking <garvinhicking>
Date: Wed, 12 Jul 2006 10:15:24 +0000 (+0000)
Subject: fix subscribed comment notifications for empty mails
X-Git-Tag: 1.0.1~11
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=ef822b116850b5312cb3aeafd8a2103a0cc3ddaf;p=s9y.git

fix subscribed comment notifications for empty mails
---

diff --git a/docs/NEWS b/docs/NEWS
index 628bfdc..9af9731 100644
--- 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 8aab683..ea1bb8f 100644
--- 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);