DecimalFormat grouping separator is broken on Nougat [37135768]

Fixed

Bug

[AOSP] FutureRelease

Status Update

No update yet.

Description

[Deleted User]

created issue #1

Feb 14, 2017 12:46PM

The DecimalFormatSymbols grouping separator is ignored on Nougat, and the number is incorrectly formatted with a comma ',' as the grouping separator even though it is configured to use a space ' '.

The following test passes on API levels <=23, but fails on Nougat (24 or 25).

DecimalFormatSymbols dfs = new DecimalFormatSymbols();
dfs.setCurrencySymbol("€");
dfs.setDecimalSeparator(',');
dfs.setMonetaryDecimalSeparator(',');
dfs.setGroupingSeparator(' ');
dfs.setInternationalCurrencySymbol("EUR");

DecimalFormat format = new java.text.DecimalFormat(pattern, dfs);
String formatted = format.format(1234.56);
assertEquals("1 234,56€", formatted);

The failure is identical with both the java.text.* and android.icu.text.* versions of DecimalFormat.

The attached project contains a small repro in the form of an instrumentation test case.

deleted

Restricted

0 B

Comments

[Deleted User] <[Deleted User]> #2Feb 14, 2017 12:49PM

We use build flavours heavily with a lot of common code. The refactoring support in AS is really good but it continually catches us out when it doesn't work across all flavours in a project. It's a big gap for serious product development.

[Deleted User] <[Deleted User]> #3Feb 14, 2017 02:36PM

We at my company need this same feature. We have a lot of white labels and need refactor the same class across flavours. :(

me...@gmail.com <me...@gmail.com> #4Mar 20, 2017 12:48PM

I need this feature too...

ar...@google.com <ar...@google.com> #5May 5, 2017 12:05PM

Assigned to ar...@google.com.

+1, I need this very badly

ar...@google.com <ar...@google.com> #6Jul 26, 2017 10:27AM

Marked as fixed.

+1 My company also need this feature.

Issue 37135768