Skip to content

Commit 7ddaf49

Browse files
committed
StringUtils.parseLocaleString accepts Java 7 variants
Issue: SPR-14718 (cherry picked from commit f24ce76)
1 parent e947363 commit 7ddaf49

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

spring-core/src/main/java/org/springframework/util/StringUtils.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -715,7 +715,7 @@ public static Locale parseLocaleString(String localeString) {
715715
private static void validateLocalePart(String localePart) {
716716
for (int i = 0; i < localePart.length(); i++) {
717717
char ch = localePart.charAt(i);
718-
if (ch != '_' && ch != ' ' && !Character.isLetterOrDigit(ch)) {
718+
if (ch != ' ' && ch != '_' && ch != '#' && !Character.isLetterOrDigit(ch)) {
719719
throw new IllegalArgumentException(
720720
"Locale part \"" + localePart + "\" contains invalid characters");
721721
}

spring-core/src/test/java/org/springframework/util/StringUtilsTests.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -692,4 +692,9 @@ public void testParseLocaleWithVariantContainingCountryCode() {
692692
assertEquals("Variant containing country code not extracted correctly", variant, locale.getVariant());
693693
}
694694

695+
@Test // SPR-14718
696+
public void testParseJava7Variant() {
697+
assertEquals("sr_#LATN", StringUtils.parseLocaleString("sr_#LATN").toString());
698+
}
699+
695700
}

0 commit comments

Comments
 (0)