From 21383d2355deb819c01978284eb96adf99125db1 Mon Sep 17 00:00:00 2001 From: Brett Smith Date: Fri, 4 Sep 2020 14:46:18 -0400 Subject: [PATCH] reports: Correct use of NumberPattern.grouping. The 2-tuple represents min/max grouping, not positive/negative. --- conservancy_beancount/reports/core.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/conservancy_beancount/reports/core.py b/conservancy_beancount/reports/core.py index 985c85f..45e5040 100644 --- a/conservancy_beancount/reports/core.py +++ b/conservancy_beancount/reports/core.py @@ -752,9 +752,6 @@ class BaseODS(BaseSpreadsheet[RT, ST], metaclass=abc.ABCMeta): fmt = fmts[fmt_index] except IndexError: fmt = fmts[0] - grouping = pattern.grouping[0] - else: - grouping = pattern.grouping[fmt_index] style = self.replace_child( root, odf.number.CurrencyStyle, @@ -769,7 +766,7 @@ class BaseODS(BaseSpreadsheet[RT, ST], metaclass=abc.ABCMeta): elif not part.strip('#0,.'): style.addElement(odf.number.Number( decimalplaces=str(pattern.frac_prec[0]), - grouping='true' if grouping else 'false', + grouping='true' if pattern.grouping[0] else 'false', minintegerdigits=str(minintegerdigits), )) elif part == 'ยค':