From 98390dae49d9ffee3030813b581dd5e5adb27dcf Mon Sep 17 00:00:00 2001 From: Suraj Shetty Date: Fri, 19 Apr 2019 21:52:08 +0530 Subject: [PATCH] fix: Do not create employee user permission if already exists --- erpnext/hr/doctype/employee/employee.py | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/erpnext/hr/doctype/employee/employee.py b/erpnext/hr/doctype/employee/employee.py index a403c393b9..48957e5819 100755 --- a/erpnext/hr/doctype/employee/employee.py +++ b/erpnext/hr/doctype/employee/employee.py @@ -80,6 +80,14 @@ class Employee(NestedSet): if not self.create_user_permission: return if not has_permission('User Permission', ptype='write'): return + employee_user_permission_exists = frappe.db.exists('User Permission', { + 'allow': 'Employee', + 'for_value': self.name, + 'user': self.user_id + }) + + if employee_user_permission_exists: return + add_user_permission("Employee", self.name, self.user_id) set_user_permission_if_allowed("Company", self.company, self.user_id)