fix: course permissions
This commit is contained in:
@@ -7,6 +7,7 @@ from lms.lms.utils import (
|
|||||||
get_courses_under_review,
|
get_courses_under_review,
|
||||||
get_average_rating,
|
get_average_rating,
|
||||||
check_multicurrency,
|
check_multicurrency,
|
||||||
|
has_course_instructor_role,
|
||||||
)
|
)
|
||||||
from lms.overrides.user import get_enrolled_courses, get_authored_courses
|
from lms.overrides.user import get_enrolled_courses, get_authored_courses
|
||||||
|
|
||||||
@@ -24,7 +25,13 @@ def get_context(context):
|
|||||||
portal_course_creation = frappe.db.get_single_value(
|
portal_course_creation = frappe.db.get_single_value(
|
||||||
"LMS Settings", "portal_course_creation"
|
"LMS Settings", "portal_course_creation"
|
||||||
)
|
)
|
||||||
context.show_creators_section = True if portal_course_creation == "Anyone" else False
|
context.show_creators_section = (
|
||||||
|
True
|
||||||
|
if portal_course_creation == "Anyone"
|
||||||
|
or has_course_moderator_role()
|
||||||
|
or has_course_instructor_role()
|
||||||
|
else False
|
||||||
|
)
|
||||||
context.show_review_section = (
|
context.show_review_section = (
|
||||||
has_course_moderator_role() and frappe.session.user != "Guest"
|
has_course_moderator_role() and frappe.session.user != "Guest"
|
||||||
)
|
)
|
||||||
|
|||||||
@@ -10,7 +10,7 @@ def get_context(context):
|
|||||||
if not frappe.db.exists("LMS Course", course_name):
|
if not frappe.db.exists("LMS Course", course_name):
|
||||||
redirect_to_courses_list()
|
redirect_to_courses_list()
|
||||||
|
|
||||||
if not can_create_courses():
|
if not can_create_courses(course_name):
|
||||||
message = "You do not have permission to access this page."
|
message = "You do not have permission to access this page."
|
||||||
if frappe.session.user == "Guest":
|
if frappe.session.user == "Guest":
|
||||||
message = "Please login to access this page."
|
message = "Please login to access this page."
|
||||||
|
|||||||
Reference in New Issue
Block a user