From ac0dd5083fd958edd03e779617c183eb7a8d8c4f Mon Sep 17 00:00:00 2001 From: Nabin Hait Date: Tue, 12 Mar 2024 12:10:55 +0530 Subject: [PATCH] fix: linter issues --- .../asset_capitalization/asset_capitalization.py | 2 +- ...e_cancelled_asset_capitalization_from_asset.py | 15 ++++++++------- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/erpnext/assets/doctype/asset_capitalization/asset_capitalization.py b/erpnext/assets/doctype/asset_capitalization/asset_capitalization.py index f3464e2aba1..76a660d6a9a 100644 --- a/erpnext/assets/doctype/asset_capitalization/asset_capitalization.py +++ b/erpnext/assets/doctype/asset_capitalization/asset_capitalization.py @@ -83,7 +83,7 @@ class AssetCapitalization(StockController): self.update_stock_ledger() self.make_gl_entries() self.restore_consumed_asset_items() - + def on_trash(self): frappe.db.set_value("Asset", self.target_asset, "capitalized_in", None) super(AssetCapitalization, self).on_trash() diff --git a/erpnext/patches/v14_0/remove_cancelled_asset_capitalization_from_asset.py b/erpnext/patches/v14_0/remove_cancelled_asset_capitalization_from_asset.py index 82f3c205582..cb39a9280e4 100644 --- a/erpnext/patches/v14_0/remove_cancelled_asset_capitalization_from_asset.py +++ b/erpnext/patches/v14_0/remove_cancelled_asset_capitalization_from_asset.py @@ -1,10 +1,11 @@ import frappe + def execute(): - cancelled_asset_capitalizations = frappe.get_all( - "Asset Capitalization", - filters={"docstatus": 2}, - fields=["name", "target_asset"], - ) - for asset_capitalization in cancelled_asset_capitalizations: - frappe.db.set_value("Asset", asset_capitalization.target_asset, "capitalized_in", None) + cancelled_asset_capitalizations = frappe.get_all( + "Asset Capitalization", + filters={"docstatus": 2}, + fields=["name", "target_asset"], + ) + for asset_capitalization in cancelled_asset_capitalizations: + frappe.db.set_value("Asset", asset_capitalization.target_asset, "capitalized_in", None)