commonfied regional function for checking delete permission

erpnext/hooks.py

@@ -201,11 +201,11 @@ doc_events = {
 	"Sales Invoice": {
 		'validate': 'erpnext.regional.india.utils.set_place_of_supply',
 		"on_submit": "erpnext.regional.france.utils.create_transaction_log",
-		"on_trash": ["erpnext.regional.france.utils.check_deletion_permission", "erpnext.regional.nepal.utils.check_deletion_permission"]
+		"on_trash": "erpnext.regional.check_deletion_permission"
 	},
 	"Payment Entry": {
 		"on_submit": ["erpnext.regional.france.utils.create_transaction_log", "erpnext.accounts.doctype.payment_request.payment_request.make_status_as_paid"],
-		"on_trash": "erpnext.regional.france.utils.check_deletion_permission"
+		"on_trash": "erpnext.regional.check_deletion_permission"
 	},
 	'Address': {
 		'validate': 'erpnext.regional.india.utils.validate_gstin_for_india'

erpnext/regional/__init__.py

@@ -0,0 +1,11 @@
+# Copyright (c) 2018, Frappe Technologies and contributors
+# For license information, please see license.txt
+
+import frappe
+from frappe import _
+from erpnext import get_region
+
+def check_deletion_permission(doc, method):
+	region = get_region()
+	if region in ["Nepal", "France"]:
+		frappe.throw(_("Deletion is not permitted for country {0}".format(region)))

erpnext/regional/france/utils.py

@@ -9,9 +9,7 @@ def create_transaction_log(doc, method):
 	region = get_region()
 	if region not in ["France"]:
 		return
-
 	else:
-
 		data = str(doc.as_dict())
 
 		frappe.get_doc({
@@ -21,14 +19,6 @@ def create_transaction_log(doc, method):
 			"data": data
 		}).insert(ignore_permissions=True)
 
-def check_deletion_permission(doc, method):
-	region = get_region()
-	if region not in ["France"]:
-		return
-
-	else:
-		frappe.throw(_("Deletion is not permitted for country {0}".format(region)))
-
 # don't remove this function it is used in tests
 def test_method():
 	'''test function'''

erpnext/regional/nepal/utils.py

@@ -1,13 +0,0 @@
-# Copyright (c) 2018, Frappe Technologies and contributors
-# For license information, please see license.txt
-
-import frappe
-from frappe import _
-from erpnext import get_region
-
-def check_deletion_permission(doc, method):
-	region = get_region()
-	if region not in ["Nepal"]:
-		return
-	else:
-		frappe.throw(_("Deletion is not permitted for country {0}".format(region)))