From 0de1835516d8e8dc871e4b4a1db3fb367e7eb80f Mon Sep 17 00:00:00 2001
From: Jonathan Weth <git@jonathanweth.de>
Date: Thu, 25 Mar 2021 17:06:28 +0100
Subject: [PATCH] Fix position of order_by in lessons_on_day query on person
 model

The order_by wasn't used in the old position because filter_person
uses an union query.
---
 aleksis/apps/chronos/model_extensions.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/aleksis/apps/chronos/model_extensions.py b/aleksis/apps/chronos/model_extensions.py
index c95fc7b0..9bdf9fa8 100644
--- a/aleksis/apps/chronos/model_extensions.py
+++ b/aleksis/apps/chronos/model_extensions.py
@@ -83,7 +83,7 @@ def lesson_periods_as_teacher(self):
 @Person.method
 def lessons_on_day(self, day: date):
     """Get all lessons of this person (either as participant or teacher) on the given day."""
-    return LessonPeriod.objects.order_by("period__period").on_day(day).filter_from_person(self)
+    return LessonPeriod.objects.on_day(day).filter_from_person(self).order_by("period__period")
 
 
 @Person.method
-- 
GitLab