mirror of
https://github.com/frappe/erpnext.git
synced 2026-08-13 14:41:53 +00:00
fix: show transaction currency symbol in Payment Request schedule dialog and reference table (#57050)
* fix: show transaction currency symbol in Payment Request schedule dialog and reference table When company currency (INR) differs from customer currency (USD), the Amount column in the Select Payment Schedule dialog and the Payment Reference table on the Payment Request form incorrectly displayed the company currency symbol (₹) instead of the transaction currency symbol ($). - Pass `currency` from the parent document on each schedule row returned by `get_available_payment_schedules` so the dialog can resolve the symbol. - Add a hidden `currency` field to the dialog table and set `options: "currency"` on `payment_amount` so Frappe renders the correct symbol. - Propagate `currency` into Payment Reference rows in `set_payment_references`. - Add a hidden `currency` Link field to the Payment Reference child DocType and set `options: "currency"` on its `amount` field so the table renders correctly. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * fix: preserve currency when serializing payment schedule rows get_available_payment_schedules set `schedule.currency` directly on the Payment Schedule Document row, but `currency` isn't a field on that DocType, so the API response serializer stripped it before it reached the client. The Select Payment Schedule dialog and the Payment Reference table therefore always fell back to the company currency symbol, even with the earlier options="currency" changes in place. Convert each row to a plain dict via as_dict() first, then set the currency key on the dict so it survives serialization. * refactor: source schedule currency in dialog instead of API serializer get_available_payment_schedules had to convert each child row with as_dict() and re-attach currency, because currency is not a field on Payment Schedule and the response serializer drops attributes set on the Document itself. The schedule dialog already has the transaction currency on frm.doc, so set it there and let the API keep returning the schedule rows unchanged. Payment Reference still stores currency per row. --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Co-authored-by: Jatin3128 <jatinsarna8@gmail.com> Co-authored-by: Jatin3128 <140256508+Jatin3128@users.noreply.github.com>
This commit is contained in:
@@ -14,7 +14,8 @@
|
||||
"section_break_mjlv",
|
||||
"due_date",
|
||||
"column_break_qghl",
|
||||
"amount"
|
||||
"amount",
|
||||
"currency"
|
||||
],
|
||||
"fields": [
|
||||
{
|
||||
@@ -55,8 +56,18 @@
|
||||
"fieldtype": "Currency",
|
||||
"in_list_view": 1,
|
||||
"label": "Amount",
|
||||
"options": "currency",
|
||||
"precision": "2"
|
||||
},
|
||||
{
|
||||
"fieldname": "currency",
|
||||
"fieldtype": "Link",
|
||||
"hidden": 1,
|
||||
"label": "Currency",
|
||||
"options": "Currency",
|
||||
"print_hide": 1,
|
||||
"read_only": 1
|
||||
},
|
||||
{
|
||||
"fieldname": "column_break_lnjp",
|
||||
"fieldtype": "Column Break"
|
||||
@@ -74,7 +85,7 @@
|
||||
"index_web_pages_for_search": 1,
|
||||
"istable": 1,
|
||||
"links": [],
|
||||
"modified": "2026-01-19 02:21:36.455830",
|
||||
"modified": "2026-07-11 00:00:00.000000",
|
||||
"modified_by": "Administrator",
|
||||
"module": "Accounts",
|
||||
"name": "Payment Reference",
|
||||
|
||||
@@ -943,6 +943,7 @@ def set_payment_references(payment_schedules):
|
||||
"description": row.get("description"),
|
||||
"due_date": row.get("due_date"),
|
||||
"amount": row.get("payment_amount"),
|
||||
"currency": row.get("currency"),
|
||||
}
|
||||
)
|
||||
|
||||
|
||||
@@ -533,7 +533,10 @@ erpnext.TransactionController = class TransactionController extends erpnext.taxe
|
||||
return;
|
||||
}
|
||||
|
||||
schedules.forEach((schedule) => (schedule.__checked = 1));
|
||||
schedules.forEach((schedule) => {
|
||||
schedule.__checked = 1;
|
||||
schedule.currency = frm.doc.currency;
|
||||
});
|
||||
|
||||
const dialog = new frappe.ui.Dialog({
|
||||
title: __("Select Payment Schedule"),
|
||||
@@ -567,10 +570,19 @@ erpnext.TransactionController = class TransactionController extends erpnext.taxe
|
||||
in_list_view: 1,
|
||||
read_only: 1,
|
||||
},
|
||||
{
|
||||
fieldtype: "Link",
|
||||
fieldname: "currency",
|
||||
label: __("Currency"),
|
||||
options: "Currency",
|
||||
hidden: 1,
|
||||
read_only: 1,
|
||||
},
|
||||
{
|
||||
fieldtype: "Currency",
|
||||
fieldname: "payment_amount",
|
||||
label: __("Amount"),
|
||||
options: "currency",
|
||||
in_list_view: 1,
|
||||
read_only: 1,
|
||||
},
|
||||
|
||||
Reference in New Issue
Block a user