changeset 418:20af29454e83

Fixing #202. Guard against an empty email recipient list to prevent mail errors. We log a warning if this happens.
author Brian Neal <bgneal@gmail.com>
date Sat, 16 Apr 2011 21:39:23 +0000
parents 538a1bd2f1f4
children fc2feaa98c58
files gpp/core/functions.py
diffstat 1 files changed, 4 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/gpp/core/functions.py	Sat Apr 16 19:30:29 2011 +0000
+++ b/gpp/core/functions.py	Sat Apr 16 21:39:23 2011 +0000
@@ -17,6 +17,10 @@
     Use this function to send email from the site. It will obey debug settings and
     log all emails. The expedite flag, when True, will bypass the mail queue.
     """
+    recipient_list = [dest for dest in recipient_list if dest]
+    if not recipient_list:
+        logging.warning("Empty recipient_list in send_mail")
+        return
 
     if settings.MAILER_ENQUEUE_MAIL and not expedite:
         mailer.enqueue_mail(subject, message, from_email, recipient_list)