Merge pull request #46683 from Abdeali099/set-employee-contact-details

fix: Set complete contact details for `Employee` in PE

erpnext/accounts/doctype/payment_entry/payment_entry.json

@@ -200,14 +200,14 @@
    "fieldtype": "Column Break"
   },
   {
-   "depends_on": "party",
+   "depends_on": "eval: doc.party && doc.party_type !== \"Employee\"",
    "fieldname": "contact_person",
    "fieldtype": "Link",
    "label": "Contact",
    "options": "Contact"
   },
   {
-   "depends_on": "contact_person",
+   "depends_on": "eval: (doc.contact_person || doc.party_type === \"Employee\") && doc.contact_email",
    "fieldname": "contact_email",
    "fieldtype": "Data",
    "label": "Email",
@@ -777,7 +777,7 @@
    "table_fieldname": "payment_entries"
   }
  ],
- "modified": "2025-01-31 11:24:58.076393",
+ "modified": "2025-03-24 16:18:19.920701",
  "modified_by": "Administrator",
  "module": "Accounts",
  "name": "Payment Entry",

erpnext/accounts/doctype/payment_entry/payment_entry.py

@@ -38,7 +38,11 @@ from erpnext.accounts.general_ledger import (
 	make_reverse_gl_entries,
 	process_gl_map,
 )
-from erpnext.accounts.party import complete_contact_details, get_party_account, set_contact_details
+from erpnext.accounts.party import (
+	complete_contact_details,
+	get_default_contact,
+	get_party_account,
+)
 from erpnext.accounts.utils import (
 	cancel_exchange_gain_loss_journal,
 	get_account_currency,
@@ -525,12 +529,12 @@ class PaymentEntry(AccountsController):
 				self.party_name = frappe.db.get_value(self.party_type, self.party, "name")
 
 		if self.party:
-			if not self.contact_person:
+			if self.party_type == "Employee":
-				set_contact_details(
+				self.contact_person = None
-					self, party=frappe._dict({"name": self.party}), party_type=self.party_type
+			elif not self.contact_person:
-				)
+				self.contact_person = get_default_contact(self.party_type, self.party)
-			else:
+
-				complete_contact_details(self)
+			complete_contact_details(self)
 
 			if not self.party_account:
 				party_account = get_party_account(self.party_type, self.party, self.company)

erpnext/accounts/party.py

@@ -280,32 +280,50 @@ def get_regional_address_details(party_details, doctype, company):
 
 
 def complete_contact_details(party_details):
-	if not party_details.contact_person:
+	contact_details = frappe._dict()
-		party_details.update(
+
-			{
+	if party_details.party_type == "Employee":
-				"contact_person": None,
+		contact_details = frappe.db.get_value(
-				"contact_display": None,
+			"Employee",
-				"contact_email": None,
+			party_details.party,
-				"contact_mobile": None,
+			[
-				"contact_phone": None,
+				"employee_name as contact_display",
-				"contact_designation": None,
+				"prefered_email as contact_email",
-				"contact_department": None,
+				"cell_number as contact_mobile",
-			}
+				"designation as contact_designation",
+				"department as contact_department",
+			],
+			as_dict=True,
+		)
+
+		contact_details.update({"contact_person": None, "contact_phone": None})
+	elif party_details.contact_person:
+		contact_details = frappe.db.get_value(
+			"Contact",
+			party_details.contact_person,
+			[
+				"name as contact_person",
+				"full_name as contact_display",
+				"email_id as contact_email",
+				"mobile_no as contact_mobile",
+				"phone as contact_phone",
+				"designation as contact_designation",
+				"department as contact_department",
+			],
+			as_dict=True,
 		)
 	else:
-		fields = [
+		contact_details = {
-			"name as contact_person",
+			"contact_person": None,
-			"full_name as contact_display",
+			"contact_display": None,
-			"email_id as contact_email",
+			"contact_email": None,
-			"mobile_no as contact_mobile",
+			"contact_mobile": None,
-			"phone as contact_phone",
+			"contact_phone": None,
-			"designation as contact_designation",
+			"contact_designation": None,
-			"department as contact_department",
+			"contact_department": None,
-		]
+		}
 
-		contact_details = frappe.db.get_value("Contact", party_details.contact_person, fields, as_dict=True)
+	party_details.update(contact_details)
-
-		party_details.update(contact_details)
 
 
 def set_contact_details(party_details, party, party_type):
@@ -780,9 +798,9 @@ def validate_account_party_type(self):
 		account_type = frappe.get_cached_value("Account", self.account, "account_type")
 		if account_type and (account_type not in ["Receivable", "Payable", "Equity"]):
 			frappe.throw(
-				_(
+				_("Party Type and Party can only be set for Receivable / Payable account<br><br>{0}").format(
-					"Party Type and Party can only be set for Receivable / Payable account<br><br>" "{0}"
+					self.account
-				).format(self.account)
+				)
 			)