Merge pull request #43184 from frappe/mergify/bp/version-15/pr-43182

fix: typo with po_date when creating remarks (backport #43182)

erpnext/accounts/doctype/sales_invoice/sales_invoice.py

@@ -951,7 +951,7 @@ def add_remarks(self):
 			if self.po_no:
 				self.remarks = _("Against Customer Order {0}").format(self.po_no)
 				if self.po_date:
-					self.remarks += " " + _("dated {0}").format(formatdate(self.po_data))
+					self.remarks += " " + _("dated {0}").format(formatdate(self.po_date))
 
 			else:
 				self.remarks = _("No Remarks")

erpnext/accounts/doctype/sales_invoice/test_sales_invoice.py

@@ -8,7 +8,7 @@
 from frappe import qb
 from frappe.model.dynamic_links import get_dynamic_link_map
 from frappe.tests.utils import FrappeTestCase, change_settings
-from frappe.utils import add_days, flt, getdate, nowdate, today
+from frappe.utils import add_days, flt, format_date, getdate, nowdate, today
 
 import erpnext
 from erpnext.accounts.doctype.account.test_account import create_account, get_inventory_account
@@ -3997,6 +3997,14 @@ def test_common_party_with_foreign_currency_jv(self):
 		self.assertTrue(jv)
 		self.assertEqual(jv[0], si.grand_total)
 
+	def test_invoice_remarks(self):
+		si = frappe.copy_doc(test_records[0])
+		si.po_no = "Test PO"
+		si.po_date = nowdate()
+		si.save()
+		si.submit()
+		self.assertEqual(si.remarks, f"Against Customer Order Test PO dated {format_date(nowdate())}")
+
 
 def set_advance_flag(company, flag, default_account):
 	frappe.db.set_value(