From be06cde5f59bec8e53d75a1608a29854308925af Mon Sep 17 00:00:00 2001 From: Nabin Hait Date: Tue, 8 Nov 2016 13:13:57 +0530 Subject: [PATCH] Update gl_entry.py --- erpnext/accounts/doctype/gl_entry/gl_entry.py | 33 ++++++++++++------- 1 file changed, 21 insertions(+), 12 deletions(-) diff --git a/erpnext/accounts/doctype/gl_entry/gl_entry.py b/erpnext/accounts/doctype/gl_entry/gl_entry.py index b052e4d141..fddabcb918 100644 --- a/erpnext/accounts/doctype/gl_entry/gl_entry.py +++ b/erpnext/accounts/doctype/gl_entry/gl_entry.py @@ -46,19 +46,22 @@ class GLEntry(Document): account_type = frappe.db.get_value("Account", self.account, "account_type") if not (self.party_type and self.party): if account_type == "Receivable": - frappe.throw(_("Customer is required against Receivable account {0}").format(self.account)) + frappe.throw(_("{0} {1}: Customer is required against Receivable account {2}") + .format(self.voucher_type, self.voucher_no, self.account)) elif account_type == "Payable": - frappe.throw(_("Supplier is required against Payable account {0}").format(self.account)) + frappe.throw(_("{0} {1}: Supplier is required against Payable account {2}") + .format(self.voucher_type, self.voucher_no, self.account)) # Zero value transaction is not allowed if not (flt(self.debit) or flt(self.credit)): - frappe.throw(_("Either debit or credit amount is required for {0}").format(self.account)) + frappe.throw(_("{0} {1}: Either debit or credit amount is required for {2}") + .format(self.voucher_type, self.voucher_no, self.account)) def pl_must_have_cost_center(self): if frappe.db.get_value("Account", self.account, "report_type") == "Profit and Loss": if not self.cost_center and self.voucher_type != 'Period Closing Voucher': - frappe.throw(_("Cost Center is required for 'Profit and Loss' account {0}. Please set up a default Cost Center for the Company.") - .format(self.account)) + frappe.throw(_("{0} {1}: Cost Center is required for 'Profit and Loss' account {2}. Please set up a default Cost Center for the Company.") + .format(self.voucher_type, self.voucher_no, self.account)) else: if self.cost_center: self.cost_center = None @@ -68,7 +71,8 @@ class GLEntry(Document): def check_pl_account(self): if self.is_opening=='Yes' and \ frappe.db.get_value("Account", self.account, "report_type")=="Profit and Loss": - frappe.throw(_("'Profit and Loss' type account {0} not allowed in Opening Entry").format(self.account)) + frappe.throw(_("{0} {1}: 'Profit and Loss' type account {2} not allowed in Opening Entry") + .format(self.voucher_type, self.voucher_no, self.account)) def validate_account_details(self, adv_adj): """Account must be ledger, active and not freezed""" @@ -77,13 +81,16 @@ class GLEntry(Document): from tabAccount where name=%s""", self.account, as_dict=1)[0] if ret.is_group==1: - frappe.throw(_("Account {0} cannot be a Group").format(self.account)) + frappe.throw(_("{0} {1}: Account {2} cannot be a Group") + .format(self.voucher_type, self.voucher_no, self.account)) if ret.docstatus==2: - frappe.throw(_("Account {0} is inactive").format(self.account)) + frappe.throw(_("{0} {1}: Account {2} is inactive") + .format(self.voucher_type, self.voucher_no, self.account)) if ret.company != self.company: - frappe.throw(_("Account {0} does not belong to Company {1}").format(self.account, self.company)) + frappe.throw(_("{0} {1}: Account {2} does not belong to Company {3}") + .format(self.voucher_type, self.voucher_no, self.account, self.company)) def validate_cost_center(self): if not hasattr(self, "cost_center_company"): @@ -97,7 +104,8 @@ class GLEntry(Document): return self.cost_center_company[self.cost_center] if self.cost_center and _get_cost_center_company() != self.company: - frappe.throw(_("Cost Center {0} does not belong to Company {1}").format(self.cost_center, self.company)) + frappe.throw(_("{0} {1}: Cost Center {2} does not belong to Company {3}") + .format(self.voucher_type, self.voucher_no, self.cost_center, self.company)) def validate_party(self): validate_party_frozen_disabled(self.party_type, self.party) @@ -110,8 +118,9 @@ class GLEntry(Document): self.account_currency = company_currency if account_currency != self.account_currency: - frappe.throw(_("Accounting Entry for {0} can only be made in currency: {1}") - .format(self.account, (account_currency or company_currency)), InvalidAccountCurrency) + frappe.throw(_("{0} {1}: Accounting Entry for {2} can only be made in currency: {3}") + .format(self.voucher_type, self.voucher_no, self.account, + (account_currency or company_currency)), InvalidAccountCurrency) if self.party_type and self.party: validate_party_gle_currency(self.party_type, self.party, self.company, self.account_currency)