refactor(sales): drop purchase-only fields from Sales Invoice/Order

Sales Invoice and Sales Order keep only the Agricultural Goods checkbox;
remove purchase_type, act_type and agricultural_country (purchase-side
fields that made no sense on sales documents). Also remove the now-pointless
sales_classification.js (purchase_type <-> agricultural_goods mutual
exclusion) and its hook registration.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

jey_erp/custom_fields.py

@@ -620,36 +620,12 @@ def create_custom_fields():
                 insert_after='is_taxes_doc',
                 hidden=1
             ),
-            # Purchase type fields
-            dict(
-                fieldname='purchase_type',
-                label='Purchase Type',
-                fieldtype='Check',
-                insert_after='taxes_doc',
-                default=0
-            ),
             dict(
                 fieldname='agricultural_goods',
                 label='Agricultural Goods',
                 fieldtype='Check',
-                insert_after='purchase_type',
+                insert_after='taxes_doc',
                 default=0
-            ),
-            dict(
-                fieldname='act_type',
-                label='Act Type',
-                fieldtype='Select',
-                options='\nPurchase Act\nEQF\nImport',
-                insert_after='agricultural_goods',
-                depends_on='eval:doc.purchase_type || doc.agricultural_goods'
-            ),
-            dict(
-                fieldname='agricultural_country',
-                label='Country',
-                fieldtype='Select',
-                options='\nAruba\nƏfqanıstan\nAnqola\nAngilya\nAlbaniya\nAndorra\nNiderland Antil adaları\nBƏƏ\nArgentina\nErmənistan\nAmerika Samoası\nAntiqua və Barbuda\nAvstraliya\nAvstriya\nBurundi\nBelçika\nBenin\nBurkina-Faso\nBanqladeş\nBolqarıstan\nBəhreyn\nBaham adaları\nBosniya və Herseqovina\nBelarus\nBeliz\nBermud adaları\nBoliviya\nBraziliya\nBarbados\nBruney\nButan\nBotsvana\nMAR\nKanada\nKokos adaları\nİsveçrə\nÇili\nÇin\nKot-d\'ivuar\nKamerun\nKDR\nKonqo Respublikası\nKuk adaları\nKolumbiya\nKomor adaları\nKabo-Verde\nKosta-Rika\nKuba\nMilad adası\nKayman adaları\nKipr\nÇexiya\nAlmaniya\nCibuti\nDominika\nDanimarka\nDominikan Respublikası\nƏlcəzair\nEkvador\nMisir\nEritreya\nQərbi Sahara\nİspaniya\nEstoniya\nEfiopiya\nFinlandiya\nFici\nFoklend adaları\nFransa\nFarer adaları\nMikroneziya\nQabon\nBöyük Britaniya\nGürcüstan\nQana\nCəbəli-Tariq\nQvineya\nQambiya\nQvineya-Bisau\nEkvatorial Qvineya\nYunanıstan\nQrenada\nQrenlandiya\nQvatemala\nQuam\nQayana\nHonkonq\nHonduras\nXorvatiya\nHaiti\nMacarıstan\nindoneziya\nHindistan\nirlandiya\nİran\nİraq\nislandiya\nisrail\nitaliya\nYamayka\nlordaniya\nYaponiya\nQazağıstan\nKeniya\nQırğızıstan\nKamboca\nKiribati\nSent-Kits və Nevis\nCənubi Koreya\nKüveyt\nLaos\nLivan\nLiberiya\nLiviya\nSent-Lüsiya\nLixtenşteyn\nŞri-Lanka\nLesoto\nLitva\nLüksemburq\nLatviya\nMakao\nMərakeş\nMonako\nMoldova\nMadaqaskar\nMaldiv adaları\nMeksika\nMarşall adaları\nMakedoniya\nMali\nMalta\nMyanma\nMonteneqro\nMonqolustan\nŞimali Marian adaları\nMozambik\nMavritaniya\nMontserrat\nMavriki\nMalavi\nMalayziya\nMayotta\nNamibiya\nYeni Kaledoniya\nNiger\nNorfolk adası\nNigeriya\nNikaraqua\nNiue\nNiderland\nNorveç\nNepal\nNauru\nYeni Zelandiya\nOman\nPakistan\nPanama\nPitkern adaları\nPeru\nFilippin\nPalau\nPapua-Yeni Qvineya\nPolşa\nPuerto-Riko\nKXDR\nPortuqaliya\nParaqvay\nFələstin Dövləti\nFransa Polineziyası\nQətər\nRuminiya\nRusiya\nRuanda\nSəudiyyə Ərəbistanı\nSudan\nSeneqal\nSinqapur\nMüqəddəs Yelena, Askenson və Tristan-da-Kunya adaları\nŞpisbergen və Yan-Mayen\nSolomon adaları\nSyerra-Leone\nSalvador\nSan-Marino\nSomali\nSen-Pyer və Mikelon\nSerbiya\nCənubi Sudan\nSan-Tome və Prinsipi\nSurinam\nSlovakiya\nSloeniya\nİsveç\nSvazilend\nSeyşel adaları\nSuriya\nTörks və Kaykos adaları\nÇad\nToqo\nTailand\nTacikistan\nTokelau\nTürkmənistan\nŞərqi Timor\nTonqa\nTrinidad və Tobaqo\nTunis\nTürkiyə\nTuvalu\nÇin Respublikası\nTanzaniya\nUqanda\nUkrayna\nUruqvay\nABŞ\nÖzbəkistan\nVatikan\nSent-Vinsent və Qrenadin\nVenesuela\nBritaniya Virgin adaları\nAmerika Virgin adaları\nVyetnam\nVanuatu\nUollis və Futuna\nSamoa\nYəmən\nCAR\nZambiya\nZimbabve',
-                insert_after='act_type',
-                depends_on='eval:(doc.purchase_type || doc.agricultural_goods) && doc.act_type == "Import"'
             )
         ],
         "Sales Order Item": [
@@ -756,37 +732,13 @@ def create_custom_fields():
                 depends_on='eval:doc.company_main_activity && ["92000", "9200003", "9200004", "9200001", "9200002", "9200005"].includes(doc.company_main_activity)',
                 allow_on_submit=1
             ),
