Projet

Général

Profil

Bug #58022

Filtres as_numeral, fonctionner avec num2words 0.5.6

Ajouté par Valentin Deniaud il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
20 octobre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

C'est la version dans buster et en fait il y a de fortes différences entre les versions, donc commencer par épingler celle-ci et faire en sorte que ça marche.


Fichiers

Révisions associées

Révision 4b3a5811 (diff)
Ajouté par Valentin Deniaud il y a plus de 2 ans

templatetags: pass float to num2words and work with 0.5.6 (#58022)

Historique

#1

Mis à jour par Valentin Deniaud il y a plus de 2 ans

À noter,

-    assert t.render(Context({'number': 42})) == 'quarante-deux euros et zéro centimes'
+    assert t.render(Context({'number': 42})) == 'quarante-deux euros'

qui est bien lié à une différence entre ces versions de num2words : la forme verbeuse arrive via la dernière version, 0.5.10 (qui est packagée pour bookworm mais pas bullseye qui est en 0.5.9).

Donc un beau jour on mettra à jour Debian et oups côté usager le montant sur les factures ne sera du jour au lendemain plus écrit pareil : est-ce qu'on accepte cette future situation ? Sinon j'imagine qu'il faut réussir à fonctionner avec la dernière version d'emblée.

#3

Mis à jour par Valentin Deniaud il y a plus de 2 ans

  • Tracker changé de Development à Bug
#4

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

Il faudrait rajouter des tests avec des chiffres plus longs jusqu'à 100000 disons), en nombre et chaînes et pour ce dernier cas avec des virgules françaises pour passer dans cette partie du code.


Pour ta question je dirai qu'on va déjà livrer la fonctionnalité avec la 0.5.6, et si ça marche de manière identique 0.5.9 en buster tant mieux, ça nous laisse encore plus de temps; dans l'absolu l'ajout de zéro centime ne me gêne pas plus que ça, mais on pourra voir pour demander upstream que ce soit éventuellement optionnel.

#5

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

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

Mis à jour par Valentin Deniaud il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 4b3a581150d68ab2f03c95dc5cb1a5780eb91250
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Wed Oct 20 17:57:34 2021 +0200

    templatetags: pass float to num2words and work with 0.5.6 (#58022)
#7

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

Formats disponibles : Atom PDF