refactor: better error messages

frappe · Jun 17, 2024 · 83ff94b · 83ff94b
1 parent 107b614
commit 83ff94b
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/erpnext/utilities/transaction_base.py b/erpnext/utilities/transaction_base.py
@@ -191,9 +191,11 @@ def validate_currency_for_receivable_payable_and_advance_account(self):
 				):
 					frappe.throw(
 						_(
-							"{0} Account must be in either customer billing currency: {1} or Company default currency: {2}"
+							"{0} Account: {1} ({2}) must be in either customer billing currency: {3} or Company default currency: {4}"
 						).format(
 							account_type,
+							frappe.bold(x.account),
+							frappe.bold(receivable_payable_account_currency),
 							frappe.bold(self.default_currency),
 							frappe.bold(company_default_currency),
 						)
@@ -205,8 +207,12 @@ def validate_currency_for_receivable_payable_and_advance_account(self):
 				):
 					frappe.throw(
 						_(
-							"Advance Account must be in either customer billing currency: {0} or Company default currency: {1}"
-						).format(frappe.bold(self.default_currency), frappe.bold(company_default_currency))
+							"Advance Account: {0} must be in either customer billing currency: {1} or Company default currency: {2}"
+						).format(
+							frappe.bold(x.advance_account),
+							frappe.bold(self.default_currency),
+							frappe.bold(company_default_currency),
+						)
 					)
 
 				if (