From 326d7a87dd72d4198f4bf52d4c16907aa2e6c312 Mon Sep 17 00:00:00 2001 From: khushi8112 Date: Mon, 13 Jul 2026 13:11:59 +0530 Subject: [PATCH] fix(asset): apply capitalize_repair_cost read-only toggle on asset change too Previously only wired to refresh, so selecting an asset in the link field left the field editable until a full page reload even though the server silently zeroes it for fully depreciated assets. Co-Authored-By: Claude Sonnet 5 --- .../doctype/asset_repair/asset_repair.js | 25 ++++++++++++------- 1 file changed, 16 insertions(+), 9 deletions(-) diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.js b/erpnext/assets/doctype/asset_repair/asset_repair.js index a477c0bfcbc..29bfc8d9560 100644 --- a/erpnext/assets/doctype/asset_repair/asset_repair.js +++ b/erpnext/assets/doctype/asset_repair/asset_repair.js @@ -74,15 +74,22 @@ frappe.ui.form.on("Asset Repair", { }; }; } - if (frm.doc.asset) { - frappe.db.get_value("Asset", frm.doc.asset, "status").then(({ message }) => { - frm.set_df_property( - "capitalize_repair_cost", - "read_only", - message && message.status === "Fully Depreciated" - ); - }); - } + frm.events.toggle_capitalize_repair_cost(frm); + }, + + asset: function (frm) { + frm.events.toggle_capitalize_repair_cost(frm); + }, + + toggle_capitalize_repair_cost: function (frm) { + if (!frm.doc.asset) return; + frappe.db.get_value("Asset", frm.doc.asset, "status").then(({ message }) => { + frm.set_df_property( + "capitalize_repair_cost", + "read_only", + message && message.status === "Fully Depreciated" + ); + }); }, repair_status: (frm) => {