From a451d0d0bdb099d38db6aeccc29dc6bb9cb5f276 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Francisco=20Rold=C3=A1n?= Date: Tue, 1 Aug 2023 12:30:14 -0300 Subject: [PATCH] =?UTF-8?q?Correcci=C3=B3n=20Conciliaci=C3=B3n=20Bancaria?= =?UTF-8?q?=20(#696)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * feat: Permitir EP mayores * mejora * feat: clearance_date * feat: refresh * fix: remove logs * feat: control * fix * fix * fix * fix: clear_simple_entry --- .../bank_reconciliation_tool.py | 61 ++++++++++++++----- .../bank_transaction/bank_transaction.py | 28 ++++++++- .../dialog_manager.js | 3 + 3 files changed, 73 insertions(+), 19 deletions(-) diff --git a/erpnext/accounts/doctype/bank_reconciliation_tool/bank_reconciliation_tool.py b/erpnext/accounts/doctype/bank_reconciliation_tool/bank_reconciliation_tool.py index c20169b7b45b..a786f78f4a69 100644 --- a/erpnext/accounts/doctype/bank_reconciliation_tool/bank_reconciliation_tool.py +++ b/erpnext/accounts/doctype/bank_reconciliation_tool/bank_reconciliation_tool.py @@ -232,22 +232,29 @@ def reconcile_vouchers(bank_transaction_name, vouchers): if transaction.unallocated_amount == 0: frappe.throw(_("This bank transaction is already fully reconciled")) - total_amount = 0 + + # total_amount = 0 + for voucher in vouchers: voucher['payment_entry'] = frappe.get_doc(voucher['payment_doctype'], voucher['payment_name']) - total_amount += get_paid_amount(frappe._dict({ - 'payment_document': voucher['payment_doctype'], - 'payment_entry': voucher['payment_name'], - }), transaction.currency, company_account) + # total_amount += get_paid_amount(frappe._dict({ + # 'payment_document': voucher['payment_doctype'], + # 'payment_entry': voucher['payment_name'], + # }), transaction.currency, company_account) - if total_amount > transaction.unallocated_amount: - frappe.throw(_("The Sum Total of Amounts of All Selected Vouchers Should be Less than the Unallocated Amount of the Bank Transaction")) + # if total_amount > transaction.unallocated_amount: + # frappe.throw(_("The Sum Total of Amounts of All Selected Vouchers Should be Less than the Unallocated Amount of the Bank Transaction")) account = frappe.db.get_value("Bank Account", transaction.bank_account, "account") for voucher in vouchers: gl_entry = frappe.db.get_value("GL Entry", dict(account=account, voucher_type=voucher['payment_doctype'], voucher_no=voucher['payment_name']), ['credit', 'debit'], as_dict=1) - gl_amount, transaction_amount = (gl_entry.credit, transaction.deposit) if gl_entry.credit > 0 else (gl_entry.debit, transaction.withdrawal) - allocated_amount = gl_amount if gl_amount >= transaction_amount else transaction_amount + gl_amount = gl_entry.credit if gl_entry.credit > 0 else gl_entry.debit + transaction_amount = transaction.deposit if transaction.deposit > 0 else transaction.withdrawal + # gl_amount, transaction_amount = (gl_entry.credit, transaction.deposit) if gl_entry.credit > 0 else (gl_entry.debit, transaction.withdrawal) + reconcilied_amount = transaction.get_reconcilied_amount(voucher['payment_entry'].doctype, voucher['payment_entry'].name) + not_reconcilied = round((gl_amount - reconcilied_amount), 2) + # allocated_amount = gl_amount if gl_amount >= transaction_amount else transaction_amount + allocated_amount = not_reconcilied if not_reconcilied <= transaction_amount else transaction_amount transaction.append("payment_entries", { "payment_document": voucher['payment_entry'].doctype, @@ -308,7 +315,7 @@ def check_matching(bank_account, company, transaction, document_types): def get_queries(bank_account, company, transaction, document_types): # get queries to get matching vouchers - amount_condition = "=" if "exact_match" in document_types else "<=" + amount_condition = "=" if "exact_match" in document_types else None account_from_to = "paid_to" if transaction.deposit > 0 else "paid_from" queries = [] @@ -341,6 +348,11 @@ def get_pe_matching_query(amount_condition, account_from_to, transaction): currency_field = "paid_to_account_currency as currency" else: currency_field = "paid_from_account_currency as currency" + + amount_filter = "True" + if amount_condition: + amount_filter = f"paid_amount {amount_condition} %(amount)s" + return f""" SELECT (CASE WHEN reference_no=%(reference_no)s THEN 1 ELSE 0 END @@ -358,7 +370,7 @@ def get_pe_matching_query(amount_condition, account_from_to, transaction): FROM `tabPayment Entry` WHERE - paid_amount {amount_condition} %(amount)s + {amount_filter} AND docstatus = 1 AND payment_type IN (%(payment_type)s, 'Internal Transfer') AND ifnull(clearance_date, '') = "" @@ -373,8 +385,12 @@ def get_je_matching_query(amount_condition, transaction): # So one bank could have both types of bank accounts like asset and liability # So cr_or_dr should be judged only on basis of withdrawal and deposit and not account type cr_or_dr = "credit" if transaction.withdrawal > 0 else "debit" - return f""" + amount_filter = "" + if amount_condition: + amount_filter = f"AND jea.{cr_or_dr}_in_account_currency {amount_condition} %(amount)s" + + return f""" SELECT (CASE WHEN je.cheque_no=%(reference_no)s THEN 1 ELSE 0 END + 1) AS rank , @@ -396,13 +412,17 @@ def get_je_matching_query(amount_condition, transaction): WHERE (je.clearance_date is null or je.clearance_date='0000-00-00') AND jea.account = %(bank_account)s - AND jea.{cr_or_dr}_in_account_currency {amount_condition} %(amount)s + {amount_filter} AND je.docstatus = 1 """ def get_si_matching_query(amount_condition): # get matchin sales invoice query + amount_filter = "" + if amount_condition: + amount_filter = f"AND sip.amount {amount_condition} %(amount)s" + return f""" SELECT ( CASE WHEN si.customer = %(party)s THEN 1 ELSE 0 END @@ -425,12 +445,16 @@ def get_si_matching_query(amount_condition): sip.parent = si.name WHERE (sip.clearance_date is null or sip.clearance_date='0000-00-00') AND sip.account = %(bank_account)s - AND sip.amount {amount_condition} %(amount)s + {amount_filter} AND si.docstatus = 1 """ def get_pi_matching_query(amount_condition): # get matching purchase invoice query + amount_filter = "True" + if amount_condition: + amount_filter = f"paid_amount {amount_condition} %(amount)s" + return f""" SELECT ( CASE WHEN supplier = %(party)s THEN 1 ELSE 0 END @@ -447,7 +471,7 @@ def get_pi_matching_query(amount_condition): FROM `tabPurchase Invoice` WHERE - paid_amount {amount_condition} %(amount)s + {amount_filter} AND docstatus = 1 AND is_paid = 1 AND ifnull(clearance_date, '') = "" @@ -460,6 +484,11 @@ def get_ec_matching_query(bank_account, company, amount_condition): filters={"default_account": bank_account}, fields=["parent"])] mode_of_payments = '(\'' + '\', \''.join(mode_of_payments) + '\' )' company_currency = get_company_currency(company) + + amount_filter = "True" + if amount_condition: + amount_filter = f"total_sanctioned_amount {amount_condition} %(amount)s" + return f""" SELECT ( CASE WHEN employee = %(party)s THEN 1 ELSE 0 END @@ -476,7 +505,7 @@ def get_ec_matching_query(bank_account, company, amount_condition): FROM `tabExpense Claim` WHERE - total_sanctioned_amount {amount_condition} %(amount)s + {amount_filter} AND docstatus = 1 AND is_paid = 1 AND ifnull(clearance_date, '') = "" diff --git a/erpnext/accounts/doctype/bank_transaction/bank_transaction.py b/erpnext/accounts/doctype/bank_transaction/bank_transaction.py index 1c04cea56438..c59f4557c917 100644 --- a/erpnext/accounts/doctype/bank_transaction/bank_transaction.py +++ b/erpnext/accounts/doctype/bank_transaction/bank_transaction.py @@ -63,9 +63,12 @@ def clear_simple_entry(self, payment_entry, for_cancel=False): return clearance_date = self.date if not for_cancel else None - frappe.db.set_value( - payment_entry.payment_document, payment_entry.payment_entry, - "clearance_date", clearance_date) + + gl_entry = frappe.db.get_value("GL Entry", dict(voucher_type=payment_entry.payment_document, voucher_no=payment_entry.payment_entry), ['credit', 'debit'], as_dict=1) + gl_amount = gl_entry.credit if gl_entry.credit > 0 else gl_entry.debit + + if self.get_reconcilied_amount(payment_entry.payment_document, payment_entry.payment_entry) >= gl_amount: + frappe.db.set_value(payment_entry.payment_document, payment_entry.payment_entry, "clearance_date", clearance_date) def clear_sales_invoice(self, payment_entry, for_cancel=False): clearance_date = self.date if not for_cancel else None @@ -77,6 +80,25 @@ def clear_sales_invoice(self, payment_entry, for_cancel=False): ), "clearance_date", clearance_date) + def get_reconcilied_amount(self, payment_document, payment_entry): + if self.withdrawal > 0: + select = "sum(bt.withdrawal) as sum" + else: + select = "sum(bt.deposit) as sum" + + amount = frappe.db.sql(f""" + select {select} + from `tabBank Transaction` as bt, `tabBank Transaction Payments` as btp + where bt.name = btp.parent + and btp.payment_entry = '{payment_entry}' + and btp.payment_document = '{payment_document}' + group by btp.payment_entry + """, as_dict=1) + + if not len(amount): + return 0 + return amount[0]['sum'] + def get_reconciled_bank_transactions(payment_entry): reconciled_bank_transactions = frappe.get_all( 'Bank Transaction Payments', diff --git a/erpnext/public/js/bank_reconciliation_tool/dialog_manager.js b/erpnext/public/js/bank_reconciliation_tool/dialog_manager.js index 6d90ba1a388e..2736adb8ea3b 100644 --- a/erpnext/public/js/bank_reconciliation_tool/dialog_manager.js +++ b/erpnext/public/js/bank_reconciliation_tool/dialog_manager.js @@ -521,6 +521,7 @@ erpnext.accounts.bank_reconciliation.DialogManager = class DialogManager { frappe.show_alert(alert_string); this.update_dt_cards(response.message); this.dialog.hide(); + cur_page.page.frm.refresh(); }, }); } @@ -545,6 +546,7 @@ erpnext.accounts.bank_reconciliation.DialogManager = class DialogManager { frappe.show_alert(alert_string); this.update_dt_cards(response.message); this.dialog.hide(); + cur_page.page.frm.refresh(); }, }); } @@ -570,6 +572,7 @@ erpnext.accounts.bank_reconciliation.DialogManager = class DialogManager { frappe.show_alert(alert_string); this.update_dt_cards(response.message); this.dialog.hide(); + cur_page.page.frm.refresh(); }, }); }