From ed8a8532e169b669d55380a3282b992b6117d7f8 Mon Sep 17 00:00:00 2001 From: Khushi Rawat <142375893+khushi8112@users.noreply.github.com> Date: Thu, 24 Apr 2025 16:14:42 +0530 Subject: [PATCH] fix: update additional cost and total asset cost after asset repair (#47233) * fix: add consumed stock's cost to the asset value after repair * fix: do not copy additional cost and total asset cost --- erpnext/assets/doctype/asset/asset.json | 4 +++- erpnext/assets/doctype/asset_repair/asset_repair.py | 12 ++++++++---- 2 files changed, 11 insertions(+), 5 deletions(-) diff --git a/erpnext/assets/doctype/asset/asset.json b/erpnext/assets/doctype/asset/asset.json index f2f196a84a2..b18f122721b 100644 --- a/erpnext/assets/doctype/asset/asset.json +++ b/erpnext/assets/doctype/asset/asset.json @@ -478,6 +478,7 @@ "fieldname": "total_asset_cost", "fieldtype": "Currency", "label": "Total Asset Cost", + "no_copy": 1, "options": "Company:company:default_currency", "read_only": 1 }, @@ -486,6 +487,7 @@ "fieldname": "additional_asset_cost", "fieldtype": "Currency", "label": "Additional Asset Cost", + "no_copy": 1, "options": "Company:company:default_currency", "read_only": 1 }, @@ -589,7 +591,7 @@ "link_fieldname": "target_asset" } ], - "modified": "2025-04-15 16:33:17.189524", + "modified": "2025-04-24 15:31:47.373274", "modified_by": "Administrator", "module": "Assets", "name": "Asset", diff --git a/erpnext/assets/doctype/asset_repair/asset_repair.py b/erpnext/assets/doctype/asset_repair/asset_repair.py index e42450e83e4..c6ae3c5d8e4 100644 --- a/erpnext/assets/doctype/asset_repair/asset_repair.py +++ b/erpnext/assets/doctype/asset_repair/asset_repair.py @@ -135,9 +135,11 @@ class AssetRepair(AccountsController): self.increase_asset_value() + total_repair_cost = self.get_total_value_of_stock_consumed() if self.capitalize_repair_cost: - self.asset_doc.total_asset_cost += self.repair_cost - self.asset_doc.additional_asset_cost += self.repair_cost + total_repair_cost += self.repair_cost + self.asset_doc.total_asset_cost += total_repair_cost + self.asset_doc.additional_asset_cost += total_repair_cost if self.get("stock_consumption"): self.check_for_stock_items_and_warehouse() @@ -176,9 +178,11 @@ class AssetRepair(AccountsController): self.decrease_asset_value() + total_repair_cost = self.get_total_value_of_stock_consumed() if self.capitalize_repair_cost: - self.asset_doc.total_asset_cost -= self.repair_cost - self.asset_doc.additional_asset_cost -= self.repair_cost + total_repair_cost += self.repair_cost + self.asset_doc.total_asset_cost -= total_repair_cost + self.asset_doc.additional_asset_cost -= total_repair_cost if self.get("capitalize_repair_cost"): self.ignore_linked_doctypes = ("GL Entry", "Stock Ledger Entry")