Project

General

Profile

Développement #79168

Utiliser partout la numérotation iso des jours de la semaine

Added by Valentin Deniaud over 1 year ago. Updated over 1 year ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
28 June 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

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

#1

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 :

#2

Updated by Robot Gitea over 1 year ago

  • Status changed from En cours to Solution proposée
#3

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 :

#4

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 :

#5

Updated by Transition automatique over 1 year ago

  • Status changed from Résolu (à déployer) to Solution déployée
#6

Updated by Transition automatique over 1 year ago

Automatic expiration

Also available in: Atom PDF