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

validation added to check if multiple sales person have been given the same employee id

Browse Source
This commit is contained in:
Neil Trini Lasrado 2015-03-11 17:07:58 +05:30
parent 1ca0b74e03
commit e7044fa0f6
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
erpnext/setup/doctype/sales_person/sales_person.py
View File

@ -14,6 +14,7 @@ class SalesPerson(NestedSet):
for d in self.get('targets') or []: for d in self.get('targets') or []:
if not flt(d.target_qty) and not flt(d.target_amount): if not flt(d.target_qty) and not flt(d.target_amount):
frappe.throw(_("Either target qty or target amount is mandatory.")) frappe.throw(_("Either target qty or target amount is mandatory."))
self.validate_employee_id()
def on_update(self): def on_update(self):
super(SalesPerson, self).on_update() super(SalesPerson, self).on_update()
@ -26,3 +27,7 @@ class SalesPerson(NestedSet):
frappe.throw(_("User ID not set for Employee {0}").format(self.employee)) frappe.throw(_("User ID not set for Employee {0}").format(self.employee))
else: else:
return frappe.db.get_value("User", user, "email") or user return frappe.db.get_value("User", user, "email") or user
def validate_employee_id(self):
if frappe.db.exists({"doctype": "Sales Person","employee": self.employee}):
frappe.throw("Another sales person with the same empoyee id exists.")