Anand Doshi | 885e074 | 2015-03-03 14:55:30 +0530 | [diff] [blame] | 1 | # Copyright (c) 2015, Frappe Technologies Pvt. Ltd. and Contributors |
Rushabh Mehta | e67d1fb | 2013-08-05 14:59:54 +0530 | [diff] [blame] | 2 | # License: GNU General Public License v3. See license.txt |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 3 | |
| 4 | from __future__ import unicode_literals |
Rohit Waghchaure | e9ff191 | 2017-06-19 12:54:59 +0530 | [diff] [blame] | 5 | import frappe, erpnext |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 6 | from frappe.utils import flt, cstr, cint, comma_and, today, getdate, formatdate, now |
Rushabh Mehta | 793ba6b | 2014-02-14 15:47:51 +0530 | [diff] [blame] | 7 | from frappe import _ |
Anurag Mishra | a11e738 | 2019-10-31 15:55:03 +0530 | [diff] [blame] | 8 | from erpnext.accounts.utils import get_stock_and_account_balance |
Nabin Hait | e2c200a | 2015-05-28 13:00:37 +0530 | [diff] [blame] | 9 | from frappe.model.meta import get_field_precision |
Nabin Hait | b9bc7d6 | 2016-05-16 14:38:47 +0530 | [diff] [blame] | 10 | from erpnext.accounts.doctype.budget.budget import validate_expense_against_budget |
deepeshgarg007 | 3d11ac0 | 2019-05-19 00:02:01 +0530 | [diff] [blame] | 11 | from erpnext.accounts.doctype.accounting_dimension.accounting_dimension import get_accounting_dimensions |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 12 | |
Nabin Hait | bb77756 | 2013-08-29 18:19:37 +0530 | [diff] [blame] | 13 | |
Mangesh-Khairnar | e5c733b | 2019-08-08 15:44:11 +0530 | [diff] [blame] | 14 | class ClosedAccountingPeriod(frappe.ValidationError): pass |
Rushabh Mehta | 793ba6b | 2014-02-14 15:47:51 +0530 | [diff] [blame] | 15 | class StockAccountInvalidTransaction(frappe.ValidationError): pass |
Anurag Mishra | a11e738 | 2019-10-31 15:55:03 +0530 | [diff] [blame] | 16 | class StockValueAndAccountBalanceOutOfSync(frappe.ValidationError): pass |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 17 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 18 | def make_gl_entries(gl_map, cancel=False, adv_adj=False, merge_entries=True, update_outstanding='Yes'): |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 19 | if gl_map: |
| 20 | if not cancel: |
Mangesh-Khairnar | e5c733b | 2019-08-08 15:44:11 +0530 | [diff] [blame] | 21 | validate_accounting_period(gl_map) |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 22 | gl_map = process_gl_map(gl_map, merge_entries) |
nabinhait | c343292 | 2014-07-29 18:06:18 +0530 | [diff] [blame] | 23 | if gl_map and len(gl_map) > 1: |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 24 | save_entries(gl_map, adv_adj, update_outstanding) |
nabinhait | c343292 | 2014-07-29 18:06:18 +0530 | [diff] [blame] | 25 | else: |
nabinhait | 4859b48 | 2014-07-30 14:28:24 +0530 | [diff] [blame] | 26 | frappe.throw(_("Incorrect number of General Ledger Entries found. You might have selected a wrong Account in the transaction.")) |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 27 | else: |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 28 | make_reverse_gl_entries(gl_map, adv_adj=adv_adj, update_outstanding=update_outstanding) |
Anand Doshi | 652bc07 | 2014-04-16 15:21:46 +0530 | [diff] [blame] | 29 | |
Mangesh-Khairnar | e5c733b | 2019-08-08 15:44:11 +0530 | [diff] [blame] | 30 | def validate_accounting_period(gl_map): |
| 31 | accounting_periods = frappe.db.sql(""" SELECT |
| 32 | ap.name as name |
| 33 | FROM |
| 34 | `tabAccounting Period` ap, `tabClosed Document` cd |
| 35 | WHERE |
| 36 | ap.name = cd.parent |
| 37 | AND ap.company = %(company)s |
| 38 | AND cd.closed = 1 |
| 39 | AND cd.document_type = %(voucher_type)s |
| 40 | AND %(date)s between ap.start_date and ap.end_date |
| 41 | """, { |
| 42 | 'date': gl_map[0].posting_date, |
| 43 | 'company': gl_map[0].company, |
| 44 | 'voucher_type': gl_map[0].voucher_type |
| 45 | }, as_dict=1) |
| 46 | |
| 47 | if accounting_periods: |
| 48 | frappe.throw(_("You can't create accounting entries in the closed accounting period {0}") |
| 49 | .format(accounting_periods[0].name), ClosedAccountingPeriod) |
| 50 | |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 51 | def process_gl_map(gl_map, merge_entries=True): |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 52 | if merge_entries: |
| 53 | gl_map = merge_similar_entries(gl_map) |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 54 | for entry in gl_map: |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 55 | # toggle debit, credit if negative entry |
Nabin Hait | 88f3cd5 | 2013-10-23 16:29:19 +0530 | [diff] [blame] | 56 | if flt(entry.debit) < 0: |
| 57 | entry.credit = flt(entry.credit) - flt(entry.debit) |
| 58 | entry.debit = 0.0 |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 59 | |
Nabin Hait | c561a49 | 2015-08-19 19:22:34 +0530 | [diff] [blame] | 60 | if flt(entry.debit_in_account_currency) < 0: |
| 61 | entry.credit_in_account_currency = \ |
| 62 | flt(entry.credit_in_account_currency) - flt(entry.debit_in_account_currency) |
| 63 | entry.debit_in_account_currency = 0.0 |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 64 | |
Nabin Hait | 88f3cd5 | 2013-10-23 16:29:19 +0530 | [diff] [blame] | 65 | if flt(entry.credit) < 0: |
| 66 | entry.debit = flt(entry.debit) - flt(entry.credit) |
| 67 | entry.credit = 0.0 |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 68 | |
Nabin Hait | c561a49 | 2015-08-19 19:22:34 +0530 | [diff] [blame] | 69 | if flt(entry.credit_in_account_currency) < 0: |
| 70 | entry.debit_in_account_currency = \ |
| 71 | flt(entry.debit_in_account_currency) - flt(entry.credit_in_account_currency) |
| 72 | entry.credit_in_account_currency = 0.0 |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 73 | |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 74 | return gl_map |
Anand Doshi | 652bc07 | 2014-04-16 15:21:46 +0530 | [diff] [blame] | 75 | |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 76 | def merge_similar_entries(gl_map): |
| 77 | merged_gl_map = [] |
deepeshgarg007 | 3d11ac0 | 2019-05-19 00:02:01 +0530 | [diff] [blame] | 78 | accounting_dimensions = get_accounting_dimensions() |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 79 | for entry in gl_map: |
Anand Doshi | 652bc07 | 2014-04-16 15:21:46 +0530 | [diff] [blame] | 80 | # if there is already an entry in this account then just add it |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 81 | # to that entry |
deepeshgarg007 | 3d11ac0 | 2019-05-19 00:02:01 +0530 | [diff] [blame] | 82 | same_head = check_if_in_list(entry, merged_gl_map, accounting_dimensions) |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 83 | if same_head: |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 84 | same_head.debit = flt(same_head.debit) + flt(entry.debit) |
Nabin Hait | c561a49 | 2015-08-19 19:22:34 +0530 | [diff] [blame] | 85 | same_head.debit_in_account_currency = \ |
| 86 | flt(same_head.debit_in_account_currency) + flt(entry.debit_in_account_currency) |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 87 | same_head.credit = flt(same_head.credit) + flt(entry.credit) |
Nabin Hait | c561a49 | 2015-08-19 19:22:34 +0530 | [diff] [blame] | 88 | same_head.credit_in_account_currency = \ |
| 89 | flt(same_head.credit_in_account_currency) + flt(entry.credit_in_account_currency) |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 90 | else: |
| 91 | merged_gl_map.append(entry) |
Anand Doshi | 652bc07 | 2014-04-16 15:21:46 +0530 | [diff] [blame] | 92 | |
thefalconx33 | bfc43d3 | 2019-12-13 15:09:51 +0530 | [diff] [blame] | 93 | company = gl_map[0].company if gl_map else erpnext.get_default_company() |
| 94 | company_currency = erpnext.get_company_currency(company) |
| 95 | precision = get_field_precision(frappe.get_meta("GL Entry").get_field("debit"), company_currency) |
| 96 | |
Nabin Hait | 815a49e | 2013-08-07 17:00:01 +0530 | [diff] [blame] | 97 | # filter zero debit and credit entries |
thefalconx33 | bfc43d3 | 2019-12-13 15:09:51 +0530 | [diff] [blame] | 98 | merged_gl_map = filter(lambda x: flt(x.debit, precision)!=0 or flt(x.credit, precision)!=0, merged_gl_map) |
Achilles Rasquinha | 908289d | 2018-03-08 13:10:51 +0530 | [diff] [blame] | 99 | merged_gl_map = list(merged_gl_map) |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 100 | |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 101 | return merged_gl_map |
| 102 | |
deepeshgarg007 | 3d11ac0 | 2019-05-19 00:02:01 +0530 | [diff] [blame] | 103 | def check_if_in_list(gle, gl_map, dimensions=None): |
| 104 | account_head_fieldnames = ['party_type', 'party', 'against_voucher', 'against_voucher_type', |
| 105 | 'cost_center', 'project'] |
| 106 | |
| 107 | if dimensions: |
| 108 | account_head_fieldnames = account_head_fieldnames + dimensions |
| 109 | |
Nabin Hait | bb77756 | 2013-08-29 18:19:37 +0530 | [diff] [blame] | 110 | for e in gl_map: |
deepeshgarg007 | 3d11ac0 | 2019-05-19 00:02:01 +0530 | [diff] [blame] | 111 | same_head = True |
| 112 | if e.account != gle.account: |
| 113 | same_head = False |
| 114 | |
| 115 | for fieldname in account_head_fieldnames: |
| 116 | if cstr(e.get(fieldname)) != cstr(gle.get(fieldname)): |
| 117 | same_head = False |
| 118 | |
| 119 | if same_head: |
| 120 | return e |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 121 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 122 | def save_entries(gl_map, adv_adj, update_outstanding): |
| 123 | validate_cwip_accounts(gl_map) |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 124 | |
Nabin Hait | e2c200a | 2015-05-28 13:00:37 +0530 | [diff] [blame] | 125 | round_off_debit_credit(gl_map) |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 126 | |
| 127 | if gl_map: |
| 128 | check_freezing_date(gl_map[0]["posting_date"], adv_adj) |
| 129 | |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 130 | for entry in gl_map: |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 131 | make_entry(entry, adv_adj, update_outstanding) |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 132 | |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 133 | # check against budget |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 134 | validate_expense_against_budget(entry) |
Anand Doshi | 652bc07 | 2014-04-16 15:21:46 +0530 | [diff] [blame] | 135 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 136 | validate_account_for_perpetual_inventory(gl_map) |
Anurag Mishra | a11e738 | 2019-10-31 15:55:03 +0530 | [diff] [blame] | 137 | |
| 138 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 139 | def make_entry(args, adv_adj, update_outstanding): |
Nabin Hait | ddc3bb2 | 2020-03-17 17:04:18 +0530 | [diff] [blame] | 140 | gle = frappe.new_doc("GL Entry") |
| 141 | gle.update(args) |
Anand Doshi | 6dfd430 | 2015-02-10 14:41:27 +0530 | [diff] [blame] | 142 | gle.flags.ignore_permissions = 1 |
Saqib | fb35a54 | 2020-02-27 18:32:19 +0530 | [diff] [blame] | 143 | gle.validate() |
Saqib | fb35a54 | 2020-02-27 18:32:19 +0530 | [diff] [blame] | 144 | gle.db_insert() |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 145 | gle.run_method("on_update_with_args", adv_adj, update_outstanding) |
Saqib | fb35a54 | 2020-02-27 18:32:19 +0530 | [diff] [blame] | 146 | gle.flags.ignore_validate = True |
Nabin Hait | aeba24e | 2013-08-23 15:17:36 +0530 | [diff] [blame] | 147 | gle.submit() |
Anand Doshi | 652bc07 | 2014-04-16 15:21:46 +0530 | [diff] [blame] | 148 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 149 | # check against budget |
| 150 | validate_expense_against_budget(args) |
| 151 | |
Rohit Waghchaure | a5f4094 | 2017-06-16 15:21:36 +0530 | [diff] [blame] | 152 | def validate_account_for_perpetual_inventory(gl_map): |
Anurag Mishra | a11e738 | 2019-10-31 15:55:03 +0530 | [diff] [blame] | 153 | if cint(erpnext.is_perpetual_inventory_enabled(gl_map[0].company)): |
| 154 | account_list = [gl_entries.account for gl_entries in gl_map] |
Anand Doshi | 652bc07 | 2014-04-16 15:21:46 +0530 | [diff] [blame] | 155 | |
Anurag Mishra | a11e738 | 2019-10-31 15:55:03 +0530 | [diff] [blame] | 156 | aii_accounts = [d.name for d in frappe.get_all("Account", |
| 157 | filters={'account_type': 'Stock', 'is_group': 0, 'company': gl_map[0].company})] |
| 158 | |
| 159 | for account in account_list: |
| 160 | if account not in aii_accounts: |
| 161 | continue |
| 162 | |
| 163 | account_bal, stock_bal, warehouse_list = get_stock_and_account_balance(account, |
| 164 | gl_map[0].posting_date, gl_map[0].company) |
| 165 | |
| 166 | if gl_map[0].voucher_type=="Journal Entry": |
| 167 | # In case of Journal Entry, there are no corresponding SL entries, |
| 168 | # hence deducting currency amount |
| 169 | account_bal -= flt(gl_map[0].debit) - flt(gl_map[0].credit) |
| 170 | if account_bal == stock_bal: |
Nabin Hait | e7f479b | 2015-06-22 07:31:49 +0530 | [diff] [blame] | 171 | frappe.throw(_("Account: {0} can only be updated via Stock Transactions") |
Anurag Mishra | a11e738 | 2019-10-31 15:55:03 +0530 | [diff] [blame] | 172 | .format(account), StockAccountInvalidTransaction) |
| 173 | |
rohitwaghchaure | 1511154 | 2019-11-28 16:43:39 +0530 | [diff] [blame] | 174 | # This has been comment for a temporary, will add this code again on release of immutable ledger |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 175 | elif account_bal != stock_bal: |
| 176 | precision = get_field_precision(frappe.get_meta("GL Entry").get_field("debit"), |
| 177 | currency=frappe.get_cached_value('Company', gl_map[0].company, "default_currency")) |
Marica | c9e8a1b | 2019-11-14 16:13:43 +0530 | [diff] [blame] | 178 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 179 | diff = flt(stock_bal - account_bal, precision) |
| 180 | error_reason = _("Stock Value ({0}) and Account Balance ({1}) are out of sync for account {2} and it's linked warehouses.").format( |
| 181 | stock_bal, account_bal, frappe.bold(account)) |
| 182 | error_resolution = _("Please create adjustment Journal Entry for amount {0} ").format(frappe.bold(diff)) |
| 183 | stock_adjustment_account = frappe.db.get_value("Company",gl_map[0].company,"stock_adjustment_account") |
Nabin Hait | 35e8d1e | 2019-11-25 14:02:51 +0530 | [diff] [blame] | 184 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 185 | db_or_cr_warehouse_account =('credit_in_account_currency' if diff < 0 else 'debit_in_account_currency') |
| 186 | db_or_cr_stock_adjustment_account = ('debit_in_account_currency' if diff < 0 else 'credit_in_account_currency') |
Nabin Hait | 35e8d1e | 2019-11-25 14:02:51 +0530 | [diff] [blame] | 187 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 188 | journal_entry_args = { |
| 189 | 'accounts':[ |
| 190 | {'account': account, db_or_cr_warehouse_account : abs(diff)}, |
| 191 | {'account': stock_adjustment_account, db_or_cr_stock_adjustment_account : abs(diff) }] |
| 192 | } |
Nabin Hait | 35e8d1e | 2019-11-25 14:02:51 +0530 | [diff] [blame] | 193 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 194 | frappe.msgprint(msg="""{0}<br></br>{1}<br></br>""".format(error_reason, error_resolution), |
| 195 | raise_exception=StockValueAndAccountBalanceOutOfSync, |
| 196 | title=_('Values Out Of Sync'), |
| 197 | primary_action={ |
| 198 | 'label': _('Make Journal Entry'), |
| 199 | 'client_action': 'erpnext.route_to_adjustment_jv', |
| 200 | 'args': journal_entry_args |
| 201 | }) |
Anand Doshi | 652bc07 | 2014-04-16 15:21:46 +0530 | [diff] [blame] | 202 | |
Anurag Mishra | 7e1987e | 2019-08-05 10:18:57 +0530 | [diff] [blame] | 203 | def validate_cwip_accounts(gl_map): |
Saqib | f37a46e | 2019-11-22 16:32:50 +0530 | [diff] [blame] | 204 | cwip_enabled = any([cint(ac.enable_cwip_accounting) for ac in frappe.db.get_all("Asset Category","enable_cwip_accounting")]) |
Marica | d00c598 | 2019-11-12 19:17:43 +0530 | [diff] [blame] | 205 | |
| 206 | if cwip_enabled and gl_map[0].voucher_type == "Journal Entry": |
Anurag Mishra | 7e1987e | 2019-08-05 10:18:57 +0530 | [diff] [blame] | 207 | cwip_accounts = [d[0] for d in frappe.db.sql("""select name from tabAccount |
| 208 | where account_type = 'Capital Work in Progress' and is_group=0""")] |
| 209 | |
| 210 | for entry in gl_map: |
| 211 | if entry.account in cwip_accounts: |
Marica | d00c598 | 2019-11-12 19:17:43 +0530 | [diff] [blame] | 212 | frappe.throw( |
| 213 | _("Account: <b>{0}</b> is capital Work in progress and can not be updated by Journal Entry").format(entry.account)) |
Anurag Mishra | 7e1987e | 2019-08-05 10:18:57 +0530 | [diff] [blame] | 214 | |
Nabin Hait | e2c200a | 2015-05-28 13:00:37 +0530 | [diff] [blame] | 215 | def round_off_debit_credit(gl_map): |
| 216 | precision = get_field_precision(frappe.get_meta("GL Entry").get_field("debit"), |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 217 | currency=frappe.get_cached_value('Company', gl_map[0].company, "default_currency")) |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 218 | |
Nabin Hait | e2c200a | 2015-05-28 13:00:37 +0530 | [diff] [blame] | 219 | debit_credit_diff = 0.0 |
| 220 | for entry in gl_map: |
| 221 | entry.debit = flt(entry.debit, precision) |
| 222 | entry.credit = flt(entry.credit, precision) |
| 223 | debit_credit_diff += entry.debit - entry.credit |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 224 | |
Nabin Hait | e2c200a | 2015-05-28 13:00:37 +0530 | [diff] [blame] | 225 | debit_credit_diff = flt(debit_credit_diff, precision) |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 226 | |
Nabin Hait | d2a966e | 2017-05-05 10:41:16 +0530 | [diff] [blame] | 227 | if gl_map[0]["voucher_type"] in ("Journal Entry", "Payment Entry"): |
Nabin Hait | fb24a27 | 2016-02-18 19:18:07 +0530 | [diff] [blame] | 228 | allowance = 5.0 / (10**precision) |
| 229 | else: |
Nabin Hait | d2a966e | 2017-05-05 10:41:16 +0530 | [diff] [blame] | 230 | allowance = .5 |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 231 | |
Nabin Hait | fb24a27 | 2016-02-18 19:18:07 +0530 | [diff] [blame] | 232 | if abs(debit_credit_diff) >= allowance: |
Nabin Hait | e2c200a | 2015-05-28 13:00:37 +0530 | [diff] [blame] | 233 | frappe.throw(_("Debit and Credit not equal for {0} #{1}. Difference is {2}.") |
| 234 | .format(gl_map[0].voucher_type, gl_map[0].voucher_no, debit_credit_diff)) |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 235 | |
Nabin Hait | e2c200a | 2015-05-28 13:00:37 +0530 | [diff] [blame] | 236 | elif abs(debit_credit_diff) >= (1.0 / (10**precision)): |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 237 | make_round_off_gle(gl_map, debit_credit_diff, precision) |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 238 | |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 239 | def make_round_off_gle(gl_map, debit_credit_diff, precision): |
Nabin Hait | 2e4de83 | 2017-09-19 14:53:16 +0530 | [diff] [blame] | 240 | round_off_account, round_off_cost_center = get_round_off_account_and_cost_center(gl_map[0].company) |
Zarrar | 3523b77 | 2018-08-14 16:28:14 +0530 | [diff] [blame] | 241 | round_off_account_exists = False |
Nabin Hait | 80069a6 | 2015-05-28 19:19:59 +0530 | [diff] [blame] | 242 | round_off_gle = frappe._dict() |
Zarrar | 3523b77 | 2018-08-14 16:28:14 +0530 | [diff] [blame] | 243 | for d in gl_map: |
| 244 | if d.account == round_off_account: |
| 245 | round_off_gle = d |
| 246 | if d.debit_in_account_currency: |
| 247 | debit_credit_diff -= flt(d.debit_in_account_currency) |
| 248 | else: |
| 249 | debit_credit_diff += flt(d.credit_in_account_currency) |
| 250 | round_off_account_exists = True |
| 251 | |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 252 | if round_off_account_exists and abs(debit_credit_diff) <= (1.0 / (10**precision)): |
| 253 | gl_map.remove(round_off_gle) |
| 254 | return |
| 255 | |
Zarrar | 3523b77 | 2018-08-14 16:28:14 +0530 | [diff] [blame] | 256 | if not round_off_gle: |
| 257 | for k in ["voucher_type", "voucher_no", "company", |
| 258 | "posting_date", "remarks", "is_opening"]: |
| 259 | round_off_gle[k] = gl_map[0][k] |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 260 | |
Nabin Hait | e2c200a | 2015-05-28 13:00:37 +0530 | [diff] [blame] | 261 | round_off_gle.update({ |
| 262 | "account": round_off_account, |
Nabin Hait | d4507ac | 2016-01-20 16:14:27 +0530 | [diff] [blame] | 263 | "debit_in_account_currency": abs(debit_credit_diff) if debit_credit_diff < 0 else 0, |
| 264 | "credit_in_account_currency": debit_credit_diff if debit_credit_diff > 0 else 0, |
Nabin Hait | e2c200a | 2015-05-28 13:00:37 +0530 | [diff] [blame] | 265 | "debit": abs(debit_credit_diff) if debit_credit_diff < 0 else 0, |
| 266 | "credit": debit_credit_diff if debit_credit_diff > 0 else 0, |
Nabin Hait | 3c67146 | 2015-05-28 13:19:01 +0530 | [diff] [blame] | 267 | "cost_center": round_off_cost_center, |
| 268 | "party_type": None, |
| 269 | "party": None, |
| 270 | "against_voucher_type": None, |
| 271 | "against_voucher": None |
Nabin Hait | e2c200a | 2015-05-28 13:00:37 +0530 | [diff] [blame] | 272 | }) |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 273 | |
Zarrar | 3523b77 | 2018-08-14 16:28:14 +0530 | [diff] [blame] | 274 | if not round_off_account_exists: |
| 275 | gl_map.append(round_off_gle) |
Nabin Hait | e2c200a | 2015-05-28 13:00:37 +0530 | [diff] [blame] | 276 | |
Nabin Hait | 2e4de83 | 2017-09-19 14:53:16 +0530 | [diff] [blame] | 277 | def get_round_off_account_and_cost_center(company): |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 278 | round_off_account, round_off_cost_center = frappe.get_cached_value('Company', company, |
Nabin Hait | 2e4de83 | 2017-09-19 14:53:16 +0530 | [diff] [blame] | 279 | ["round_off_account", "round_off_cost_center"]) or [None, None] |
| 280 | if not round_off_account: |
| 281 | frappe.throw(_("Please mention Round Off Account in Company")) |
| 282 | |
| 283 | if not round_off_cost_center: |
| 284 | frappe.throw(_("Please mention Round Off Cost Center in Company")) |
| 285 | |
| 286 | return round_off_account, round_off_cost_center |
| 287 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 288 | def make_reverse_gl_entries(gl_entries=None, voucher_type=None, voucher_no=None, |
| 289 | adv_adj=False, update_outstanding="Yes"): |
| 290 | """ |
| 291 | Get original gl entries of the voucher |
| 292 | and make reverse gl entries by swapping debit and credit |
| 293 | """ |
Anand Doshi | 652bc07 | 2014-04-16 15:21:46 +0530 | [diff] [blame] | 294 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 295 | if not gl_entries: |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 296 | gl_entries = frappe.get_all("GL Entry", |
| 297 | fields = ["*"], |
| 298 | filters = { |
| 299 | "voucher_type": voucher_type, |
| 300 | "voucher_no": voucher_no |
| 301 | }) |
Nabin Hait | 56548cb | 2016-07-01 15:58:39 +0530 | [diff] [blame] | 302 | |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 303 | if gl_entries: |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 304 | set_as_cancel(gl_entries[0]['voucher_type'], gl_entries[0]['voucher_no']) |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 305 | check_freezing_date(gl_entries[0]["posting_date"], adv_adj) |
Anand Doshi | 652bc07 | 2014-04-16 15:21:46 +0530 | [diff] [blame] | 306 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 307 | for entry in gl_entries: |
| 308 | entry['name'] = None |
| 309 | debit = entry.get('debit', 0) |
| 310 | credit = entry.get('credit', 0) |
Anand Doshi | 652bc07 | 2014-04-16 15:21:46 +0530 | [diff] [blame] | 311 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 312 | debit_in_account_currency = entry.get('debit_in_account_currency', 0) |
| 313 | credit_in_account_currency = entry.get('credit_in_account_currency', 0) |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 314 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 315 | entry['debit'] = credit |
| 316 | entry['credit'] = debit |
| 317 | entry['debit_in_account_currency'] = credit_in_account_currency |
| 318 | entry['credit_in_account_currency'] = debit_in_account_currency |
| 319 | |
| 320 | entry['remarks'] = "On cancellation of " + entry['voucher_no'] |
| 321 | entry['is_cancelled'] = 1 |
| 322 | entry['posting_date'] = today() |
| 323 | |
| 324 | if entry['debit'] or entry['credit']: |
| 325 | make_entry(entry, adv_adj, "Yes") |
| 326 | |
| 327 | |
| 328 | def check_freezing_date(posting_date, adv_adj=False): |
| 329 | """ |
| 330 | Nobody can do GL Entries where posting date is before freezing date |
| 331 | except authorized person |
| 332 | """ |
| 333 | if not adv_adj: |
| 334 | acc_frozen_upto = frappe.db.get_value('Accounts Settings', None, 'acc_frozen_upto') |
| 335 | if acc_frozen_upto: |
| 336 | frozen_accounts_modifier = frappe.db.get_value( 'Accounts Settings', None,'frozen_accounts_modifier') |
| 337 | if getdate(posting_date) <= getdate(acc_frozen_upto) \ |
| 338 | and not frozen_accounts_modifier in frappe.get_roles(): |
| 339 | frappe.throw(_("You are not authorized to add or update entries before {0}").format(formatdate(acc_frozen_upto))) |
| 340 | |
| 341 | def set_as_cancel(voucher_type, voucher_no): |
| 342 | """ |
| 343 | Set is_cancelled=1 in all original gl entries for the voucher |
| 344 | """ |
| 345 | frappe.db.sql("""update `tabGL Entry` set is_cancelled = 1, |
| 346 | modified=%s, modified_by=%s |
| 347 | where voucher_type=%s and voucher_no=%s and is_cancelled = 0""", |
| 348 | (now(), frappe.session.user, voucher_type, voucher_no)) |