Merge pull request #1150 from 8848digital/trends

fix: PI and SI trends report group by issue
8848digital · Jan 29, 2025 · 0a09d23 · 0a09d23
2 parents 6a6e2dc + 694a252
commit 0a09d23
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/erpnext/controllers/trends.py b/erpnext/controllers/trends.py
@@ -203,13 +203,13 @@ def get_data(filters, conditions):
 					(filters.get("company"), year_start_date, year_end_date, row[i][0], data1[d][0]),
 					as_list=1,
 				)
+				if row1:
+					des[ind] = row[i][0]
 
-				des[ind] = row[i][0]
+					for j in range(1, len(conditions["columns"]) - inc):
+						des[j + inc] = row1[0][j]
 
-				for j in range(1, len(conditions["columns"]) - inc):
-					des[j + inc] = row1[0][j]
-
-				data.append(des)
+					data.append(des)
 	else:
 		data = frappe.db.sql(
 			"""