From 7d2b86c5484d4a8feb95620c3435cb3bec6bc72b Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Thu, 13 Oct 2022 13:04:22 +0530
Subject: [PATCH] fix: dont show start learning from course page if disabled

---
 lms/lms/widgets/CourseOutline.html  | 2 +-
 lms/lms/widgets/NoPreviewModal.html | 2 +-
 lms/www/courses/index.py            | 4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/lms/lms/widgets/CourseOutline.html b/lms/lms/widgets/CourseOutline.html
index 41c22908..17ce8c43 100644
--- a/lms/lms/widgets/CourseOutline.html
+++ b/lms/lms/widgets/CourseOutline.html
@@ -111,7 +111,7 @@
 
 </div>
 <!-- No Preview Modal -->
-{{ widgets.NoPreviewModal(course=course) }}
+{{ widgets.NoPreviewModal(course=course, membership=membership) }}
 
 <script>
 
diff --git a/lms/lms/widgets/NoPreviewModal.html b/lms/lms/widgets/NoPreviewModal.html
index 0f2144ad..0d9c5767 100644
--- a/lms/lms/widgets/NoPreviewModal.html
+++ b/lms/lms/widgets/NoPreviewModal.html
@@ -29,7 +29,7 @@
                 <button class="btn btn-primary btn-sm notify-me pull-right" data-course="{{course.name | urlencode}}">
                     {{ _("Notify me when available") }}
                 </button>
-                {% else %}
+                {% elif show_start_learing_cta(course, membership) %}
                 <button class="btn btn-primary btn-sm join-batch pull-right" data-course="{{ course.name | urlencode}}">
                     {{ _("Start Learning") }}
                 </button>
diff --git a/lms/www/courses/index.py b/lms/www/courses/index.py
index 3a92b724..f7797091 100644
--- a/lms/www/courses/index.py
+++ b/lms/www/courses/index.py
@@ -10,13 +10,13 @@ def get_context(context):
     portal_course_creation = frappe.db.get_single_value("LMS Settings", "portal_course_creation")
     context.show_creators_section = frappe.session.user != "Guest" and \
         (portal_course_creation == "Anyone" or has_course_instructor_role())
-    context.show_review_section = has_course_moderator_role() and frappe.session.user != "Gurst"
+    context.show_review_section = has_course_moderator_role() and frappe.session.user != "Guest"
 
     if context.restriction:
         context.restriction_details = get_restriction_details()
 
     context.metatags = {
-        "title": _("Courses List"),
+        "title": _("Course List"),
         "image": frappe.db.get_single_value("Website Settings", "banner_image"),
         "description": "This page lists all the courses published on our website",
         "keywords": "All Courses, Courses, Learn"