fix: on changes of paid from/to account fetch company bank account

Author: Abdeali099

erpnext/accounts/doctype/bank_account/bank_account.py

@@ -115,3 +115,10 @@ def get_bank_account_details(bank_account):
 	return frappe.get_cached_value(
 		"Bank Account", bank_account, ["account", "bank", "bank_account_no"], as_dict=1
 	)
+
+
+@frappe.whitelist()
+def get_company_bank_account(company: str, account: str):
+	frappe.has_permission("Bank Account", throw=True)
+
+	return frappe.get_cached_value("Bank Account", {"company": company, "account": account, "disabled": 0})

erpnext/accounts/doctype/payment_entry/payment_entry.js

@@ -575,6 +575,8 @@ frappe.ui.form.on("Payment Entry", {
 	},
 
 	paid_from: function (frm) {
+		frm.events.set_company_bank_account(frm);
+
 		if (frm.set_party_account_based_on_party) return;
 
 		frm.events.set_account_currency_and_balance(
@@ -591,6 +593,8 @@ frappe.ui.form.on("Payment Entry", {
 	},
 
 	paid_to: function (frm) {
+		frm.events.set_company_bank_account(frm);
+
 		if (frm.set_party_account_based_on_party) return;
 
 		frm.events.set_account_currency_and_balance(
@@ -1363,6 +1367,32 @@ frappe.ui.form.on("Payment Entry", {
 		}
 	},
 
+	set_company_bank_account: function (frm) {
+		if (!["Pay", "Receive"].includes(frm.doc.payment_type)) return;
+
+		const field = frm.doc.payment_type == "Pay" ? "paid_from" : "paid_to";
+
+		if (!frm.doc[field]) {
+			frm.set_value(field, null);
+			return;
+		}
+
+		if (frm.doc.bank_account) return;
+
+		frappe.call({
+			method: "erpnext.accounts.doctype.bank_account.bank_account.get_company_bank_account",
+			args: {
+				company: frm.doc.company,
+				account: frm.doc[field],
+			},
+			callback: function (r) {
+				if (r.message) {
+					frm.set_value("bank_account", r.message);
+				}
+			},
+		});
+	},
+
 	sales_taxes_and_charges_template: function (frm) {
 		frm.trigger("fetch_taxes_from_template");
 	},