Merge pull request #572 from tahir-zaqout/fix-instructors

fix: course when instructor is not set
frappe · Aug 25, 2023 · 1676329 · 1676329
2 parents 7f67c6c + 86434ea
commit 1676329
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lms/lms/utils.py b/lms/lms/utils.py
@@ -173,15 +173,15 @@ def get_tags(course):
 def get_instructors(course):
 	instructor_details = []
 	instructors = frappe.get_all(
-		"Course Instructor", {"parent": course}, ["instructor"], order_by="idx"
+		"Course Instructor", {"parent": course}, order_by="idx", pluck="instructor"
 	)
 	if not instructors:
 		instructors = frappe.db.get_value("LMS Course", course, "owner").split(" ")
 	for instructor in instructors:
 		instructor_details.append(
 			frappe.db.get_value(
 				"User",
-				instructor.instructor,
+				instructor,
 				["name", "username", "full_name", "user_image"],
 				as_dict=True,
 			)