-            # Purchase type fields
-            dict(
-                fieldname='purchase_type',
-                label='Purchase Type',
-                fieldtype='Check',
-                insert_after='seller',
-                default=0
-            ),
             dict(
                 fieldname='agricultural_goods',
                 label='Agricultural Goods',
                 fieldtype='Check',
-                insert_after='purchase_type',
+                insert_after='seller',
                 default=0
             ),
-            dict(
-                fieldname='act_type',
-                label='Act Type',
-                fieldtype='Select',
-                options='\nPurchase Act\nEQF\nImport',
-                insert_after='agricultural_goods',
-                depends_on='eval:doc.purchase_type || doc.agricultural_goods'
-            ),
-            dict(
-                fieldname='agricultural_country',
-                label='Country',
-                fieldtype='Select',
-                options='\nAruba\nƏfqanıstan\nAnqola\nAngilya\nAlbaniya\nAndorra\nNiderland Antil adaları\nBƏƏ\nArgentina\nErmənistan\nAmerika Samoası\nAntiqua və Barbuda\nAvstraliya\nAvstriya\nBurundi\nBelçika\nBenin\nBurkina-Faso\nBanqladeş\nBolqarıstan\nBəhreyn\nBaham adaları\nBosniya və Herseqovina\nBelarus\nBeliz\nBermud adaları\nBoliviya\nBraziliya\nBarbados\nBruney\nButan\nBotsvana\nMAR\nKanada\nKokos adaları\nİsveçrə\nÇili\nÇin\nKot-d\'ivuar\nKamerun\nKDR\nKonqo Respublikası\nKuk adaları\nKolumbiya\nKomor adaları\nKabo-Verde\nKosta-Rika\nKuba\nMilad adası\nKayman adaları\nKipr\nÇexiya\nAlmaniya\nCibuti\nDominika\nDanimarka\nDominikan Respublikası\nƏlcəzair\nEkvador\nMisir\nEritreya\nQərbi Sahara\nİspaniya\nEstoniya\nEfiopiya\nFinlandiya\nFici\nFoklend adaları\nFransa\nFarer adaları\nMikroneziya\nQabon\nBöyük Britaniya\nGürcüstan\nQana\nCəbəli-Tariq\nQvineya\nQambiya\nQvineya-Bisau\nEkvatorial Qvineya\nYunanıstan\nQrenada\nQrenlandiya\nQvatemala\nQuam\nQayana\nHonkonq\nHonduras\nXorvatiya\nHaiti\nMacarıstan\nindoneziya\nHindistan\nirlandiya\nİran\nİraq\nislandiya\nisrail\nitaliya\nYamayka\nlordaniya\nYaponiya\nQazağıstan\nKeniya\nQırğızıstan\nKamboca\nKiribati\nSent-Kits və Nevis\nCənubi Koreya\nKüveyt\nLaos\nLivan\nLiberiya\nLiviya\nSent-Lüsiya\nLixtenşteyn\nŞri-Lanka\nLesoto\nLitva\nLüksemburq\nLatviya\nMakao\nMərakeş\nMonako\nMoldova\nMadaqaskar\nMaldiv adaları\nMeksika\nMarşall adaları\nMakedoniya\nMali\nMalta\nMyanma\nMonteneqro\nMonqolustan\nŞimali Marian adaları\nMozambik\nMavritaniya\nMontserrat\nMavriki\nMalavi\nMalayziya\nMayotta\nNamibiya\nYeni Kaledoniya\nNiger\nNorfolk adası\nNigeriya\nNikaraqua\nNiue\nNiderland\nNorveç\nNepal\nNauru\nYeni Zelandiya\nOman\nPakistan\nPanama\nPitkern adaları\nPeru\nFilippin\nPalau\nPapua-Yeni Qvineya\nPolşa\nPuerto-Riko\nKXDR\nPortuqaliya\nParaqvay\nFələstin Dövləti\nFransa Polineziyası\nQətər\nRuminiya\nRusiya\nRuanda\nSəudiyyə Ərəbistanı\nSudan\nSeneqal\nSinqapur\nMüqəddəs Yelena, Askenson və Tristan-da-Kunya adaları\nŞpisbergen və Yan-Mayen\nSolomon adaları\nSyerra-Leone\nSalvador\nSan-Marino\nSomali\nSen-Pyer və Mikelon\nSerbiya\nCənubi Sudan\nSan-Tome və Prinsipi\nSurinam\nSlovakiya\nSloeniya\nİsveç\nSvazilend\nSeyşel adaları\nSuriya\nTörks və Kaykos adaları\nÇad\nToqo\nTailand\nTacikistan\nTokelau\nTürkmənistan\nŞərqi Timor\nTonqa\nTrinidad və Tobaqo\nTunis\nTürkiyə\nTuvalu\nÇin Respublikası\nTanzaniya\nUqanda\nUkrayna\nUruqvay\nABŞ\nÖzbəkistan\nVatikan\nSent-Vinsent və Qrenadin\nVenesuela\nBritaniya Virgin adaları\nAmerika Virgin adaları\nVyetnam\nVanuatu\nUollis və Futuna\nSamoa\nYəmən\nCAR\nZambiya\nZimbabve',
-                insert_after='act_type',
-                depends_on='eval:(doc.purchase_type || doc.agricultural_goods) && doc.act_type == "Import"'
-            ),
             # Per-item multiple tax articles: parent-level backing table,
             # linked to each item row by custom_si_row_key (Frappe has no
             # nested child tables, so this lives on the parent, not the row).

