Superfluous comma in datetimeformat-short (cldr/nls/en/gregorian.js)

[wol]

In cldr/nls/en/gregorian.js, datetimeformat-short is defined as follows:

"dateTimeFormat-short": "{1}, {0}"

The comma after the {1} is new, introduced with

---

r30140 | peller | 2012-12-07 23:07:01 +0100 (Fre, 07. Dez 2012) | 176 Zeilen

run scripts against CLDR v21 Refs #13906, #15424, #16281 -This line, and those below, will be ignored--

We have a custom textbox widget, and in the end, datetimeformat-short is used by _parseInfo in dojo/date/locale.js to assemble a complete datetime pattern based on (1) a date pattern and (2) a time pattern.

E.g. we have dd.MM.yyyy and hh:mm, and the expected result is "dd.MM.yyyy hh:mm", however since r30140, the result is "dd.MM.yyyy, hh:mm" which obviously is not the desired result.

In language files like de/gregorian.js, that comma is not present, just our code uses the en/gregorian.js language file.

[Adam Peller]

This was apparently a deliberate change made by the CLDR maintainers

[sasdxm]

I got caught by this also. Seems like it would be really useful to be able to specify the separation character for user defined date/time formatting and parsing.

[Adam Peller]

#17342 is a duplicate of this ticket.

[vapier]

i've filed #17544 to add an enhancement to make this possible (it also includes a simple hack for people working with the current code base)