From 707b630851659eea04c4c71740689c11ec94247f Mon Sep 17 00:00:00 2001
From: Smit Vora <mailsmitvora@gmail.com>
Date: Thu, 19 Nov 2020 18:42:39 +0530
Subject: [PATCH] fix: add remarks to sales invoice

---
 .../doctype/sales_invoice/sales_invoice.py    |  9 +++--
 erpnext/patches.txt                           |  3 +-
 .../v12_0/update_sales_invoice_remarks.py     | 35 +++++++++++++++++++
 3 files changed, 44 insertions(+), 3 deletions(-)
 create mode 100644 erpnext/patches/v12_0/update_sales_invoice_remarks.py

diff --git a/erpnext/accounts/doctype/sales_invoice/sales_invoice.py b/erpnext/accounts/doctype/sales_invoice/sales_invoice.py
index 2f8b782356d..94bc400fe5b 100644
--- a/erpnext/accounts/doctype/sales_invoice/sales_invoice.py
+++ b/erpnext/accounts/doctype/sales_invoice/sales_invoice.py
@@ -4,7 +4,7 @@
 from __future__ import unicode_literals
 import frappe, erpnext
 import frappe.defaults
-from frappe.utils import cint, flt, add_months, today, date_diff, getdate, add_days, cstr, nowdate
+from frappe.utils import cint, flt, add_months, today, date_diff, getdate, add_days, cstr, nowdate, formatdate
 from frappe import _, msgprint, throw
 from erpnext.accounts.party import get_party_account, get_due_date
 from erpnext.controllers.stock_controller import update_gl_entries_after
@@ -530,7 +530,12 @@ class SalesInvoice(SellingController):
 		self.against_income_account = ','.join(against_acc)
 
 	def add_remarks(self):
-		if not self.remarks: self.remarks = 'No Remarks'
+		if not self.remarks:
+			if self.po_no and self.po_date:
+				self.remarks = _("Against Customer Order {0} dated {1}").format(self.po_no, 
+					formatdate(self.po_date))
+			else:
+				self.remarks = _("No Remarks")
 
 	def validate_auto_set_posting_time(self):
 		# Don't auto set the posting date and time if invoice is amended
diff --git a/erpnext/patches.txt b/erpnext/patches.txt
index b5f31bafa7e..2615830eb0a 100644
--- a/erpnext/patches.txt
+++ b/erpnext/patches.txt
@@ -677,4 +677,5 @@ erpnext.patches.v12_0.set_multi_uom_in_rfq
 erpnext.patches.v12_0.update_state_code_for_daman_and_diu
 erpnext.patches.v12_0.rename_lost_reason_detail
 erpnext.patches.v12_0.update_leave_application_status
-erpnext.patches.v12_0.update_payment_entry_status
\ No newline at end of file
+erpnext.patches.v12_0.update_payment_entry_status
+erpnext.patches.v12_0.update_sales_invoice_remarks
\ No newline at end of file
diff --git a/erpnext/patches/v12_0/update_sales_invoice_remarks.py b/erpnext/patches/v12_0/update_sales_invoice_remarks.py
new file mode 100644
index 00000000000..5443d133a2e
--- /dev/null
+++ b/erpnext/patches/v12_0/update_sales_invoice_remarks.py
@@ -0,0 +1,35 @@
+from __future__ import unicode_literals
+from mmap import PAGESIZE
+import frappe
+
+from frappe import _
+from frappe.utils import formatdate
+
+def execute():
+    si_list = frappe.db.get_all('Sales Invoice', filters = {
+        'docstatus': 1,
+        'remarks': 'No Remarks',
+        'po_no' : ['!=', ''],
+        'po_date' : ['!=', '']
+        }, 
+        fields = ['name', 'po_no', 'po_date']
+    )
+
+    for doc in si_list:
+        remarks = _("Against Customer Order {0} dated {1}").format(doc.po_no, 
+					formatdate(doc.po_date))
+        
+        frappe.db.set_value('Sales Invoice', doc.name, 'remarks', remarks)
+        
+        gl_entry_list = frappe.db.get_all('GL Entry', filters = {
+            'voucher_type': 'Sales Invoice',
+            'remarks': 'No Remarks',
+            'voucher_no' : doc.name
+            },
+            fields = ['name']
+        )
+        
+        for entry in gl_entry_list:
+            frappe.db.set_value('GL Entry', entry.name, 'remarks', remarks)
+
+    
\ No newline at end of file