From 7854779c221bdb3ee99253ea2d91d8af947d71bf Mon Sep 17 00:00:00 2001 From: Anand Doshi Date: Sun, 5 Apr 2015 13:44:52 +0530 Subject: [PATCH] [hotfix] strip email when importing newsletter list --- erpnext/crm/doctype/newsletter_list/newsletter_list.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/erpnext/crm/doctype/newsletter_list/newsletter_list.py b/erpnext/crm/doctype/newsletter_list/newsletter_list.py index cee68c9523..3116bbe48f 100644 --- a/erpnext/crm/doctype/newsletter_list/newsletter_list.py +++ b/erpnext/crm/doctype/newsletter_list/newsletter_list.py @@ -5,7 +5,7 @@ from __future__ import unicode_literals import frappe from frappe.model.document import Document -from frappe.utils import validate_email_add +from frappe.utils import validate_email_add, strip from frappe import _ class NewsletterList(Document): @@ -23,11 +23,12 @@ class NewsletterList(Document): for user in frappe.db.get_all(doctype, [email_field, unsubscribed_field or "name"]): try: - if user.get(email_field): + email = strip(user.get(email_field)) + if email: frappe.get_doc({ "doctype": "Newsletter List Subscriber", "newsletter_list": self.name, - "email": user.get(email_field), + "email": email, "unsubscribed": user.get(unsubscribed_field) if unsubscribed_field else 0 }).insert(ignore_permissions=True)