Développement #79168
Utiliser partout la numérotation iso des jours de la semaine
0%
Description
Python numérote de 0 à 6 avec 0 le lundi, les lookups db de Django numérotent de 1 à 7 avec 1 le dimanche.
Dans le code ça donne des lignes genre
dj_weekday=ExtractWeekDay('start_datetime'), dj_weekday_int=Cast('dj_weekday', models.IntegerField()), weekday=(F('dj_weekday_int') - 2) % 7,
et c'est pas mal de nœuds aux cerveau à chaque fois (#79112 dernier en date).
Or depuis Django 3.1 on dispose d'un lookup iso_week_day https://docs.djangoproject.com/en/3.2/ref/models/querysets/#iso-week-day, les objets datetime ont également une méthode isoweekday.
Tentons d'utiliser ça partout (ça nécessitera au moins une migration pour les évènements récurrents, dont le champ recurrence_days contient les numéros des jours de la semaine).
History
Updated by Robot Gitea over 1 year ago
- Status changed from Nouveau to En cours
Valentin Deniaud (vdeniaud) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/chrono/pulls/111
- Titre : WIP: agendas: use iso week days in events and shared custody (#79168)
- Modifications : https://git.entrouvert.org/entrouvert/chrono/pulls/111/files
Updated by Robot Gitea over 1 year ago
- Status changed from Solution proposée to Solution validée
Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :
Updated by Robot Gitea over 1 year ago
- Status changed from Solution validée to Résolu (à déployer)
Valentin Deniaud (vdeniaud) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/chrono/pulls/111
- Titre : agendas: use iso week days in events and shared custody (#79168)
- Modifications : https://git.entrouvert.org/entrouvert/chrono/pulls/111/files
Updated by Transition automatique over 1 year ago
- Status changed from Résolu (à déployer) to Solution déployée