From 76e344f68c1616693e93791f55348272cc61267e Mon Sep 17 00:00:00 2001
From: Alban Gruin
Date: Thu, 5 Oct 2017 22:02:05 +0200
Subject: Simplification du filtrage de process_timetable_week() (pourquoi
 réinventer la roue carrée ?…) Ajout de commentaires

---
 management/commands/timetables.py | 34 ++++++++++++++++++++++++++--------
 1 file changed, 26 insertions(+), 8 deletions(-)

diff --git a/management/commands/timetables.py b/management/commands/timetables.py
index 8ba8768..8945950 100644
--- a/management/commands/timetables.py
+++ b/management/commands/timetables.py
@@ -26,31 +26,49 @@ from ._private import delete_courses_in_week, get_events, get_update_date, get_w
 
 @transaction.atomic
 def process_timetable_week(timetable, soup, weeks_in_soup, force, year=None, week=None):
-    criteria = {}
     today = timezone.make_aware(datetime.datetime.now())
+
+    # On récupère la mise à jour la plus ancienne dans les cours de l’emploi du temps
+    # commençant à partir de maintenant
+    last_update_date = Course.objects.filter(timetable=timetable, begin__gte=today)
     if year is not None and week is not None:
-        begin, end = get_week(year, week)
-        criteria["begin__gte"] = begin
-        criteria["begin__lt"] = end
+        # Si jamais on traite une semaine spécifique, on limite les cours sélectionnés
+        # à ceux qui commencent entre le début du traitement et la fin de la semaine
+        _, end = get_week(year, week)
+        last_update_date = last_update_date.filter(begin__lt=end)
+
+    last_update_date = last_update_date.aggregate(Min("last_update")) \
+                       ["last_update__min"]
 
-    last_update_date = Course.objects.filter(timetable=timetable, **criteria) \
-                                     .aggregate(Min("last_update")) \
-                                     ["last_update__min"]
+    # Date de mise à jour de Celcat, utilisée à des fins de statistiques
     new_update_date = get_update_date(soup)
 
+    # On ne fait pas la mise à jour si jamais la dernière date de MàJ est plus récente
+    # que celle indiquée par Celcat.
+    # Attention, le champ last_update de la classe Course représente l’heure à laquelle
+    # le cours a été inséré dans la base de données, et non pas la date indiquée par
+    # Celcat.
     if not force and last_update_date is not None and new_update_date is not None and \
        last_update_date >= new_update_date:
         return
 
     if year is not None and week is not None:
+        # On efface la semaine à partir de maintenant si jamais
+        # on demande le traitement d’une seule semaine
         delete_courses_in_week(timetable, year, week, today)
     else:
-        delete_from = max(min(weeks_in_soup.values()), today)
+        # Sinon, on efface tous les cours à partir de maintenant.
+        # Précisément, on prend la plus grande valeur entre la première semaine
+        # présente dans Celcat et maintenant.
+        delete_from = max(min(weeks_in_soup.values()), today) # Vraiment utile ?
         Course.objects.filter(timetable=timetable, begin__gte=delete_from).delete()
 
+    # Tous les cours commençant sur la période traitée
+    # sont parsés, puis enregistrés dans la base de données.
     for course in get_events(timetable, soup, weeks_in_soup, today, year, week):
         course.save()
 
+    # On renseigne la date de mise à jour de Celcat, à des fins de statistiques
     timetable.last_update_date = new_update_date
     timetable.save()
 
-- 
cgit v1.2.1