c9e8a1bf96
* fix: Account Balance and Stock Value out of sync error message Added 'Make Adjustment Entry' button and enhanced message * fix: Split message and changed routing for translation
294 lines
11 KiB
Python
294 lines
11 KiB
Python
# Copyright (c) 2015, Frappe Technologies Pvt. Ltd. and Contributors
|
|
# License: GNU General Public License v3. See license.txt
|
|
|
|
from __future__ import unicode_literals
|
|
import frappe, erpnext
|
|
from frappe.utils import flt, cstr, cint, comma_and
|
|
from frappe import _
|
|
from erpnext.accounts.utils import get_stock_and_account_balance
|
|
from frappe.model.meta import get_field_precision
|
|
from erpnext.accounts.doctype.budget.budget import validate_expense_against_budget
|
|
from erpnext.accounts.doctype.accounting_dimension.accounting_dimension import get_accounting_dimensions
|
|
|
|
|
|
class ClosedAccountingPeriod(frappe.ValidationError): pass
|
|
class StockAccountInvalidTransaction(frappe.ValidationError): pass
|
|
class StockValueAndAccountBalanceOutOfSync(frappe.ValidationError): pass
|
|
|
|
def make_gl_entries(gl_map, cancel=False, adv_adj=False, merge_entries=True, update_outstanding='Yes', from_repost=False):
|
|
if gl_map:
|
|
if not cancel:
|
|
validate_accounting_period(gl_map)
|
|
gl_map = process_gl_map(gl_map, merge_entries)
|
|
if gl_map and len(gl_map) > 1:
|
|
save_entries(gl_map, adv_adj, update_outstanding, from_repost)
|
|
else:
|
|
frappe.throw(_("Incorrect number of General Ledger Entries found. You might have selected a wrong Account in the transaction."))
|
|
else:
|
|
delete_gl_entries(gl_map, adv_adj=adv_adj, update_outstanding=update_outstanding)
|
|
|
|
def validate_accounting_period(gl_map):
|
|
accounting_periods = frappe.db.sql(""" SELECT
|
|
ap.name as name
|
|
FROM
|
|
`tabAccounting Period` ap, `tabClosed Document` cd
|
|
WHERE
|
|
ap.name = cd.parent
|
|
AND ap.company = %(company)s
|
|
AND cd.closed = 1
|
|
AND cd.document_type = %(voucher_type)s
|
|
AND %(date)s between ap.start_date and ap.end_date
|
|
""", {
|
|
'date': gl_map[0].posting_date,
|
|
'company': gl_map[0].company,
|
|
'voucher_type': gl_map[0].voucher_type
|
|
}, as_dict=1)
|
|
|
|
if accounting_periods:
|
|
frappe.throw(_("You can't create accounting entries in the closed accounting period {0}")
|
|
.format(accounting_periods[0].name), ClosedAccountingPeriod)
|
|
|
|
def process_gl_map(gl_map, merge_entries=True):
|
|
if merge_entries:
|
|
gl_map = merge_similar_entries(gl_map)
|
|
for entry in gl_map:
|
|
# toggle debit, credit if negative entry
|
|
if flt(entry.debit) < 0:
|
|
entry.credit = flt(entry.credit) - flt(entry.debit)
|
|
entry.debit = 0.0
|
|
|
|
if flt(entry.debit_in_account_currency) < 0:
|
|
entry.credit_in_account_currency = \
|
|
flt(entry.credit_in_account_currency) - flt(entry.debit_in_account_currency)
|
|
entry.debit_in_account_currency = 0.0
|
|
|
|
if flt(entry.credit) < 0:
|
|
entry.debit = flt(entry.debit) - flt(entry.credit)
|
|
entry.credit = 0.0
|
|
|
|
if flt(entry.credit_in_account_currency) < 0:
|
|
entry.debit_in_account_currency = \
|
|
flt(entry.debit_in_account_currency) - flt(entry.credit_in_account_currency)
|
|
entry.credit_in_account_currency = 0.0
|
|
|
|
return gl_map
|
|
|
|
def merge_similar_entries(gl_map):
|
|
merged_gl_map = []
|
|
accounting_dimensions = get_accounting_dimensions()
|
|
for entry in gl_map:
|
|
# if there is already an entry in this account then just add it
|
|
# to that entry
|
|
same_head = check_if_in_list(entry, merged_gl_map, accounting_dimensions)
|
|
if same_head:
|
|
same_head.debit = flt(same_head.debit) + flt(entry.debit)
|
|
same_head.debit_in_account_currency = \
|
|
flt(same_head.debit_in_account_currency) + flt(entry.debit_in_account_currency)
|
|
same_head.credit = flt(same_head.credit) + flt(entry.credit)
|
|
same_head.credit_in_account_currency = \
|
|
flt(same_head.credit_in_account_currency) + flt(entry.credit_in_account_currency)
|
|
else:
|
|
merged_gl_map.append(entry)
|
|
|
|
# filter zero debit and credit entries
|
|
merged_gl_map = filter(lambda x: flt(x.debit, 9)!=0 or flt(x.credit, 9)!=0, merged_gl_map)
|
|
merged_gl_map = list(merged_gl_map)
|
|
|
|
return merged_gl_map
|
|
|
|
def check_if_in_list(gle, gl_map, dimensions=None):
|
|
account_head_fieldnames = ['party_type', 'party', 'against_voucher', 'against_voucher_type',
|
|
'cost_center', 'project']
|
|
|
|
if dimensions:
|
|
account_head_fieldnames = account_head_fieldnames + dimensions
|
|
|
|
for e in gl_map:
|
|
same_head = True
|
|
if e.account != gle.account:
|
|
same_head = False
|
|
|
|
for fieldname in account_head_fieldnames:
|
|
if cstr(e.get(fieldname)) != cstr(gle.get(fieldname)):
|
|
same_head = False
|
|
|
|
if same_head:
|
|
return e
|
|
|
|
def save_entries(gl_map, adv_adj, update_outstanding, from_repost=False):
|
|
if not from_repost:
|
|
validate_cwip_accounts(gl_map)
|
|
|
|
round_off_debit_credit(gl_map)
|
|
for entry in gl_map:
|
|
make_entry(entry, adv_adj, update_outstanding, from_repost)
|
|
|
|
# check against budget
|
|
if not from_repost:
|
|
validate_expense_against_budget(entry)
|
|
|
|
if not from_repost:
|
|
validate_account_for_perpetual_inventory(gl_map)
|
|
|
|
|
|
def make_entry(args, adv_adj, update_outstanding, from_repost=False):
|
|
args.update({"doctype": "GL Entry"})
|
|
gle = frappe.get_doc(args)
|
|
gle.flags.ignore_permissions = 1
|
|
gle.flags.from_repost = from_repost
|
|
gle.insert()
|
|
gle.run_method("on_update_with_args", adv_adj, update_outstanding, from_repost)
|
|
gle.submit()
|
|
|
|
def validate_account_for_perpetual_inventory(gl_map):
|
|
if cint(erpnext.is_perpetual_inventory_enabled(gl_map[0].company)):
|
|
account_list = [gl_entries.account for gl_entries in gl_map]
|
|
|
|
aii_accounts = [d.name for d in frappe.get_all("Account",
|
|
filters={'account_type': 'Stock', 'is_group': 0, 'company': gl_map[0].company})]
|
|
|
|
for account in account_list:
|
|
if account not in aii_accounts:
|
|
continue
|
|
|
|
account_bal, stock_bal, warehouse_list = get_stock_and_account_balance(account,
|
|
gl_map[0].posting_date, gl_map[0].company)
|
|
|
|
if gl_map[0].voucher_type=="Journal Entry":
|
|
# In case of Journal Entry, there are no corresponding SL entries,
|
|
# hence deducting currency amount
|
|
account_bal -= flt(gl_map[0].debit) - flt(gl_map[0].credit)
|
|
if account_bal == stock_bal:
|
|
frappe.throw(_("Account: {0} can only be updated via Stock Transactions")
|
|
.format(account), StockAccountInvalidTransaction)
|
|
|
|
elif account_bal != stock_bal:
|
|
error_reason = _("Account Balance ({0}) and Stock Value ({1}) is out of sync for account {2} and it's linked warehouses.").format(
|
|
account_bal, stock_bal, frappe.bold(account))
|
|
error_resolution = _("Please create adjustment Journal Entry for amount {0} ").format(frappe.bold(stock_bal - account_bal))
|
|
button_text = _("Make Adjustment Entry")
|
|
|
|
frappe.throw("""{0}<br></br>{1}<br></br>
|
|
<div style="text-align:right;">
|
|
<button class="btn btn-primary" onclick="frappe.new_doc('Journal Entry')">{2}</button>
|
|
</div>""".format(error_reason, error_resolution, button_text),
|
|
StockValueAndAccountBalanceOutOfSync, title=_('Account Balance Out Of Sync'))
|
|
|
|
def validate_cwip_accounts(gl_map):
|
|
cwip_enabled = cint(frappe.get_cached_value("Company",
|
|
gl_map[0].company, "enable_cwip_accounting"))
|
|
|
|
if not cwip_enabled:
|
|
cwip_enabled = any([cint(ac.enable_cwip_accounting) for ac in frappe.db.get_all("Asset Category","enable_cwip_accounting")])
|
|
|
|
if cwip_enabled and gl_map[0].voucher_type == "Journal Entry":
|
|
cwip_accounts = [d[0] for d in frappe.db.sql("""select name from tabAccount
|
|
where account_type = 'Capital Work in Progress' and is_group=0""")]
|
|
|
|
for entry in gl_map:
|
|
if entry.account in cwip_accounts:
|
|
frappe.throw(
|
|
_("Account: <b>{0}</b> is capital Work in progress and can not be updated by Journal Entry").format(entry.account))
|
|
|
|
def round_off_debit_credit(gl_map):
|
|
precision = get_field_precision(frappe.get_meta("GL Entry").get_field("debit"),
|
|
currency=frappe.get_cached_value('Company', gl_map[0].company, "default_currency"))
|
|
|
|
debit_credit_diff = 0.0
|
|
for entry in gl_map:
|
|
entry.debit = flt(entry.debit, precision)
|
|
entry.credit = flt(entry.credit, precision)
|
|
debit_credit_diff += entry.debit - entry.credit
|
|
|
|
debit_credit_diff = flt(debit_credit_diff, precision)
|
|
|
|
if gl_map[0]["voucher_type"] in ("Journal Entry", "Payment Entry"):
|
|
allowance = 5.0 / (10**precision)
|
|
else:
|
|
allowance = .5
|
|
|
|
if abs(debit_credit_diff) >= allowance:
|
|
frappe.throw(_("Debit and Credit not equal for {0} #{1}. Difference is {2}.")
|
|
.format(gl_map[0].voucher_type, gl_map[0].voucher_no, debit_credit_diff))
|
|
|
|
elif abs(debit_credit_diff) >= (1.0 / (10**precision)):
|
|
make_round_off_gle(gl_map, debit_credit_diff, precision)
|
|
|
|
def make_round_off_gle(gl_map, debit_credit_diff, precision):
|
|
round_off_account, round_off_cost_center = get_round_off_account_and_cost_center(gl_map[0].company)
|
|
round_off_account_exists = False
|
|
round_off_gle = frappe._dict()
|
|
for d in gl_map:
|
|
if d.account == round_off_account:
|
|
round_off_gle = d
|
|
if d.debit_in_account_currency:
|
|
debit_credit_diff -= flt(d.debit_in_account_currency)
|
|
else:
|
|
debit_credit_diff += flt(d.credit_in_account_currency)
|
|
round_off_account_exists = True
|
|
|
|
if round_off_account_exists and abs(debit_credit_diff) <= (1.0 / (10**precision)):
|
|
gl_map.remove(round_off_gle)
|
|
return
|
|
|
|
if not round_off_gle:
|
|
for k in ["voucher_type", "voucher_no", "company",
|
|
"posting_date", "remarks", "is_opening"]:
|
|
round_off_gle[k] = gl_map[0][k]
|
|
|
|
round_off_gle.update({
|
|
"account": round_off_account,
|
|
"debit_in_account_currency": abs(debit_credit_diff) if debit_credit_diff < 0 else 0,
|
|
"credit_in_account_currency": debit_credit_diff if debit_credit_diff > 0 else 0,
|
|
"debit": abs(debit_credit_diff) if debit_credit_diff < 0 else 0,
|
|
"credit": debit_credit_diff if debit_credit_diff > 0 else 0,
|
|
"cost_center": round_off_cost_center,
|
|
"party_type": None,
|
|
"party": None,
|
|
"against_voucher_type": None,
|
|
"against_voucher": None
|
|
})
|
|
|
|
if not round_off_account_exists:
|
|
gl_map.append(round_off_gle)
|
|
|
|
def get_round_off_account_and_cost_center(company):
|
|
round_off_account, round_off_cost_center = frappe.get_cached_value('Company', company,
|
|
["round_off_account", "round_off_cost_center"]) or [None, None]
|
|
if not round_off_account:
|
|
frappe.throw(_("Please mention Round Off Account in Company"))
|
|
|
|
if not round_off_cost_center:
|
|
frappe.throw(_("Please mention Round Off Cost Center in Company"))
|
|
|
|
return round_off_account, round_off_cost_center
|
|
|
|
def delete_gl_entries(gl_entries=None, voucher_type=None, voucher_no=None,
|
|
adv_adj=False, update_outstanding="Yes"):
|
|
|
|
from erpnext.accounts.doctype.gl_entry.gl_entry import validate_balance_type, \
|
|
check_freezing_date, update_outstanding_amt, validate_frozen_account
|
|
|
|
if not gl_entries:
|
|
gl_entries = frappe.db.sql("""
|
|
select account, posting_date, party_type, party, cost_center, fiscal_year,voucher_type,
|
|
voucher_no, against_voucher_type, against_voucher, cost_center, company
|
|
from `tabGL Entry`
|
|
where voucher_type=%s and voucher_no=%s""", (voucher_type, voucher_no), as_dict=True)
|
|
|
|
if gl_entries:
|
|
check_freezing_date(gl_entries[0]["posting_date"], adv_adj)
|
|
|
|
frappe.db.sql("""delete from `tabGL Entry` where voucher_type=%s and voucher_no=%s""",
|
|
(voucher_type or gl_entries[0]["voucher_type"], voucher_no or gl_entries[0]["voucher_no"]))
|
|
|
|
for entry in gl_entries:
|
|
validate_frozen_account(entry["account"], adv_adj)
|
|
validate_balance_type(entry["account"], adv_adj)
|
|
if not adv_adj:
|
|
validate_expense_against_budget(entry)
|
|
|
|
if entry.get("against_voucher") and update_outstanding == 'Yes' and not adv_adj:
|
|
update_outstanding_amt(entry["account"], entry.get("party_type"), entry.get("party"), entry.get("against_voucher_type"),
|
|
entry.get("against_voucher"), on_cancel=True)
|