Skip to content

Commit

Permalink
fix: don't overwrite existing terms in transaction
Browse files Browse the repository at this point in the history
(cherry picked from commit 77b044f)

# Conflicts:
#	erpnext/public/js/controllers/transaction.js
  • Loading branch information
barredterra authored and mergify[bot] committed Jan 31, 2024
1 parent d785c6c commit dcdd3e8
Showing 1 changed file with 34 additions and 0 deletions.
34 changes: 34 additions & 0 deletions erpnext/public/js/controllers/transaction.js
Original file line number Diff line number Diff line change
Expand Up @@ -809,6 +809,40 @@ erpnext.TransactionController = class TransactionController extends erpnext.taxe
me.frm.set_value("tc_name", company_doc.default_buying_terms);
}
}
<<<<<<< HEAD
=======

if (
company_doc.default_selling_terms &&
frappe.meta.has_field(me.frm.doc.doctype, "tc_name") &&
[
"Sales Invoice",
"Quotation",
"Sales Order",
"Delivery Note",
].includes(me.frm.doc.doctype) &&
!me.frm.doc.tc_name
) {
me.frm.set_value("tc_name", company_doc.default_selling_terms);
}

if (
company_doc.default_buying_terms &&
frappe.meta.has_field(me.frm.doc.doctype, "tc_name") &&
[
"Request for Quotation",
"Supplier Quotation",
"Purchase Order",
// Purchase Invoice is excluded as per issue #3345
"Material Request",
"Purchase Receipt",
].includes(me.frm.doc.doctype) &&
!me.frm.doc.tc_name
) {
me.frm.set_value("tc_name", company_doc.default_buying_terms);
}

>>>>>>> 77b044f1a6 (fix: don't overwrite existing terms in transaction)
frappe.run_serially([
() => me.frm.script_manager.trigger("currency"),
() => me.update_item_tax_map(),
Expand Down

0 comments on commit dcdd3e8

Please sign in to comment.