From e630cd69a3147dffd8e22a4dc83cc5ae7b3a3115 Mon Sep 17 00:00:00 2001 From: "Steven R. Loomis" Date: Mon, 17 Jun 2024 16:53:12 -0500 Subject: [PATCH] CLDR-17532 improve CheckWidths message for annotations - mention both the warning and maximum values, not just the warning value. --- .../main/java/org/unicode/cldr/test/CheckWidths.java | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/tools/cldr-code/src/main/java/org/unicode/cldr/test/CheckWidths.java b/tools/cldr-code/src/main/java/org/unicode/cldr/test/CheckWidths.java index 01371c40a9e..b753bd77c9e 100644 --- a/tools/cldr-code/src/main/java/org/unicode/cldr/test/CheckWidths.java +++ b/tools/cldr-code/src/main/java/org/unicode/cldr/test/CheckWidths.java @@ -117,7 +117,8 @@ public Limit( this.subtype = Subtype.valueTooWide; break; case SET_ELEMENTS: - this.message = "Expected no more than {0} items(s), but was {1}."; + this.message = + "There cannot be more than {3} item(s), and it is recommended to not have more than {0} item(s). Found {1} item(s)."; this.subtype = Subtype.tooManyValues; break; default: @@ -220,7 +221,12 @@ boolean hasProblem( .setCause(cause) .setMainType(errorType) .setSubtype(subtype) - .setMessage(message, warningReference, valueMeasure, percent)); + .setMessage( + message, + warningReference, + valueMeasure, + percent, + errorReference)); return true; } }