Project

General

Profile

Bug #58022

Filtres as_numeral, fonctionner avec num2words 0.5.6

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

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

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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.


Files

Associated revisions

Revision 4b3a5811 (diff)
Added by Valentin Deniaud about 3 years ago

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

History

#1

Updated by Valentin Deniaud about 3 years ago

À 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

Updated by Valentin Deniaud about 3 years ago

  • Tracker changed from Développement to Bug
#4

Updated by Benjamin Dauvergne about 3 years ago

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

Updated by Benjamin Dauvergne about 3 years ago

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

Updated by Valentin Deniaud about 3 years ago

  • Status changed from Solution validée to 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

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

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

Also available in: Atom PDF