Projet

Général

Profil

Bug #20203

make_date ne gère pas les dates iso avec une timezone

Ajouté par Thomas Noël il y a plus de 6 ans. Mis à jour il y a 4 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
21 novembre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Plante sur 2005-10-18T00:00:00+02:00

Ma piste serait d'ajouter les outils de Django parse_date et parse_datetime en dernier recours sur get_as_datetime


Demandes liées

Lié à w.c.s. - Development #20842: Avoir un templatetag pour parser une dateFermé21 décembre 2017

Actions

Révisions associées

Révision 9cf4fa96 (diff)
Ajouté par Frédéric Péters il y a 4 mois

misc: add support for more datetime formats (#20203)

Historique

#2

Mis à jour par Thomas Noël il y a environ 6 ans

#3

Mis à jour par Thomas Noël il y a environ 6 ans

Avec le format de template Django et #20842 je crois qu'on doit pouvoir se passer de ça ; mais le ticket ne donne pas plus de contexte, ma faute.

#4

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

Je relance on a eu le souci sur CD13 aujourd'hui; en même temps c'est une erreur de leur part, ils nous renvoient 2018-09-30T00:00:00.000+02:00 au lieu d'une date simple 2018-09-30 et donc on a pu le voir, néanmoins il serait bien qu'on puisse parser ce genre de dates et les mettre de force dans la timezone locale (avec make_naive()).

#5

Mis à jour par Thomas Noël il y a environ 5 ans

Je cherche le contexte de ton soucis

Est-ce que ton pépin dans cette affaire c'est que 2018-09-30T00:00:00.000+02:00 c'est en fait le 29/09/2018 à 22h ?

Sinon, avec le code actuel {{ "2018-09-30T00:00:00.000+02:00"|datetime|date }} renvoie bien le 30 septembre.

En revanche {{ "2018-09-30T00:00:00.000+02:00"|date }} plante, alors que {{ "2018-09-30T00:00:00"|date }} passe.

Donc en fait ma question : qu'est-ce que tu cherchais à faire comme traitement sur "2018-09-30T00:00:00.000+02:00" ?

#6

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Statut changé de Nouveau à Fermé

Si datetime passe c'est ok pour moi, je pensais que ce n'était pas le cas mais je n'ai testé qu'en python, via le vieux make_datetime pensant que c'était équivalent, mais les choses commencent à bien diverger entre evalutils et les templatetags, il y a une certaine volonté à rendre le python inutilisable ce que je trouve mal mais j'ai peu de chance d'être entendu.

Aussi le comportement diffère si on fait un |date:"xxx" ou un |date dans le premier cas parse_datetime est utilisé pas dans le second, donc la chaîne en question va passer avec |date:"xxx" mais pas |date ce qui peut paraître incohérent.

#7

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Statut changé de Fermé à Information nécessaire
#8

Mis à jour par Frédéric Péters il y a 4 mois

  • Statut changé de Information nécessaire à En cours
  • Assigné à mis à Frédéric Péters
  • Planning mis à Non

En revanche {{ "2018-09-30T00:00:00.000+02:00"|date }} plante

Ça n'est pas totalement une affaire de timezone, ça échoue aussi sur {{ "2018-09-30T00:00:00.000"|date }},

#9

Mis à jour par Robot Gitea il y a 4 mois

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

#10

Mis à jour par Robot Gitea il y a 4 mois

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

Mis à jour par Robot Gitea il y a 4 mois

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

Thomas NOËL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :

#12

Mis à jour par Robot Gitea il y a 4 mois

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

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

#13

Mis à jour par Transition automatique il y a 4 mois

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

Mis à jour par Transition automatique il y a 2 mois

Automatic expiration

Formats disponibles : Atom PDF