Shiloh/brotherton-erpnext
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[hotfix] strip email when importing newsletter list

Browse Source
This commit is contained in:
Anand Doshi 2015-04-05 13:44:52 +05:30
parent e4580447d0
commit 7854779c22
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
erpnext/crm/doctype/newsletter_list/newsletter_list.py
View File

@ -5,7 +5,7 @@
from __future__ import unicode_literals from __future__ import unicode_literals
import frappe import frappe
from frappe.model.document import Document from frappe.model.document import Document
from frappe.utils import validate_email_add from frappe.utils import validate_email_add, strip
from frappe import _ from frappe import _
class NewsletterList(Document): class NewsletterList(Document):
@ -23,11 +23,12 @@ class NewsletterList(Document):
for user in frappe.db.get_all(doctype, [email_field, unsubscribed_field or "name"]): for user in frappe.db.get_all(doctype, [email_field, unsubscribed_field or "name"]):
try: try:
if user.get(email_field): email = strip(user.get(email_field))
if email:
frappe.get_doc({ frappe.get_doc({
"doctype": "Newsletter List Subscriber", "doctype": "Newsletter List Subscriber",
"newsletter_list": self.name, "newsletter_list": self.name,
"email": user.get(email_field), "email": email,
"unsubscribed": user.get(unsubscribed_field) if unsubscribed_field else 0 "unsubscribed": user.get(unsubscribed_field) if unsubscribed_field else 0
}).insert(ignore_permissions=True) }).insert(ignore_permissions=True)