aboutsummaryrefslogtreecommitdiff
path: root/management/commands/timetables.py
diff options
context:
space:
mode:
authorAlban Gruin2017-10-05 22:11:59 +0200
committerAlban Gruin2017-10-05 22:11:59 +0200
commitf6e81c1a165b983316b49bdd433cd12cfebe1897 (patch)
treed638e3cf6335dcab46a415c5cb9c5946ae02dbac /management/commands/timetables.py
parentae3358c1296a02352409910a9ffcc2307d5ea87a (diff)
parent76e344f68c1616693e93791f55348272cc61267e (diff)
Merge branch 'stable/0.10.z' into prod/pa1ch/0.10.zv0.10.1-pa1ch
Diffstat (limited to 'management/commands/timetables.py')
-rw-r--r--management/commands/timetables.py43
1 files changed, 31 insertions, 12 deletions
diff --git a/management/commands/timetables.py b/management/commands/timetables.py
index 76f0a7c..8945950 100644
--- a/management/commands/timetables.py
+++ b/management/commands/timetables.py
@@ -26,30 +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:
- delete_courses_in_week(timetable, year, week)
+ # 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:
- Course.objects.filter(timetable=timetable,
- begin__gte=min(weeks_in_soup.values())).delete()
-
- for course in get_events(timetable, soup, weeks_in_soup, year, week):
+ # 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()