nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-02-16 08:05:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #38580 from ruthra-kumar/finance_book_filter_bug_in_general_ledger

Browse Source 
fix: sql error while filtering on finance book in GL
This commit is contained in:
ruthra kumar
2023-12-05 14:40:34 +05:30
committed by GitHub
parent aaa9036eca b1d9f3132d
commit 5da3e532c9
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
erpnext/accounts/report/general_ledger/general_ledger.py
View File

@@ -289,7 +289,8 @@ def get_conditions(filters):
if accounting_dimensions:
for dimension in accounting_dimensions:
if not dimension.disabled:
# Ignore 'Finance Book' set up as dimension in below logic, as it is already handled in above section
if not dimension.disabled and dimension.document_type != "Finance Book":
if filters.get(dimension.fieldname):
if frappe.get_cached_value("DocType", dimension.document_type, "is_tree"):
filters[dimension.fieldname] = get_dimension_with_children(