CalvinBackup/erpnext
1
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-03-11 14:29:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: Mandatory bank account error fix (#20734)

Browse Source 
* fix: Mandatory bank account error fix

* fix: SQL condition
This commit is contained in:
Deepesh Garg
2020-02-26 17:17:06 +05:30
committed by GitHub
parent 5475b5f562
commit acbc4f648d
1 changed files with 12 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
erpnext/accounts/doctype/bank_reconciliation/bank_reconciliation.py
View File

@@ -13,9 +13,11 @@ form_grid_templates = {
class BankReconciliation(Document):
def get_payment_entries(self):
if not (self.bank_account and self.from_date and self.to_date):
msgprint(_("Bank Account, From Date and To Date are Mandatory"))
return
if not (self.from_date and self.to_date):
frappe.throw(_("From Date and To Date are Mandatory"))
if not self.account:
frappe.throw(_("Account is mandatory to get payment entries"))
condition = ""
if not self.include_reconciled_entries:
@@ -37,6 +39,11 @@ class BankReconciliation(Document):
order by t1.posting_date ASC, t1.name DESC
""", {"condition":condition, "account": self.account, "from": self.from_date, "to": self.to_date}, as_dict=1)
condition = ''
if self.bank_account:
condition += 'and bank_account = %(bank_account)s'
payment_entries = frappe.db.sql("""
select
"Payment Entry" as payment_document, name as payment_entry,
@@ -49,10 +56,10 @@ class BankReconciliation(Document):
where
(paid_from=%(account)s or paid_to=%(account)s) and docstatus=1
and posting_date >= %(from)s and posting_date <= %(to)s
and bank_account = %(bank_account)s
{condition}
order by
posting_date ASC, name DESC
""", {"account": self.account, "from":self.from_date,
""".format(condition=condition), {"account": self.account, "from":self.from_date,
"to": self.to_date, "bank_account": self.bank_account}, as_dict=1)
pos_entries = []