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

[hotfix] fixed the recursion error while saving the User (#8696)

Browse Source
This commit is contained in:
Makarand Bauskar 2017-05-05 11:53:00 +05:30 committed by Nabin Hait
parent c4ec937835
commit 6b3bc8a8e1
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
erpnext/portal/utils.py
View File

@ -4,14 +4,17 @@ def set_default_role(doc, method):
'''Set customer, supplier, student based on email''' '''Set customer, supplier, student based on email'''
if frappe.flags.setting_role or frappe.flags.in_migrate: if frappe.flags.setting_role or frappe.flags.in_migrate:
return return
roles = frappe.get_roles(doc.name)
contact_name = frappe.get_value('Contact', dict(email_id=doc.email)) contact_name = frappe.get_value('Contact', dict(email_id=doc.email))
if contact_name: if contact_name:
contact = frappe.get_doc('Contact', contact_name) contact = frappe.get_doc('Contact', contact_name)
for link in contact.links: for link in contact.links:
frappe.flags.setting_role = True frappe.flags.setting_role = True
if link.link_doctype=='Customer': if link.link_doctype=='Customer' and 'Customer' not in roles:
doc.add_roles('Customer') doc.add_roles('Customer')
elif link.link_doctype=='Supplier': elif link.link_doctype=='Supplier' and 'Supplier' not in roles:
doc.add_roles('Supplier') doc.add_roles('Supplier')
elif frappe.get_value('Student', dict(student_email_id=doc.email)): elif frappe.get_value('Student', dict(student_email_id=doc.email)) and 'Student' not in roles:
doc.add_roles('Student') doc.add_roles('Student')