fix: gst_hsn_code is mandatory if gst_category is overseas

(cherry picked from commit 024e6f2)

india_compliance/gst_india/client_scripts/purchase_invoice.js

@@ -20,7 +20,10 @@ frappe.ui.form.on(DOCTYPE, {
 
     onload: toggle_reverse_charge,
 
-    gst_category: toggle_reverse_charge,
+    gst_category(frm) {
+        toggle_reverse_charge(frm);
+        validate_gst_hsn_code(frm);
+    },
 
     after_save(frm) {
         if (
@@ -90,9 +93,14 @@ frappe.ui.form.on(DOCTYPE, {
 });
 
 frappe.ui.form.on("Purchase Invoice Item", {
-    item_code: toggle_reverse_charge,
+    item_code(frm) {
+        toggle_reverse_charge(frm);
+        validate_gst_hsn_code(frm);
+    },
 
     items_remove: toggle_reverse_charge,
+
+    gst_hsn_code: validate_gst_hsn_code,
 });
 
 function toggle_reverse_charge(frm) {
@@ -104,3 +112,11 @@ function toggle_reverse_charge(frm) {
 
     frm.set_df_property("is_reverse_charge", "read_only", is_read_only);
 }
+
+function validate_gst_hsn_code(frm) {
+    if (frm.doc.gst_category !== "Overseas") return;
+
+    if (frm.doc.items.some(item => !item.gst_hsn_code)) {
+        frappe.throw(__("GST HSN Code is mandatory for Overseas Purchase Invoice."));
+    }
+}

india_compliance/gst_india/overrides/purchase_invoice.py

@@ -43,6 +43,7 @@ def validate(doc, method=None):
     if validate_transaction(doc) is False:
         return
 
+    validate_gst_hsn_code(doc)
     set_ineligibility_reason(doc)
     update_itc_totals(doc)
     validate_supplier_invoice_number(doc)
@@ -51,7 +52,6 @@ def validate(doc, method=None):
 
 
 def on_cancel(doc, method=None):
-
     frappe.db.set_value(
         "GST Inward Supply",
         {"link_doctype": "Purchase Invoice", "link_name": doc.name},
@@ -263,3 +263,12 @@ def validate_reverse_charge(doc):
         return
 
     frappe.throw(_("Reverse Charge is not applicable on Import of Goods"))
+
+
+def validate_gst_hsn_code(doc):
+    if doc.gst_category != "Overseas":
+        return
+
+    for item in doc.items:
+        if not item.get("gst_hsn_code"):
+            frappe.throw(_("GST HSN Code is mandatory for Overseas Purchase Invoice"))