Skip to content
Snippets Groups Projects
Verified Commit 32ad8b40 authored by Jonathan Weth's avatar Jonathan Weth :keyboard:
Browse files

Fix view_week_personalnote

parent 5a900697
No related branches found
No related tags found
1 merge request!49Resolve "Add rules and permissions"
Pipeline #3512 passed
...@@ -21,6 +21,7 @@ from .util.predicates import ( ...@@ -21,6 +21,7 @@ from .util.predicates import (
is_person_group_owner, is_person_group_owner,
is_personal_note_lesson_parent_group_owner, is_personal_note_lesson_parent_group_owner,
is_personal_note_lesson_teacher, is_personal_note_lesson_teacher,
is_teacher,
) )
# View lesson # View lesson
...@@ -99,6 +100,7 @@ view_week_personal_notes_predicate = has_person & ( ...@@ -99,6 +100,7 @@ view_week_personal_notes_predicate = has_person & (
has_global_perm("alsijil.view_personalnote") has_global_perm("alsijil.view_personalnote")
| has_object_perm("core.view_personalnote_group") | has_object_perm("core.view_personalnote_group")
| is_group_owner | is_group_owner
| (is_current_person & is_teacher)
) )
add_perm("alsijil.view_week_personalnote", view_week_personal_notes_predicate) add_perm("alsijil.view_week_personalnote", view_week_personal_notes_predicate)
......
...@@ -208,3 +208,9 @@ def has_any_object_absence(user: User) -> bool: ...@@ -208,3 +208,9 @@ def has_any_object_absence(user: User) -> bool:
member_of__in=get_objects_for_user(user, "core.register_absence_group", Group) member_of__in=get_objects_for_user(user, "core.register_absence_group", Group)
).exists(): ).exists():
return True return True
@predicate
def is_teacher(user: User, obj: Person) -> bool:
"""Predicate which checks if the provided object is a teacher."""
return user.person.is_teacher()
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