WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Commit d27b699

Browse files
committed
chore(Employee Referral): Better Feedback message
1 parent 764e5ca commit d27b699

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

hrms/hr/doctype/employee_referral/employee_referral.py

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,15 @@ def validate(self):
1919
self.set_referral_bonus_payment_status()
2020

2121
def validate_unique_referral(self):
22-
if frappe.db.exists(
22+
if referral := frappe.db.exists(
2323
"Employee Referral", {"name": ("!=", self.name), "email": self.email, "docstatus": ("!=", 2)}
2424
):
25-
frappe.throw("Referral for this email already exists.")
25+
frappe.throw(
26+
_("Employee Referral {0} already exists for email: {1}").format(
27+
get_link_to_form("Employee Referral", referral), frappe.bold(self.email)
28+
),
29+
frappe.DuplicateEntryError,
30+
)
2631

2732
def set_full_name(self):
2833
self.full_name = " ".join(filter(None, [self.first_name, self.last_name]))

hrms/hr/doctype/employee_referral/test_employee_referral.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ def test_status_on_discard(self):
5858

5959
def test_unique_referral(self):
6060
referral_1 = create_employee_referral(email="[email protected]")
61-
self.assertRaises(frappe.ValidationError, create_employee_referral, email="[email protected]")
61+
self.assertRaises(frappe.DuplicateEntryError, create_employee_referral, email="[email protected]")
6262
referral_1.cancel()
6363
referral_2 = create_employee_referral(email="[email protected]")
6464
self.assertTrue(referral_2)

0 commit comments

Comments
 (0)