CalvinBackup/erpnext
1
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-03-24 05:32:46 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: set default repair cost to 0 if no value is returned

Browse Source
This commit is contained in:
ljain112
2026-03-09 18:34:46 +05:30
parent c71557f432
commit 0b1746a4c8
2 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
erpnext/assets/doctype/asset_repair/asset_repair.js
View File

@@ -116,9 +116,7 @@ frappe.ui.form.on("Asset Repair", {
purchase_invoice: frm.doc.purchase_invoice,
},
callback: function (r) {
if (r.message) {
frm.set_value("repair_cost", r.message);
}
frm.set_value("repair_cost", r.message || 0);
},
});
} else {

12
erpnext/assets/doctype/asset_repair/test_asset_repair.py
View File

@@ -326,20 +326,21 @@ class TestAssetRepair(unittest.TestCase):
def test_repair_cost_fetches_only_service_item_amount(self):
"""Test that repair cost only includes service (non-stock) item amounts from purchase invoice."""
company = "_Test Company with perpetual inventory"
service_item = create_item(
"_Test Service Item for Repair",
is_stock_item=0,
company="_Test Company",
company=company,
)
stock_item = create_item(
"_Test Stock Item for Repair",
is_stock_item=1,
company="_Test Company",
company=company,
)
expense_account = frappe.db.get_value("Company", "_Test Company", "default_expense_account")
cost_center = frappe.db.get_value("Company", "_Test Company", "cost_center")
expense_account = frappe.db.get_value("Company", company, "default_expense_account")
cost_center = frappe.db.get_value("Company", company, "cost_center")
pi = make_purchase_invoice(
item_code=service_item.name,
@@ -349,6 +350,7 @@ class TestAssetRepair(unittest.TestCase):
cost_center=cost_center,
update_stock=0,
do_not_submit=1,
company=company,
)
pi.update_stock = 1
@@ -358,7 +360,7 @@ class TestAssetRepair(unittest.TestCase):
"item_code": stock_item.name,
"qty": 2,
"rate": 300,
"warehouse": "_Test Warehouse - _TC",
"warehouse": "Stores - TCP1",
"cost_center": cost_center,
},
)