From ef822b116850b5312cb3aeafd8a2103a0cc3ddaf Mon Sep 17 00:00:00 2001
From: garvinhicking <garvinhicking>
Date: Wed, 12 Jul 2006 10:15:24 +0000
Subject: [PATCH] fix subscribed comment notifications for empty mails

---
 docs/NEWS                          | 6 +++++-
 include/functions_comments.inc.php | 1 +
 2 files changed, 6 insertions(+), 1 deletion(-)

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);
 
-- 
2.39.5