From 3e3afa63c25b32ae3ea442f0902e729f962e2393 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Thu, 12 Feb 2026 10:44:01 +0530
Subject: [PATCH] fix: issues on home page

---
 lms/lms/api.py   |  8 ++++----
 lms/lms/utils.py | 12 +++---------
 2 files changed, 7 insertions(+), 13 deletions(-)

diff --git a/lms/lms/api.py b/lms/lms/api.py
index 84155f3e..00097e83 100644
--- a/lms/lms/api.py
+++ b/lms/lms/api.py
@@ -618,14 +618,14 @@ def check_app_permission():
 def save_evaluation_details(
 	member: str,
 	course: str,
-	batch_name: str,
-	evaluator: str,
 	date: str,
 	start_time: str,
 	end_time: str,
 	status: str,
-	rating: float,
-	summary: str,
+	batch_name: str = None,
+	evaluator: str = None,
+	rating: float = 0,
+	summary: str = None,
 ):
 	"""
 	Save evaluation details for a member against a course.
diff --git a/lms/lms/utils.py b/lms/lms/utils.py
index 5631aa1c..56739992 100644
--- a/lms/lms/utils.py
+++ b/lms/lms/utils.py
@@ -882,7 +882,8 @@ def get_course_details(course: str):
 		return {}
 
 	is_course_published = frappe.db.get_value("LMS Course", course, "published")
-	if not is_course_published and not can_modify_course(course):
+	membership = get_membership(course)
+	if not is_course_published and not can_modify_course(course) and not membership:
 		return {}
 
 	fields = get_course_fields()
@@ -894,6 +895,7 @@ def get_course_details(course: str):
 	)
 
 	course_details.instructors = get_instructors("LMS Course", course_details.name)
+	course_details.membership = membership
 	# course_details.is_instructor = is_instructor(course_details.name)
 	if course_details.paid_course or course_details.paid_certificate:
 		"""course_details.course_price, course_details.currency = check_multicurrency(
@@ -902,15 +904,7 @@ def get_course_details(course: str):
 		course_details.price = fmt_money(course_details.course_price, 0, course_details.currency)
 
 	if frappe.session.user == "Guest":
-		course_details.membership = None
 		course_details.is_instructor = False
-	else:
-		course_details.membership = frappe.db.get_value(
-			"LMS Enrollment",
-			{"member": frappe.session.user, "course": course_details.name},
-			["name", "course", "current_lesson", "progress", "member"],
-			as_dict=1,
-		)
 
 	if course_details.membership and course_details.membership.current_lesson:
 		course_details.current_lesson = get_lesson_index(course_details.membership.current_lesson)