From 9198302f7ef4a8f12efd8e77adef833dba0a3fe2 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Wed, 7 Jun 2023 16:53:46 +0530
Subject: [PATCH] fix: show course settings only to moderators

---
 lms/www/courses/create.html | 2 ++
 lms/www/courses/create.py   | 7 ++++++-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/lms/www/courses/create.html b/lms/www/courses/create.html
index fb803672..d8a88c65 100644
--- a/lms/www/courses/create.html
+++ b/lms/www/courses/create.html
@@ -127,6 +127,7 @@
         </div>
     </div>
 
+    {% if is_moderator %}
     <div class="field-group vertically-center">
         <label for="published" class="vertically-center mb-0">
             <input type="checkbox" id="published" {% if course.published %} checked {% endif %}>
@@ -137,6 +138,7 @@
             {{ _("Upcoming") }}
         </label>
     </div>
+    {% endif %}
 
     <div class="field-group">
         <div>
diff --git a/lms/www/courses/create.py b/lms/www/courses/create.py
index a20388ee..569dea3c 100644
--- a/lms/www/courses/create.py
+++ b/lms/www/courses/create.py
@@ -1,5 +1,9 @@
 import frappe
-from lms.lms.utils import redirect_to_courses_list, can_create_courses
+from lms.lms.utils import (
+	redirect_to_courses_list,
+	can_create_courses,
+	has_course_moderator_role,
+)
 from frappe import _
 
 
@@ -27,6 +31,7 @@ def get_context(context):
 	else:
 		set_course_context(context, course_name)
 
+	context.is_moderator = has_course_moderator_role()
 	context.member = frappe.db.get_value(
 		"User", frappe.session.user, ["full_name", "username"], as_dict=True
 	)