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

refactor: change validate email add to validate email address

Browse Source
This commit is contained in:
Mangesh-Khairnar 2019-03-16 13:38:53 +05:30
parent 349a1157ef
commit b343407236
4 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
erpnext/crm/doctype/lead/lead.py
View File

@ -4,7 +4,7 @@
from __future__ import unicode_literals from __future__ import unicode_literals
import frappe import frappe
from frappe import _ from frappe import _
from frappe.utils import (cstr, validate_email_add, cint, comma_and, has_gravatar, now, getdate, nowdate) from frappe.utils import (cstr, validate_email_address, cint, comma_and, has_gravatar, now, getdate, nowdate)
from frappe.model.mapper import get_mapped_doc from frappe.model.mapper import get_mapped_doc
from erpnext.controllers.selling_controller import SellingController from erpnext.controllers.selling_controller import SellingController
@ -38,7 +38,7 @@ class Lead(SellingController):
if self.email_id: if self.email_id:
if not self.flags.ignore_email_validation: if not self.flags.ignore_email_validation:
validate_email_add(self.email_id, True) validate_email_address(self.email_id, True)
if self.email_id == self.lead_owner: if self.email_id == self.lead_owner:
frappe.throw(_("Lead Owner cannot be same as the Lead")) frappe.throw(_("Lead Owner cannot be same as the Lead"))

6
erpnext/hr/doctype/employee/employee.py
View File

@ -4,7 +4,7 @@
from __future__ import unicode_literals from __future__ import unicode_literals
import frappe import frappe
from frappe.utils import getdate, validate_email_add, today, add_years, format_datetime from frappe.utils import getdate, validate_email_address, today, add_years, format_datetime
from frappe.model.naming import set_name_by_naming_series from frappe.model.naming import set_name_by_naming_series
from frappe import throw, _, scrub from frappe import throw, _, scrub
from frappe.permissions import add_user_permission, remove_user_permission, \ from frappe.permissions import add_user_permission, remove_user_permission, \
@ -142,9 +142,9 @@ class Employee(NestedSet):
def validate_email(self): def validate_email(self):
if self.company_email: if self.company_email:
validate_email_add(self.company_email, True) validate_email_address(self.company_email, True)
if self.personal_email: if self.personal_email:
validate_email_add(self.personal_email, True) validate_email_address(self.personal_email, True)
def validate_status(self): def validate_status(self):
if self.status == 'Left': if self.status == 'Left':

4
erpnext/hr/doctype/job_applicant/job_applicant.py
View File

@ -7,7 +7,7 @@ from __future__ import unicode_literals
from frappe.model.document import Document from frappe.model.document import Document
import frappe import frappe
from frappe import _ from frappe import _
from frappe.utils import comma_and, validate_email_add from frappe.utils import comma_and, validate_email_address
sender_field = "email_id" sender_field = "email_id"
@ -28,7 +28,7 @@ class JobApplicant(Document):
def validate(self): def validate(self):
self.check_email_id_is_unique() self.check_email_id_is_unique()
if self.email_id: if self.email_id:
validate_email_add(self.email_id, True) validate_email_address(self.email_id, True)
if not self.applicant_name and self.email_id: if not self.applicant_name and self.email_id:
guess = self.email_id.split('@')[0] guess = self.email_id.split('@')[0]

4
erpnext/non_profit/doctype/member/member.py
View File

@ -17,5 +17,5 @@ class Member(Document):
self.validate_email_type(self.email) self.validate_email_type(self.email)
def validate_email_type(self, email): def validate_email_type(self, email):
from frappe.utils import validate_email_add from frappe.utils import validate_email_address
validate_email_add(email.strip(), True) validate_email_address(email.strip(), True)