jey_erp/hooks.py

@@ -33,12 +33,10 @@ doctype_js = {
         "public/js/sales_invoice_vat.js",
         "public/js/sales_invoice_tax_articles.js",
         "public/js/sales_invoice_certificates.js",
-        "public/js/sales_classification.js",
     ],
     "Sales Order": [
         "public/js/vat_calculator.js",
         "public/js/sales_order_vat.js",
-        "public/js/sales_classification.js",
     ],
     "Asset": "public/js/asset.js",
     "Purchase Invoice": "public/js/purchase_invoice.js",

jey_erp/public/js/sales_classification.js

@@ -1,18 +0,0 @@
-// Mutual exclusion between the two purchase-classification checkboxes on
-// Sales Invoice and Sales Order: only one of `purchase_type` /
-// `agricultural_goods` may be set at a time. Checking one clears the other.
-// Mirrors the Purchase Invoice behaviour in purchase_invoice.js.
-['Sales Invoice', 'Sales Order'].forEach(function (doctype) {
-    frappe.ui.form.on(doctype, {
-        purchase_type: function (frm) {
-            if (frm.doc.purchase_type && frm.doc.agricultural_goods) {
-                frm.set_value('agricultural_goods', 0);
-            }
-        },
-        agricultural_goods: function (frm) {
-            if (frm.doc.agricultural_goods && frm.doc.purchase_type) {
-                frm.set_value('purchase_type', 0);
-            }
-        },
-    });
-});