[Collective-checkins] r245063 - Products.EasyNewsletter/branches/bpi-rouet-2011-09/Products/EasyNewsletter/browser

[Régis R. <svn...@pl...>]

Author: regisrouet
Date: Tue Oct  4 15:35:38 2011
New Revision: 245063

Modified:
   Products.EasyNewsletter/branches/bpi-rouet-2011-09/Products/EasyNewsletter/browser/registration.py
Log:
better comment

Modified: Products.EasyNewsletter/branches/bpi-rouet-2011-09/Products/EasyNewsletter/browser/registration.py
==============================================================================
--- Products.EasyNewsletter/branches/bpi-rouet-2011-09/Products/EasyNewsletter/browser/registration.py	(original)
+++ Products.EasyNewsletter/branches/bpi-rouet-2011-09/Products/EasyNewsletter/browser/registration.py	Tue Oct  4 15:35:38 2011
@@ -100,7 +100,8 @@
         props = getToolByName(self, "portal_properties").site_properties
         charset = props.getProperty("default_charset")
         msg = MIMEText(msg_text, "plain", charset)
-        # don't use Header() charset arg, even if it is correct, mail server may not support encoded email address
+        # don't use Header() with a str and a charset arg, even if it is correct
+        # this would generate a encoded header and mail server may not support utf-8 encoded email address
         msg['To']= Header('<%s>' % unicode(subscriber, charset))
         msg['From'] = newsletter_container.getRegistrationSender()
         msg['Subject'] = Header(msg_subject)