CalvinBackup/erpnext
1
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-02-13 01:34:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #47118 from aerele/tb-group-by-account

Browse Source 
fix: add group by after user permission condition
This commit is contained in:
ruthra kumar
2025-04-17 16:05:19 +05:30
committed by GitHub
parent e5eafc49ee 756d496235
commit 048997f809
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
erpnext/accounts/report/financial_statements.py
View File

@@ -519,9 +519,6 @@ def get_accounting_entries(
.where(gl_entry.company == filters.company)
)
if group_by_account:
query = query.groupby(gl_entry.account)
ignore_is_opening = frappe.db.get_single_value(
"Accounts Settings", "ignore_is_opening_check_for_reporting"
)
@@ -551,6 +548,9 @@ def get_accounting_entries(
if match_conditions:
query += "and" + match_conditions
if group_by_account:
query += " GROUP BY `account`"
return frappe.db.sql(query, params, as_dict=True)