Project

General

Profile

Développement #58158

Filtres as_numeral, corriger le fonctionnement pour les nombres à virgule

Added by Valentin Deniaud about 3 years ago. Updated almost 3 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
25 October 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

assert t.render(Context({'number': 42.15})) == 'quarante-deux virgule un cinq'
Ce n'est pas très français.
On peut :
  • Virer ce filtre (suggestion de Benj, #57983#note-13)
  • Prendre à notre charge de convertir « 42 » et « 15 » séparément en splittant sur la virgule (suggestion de Paul, #57983#note-14)
    • Ce faisant casser le comportement en anglais ('forty-two point one five' étant le résultat attendu ?) ou ne faire ça que pour la locale fr

Files

Associated revisions

Revision 38189e9d (diff)
Added by Valentin Deniaud almost 3 years ago

templatetags: handle only integers with as_numeral filter (#58158)

History

#1

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

Faire un ticket upstream pour corriger pour tout le monde ?

#2

Updated by Valentin Deniaud about 3 years ago

Frédéric Péters a écrit :

Faire un ticket upstream pour corriger pour tout le monde ?

Ouep mais ça vient en plus et pas à la place, on aura de toute façon pas le fix packagé avant des années (outre qu'il y a 62 tickets et 75 PR ouvertes sur le projet, il faudra d'abord voir si ça n'a pas déjà été discuté).

#3

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

Ok, autre suggestion alors gérer uniquement les entiers, ça permet de parler de ce filtre, et de ne pas risquer des gens faisant le rendu monétaire + hack pour virer la mention euro.

#4

Updated by Valentin Deniaud about 3 years ago

Frédéric Péters a écrit :

gérer uniquement les entiers

Dac, allons-y comme ça.

#5

Updated by Paul Marillonnet about 3 years ago

Valentin Deniaud a écrit :

  • Prendre à notre charge de convertir « 42 » et « 15 » séparément en splittant sur la virgule (suggestion de Paul, #57983#note-14)
    • Ce faisant casser le comportement en anglais ('forty-two point one five' étant le résultat attendu ?) ou ne faire ça que pour la locale fr

Iirc en anglais américain ça se fait d’écrire "three point seventy-two", au delà de deux chiffres après la virgule c’est étrange de l’écrire autrement que chiffre par chiffre oui :)

Ne laisser que la partie entière c’est très bien aussi (mais peut-être renommer le filtre pour qu’on comprenne que s’il s’agit d’un flottant la partie décimale sera tronquée ?).

#6

Updated by Benjamin Dauvergne about 3 years ago

Je retente ma chance avec l'idée de le virer, il y a un seul cas d'usage pour l'instant Toulouse, et c'est pour l'autre filtre.

#7

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

Certes mais "ne pas risquer des gens faisant le rendu monétaire + hack pour virer la mention euro".

#8

Updated by Benjamin Dauvergne about 3 years ago

Frédéric Péters a écrit :

Certes mais "ne pas risquer des gens faisant le rendu monétaire + hack pour virer la mention euro".

Ça ne me parait pas suffisant pour justifier de conserver ce filtre.

#9

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

Ça me parait.

DUEL!

#10

Updated by Paul Marillonnet about 3 years ago

Frédéric Péters a écrit :

DUEL!

(La cote des paris en faveur de Benjamin est de un virgule six quatre.)

Blague à part, est-ce qu’on a des cas d’usages hors rendu monétaire ? Genre des convocations officielles de la collectivité où la date et heure devraient appairaître en toutes lettres ? ("Veuillez vous présenter le six juillet à quatorze heures etc".)

#11

Updated by Benjamin Dauvergne about 3 years ago

Paul Marillonnet a écrit :

Blague à part, est-ce qu’on a des cas d’usages hors rendu monétaire ? Genre des convocations officielles de la collectivité où la date et heure devraient appairaître en toutes lettres ? ("Veuillez vous présenter le six juillet à quatorze heures etc".)

Non, le seul cas d'usage c'est celui qui m'a amené à demander ce développement, cf. #57146-3, reçu fiscal pour un don en ligne, et dans ma courte vie je n'ai vu que des montants monétaires qu'il fallait écrire en toutes lettres.

#13

Updated by Benjamin Dauvergne almost 3 years ago

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

Allez go.

#14

Updated by Valentin Deniaud almost 3 years ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 38189e9de717315411839d6de81dc03ecfe16c97
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Mon Oct 25 11:59:30 2021 +0200

    templatetags: handle only integers with as_numeral filter (#58158)
#15

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

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

Updated by Transition automatique almost 3 years ago

Automatic expiration

Also available in: Atom PDF