From d954262184e15ca93d512b61e15b99ba6c421aa0 Mon Sep 17 00:00:00 2001 From: Jannat Patel Date: Mon, 4 Apr 2022 12:46:35 +0530 Subject: [PATCH] fix: redirect to list when course does not exist --- lms/www/courses/course.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lms/www/courses/course.py b/lms/www/courses/course.py index ef969cdd..4e1d923e 100644 --- a/lms/www/courses/course.py +++ b/lms/www/courses/course.py @@ -16,16 +16,16 @@ def get_context(context): "disable_self_learning", "video_link", "enable_certification", "status"], as_dict=True) + if course is None: + frappe.local.flags.redirect_location = "/courses" + raise frappe.Redirect + related_courses = frappe.get_all("Related Courses", {"parent": course.name}, ["course"]) for csr in related_courses: csr.update(frappe.db.get_value("LMS Course", csr.course, ["name", "upcoming", "title", "image", "enable_certification"], as_dict=True)) course.related_courses = related_courses - if course is None: - frappe.local.flags.redirect_location = "/courses" - raise frappe.Redirect - context.course = course membership = get_membership(course.name, frappe.session.user) context.course.query_parameter = "?batch=" + membership.batch if membership and membership.batch else ""