Shiloh/brotherton-erpnext
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: composite asset capitalization using asset components

Browse Source
This commit is contained in:
Nabin Hait 2024-01-17 16:17:53 +05:30
parent 1148ed1566
commit 5df40661d2
3 changed files with 38 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
erpnext/assets/doctype/asset/asset.py
View File

@ -1042,6 +1042,8 @@ def is_cwip_accounting_enabled(asset_category):
@frappe.whitelist() @frappe.whitelist()
def get_asset_value_after_depreciation(asset_name, finance_book=None): def get_asset_value_after_depreciation(asset_name, finance_book=None):
asset = frappe.get_doc("Asset", asset_name) asset = frappe.get_doc("Asset", asset_name)
if not asset.calculate_depreciation:
return flt(asset.value_after_depreciation)
return asset.get_value_after_depreciation(finance_book) return asset.get_value_after_depreciation(finance_book)

27
erpnext/assets/doctype/asset_capitalization/asset_capitalization.js
View File

@ -21,10 +21,10 @@ erpnext.assets.AssetCapitalization = class AssetCapitalization extends erpnext.s
this.show_stock_ledger(); this.show_stock_ledger();
} }
if (this.frm.doc.stock_items && !this.frm.doc.stock_items.length && this.frm.doc.target_asset && this.frm.doc.capitalization_method === "Choose a WIP composite asset") { // if (this.frm.doc.stock_items && !this.frm.doc.stock_items.length && this.frm.doc.target_asset && this.frm.doc.capitalization_method === "Choose a WIP composite asset") {
this.set_consumed_stock_items_tagged_to_wip_composite_asset(this.frm.doc.target_asset); // this.set_consumed_stock_items_tagged_to_wip_composite_asset(this.frm.doc.target_asset);
this.get_target_asset_details(); // this.get_target_asset_details();
} // }
} }
setup_queries() { setup_queries() {
@ -143,13 +143,20 @@ erpnext.assets.AssetCapitalization = class AssetCapitalization extends erpnext.s
}, },
callback: function (r) { callback: function (r) {
if (!r.exc && r.message) { if (!r.exc && r.message) {
me.frm.clear_table("stock_items"); if(r.message[0] && r.message[0].length) {
me.frm.clear_table("stock_items");
for (let item of r.message) { for (let item of r.message[0]) {
me.frm.add_child("stock_items", item); me.frm.add_child("stock_items", item);
}
refresh_field("stock_items");
}
if (r.message[1] && r.message[1].length) {
me.frm.clear_table("asset_items");
for (let item of r.message[1]) {
me.frm.add_child("asset_items", item);
}
me.frm.refresh_field("asset_items");
} }
refresh_field("stock_items");
me.calculate_totals(); me.calculate_totals();
} }

22
erpnext/assets/doctype/asset_capitalization/asset_capitalization.py
View File

@ -900,7 +900,6 @@ def get_consumed_asset_details(args):
out.cost_center = get_default_cost_center( out.cost_center = get_default_cost_center(
args, item_defaults, item_group_defaults, brand_defaults args, item_defaults, item_group_defaults, brand_defaults
) )
return out return out
@ -948,10 +947,27 @@ def get_items_tagged_to_wip_composite_asset(asset):
"qty", "qty",
"valuation_rate", "valuation_rate",
"amount", "amount",
"is_fixed_asset",
"parent",
] ]
pr_items = frappe.get_all( pr_items = frappe.get_all(
"Purchase Receipt Item", filters={"wip_composite_asset": asset}, fields=fields "Purchase Receipt Item", filters={"wip_composite_asset": asset, "docstatus": 1}, fields=fields
) )
return pr_items stock_items = []
asset_items = []
for d in pr_items:
if not d.is_fixed_asset:
stock_items.append(frappe._dict(d))
else:
asset_details = frappe.db.get_value(
"Asset",
{"item_code": d.item_code, "purchase_receipt": d.parent},
["name as asset", "asset_name"],
as_dict=1,
)
d.update(asset_details)
asset_items.append(frappe._dict(d))
return stock_items, asset_items