[Gforge-commits] gforge/cronjobs massmail.php,1.13,1.14

[tom]

Update of /cvsroot/gforge/gforge/cronjobs
In directory db.perdue.net:/tmp/cvs-serv1167/cronjobs

Modified Files:
        massmail.php 
Log Message:
Fixed bug #837: Mass mailing sends duplicate emails.  This won't fix all cases 
- for example, if a person signs up for two different accounts, and starts a 
new project with each, he'll get two emails because the SELECT DISTINCT checks 
the users.confirmation_hash field.  But this will catch the most common cases...

Index: massmail.php
===================================================================
RCS file: /cvsroot/gforge/gforge/cronjobs/massmail.php,v
retrieving revision 1.13
retrieving revision 1.14
diff -u -d -r1.13 -r1.14
--- massmail.php        16 Jun 2004 15:52:37 -0000      1.13
+++ massmail.php        20 Jul 2004 18:54:09 -0000      1.14
@@ -103,7 +103,7 @@
 
 //$err .= "Got mail to send: ".$subj."\n";
 
-$sql = "SELECT 
users.user_id,users.user_name,users.realname,users.email,users.confirm_hash
+$sql = "SELECT DISTINCT 
users.user_id,users.user_name,users.realname,users.email,users.confirm_hash
        FROM $table_mapping[$type]
        WHERE users.user_id>".db_result($mail_res, 0, 'last_userid')."
        AND users.status='A'