nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-09-16 18:24:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(asset): skip missing checkbox columns in asset type patch (backport #58416) (#58417)

Browse Source 
Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>
This commit is contained in:
mergify[bot]
2026-09-16 01:39:55 +02:00
committed by GitHub
parent 78cac8ed09
commit 3c9d9d62aa
1 changed files with 12 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
erpnext/patches/v16_0/migrate_asset_type_checkboxes_to_select.py
View File

@@ -3,23 +3,19 @@ from frappe.query_builder import Case
def execute():
required_columns = [
"is_existing_asset",
"is_composite_asset",
"is_composite_component",
]
# Skip patch if any required column is missing
if not all(frappe.db.has_column("Asset", col) for col in required_columns):
# v15 sites never had is_composite_component; only migrate columns that exist.
column_values = (
("is_existing_asset", "Existing Asset"),
("is_composite_asset", "Composite Asset"),
("is_composite_component", "Composite Component"),
)
existing = [(col, value) for col, value in column_values if frappe.db.has_column("Asset", col)]
if not existing:
return
Asset = frappe.qb.DocType("Asset")
case = Case()
for column, value in existing:
case = case.when(getattr(Asset, column) == 1, value)
frappe.qb.update(Asset).set(
Asset.asset_type,
Case()
.when(Asset.is_existing_asset == 1, "Existing Asset")
.when(Asset.is_composite_asset == 1, "Composite Asset")
.when(Asset.is_composite_component == 1, "Composite Component")
.else_(""),
).run()
frappe.qb.update(Asset).set(Asset.asset_type, case.else_("")).run()