ICU-22856 Fix ICUCurrencyDisplayInfoProvider.fetchUnitPatterns() so t…

…hat it doesn't skip inheriting patterns from root.
unicode-org · Sep 16, 2024 · ee7cc23 · ee7cc23
1 parent 688957e
commit ee7cc23
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/util/CurrencyTest.java b/icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/util/CurrencyTest.java
@@ -345,9 +345,7 @@ public void TestCurrencyDisplayNames() {
         // known currency, behavior unchanged
         assertEquals("de_USD_name", "US-Dollar", cdn.getName("USD"));
         assertEquals("de_USD_symbol", "$", cdn.getSymbol("USD"));
-        if (!logKnownIssue("ICU-22856", "Problem with inheritance for plural rules in currencies")) {
-        	assertEquals("de_USD_plural_other", "US-Dollar", cdn.getPluralName("USD", "other"));
-        }
+        assertEquals("de_USD_plural_other", "US-Dollar", cdn.getPluralName("USD", "other"));
 
         // known currency but unknown plural category
         assertNull("de_USD_plural_foo", cdn.getPluralName("USD", "foo"));

diff --git a/icu4j/main/currdata/src/main/java/com/ibm/icu/impl/ICUCurrencyDisplayInfoProvider.java b/icu4j/main/currdata/src/main/java/com/ibm/icu/impl/ICUCurrencyDisplayInfoProvider.java
@@ -325,7 +325,7 @@ Map<String, String> fetchUnitPatterns() {
             Map<String, String> result = unitPatternsCache;
             if (result == null) {
                 result = new HashMap<>();
-                CurrencySink sink = new CurrencySink(!fallback, CurrencySink.EntrypointTable.CURRENCY_UNIT_PATTERNS);
+                CurrencySink sink = new CurrencySink(false, CurrencySink.EntrypointTable.CURRENCY_UNIT_PATTERNS);
                 sink.unitPatterns = result;
                 rb.getAllItemsWithFallback("CurrencyUnitPatterns", sink);
                 unitPatternsCache = result;