Skip to content

Commit

Permalink
Merge pull request #43235 from vishakhdesai/general-ledger-duplicate-…
Browse files Browse the repository at this point in the history
…rows-issue

fix: set group_by condition to "Group by Voucher (Consolidated)" if `None` and voucher_no is set
  • Loading branch information
ruthra-kumar authored Sep 24, 2024
2 parents c671a6c + a6b0cb6 commit 6954bfc
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions erpnext/accounts/report/general_ledger/general_ledger.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,9 @@ def execute(filters=None):
if filters.get("party"):
filters.party = frappe.parse_json(filters.get("party"))

if filters.get("voucher_no") and not filters.get("group_by"):
filters.group_by = "Group by Voucher (Consolidated)"

validate_filters(filters, account_details)

validate_party(filters)
Expand Down

0 comments on commit 6954bfc

Please sign in to comment.