Changeset 7402 for main/waeup.sirp/trunk/src/waeup/sirp/utils

Timestamp:

20 Dec 2011, 09:18:02 (13 years ago)

Author:

Henrik Bettermann

Message:

Harmonize sendMail method (work in progress).

File:

• main/waeup.sirp/trunk/src/waeup/sirp/utils/utils.py (modified) (6 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/utils/utils.py

@@ -28 +28 @@
 from email.Utils import parseaddr, formataddr
 
-def send_mail(from_name,from_email,rcpt_name,rcpt_email,subject,body,config):
+def send_mail(from_name,from_addr,rcpt_name,rcpt_addr,subject,body,config):
     """
     XXX: While this method is unicode-proof (i.e. it can handle
@@ -37 +37 @@
 
     header_charset = 'ISO-8859-1'
-    from_ = '%s <%s>' % (from_name,from_email)
-    to_ = '%s <%s>' % (rcpt_name,rcpt_email)
-    from_name, from_addr = parseaddr(from_)
-    rcpt_name, rcpt_addr = parseaddr(to_)
     from_name = str(Header(unicode(from_name), header_charset))
     rcpt_name = str(Header(unicode(rcpt_name), header_charset))
@@ -63 +59 @@
         server.login(config.smtp_username,config.smtp_password)
     try:
-        server.sendmail(from_email,rcpt_email,msg.as_string())
+        server.sendmail(from_addr,rcpt_addr,msg.as_string())
     except Exception, e:
         # XXX: log error message
@@ -76 +72 @@
     grok.implements(ISIRPUtils)
 
-    def sendMail(self,fullname,username,usertype,portal,body,
-                 email_from,email_to,subject):
+    def sendMail(self,from_name,from_addr,rcpt_name,rcpt_addr,
+                from_username,usertype,portal,body,subject):
         """Send an email with data provided by forms.
         """
@@ -88 +84 @@
 %s
 """
-        from_name = fullname
-        from_email = email_from
-        rcpt_name = 'xyz'
-        rcpt_email = email_to
-        body = text % (fullname,username,usertype,portal,body)
+        body = text % (from_name,from_username,usertype,portal,body)
         return send_mail(
-            from_name,from_email,rcpt_name,rcpt_email,subject,body,config)
+            from_name,from_addr,rcpt_name,rcpt_addr,subject,body,config)
 
     def fullname(self,firstname,lastname,middlename=None):
@@ -139 +131 @@
         config = grok.getSite()['configuration']
         from_name = config.name_admin
-        from_email = config.email_admin
+        from_addr = config.email_admin
         rcpt_name = applicant.display_fullname
-        rcpt_email = applicant.email
+        rcpt_addr = applicant.email
         body = text % (
                 rcpt_name, msg,config.name,username,password,login_url)
         return send_mail(
-            from_name,from_email,rcpt_name,rcpt_email,subject,body,config)
-
-
-
+            from_name,from_addr,rcpt_name,rcpt_addr,subject,body,config)