nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-04-15 04:45:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: update repair cost logic to set value only for positive amounts

Browse Source
This commit is contained in:
ljain112
2025-11-28 18:45:07 +05:30
parent b9aaae6343
commit 00ffdee928
1 changed files with 12 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -135,7 +135,7 @@ frappe.ui.form.on("Asset Repair Purchase Invoice", {
},
expense_account: function (frm, cdt, cdn) {
var row = locals[cdt][cdn];
let row = locals[cdt][cdn];
if (!row.purchase_invoice || !row.expense_account) {
frappe.model.set_value(cdt, cdn, "repair_cost", 0);
@@ -150,13 +150,19 @@ frappe.ui.form.on("Asset Repair Purchase Invoice", {
},
callback: function (r) {
if (r.message !== undefined) {
frappe.model.set_value(cdt, cdn, "repair_cost", r.message);
if (r.message <= 0) {
if (r.message > 0) {
frappe.model.set_value(cdt, cdn, "repair_cost", r.message);
} else {
frappe.model.set_value(cdt, cdn, "repair_cost", 0);
let pi_link = frappe.utils.get_form_link(
"Purchase Invoice",
row.purchase_invoice,
true
);
frappe.msgprint({
title: __("No Available Amount"),
message: __(
"There is no available repair cost for this Purchase Invoice and Expense Account combination."
"Row {0}: The entire expense amount for account {1} in {2} has already been allocated.",
[row.idx, row.expense_account.bold(), pi_link]
),
indicator: "orange",
});