From 83ff94b9b87772251f663f3be59c44cadb2c3a67 Mon Sep 17 00:00:00 2001 From: ruthra kumar Date: Tue, 11 Jun 2024 10:18:59 +0530 Subject: [PATCH] refactor: better error messages --- erpnext/utilities/transaction_base.py | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/erpnext/utilities/transaction_base.py b/erpnext/utilities/transaction_base.py index 9559c9170cf..6fab5380c38 100644 --- a/erpnext/utilities/transaction_base.py +++ b/erpnext/utilities/transaction_base.py @@ -191,9 +191,11 @@ class TransactionBase(StatusUpdater): ): 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 @@ class TransactionBase(StatusUpdater): ): 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 (