diff options
author | Alban Gruin | 2018-12-31 13:31:16 +0100 |
---|---|---|
committer | Alban Gruin | 2018-12-31 13:31:16 +0100 |
commit | cc92838a22eeb406474f7f5fa48eb42fa5ee4e4f (patch) | |
tree | 4941e32f67d20842cbf8ce41e37e5d3e305094de /urls.py | |
parent | 171472d7dc42e2d3b390ad8b052c7e88fca21722 (diff) |
ups2018: get_weeks() retourne la vraie liste des semaines présentes
Change la fonction get_weeks() du parseur UPS2018 pour créer la liste
des semaines présentes à partir de la liste des événements.
Avant, cette liste contenait seulement la semaine courante, ce qui
posait problème quand une mise à jour était faite avec les paramètres
`--all --force' : les cours à partir de la semaine courante étaient
supprimés, mais les cours rajoutés étaient ceux du mois entier. Il
était donc possible de se retrouver avec des cours en doublons entre
la première semaine du mois courant et la semaine courante.
Il peut éventuellement y avoir un défaut avec cette technique : si
tous les cours des premières semaines du mois ont été retirées de
l’emploi du temps parce qu’ils ont étés annulés, les premières
semaines ne se trouveront pas dans la liste des semaines disponibles,
et ces cours resterons dans la base de celcatsanitizer, alors qu’ils
ont étés annulés. Il serait peut-être plus judicieux de créer une
liste de manière statique, en prenant comme seule valeur la première
semaine du mois.
La fonction get_weeks() calcule donc le début de chaque événement et
vérifie si son titre ne contient pas « global event » pour ne pas le
prendre en compte. Pour éviter de faire ces opérations deux fois, ces
données sont stockées dans l’événement.
Signed-off-by: Alban Gruin <alban at pa1ch dot fr>
Diffstat (limited to 'urls.py')
0 files changed, 0 insertions, 0 deletions