Développement #58158
Filtres as_numeral, corriger le fonctionnement pour les nombres à virgule
0%
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
History
Updated by Frédéric Péters about 3 years ago
Faire un ticket upstream pour corriger pour tout le monde ?
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é).
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.
Updated by Valentin Deniaud about 3 years ago
- File 0001-templatetags-handle-only-integers-with-as_numeral-fi.patch 0001-templatetags-handle-only-integers-with-as_numeral-fi.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Frédéric Péters a écrit :
gérer uniquement les entiers
Dac, allons-y comme ça.
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 ?).
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.
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".
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.
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".)
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.
Updated by Benjamin Dauvergne almost 3 years ago
- Status changed from Solution proposée to Solution validée
Allez go.
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)Updated by Frédéric Péters almost 3 years ago
- Status changed from Résolu (à déployer) to Solution déployée
templatetags: handle only integers with as_numeral filter (#58158)