fix: check if approver is set before sharing doc (backport #590) (#605)

Co-authored-by: MOHAMMED NIYAS <[email protected]>
frappe · Jun 16, 2023 · 1902872 · 1902872
1 parent 4066445
commit 1902872
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/hrms/hr/utils.py b/hrms/hr/utils.py
@@ -643,6 +643,9 @@ def get_previous_claimed_amount(employee, payroll_period, non_pro_rata=False, co
 
 
 def share_doc_with_approver(doc, user):
+	if not user:
+		return
+
 	# if approver does not have permissions, share
 	if not frappe.has_permission(doc=doc, ptype="submit", user=user):
 		frappe.share.add_docshare(