Project

General

Profile

Développement #52142

test évènements récurrents qui ne passe pas à minuit

Added by Valentin Deniaud over 3 years ago. Updated over 3 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Valentin Deniaud
Category:
-
Target version:
-
Start date:
17 March 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Noté par Benj ici #17685#note-21, https://jenkins.entrouvert.org/job/chrono-wip/job/wip%252F17685--preblocage-d-une-reservation/2/testReport/junit/py3-django111-pg-black-coverage-pylint.tests/test_manager/test_delete_recurring_event/.

Files

0001-agendas-use-localtime-when-computing-recurrence-rule.patch (1.94 KB) 0001-agendas-use-localtime-when-computing-recurrence-rule.patch Valentin Deniaud, 17 March 2021 04:04 PM

Associated revisions

Revision b7f1a2c5 (diff)
Added by Valentin Deniaud over 3 years ago

agendas: use localtime when computing recurrence rule (#52142)

History

#1

Updated by Valentin Deniaud over 3 years ago

Vrai bug : on calcule les paramètres de répétition dans la méthode save(), paramètres incluant notamment le jour de répétition. Donc évènement à 00:30 -> 23:30 en UTC -> répétition le mardi au lieu du mercredi, genre. Ça c'est la théorie, en pratique on est sauvé par le fait que l'heure arrive en timezone locale quand on sauvegarde un évènement depuis le manager, et donc le calcul du jour utilise bien 00:30.

Il n'y a qu'un test qui plante à minuit parce que la plupart des autres ont un freezer, sinon ils casseraient aussi.

#2

Updated by Paul Marillonnet over 3 years ago

  • Status changed from Solution proposée to Solution validée
#3

Updated by Valentin Deniaud over 3 years ago

  • Status changed from Solution validée to Résolu (à déployer)
commit b7f1a2c5ec9ac1985ca8c7ab213cb50e1b08f200
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Wed Mar 17 15:30:31 2021 +0100

    agendas: use localtime when computing recurrence rule (#52142)
#4

Updated by Frédéric Péters over 3 years ago

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

Also available in: Atom PDF