Skip to content

Commit

Permalink
Merge pull request #38585 from frappe/mergify/bp/version-14-hotfix/pr…
Browse files Browse the repository at this point in the history
…-38580

fix: sql error while filtering on finance book in GL (backport #38580)
  • Loading branch information
ruthra-kumar authored Dec 5, 2023
2 parents 14b908f + 25ec1fc commit b858d1c
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion erpnext/accounts/report/general_ledger/general_ledger.py
Original file line number Diff line number Diff line change
Expand Up @@ -282,7 +282,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(
Expand Down

0 comments on commit b858d1c

Please sign in to comment.