Merge pull request #19576 from marination/cwip-patch-fix
fix(patch): Enable CWIP Accounting
diff --git a/erpnext/patches/v12_0/set_cwip_and_delete_asset_settings.py b/erpnext/patches/v12_0/set_cwip_and_delete_asset_settings.py
index 3d07fe5..5842e9e 100644
--- a/erpnext/patches/v12_0/set_cwip_and_delete_asset_settings.py
+++ b/erpnext/patches/v12_0/set_cwip_and_delete_asset_settings.py
@@ -9,13 +9,12 @@
if frappe.db.exists("DocType","Asset Settings"):
frappe.reload_doctype("Company")
- cwip_value = frappe.db.sql(""" SELECT value FROM `tabSingles` WHERE doctype='Asset Settings'
- and field='disable_cwip_accounting' """, as_dict=1)
+ cwip_value = frappe.db.get_single_value("Asset Settings","disable_cwip_accounting")
companies = [x['name'] for x in frappe.get_all("Company", "name")]
for company in companies:
- enable_cwip_accounting = cint(not cint(cwip_value[0]['value']))
- frappe.set_value("Company", company, "enable_cwip_accounting", enable_cwip_accounting)
+ enable_cwip_accounting = cint(not cint(cwip_value))
+ frappe.db.set_value("Company", company, "enable_cwip_accounting", enable_cwip_accounting)
frappe.db.sql(
""" DELETE FROM `tabSingles` where doctype = 'Asset Settings' """)