Update gl_entry.py
diff --git a/erpnext/accounts/doctype/gl_entry/gl_entry.py b/erpnext/accounts/doctype/gl_entry/gl_entry.py
index b052e4d..fddabcb 100644
--- a/erpnext/accounts/doctype/gl_entry/gl_entry.py
+++ b/erpnext/accounts/doctype/gl_entry/gl_entry.py
@@ -46,19 +46,22 @@
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 @@
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 @@
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 @@
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 @@
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)