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

refactor(sales_invoice): replace sql with qb in get_all_mode_of_payments (#55377)

Browse Source 
(cherry picked from commit 618045ec98)
This commit is contained in:
Loic Oberle
2026-05-29 14:04:20 +02:00
committed by Mergify
parent e42c3961a9
commit db4e459a5a
1 changed files with 14 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
erpnext/accounts/doctype/sales_invoice/sales_invoice.py
View File

@@ -3036,14 +3036,21 @@ def update_multi_mode_option(doc, pos_profile):
def get_all_mode_of_payments(doc): def get_all_mode_of_payments(doc):
return frappe.db.sql( ModeOfPaymentAccount = frappe.qb.DocType("Mode of Payment Account")
""" ModeOfPayment = frappe.qb.DocType("Mode of Payment")
select mpa.default_account, mpa.parent, mp.type as type
from `tabMode of Payment Account` mpa,`tabMode of Payment` mp query = (
where mpa.parent = mp.name and mpa.company = %(company)s and mp.enabled = 1""", frappe.qb.from_(ModeOfPaymentAccount)
{"company": doc.company}, .join(ModeOfPayment)
as_dict=1, .on(ModeOfPaymentAccount.parent == ModeOfPayment.name)
.select(
ModeOfPaymentAccount.default_account, ModeOfPaymentAccount.parent, ModeOfPayment.type.as_("type")
) )
.where(ModeOfPaymentAccount.company == doc.company)
.where(ModeOfPayment.enabled == 1)
)
return query.run(as_dict=1)
def get_mode_of_payments_info(mode_of_payments, company): def get_mode_of_payments_info(mode_of_payments, company):