From 54385dde2477f58b7a3c0c9c59668664f36cb4b2 Mon Sep 17 00:00:00 2001
From: Sanket322 <shahsanket322003.com>
Date: Thu, 10 Apr 2025 01:15:27 +0530
Subject: [PATCH] fix: changed postalcode order and correct the condition for
 party_name

---
 .../address_and_contacts.py                   | 40 ++++++++++++++-----
 1 file changed, 29 insertions(+), 11 deletions(-)

diff --git a/erpnext/selling/report/address_and_contacts/address_and_contacts.py b/erpnext/selling/report/address_and_contacts/address_and_contacts.py
index 11db4176702..6a507c94773 100644
--- a/erpnext/selling/report/address_and_contacts/address_and_contacts.py
+++ b/erpnext/selling/report/address_and_contacts/address_and_contacts.py
@@ -9,9 +9,9 @@ field_map = {
 	"Address": [
 		"address_line1",
 		"address_line2",
+		"pincode",
 		"city",
 		"state",
-		"pincode",
 		"country",
 		"is_primary_address",
 	],
@@ -43,14 +43,10 @@ def get_columns(filters):
 		"Email Id",
 		"Is Primary Contact:Check",
 	]
-	if filters.get("party_type") == "Supplier" and frappe.db.get_single_value(
-		"Buying Settings", "supp_master_name"
-	) == ["Naming Series", "Auto Name"]:
-		columns.insert(1, "Supplier Name:Data:150")
-	if filters.get("party_type") == "Customer" and frappe.db.get_single_value(
-		"Selling Settings", "cust_master_name"
-	) == ["Naming Series", "Auto Name"]:
-		columns.insert(1, "Customer Name:Data:150")
+
+	if should_add_party_name(party_type):
+		columns.insert(2, f"{party_type} Name:Data:150")
+
 	return columns
 
 
@@ -72,6 +68,7 @@ def get_party_addresses_and_contact(party_type, party, party_group, filters):
 
 	if party:
 		query_filters = {"name": party}
+
 	if filters.get("party_type") in ["Customer", "Supplier"]:
 		field = filters.get("party_type").lower() + "_name"
 	else:
@@ -93,14 +90,18 @@ def get_party_addresses_and_contact(party_type, party, party_group, filters):
 	party_details = get_party_details(party_type, party_list, "Address", party_details)
 	party_details = get_party_details(party_type, party_list, "Contact", party_details)
 
+	add_party_name = should_add_party_name(party_type)
+
 	for party, details in party_details.items():
 		addresses = details.get("address", [])
 		contacts = details.get("contact", [])
 		if not any([addresses, contacts]):
 			result = [party]
 			result.append(party_groups[party])
-			if filters.get("party_type") in ["Customer", "Supplier"]:
+
+			if add_party_name:
 				result.append(party_name_map[party])
+
 			result.extend(add_blank_columns_for("Contact"))
 			result.extend(add_blank_columns_for("Address"))
 			data.append(result)
@@ -112,8 +113,10 @@ def get_party_addresses_and_contact(party_type, party, party_group, filters):
 			for idx in range(0, max_length):
 				result = [party]
 				result.append(party_groups[party])
-				if filters.get("party_type") in ["Customer", "Supplier"]:
+
+				if add_party_name:
 					result.append(party_name_map[party])
+
 				address = addresses[idx] if idx < len(addresses) else add_blank_columns_for("Address")
 				contact = contacts[idx] if idx < len(contacts) else add_blank_columns_for("Contact")
 				result.extend(address)
@@ -130,9 +133,11 @@ def get_party_details(party_type, party_list, doctype, party_details):
 	fields = ["`tabDynamic Link`.link_name", *field_map.get(doctype, [])]
 
 	records = frappe.get_list(doctype, filters=filters, fields=fields, as_list=True)
+
 	for d in records:
 		details = party_details.get(d[0])
 		details.setdefault(frappe.scrub(doctype), []).append(d[1:])
+
 	return party_details
 
 
@@ -151,3 +156,16 @@ def get_party_group(party_type):
 	}
 
 	return group[party_type]
+
+
+def should_add_party_name(party_type):
+	settings_map = {
+		"Supplier": ("Buying Settings", "supp_master_name"),
+		"Customer": ("Selling Settings", "cust_master_name"),
+	}
+
+	if party_type in settings_map:
+		doctype, fieldname = settings_map.get(party_type)
+		return frappe.db.get_single_value(doctype, fieldname) in ["Naming Series", "Auto Name"]
+
+	return False