Skip to content
Snippets Groups Projects
Commit b4c3648d authored by Hangzhi Yu's avatar Hangzhi Yu
Browse files

Change evaluation order in coursebook predicates

parent 5a7206de
No related branches found
No related tags found
2 merge requests!352Draft: Resolve "Add dialog with each lesson's students",!350Resolve "Add simple course book list"
Pipeline #175568 failed
......@@ -387,18 +387,18 @@ def is_lesson_event_group_owner(user: User, obj: LessonEvent):
def can_view_documentation(user: User, obj: Documentation):
"""Predicate which checks if the user is allowed to view a documentation."""
if obj:
if obj.course:
return (
is_course_teacher(user, obj.course)
| is_course_member(user, obj.course)
| is_course_group_owner(user, obj.course)
)
if obj.lesson_event:
return (
is_lesson_event_teacher(user, obj.lesson_event)
| is_lesson_event_member(user, obj.lesson_event)
| is_lesson_event_group_owner(user, obj.lesson_event)
)
if obj.course:
return (
is_course_teacher(user, obj.course)
| is_course_member(user, obj.course)
| is_course_group_owner(user, obj.course)
)
return False
......@@ -406,10 +406,10 @@ def can_view_documentation(user: User, obj: Documentation):
def can_edit_documentation(user: User, obj: Documentation):
"""Predicate which checks if the user is allowed to edit or delete a documentation."""
if obj:
if obj.course:
return is_course_teacher(user, obj.course) | is_course_group_owner(user, obj.course)
if obj.lesson_event:
return is_lesson_event_teacher(user, obj.lesson_event) | is_lesson_event_group_owner(
user, obj.lesson_event
)
if obj.course:
return is_course_teacher(user, obj.course) | is_course_group_owner(user, obj.course)
return False
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment