From 3c9d9d62aa0fe37658180a548784d8cd6b8a55b6 Mon Sep 17 00:00:00 2001
From: "mergify[bot]" <37929162+mergify[bot]@users.noreply.github.com>
Date: Wed, 16 Sep 2026 01:39:55 +0200
Subject: [PATCH] fix(asset): skip missing checkbox columns in asset type patch
 (backport #58416) (#58417)

Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>
---
 ...migrate_asset_type_checkboxes_to_select.py | 28 ++++++++-----------
 1 file changed, 12 insertions(+), 16 deletions(-)

diff --git a/erpnext/patches/v16_0/migrate_asset_type_checkboxes_to_select.py b/erpnext/patches/v16_0/migrate_asset_type_checkboxes_to_select.py
index 11f341ce65a..897f25519f1 100644
--- a/erpnext/patches/v16_0/migrate_asset_type_checkboxes_to_select.py
+++ b/erpnext/patches/v16_0/migrate_asset_type_checkboxes_to_select.py
@@ -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()