Projet

Général

Profil

Development #79168

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

Ajouté par Valentin Deniaud il y a 10 mois. Mis à jour il y a 10 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
28 juin 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

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).

Historique

#1

Mis à jour par Robot Gitea il y a 10 mois

  • Statut changé de Nouveau à En cours

Valentin Deniaud (vdeniaud) a ouvert une pull request sur Gitea concernant cette demande :

#2

Mis à jour par Robot Gitea il y a 10 mois

  • Statut changé de En cours à Solution proposée
#3

Mis à jour par Robot Gitea il y a 10 mois

  • Statut changé de Solution proposée à Solution validée

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#4

Mis à jour par Robot Gitea il y a 10 mois

  • Statut changé de Solution validée à Résolu (à déployer)

Valentin Deniaud (vdeniaud) a mergé une pull request sur Gitea concernant cette demande :

#5

Mis à jour par Transition automatique il y a 10 mois

  • Statut changé de Résolu (à déployer) à Solution déployée
#6

Mis à jour par Transition automatique il y a 8 mois

Automatic expiration

Formats disponibles : Atom PDF