fix: check return_against exists before api call

(cherry picked from commit 00b6b97197)
This commit is contained in:
Sanket322
2025-04-13 13:17:15 +05:30
committed by Mergify
parent 73bc57f53e
commit 8623a5650b

View File

@@ -896,23 +896,25 @@ erpnext.taxes_and_totals = class TaxesAndTotals extends erpnext.payments {
it should set the return to that mode of payment only. it should set the return to that mode of payment only.
*/ */
let return_against_mop = await frappe.call({ if(this.frm.doc.return_against){
method: 'erpnext.controllers.sales_and_purchase_return.get_payment_data', let {message : return_against_mop } = await frappe.call({
args: { method: 'erpnext.controllers.sales_and_purchase_return.get_payment_data',
invoice: this.frm.doc.return_against args: {
} invoice: this.frm.doc.return_against
});
if (return_against_mop.message.length === 1) {
this.frm.doc.payments.forEach(payment => {
if (payment.mode_of_payment == return_against_mop.message[0].mode_of_payment) {
payment.amount = total_amount_to_pay;
} else {
payment.amount = 0;
} }
}); });
this.frm.refresh_fields();
return; if (return_against_mop.length === 1) {
this.frm.doc.payments.forEach(payment => {
if (payment.mode_of_payment == return_against_mop[0].mode_of_payment) {
payment.amount = total_amount_to_pay;
} else {
payment.amount = 0;
}
});
this.frm.refresh_fields();
return;
}
} }
this.frm.doc.payments.find(payment => { this.frm.doc.payments.find(payment => {