Project

General

Profile

Development #93896

LazyDateObject fait mauvais ménage avec Django 4

Added by Valentin Deniaud 2 months ago. Updated 2 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
06 August 2024
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Django 4 passe de pytz à zoneinfo et ça change pas mal par où passe le code.

Sur un simple {{ now }} on passe désormais par une méthode Django qui va fait un tz.utcoffset(dt) avec dt de type LazyDateObject, et ça produit une erreur de segmentation.

On pourrait dire que le problème est plus général, par exemple j'arrive à faire planter le code actuel en 3.2 en tapant {{ now|time:"O" }}, qui produit « unsupported operand type(s) for +: 'LazyDateObject' and 'datetime.timedelta' ».

#29406 a introduit ces objets, de la description

Via des SimpleLazyObject proposés par Django, on peut surcharger eq/ne/gt/lt/etc. et faire en sorte que "now" et "today" acceptent d'être comparer à date ou datetime, indifféremment.

je ne comprends pas bien pourquoi hériter de SimpleLazyObject, pourquoi pas juste hériter de datetime et surcharger les opérateur pareillement.


Related issues

Related to w.c.s. - Development #93785: Compatibilité Django 4.2Solution déployée01 August 2024

Actions

Associated revisions

Revision ae3f8e84 (diff)
Added by Valentin Deniaud 2 months ago

variables: remove unneeded laziness from custom date objects (#93896)

Revision 48656421 (diff)
Added by Valentin Deniaud 2 months ago

misc: use accurate naming for custom date objects (#93896)

History

#1

Updated by Robot Gitea 2 months 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 2 months ago

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

Updated by Robot Gitea 2 months ago

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

Frédéric Péters (fpeters) a approuvé une pull request sur Gitea concernant cette demande :

#4

Updated by Robot Gitea 2 months ago

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

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#5

Updated by Transition automatique 2 months ago

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

Updated by Valentin Deniaud about 2 months ago

Also available in: Atom PDF