locale: Fix 24 hour format for DateTimeFormats.

Signed-off-by: Augustin Cavalier <waddlesplash@gmail.com> Fixes #12575. I checked with PulkoMandy on IRC before merging this, and fixed some whitespace violations of the coding style.
haiku · Jun 18, 2016 · ac315db · ac315db
1 parent 8981510
commit ac315db
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 13 deletions.
diff --git a/headers/os/locale/FormattingConventions.h b/headers/os/locale/FormattingConventions.h
@@ -141,6 +141,7 @@ class BFormattingConventions : public BArchivable {
 			bool				fUseStringsFromPreferredLanguage;
 
 			U_ICU_NAMESPACE::Locale*		fICULocale;
+			void				CoerceFormatForClock(BString& outFormat) const;
 };
 
 

diff --git a/src/kits/locale/FormattingConventions.cpp b/src/kits/locale/FormattingConventions.cpp
@@ -475,19 +475,7 @@ BFormattingConventions::GetTimeFormat(BTimeFormatStyle style,
 	BStringByteSink stringConverter(&outFormat);
 	icuString.toUTF8(stringConverter);
 
-	int8 use24HourClock = fExplicitUse24HourClock != CLOCK_HOURS_UNSET
-		? fExplicitUse24HourClock : fCachedUse24HourClock;
-	if (use24HourClock != CLOCK_HOURS_UNSET) {
-		// adjust to 12/24-hour clock as requested
-		bool localeUses24HourClock = !FormatUsesAmPm(outFormat);
-		if (localeUses24HourClock) {
-			if (use24HourClock == CLOCK_HOURS_12)
-				CoerceFormatTo12HourClock(outFormat);
-		} else {
-			if (use24HourClock == CLOCK_HOURS_24)
-				CoerceFormatTo24HourClock(outFormat);
-		}
-	}
+	CoerceFormatForClock(outFormat);
 
 	if (style != B_FULL_TIME_FORMAT) {
 		// use abbreviated timezone in short timezone format
@@ -531,6 +519,13 @@ BFormattingConventions::GetDateTimeFormat(BDateFormatStyle dateStyle,
 	BStringByteSink stringConverter(&outFormat);
 	icuString.toUTF8(stringConverter);
 
+	CoerceFormatForClock(outFormat);
+
+	if (dateStyle != B_FULL_DATE_FORMAT) {
+		// use abbreviated timezone in short timezone format
+		CoerceFormatToAbbreviatedTimezone(outFormat);
+	}
+
 	fCachedDateTimeFormats[dateStyle][timeStyle] = outFormat;
 
 	return B_OK;
@@ -665,3 +660,23 @@ BFormattingConventions::Archive(BMessage* archive, bool deep) const
 
 	return status;
 }
+
+
+void
+BFormattingConventions::CoerceFormatForClock(BString& outFormat) const
+{
+	int8 use24HourClock = fExplicitUse24HourClock != CLOCK_HOURS_UNSET
+		? fExplicitUse24HourClock : fCachedUse24HourClock;
+	if (use24HourClock != CLOCK_HOURS_UNSET) {
+		// adjust to 12/24-hour clock as requested
+		bool localeUses24HourClock = !FormatUsesAmPm(outFormat);
+		if (localeUses24HourClock) {
+			if (use24HourClock == CLOCK_HOURS_12)
+				CoerceFormatTo12HourClock(outFormat);
+		} else {
+			if (use24HourClock == CLOCK_HOURS_24)
+				CoerceFormatTo24HourClock(outFormat);
+		}
+	}
+}
+