commit f24ed6723ebc16282b3968e9ef5d58e629f7deb1
author hendrik <hendrik.zeta@gmail.com> Wed Oct 27 16:08:20 2021 +0700
committer GitHub <noreply@github.com> Wed Oct 27 14:38:20 2021 +0530
tree a0fca2282852761238472bff993c9973cd73b0be
parent af9ead84e98a92062d401868557fa79e3a92818f

fix(general_ledger): Order by in case Group by Account (#28093)

* Update general_ledger.py

Fix order_by_statement if filter group by: Group by Account

* chore: whitespace

Co-authored-by: Ankush Menat <ankushmenat@gmail.com>
Co-authored-by: Afshan <33727827+AfshanKhan@users.noreply.github.com>

erpnext/accounts/report/general_ledger/general_ledger.py

diff --git a/erpnext/accounts/report/general_ledger/general_ledger.py b/erpnext/accounts/report/general_ledger/general_ledger.py
index 0094bc2..31416da 100644
--- a/erpnext/accounts/report/general_ledger/general_ledger.py
+++ b/erpnext/accounts/report/general_ledger/general_ledger.py
@@ -155,6 +155,8 @@
 
 	if filters.get("group_by") == "Group by Voucher":
 		order_by_statement = "order by posting_date, voucher_type, voucher_no"
+	if filters.get("group_by") == "Group by Account":
+		order_by_statement = "order by account, posting_date, creation"
 
 	if filters.get("include_default_book_entries"):
 		filters['company_fb'] = frappe.db.get_value("Company",