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

fix(accounts): resolve subscription plans for any reference doctype in Payment Request (backport #58438)

Browse Source
This commit is contained in:
jatin3128
2026-09-01 11:38:15 +05:30
parent 22755c4ad6
commit a12eee1272
1 changed files with 11 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
erpnext/accounts/doctype/payment_request/payment_request.py
View File

@@ -710,7 +710,8 @@ def make_payment_request(**args):
if not party_account_currency:
party_account = get_party_account(party_type, ref_doc.get(party_type.lower()), ref_doc.company)
party_account_currency = get_account_currency(party_account)
is_a_subscription = 1 if ref_doc.get("subscription") else 0
subscription_plans = get_subscription_details(ref_doc.doctype, ref_doc.name)
pr.update(
{
"payment_gateway_account": gateway_account.get("name"),
@@ -742,15 +743,14 @@ def make_payment_request(**args):
or gateway_account.get("payment_channel", "Email") != "Email"
),
"phone_number": args.get("phone_number") if args.get("phone_number") else None,
"is_a_subscription": is_a_subscription,
"is_a_subscription": 1 if subscription_plans else 0,
}
)
if selected_payment_schedules:
apply_payment_references(pr, payment_reference)
if is_a_subscription:
values = get_subscription_details(ref_doc.doctype, ref_doc.name)
if subscription_plans:
pr.set(
"subscription_plans",
[
@@ -758,7 +758,7 @@ def make_payment_request(**args):
"plan": row.plan,
"qty": row.qty,
}
for row in values
for row in subscription_plans
],
)
# Dimensions
@@ -1074,16 +1074,18 @@ def get_dummy_message(doc):
@frappe.whitelist()
def get_subscription_details(reference_doctype: str, reference_name: str):
if reference_doctype != "Sales Invoice":
def get_subscription_details(reference_doctype: str, reference_name: str) -> list[dict]:
frappe.has_permission(reference_doctype, "read", reference_name, throw=True)
if not frappe.get_meta(reference_doctype).has_field("subscription"):
return []
subscription = frappe.db.get_value("Sales Invoice", reference_name, "subscription")
subscription = frappe.db.get_value(reference_doctype, reference_name, "subscription")
if not subscription:
return []
subscription_plan = frappe.get_all(
return frappe.get_all(
"Subscription Plan Detail",
filters={"parent": subscription, "parenttype": "Subscription", "parentfield": "plans"},
fields=[
@@ -1092,8 +1094,6 @@ def get_subscription_details(reference_doctype: str, reference_name: str):
],
)
return subscription_plan
@frappe.whitelist()
def make_payment_order(source_name, target_doc=None):