nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-08-14 15:11:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(setup): scope regional-tax-settings rollback to a savepoint (review)

Browse Source 
from_detailed_data inserts tax templates/accounts before update_regional_tax_settings in the same transaction; a full frappe.db.rollback() on regional-setup failure discarded those templates while the wizard continued. Take a savepoint before the regional call and roll back only to it.
This commit is contained in:
Mihir Kandoi
2026-06-30 12:17:38 +05:30
parent 16a6a4913e
commit bd57e43446
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
erpnext/setup/setup_wizard/operations/taxes_setup.py
View File

@@ -120,6 +120,7 @@ def from_detailed_data(company_name, data):
def update_regional_tax_settings(country, company):
path = frappe.get_app_path("erpnext", "regional", frappe.scrub(country))
if os.path.exists(path.encode("utf-8")):
frappe.db.savepoint("regional_tax_settings")
try:
module_name = f"erpnext.regional.{frappe.scrub(country)}.setup.update_regional_tax_settings"
frappe.get_attr(module_name)(country, company)
@@ -127,7 +128,7 @@ def update_regional_tax_settings(country, company):
pass
except Exception:
# Log error and ignore if failed to setup regional tax settings
frappe.db.rollback()
frappe.db.rollback(save_point="regional_tax_settings")
frappe.log_error("Unable to setup regional tax settings")