fix: certificate download template

frappe · Oct 26, 2023 · ad39530 · ad39530
1 parent a6c2378
commit ad39530
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/lms/www/courses/certificate.html b/lms/www/courses/certificate.html
@@ -19,9 +19,9 @@
 
       <div>
 
-        {% if doc.member == frappe.session.user %}
+        {% if doc.member == frappe.session.user or is_moderator %}
         <div class="">
-          <a class="btn btn-default btn-sm" target="_blank" href="/api/method/frappe.utils.print_format.download_pdf?doctype=LMS%20Certificate&name={{ doc.name }}&_lang=en">
+          <a class="btn btn-default btn-sm" target="_blank" href="/api/method/frappe.utils.print_format.download_pdf?doctype=LMS%20Certificate&name={{ doc.name }}&format={{ print_format }}&_lang=en">
             {{ _("Download") }}
           </a>
 

diff --git a/lms/www/courses/certificate.py b/lms/www/courses/certificate.py
@@ -2,6 +2,7 @@
 from frappe import _
 from frappe.utils.jinja import render_template
 from frappe.utils import get_url
+from lms.lms.utils import has_course_moderator_role
 
 
 def get_context(context):
@@ -30,12 +31,14 @@ def get_context(context):
 		"User", context.doc.member, ["full_name", "username"], as_dict=True
 	)
 	context.url = f"{get_url()}/courses/{context.course.name}/{context.doc.name}"
+	context.is_moderator = has_course_moderator_role()
 
 	if context.doc.template:
 		print_format = context.doc.template
 	else:
 		print_format = get_print_format()
 
+	context.print_format = print_format
 	template = frappe.db.get_value(
 		"Print Format", print_format, ["html", "css"], as_dict=True
 	)