Merge pull request #36261 from ruthra-kumar/possible_missing_field_error_on_payment_entry

refactor: apply terms based validation only on sales/purchase doctypes
This commit is contained in:
ruthra kumar 2023-07-24 15:22:15 +05:30 committed by GitHub
commit 4464b2a21b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -210,7 +210,11 @@ class PaymentEntry(AccountsController):
def term_based_allocation_enabled_for_reference(
self, reference_doctype: str, reference_name: str
) -> bool:
if reference_doctype and reference_name:
if (
reference_doctype
and reference_doctype in ["Sales Invoice", "Sales Order", "Purchase Order", "Purchase Invoice"]
and reference_name
):
if template := frappe.db.get_value(reference_doctype, reference_name, "payment_terms_template"):
return frappe.db.get_value(
"Payment Terms Template", template, "allocate_payment_based_on_payment_terms"