refactor: enable legacy controller by default for pcv

This commit is contained in:
ruthra kumar
2025-10-20 13:16:18 +05:30
parent 0b88f98a86
commit fe39ce03bb
4 changed files with 16 additions and 2 deletions

View File

@@ -654,7 +654,7 @@
"label": "Use Legacy Budget Controller"
},
{
"default": "0",
"default": "1",
"fieldname": "use_legacy_controller_for_pcv",
"fieldtype": "Check",
"label": "Use Legacy Controller For Period Closing Voucher"
@@ -666,7 +666,7 @@
"index_web_pages_for_search": 1,
"issingle": 1,
"links": [],
"modified": "2025-10-01 15:17:47.168354",
"modified": "2025-10-20 14:06:08.870427",
"modified_by": "Administrator",
"module": "Accounts",
"name": "Accounts Settings",

View File

@@ -13,6 +13,10 @@ from erpnext.accounts.utils import get_fiscal_year
class TestPeriodClosingVoucher(IntegrationTestCase):
def setUp(self):
super().setUp()
frappe.db.set_single_value("Accounts Settings", "use_legacy_controller_for_pcv", 1)
def test_closing_entry(self):
frappe.db.sql("delete from `tabGL Entry` where company='Test PCV Company'")
frappe.db.sql("delete from `tabPeriod Closing Voucher` where company='Test PCV Company'")

View File

@@ -443,3 +443,4 @@ erpnext.patches.v16_0.update_serial_no_reference_name
erpnext.patches.v16_0.rename_subcontracted_quantity
erpnext.patches.v16_0.add_new_stock_entry_types
erpnext.patches.v15_0.set_asset_status_if_not_already_set
erpnext.patches.v15_0.toggle_legacy_controller_for_period_closing

View File

@@ -0,0 +1,9 @@
import frappe
def execute():
"""
Description:
Enable Legacy controller for Period Closing Voucher
"""
frappe.db.set_single_value("Accounts Settings", "use_legacy_controller_for_pcv", 1)