From 79fd881ad51e9cc0fa7a209c91eb2eaa33215a5d Mon Sep 17 00:00:00 2001
From: Sugesh393 <sugeshgk002@gmail.com>
Date: Wed, 11 Dec 2024 11:29:21 +0530
Subject: [PATCH] chore: update variable names

---
 .../customer_ledger_summary.py                | 56 +++++++++++--------
 1 file changed, 33 insertions(+), 23 deletions(-)

diff --git a/erpnext/accounts/report/customer_ledger_summary/customer_ledger_summary.py b/erpnext/accounts/report/customer_ledger_summary/customer_ledger_summary.py
index 39282152530..f11c4c24863 100644
--- a/erpnext/accounts/report/customer_ledger_summary/customer_ledger_summary.py
+++ b/erpnext/accounts/report/customer_ledger_summary/customer_ledger_summary.py
@@ -278,11 +278,19 @@ class PartyLedgerSummaryReport:
 		)
 
 		if self.filters.party_type == "Customer":
-			ctr = qb.DocType("Customer")
-			query = query.select(ctr.customer_name.as_("party_name")).left_join(ctr).on(ctr.name == gle.party)
+			customer = qb.DocType("Customer")
+			query = (
+				query.select(customer.customer_name.as_("party_name"))
+				.left_join(customer)
+				.on(customer.name == gle.party)
+			)
 		elif self.filters.party_type == "Supplier":
-			spr = qb.DocType("Supplier")
-			query = query.select(spr.supplier_name.as_("party_name")).left_join(spr).on(spr.name == gle.party)
+			supplier = qb.DocType("Supplier")
+			query = (
+				query.select(supplier.supplier_name.as_("party_name"))
+				.left_join(supplier)
+				.on(supplier.name == gle.party)
+			)
 
 		query = self.prepare_conditions(query)
 		self.gl_entries = query.run(as_dict=True)
@@ -299,59 +307,61 @@ class PartyLedgerSummaryReport:
 			query = query.where(gle.party == self.filters.party)
 
 		if self.filters.party_type == "Customer":
-			ctr = qb.DocType("Customer")
+			customer = qb.DocType("Customer")
 			if self.filters.customer_group:
 				query = query.where(
 					(gle.party).isin(
-						qb.from_(ctr)
-						.select(ctr.name)
-						.where(ctr.customer_group == self.filters.customer_group)
+						qb.from_(customer)
+						.select(customer.name)
+						.where(customer.customer_group == self.filters.customer_group)
 					)
 				)
 
 			if self.filters.territory:
 				query = query.where(
 					(gle.party).isin(
-						qb.from_(ctr).select(ctr.name).where(ctr.territory == self.filters.territory)
+						qb.from_(customer)
+						.select(customer.name)
+						.where(customer.territory == self.filters.territory)
 					)
 				)
 
 			if self.filters.payment_terms_template:
 				query = query.where(
 					(gle.party).isin(
-						qb.from_(ctr)
-						.select(ctr.name)
-						.where(ctr.payment_terms == self.filters.payment_terms_template)
+						qb.from_(customer)
+						.select(customer.name)
+						.where(customer.payment_terms == self.filters.payment_terms_template)
 					)
 				)
 
 			if self.filters.sales_partner:
 				query = query.where(
 					(gle.party).isin(
-						qb.from_(ctr)
-						.select(ctr.name)
-						.where(ctr.default_sales_partner == self.filters.sales_partner)
+						qb.from_(customer)
+						.select(customer.name)
+						.where(customer.default_sales_partner == self.filters.sales_partner)
 					)
 				)
 
 			if self.filters.sales_person:
-				sl_team = qb.DocType("Sales Team")
+				sales_team = qb.DocType("Sales Team")
 				query = query.where(
 					(gle.party).isin(
-						qb.from_(sl_team)
-						.select(sl_team.parent)
-						.where(sl_team.sales_person == self.filters.sales_person)
+						qb.from_(sales_team)
+						.select(sales_team.parent)
+						.where(sales_team.sales_person == self.filters.sales_person)
 					)
 				)
 
 		if self.filters.party_type == "Supplier":
 			if self.filters.supplier_group:
-				spr = qb.DocType("Supplier")
+				supplier = qb.DocType("Supplier")
 				query = query.where(
 					(gle.party).isin(
-						qb.from_(spr)
-						.select(spr.name)
-						.where(spr.supplier_group == self.filters.supplier_group)
+						qb.from_(supplier)
+						.select(supplier.name)
+						.where(supplier.supplier_group == self.filters.supplier_group)
 					)
 				)