Skip to content
Snippets Groups Projects
Commit e319b916 authored by Julian's avatar Julian
Browse files

Hide persons without courses on timetable list

parent bd902488
No related branches found
No related tags found
1 merge request!382Confidential Issue
Pipeline #193576 passed
from datetime import date, datetime, timedelta
from typing import TYPE_CHECKING, Optional
from django.db.models import Q
from django.db.models import Count, Q
from guardian.shortcuts import get_objects_for_user
......@@ -19,7 +19,11 @@ if TYPE_CHECKING:
def get_teachers(user: "User"):
"""Get the teachers whose timetables are allowed to be seen by current user."""
teachers = Person.objects.all().order_by("short_name", "last_name")
teachers = (
Person.objects.annotate(course_count=Count("courses_as_teacher"))
.filter(course_count__gt=0)
.order_by("short_name", "last_name")
)
if not user.has_perm("chronos.view_all_person_timetables"):
teachers.filter(
......
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