Merge pull request #2944 from neilLasrado/po
fixes in Production Order
diff --git a/erpnext/setup/doctype/sales_person/sales_person.py b/erpnext/setup/doctype/sales_person/sales_person.py
index b4a8b7c..9098aae 100644
--- a/erpnext/setup/doctype/sales_person/sales_person.py
+++ b/erpnext/setup/doctype/sales_person/sales_person.py
@@ -14,6 +14,7 @@
for d in self.get('targets') or []:
if not flt(d.target_qty) and not flt(d.target_amount):
frappe.throw(_("Either target qty or target amount is mandatory."))
+ self.validate_employee_id()
def on_update(self):
super(SalesPerson, self).on_update()
@@ -26,3 +27,7 @@
frappe.throw(_("User ID not set for Employee {0}").format(self.employee))
else:
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 employee id exists.")