Skip to content

Commit 9ed8c14

Browse files
authored
Merge pull request #3752 from resilient-tech/mergify/bp/version-14-hotfix/pr-3751
fix: always set correct gst treatment (backport #3751)
2 parents f64df9f + 55f6524 commit 9ed8c14

1 file changed

Lines changed: 5 additions & 15 deletions

File tree

india_compliance/gst_india/overrides/transaction.py

Lines changed: 5 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1422,34 +1422,24 @@ def set_for_no_taxes(self):
14221422

14231423
def update_gst_treatment_map(self):
14241424
item_templates = set()
1425-
gst_treatments = set()
1426-
gst_treatment_map = {}
14271425

14281426
for item in self.doc.items:
14291427
item_templates.add(item.item_tax_template)
1430-
gst_treatments.add(item.gst_treatment)
14311428

1432-
if any(
1433-
gst_treatment in gst_treatments
1434-
for gst_treatment in ["Zero-Rated", "Nil-Rated"]
1435-
):
1436-
# doc changed from overseas to local sale post
1437-
# taxes added after save
1438-
_gst_treatments = frappe.get_all(
1429+
self.gst_treatment_map = frappe._dict(
1430+
frappe.get_all(
14391431
"Item Tax Template",
14401432
filters={"name": ("in", item_templates)},
14411433
fields=["name", "gst_treatment"],
1434+
as_list=True,
14421435
)
1443-
gst_treatment_map = {row.name: row.gst_treatment for row in _gst_treatments}
1444-
1445-
self.gst_treatment_map = gst_treatment_map
1436+
)
14461437

14471438
def set_default_treatment(self):
14481439
default_treatment = self.get_default_treatment()
14491440

14501441
for item in self.doc.items:
1451-
if item.gst_treatment in ("Zero-Rated", "Nil-Rated"):
1452-
item.gst_treatment = self.gst_treatment_map.get(item.item_tax_template)
1442+
item.gst_treatment = self.gst_treatment_map.get(item.item_tax_template)
14531443

14541444
if not item.gst_treatment or not item.item_tax_template:
14551445
item.gst_treatment = default_treatment

0 commit comments

Comments
 (0)