Merge pull request #28273 from frappe/mergify/bp/version-13-hotfix/pr-28065

fix(minor): check if gst_category exists while validating GSTIN (backport #28065)
This commit is contained in:
Deepesh Garg
2021-11-08 18:33:37 +05:30
committed by GitHub

View File

@@ -29,6 +29,7 @@ def validate_gstin_for_india(doc, method):
gst_category = [] gst_category = []
if hasattr(doc, 'gst_category'):
if len(doc.links): if len(doc.links):
link_doctype = doc.links[0].get("link_doctype") link_doctype = doc.links[0].get("link_doctype")
link_name = doc.links[0].get("link_name") link_name = doc.links[0].get("link_name")
@@ -76,12 +77,11 @@ def validate_tax_category(doc, method):
frappe.throw(_("Intra State tax category for GST State {0} already exists").format(doc.gst_state)) frappe.throw(_("Intra State tax category for GST State {0} already exists").format(doc.gst_state))
def update_gst_category(doc, method): def update_gst_category(doc, method):
if hasattr(doc, 'gst_category'):
for link in doc.links: for link in doc.links:
if link.link_doctype in ['Customer', 'Supplier']: if link.link_doctype in ['Customer', 'Supplier']:
if doc.get('gstin'): if doc.get('gstin'):
frappe.db.sql(""" frappe.db.set_value(link.link_doctype, {'name': link.link_name, 'gst_category': 'Unregistered'}, 'gst_category', 'Registered Regular')
UPDATE `tab{0}` SET gst_category = %s WHERE name = %s AND gst_category = 'Unregistered'
""".format(link.link_doctype), ("Registered Regular", link.link_name)) #nosec
def set_gst_state_and_state_number(doc): def set_gst_state_and_state_number(doc):
if not doc.gst_state: if not doc.gst_state: