Development #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
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 2 ans
Faire un ticket upstream pour corriger pour tout le monde ?
Mis à jour par Valentin Deniaud il y a plus de 2 ans
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é).
Mis à jour par Frédéric Péters il y a plus de 2 ans
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.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-templatetags-handle-only-integers-with-as_numeral-fi.patch 0001-templatetags-handle-only-integers-with-as_numeral-fi.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Frédéric Péters a écrit :
gérer uniquement les entiers
Dac, allons-y comme ça.
Mis à jour par Paul Marillonnet il y a plus de 2 ans
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 ?).
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
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.
Mis à jour par Frédéric Péters il y a plus de 2 ans
Certes mais "ne pas risquer des gens faisant le rendu monétaire + hack pour virer la mention euro".
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
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.
Mis à jour par Paul Marillonnet il y a plus de 2 ans
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".)
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
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.
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Allez go.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution validée à 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)Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
templatetags: handle only integers with as_numeral filter (#58158)