3
0
Fork 0

fix: include thousands separator for app.render_quantity()

This commit is contained in:
Lance Edgar 2025-12-19 17:20:44 -06:00
parent 37cf3c4400
commit 78600c8cc2
2 changed files with 10 additions and 2 deletions

View file

@ -962,8 +962,8 @@ class AppHandler: # pylint: disable=too-many-public-methods
value = int(value) value = int(value)
if empty_zero and value == 0: if empty_zero and value == 0:
return "" return ""
return str(value) return f"{value:,}"
return str(value).rstrip("0") return f"{value:,}".rstrip("0")
def render_time_ago(self, value): def render_time_ago(self, value):
""" """

View file

@ -672,6 +672,14 @@ app_title = WuttaTest
self.assertEqual(self.app.render_quantity(0), "0") self.assertEqual(self.app.render_quantity(0), "0")
self.assertEqual(self.app.render_quantity(0, empty_zero=True), "") self.assertEqual(self.app.render_quantity(0, empty_zero=True), "")
# has thousands separator
value = 1234
self.assertEqual(self.app.render_quantity(value), "1,234")
value = decimal.Decimal("1234.567")
self.assertEqual(self.app.render_quantity(value), "1,234.567")
value = decimal.Decimal("1234.567000")
self.assertEqual(self.app.render_quantity(value), "1,234.567")
def test_render_time_ago(self): def test_render_time_ago(self):
with patch.object(mod, "humanize") as humanize: with patch.object(mod, "humanize") as humanize:
humanize.naturaltime.return_value = "now" humanize.naturaltime.return_value = "now"