fix: rendering the email template subject in leave notification (back…

…port #2027) (#2046) (cherry picked from commit 816a50e) Co-authored-by: Nihantra C. Patel <[email protected]>
frappe · Aug 6, 2024 · 1e576fc · 1e576fc
1 parent adeb482
commit 1e576fc
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/hrms/hr/doctype/leave_application/leave_application.py b/hrms/hr/doctype/leave_application/leave_application.py
@@ -579,6 +579,7 @@ def notify_employee(self):
 			frappe.msgprint(_("Please set default template for Leave Status Notification in HR Settings."))
 			return
 		email_template = frappe.get_doc("Email Template", template)
+		subject = frappe.render_template(email_template.subject, args)
 		message = frappe.render_template(email_template.response_, args)
 
 		self.notify(
@@ -587,7 +588,7 @@ def notify_employee(self):
 				"message": message,
 				"message_to": employee.user_id,
 				# for email
-				"subject": email_template.subject,
+				"subject": subject,
 				"notify": "employee",
 			}
 		)
@@ -604,6 +605,7 @@ def notify_leave_approver(self):
 				)
 				return
 			email_template = frappe.get_doc("Email Template", template)
+			subject = frappe.render_template(email_template.subject, args)
 			message = frappe.render_template(email_template.response_, args)
 
 			self.notify(
@@ -612,7 +614,7 @@ def notify_leave_approver(self):
 					"message": message,
 					"message_to": self.leave_approver,
 					# for email
-					"subject": email_template.subject,
+					"subject": subject,
 				}
 			)