Bug 159829 - Update en_ZA locale (ThousandSeparator and DecimalSeparator)
Summary: Update en_ZA locale (ThousandSeparator and DecimalSeparator)
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Localization (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: All All
: medium enhancement
Assignee: Not Assigned
URL:
Whiteboard: target:24.8.0 inReleaseNotes:24.8
Keywords:
Depends on:
Blocks:
 
Reported: 2024-02-21 19:47 UTC by Glen A.
Modified: 2024-10-20 07:43 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Glen A. 2024-02-21 19:47:16 UTC
Description:
The ThousandSeparator and DecimalSeparator has been updated for the en_ZA locale in CLDR v43.1.

See:

- https://unicode-org.atlassian.net/browse/CLDR-16247
- https://cldr.unicode.org/index/downloads/cldr-43#h.qobmda543waj

Actual Results:
en_ZA locale is not up-to-date with CLDR data.

<ThousandSeparator> </ThousandSeparator>
<DecimalSeparator>,</DecimalSeparator>

Expected Results:
en_ZA locale is up-to-date with CLDR data.

<ThousandSeparator>,</ThousandSeparator>
<DecimalSeparator>.</DecimalSeparator>


Reproducible: Always


User Profile Reset: No

Additional Info:
Version: 24.2.0.3 (X86_64) / LibreOffice Community
Build ID: da48488a73ddd66ea24cf16bbc4f7b9c08e9bea1
CPU threads: 20; OS: Windows 10.0 Build 22621; UI render: Skia/Raster; VCL: win
Locale: en-ZA (en_ZA); UI: en-ZA
Calc: threaded
Comment 1 Xisco Faulí 2024-02-21 20:14:56 UTC
Hello Eike,
I thought you might be interested in this issue
Comment 2 Eike Rathke 2024-02-21 22:55:22 UTC
Please see bug 119613 for why LO uses the current separators.
Comment 3 Glen A. 2024-02-22 06:46:12 UTC
Eike,

I understand that, but even though these are the "official" separators, they are very seldom used.

Have a look at this: https://unicode-org.atlassian.net/browse/CLDR-14707?focusedCommentId=161272

All 4 of the top banks, the largest retailers, and even the SA Revenue Service (tax authority).

The CLDR has been updated to follow widespread usage, and I hope that you will do the same.

If you like, I can try to get in touch with the OP of that issue, to see if he is in agreement.
Comment 4 Eike Rathke 2024-04-10 19:07:04 UTC
I'm inclined to follow CLDR here, with what we already had before their switch, even if contradictory to official style guides.

Meanwhile I just learned today that there seems to be a 3rd mix in the wild,
group separator _space_ and decimal separator _period_
https://ask.libreoffice.org/t/tricky-almost-impossible-csv-import-due-to-thousands-separator-and-decimal-symbol/104593

Anyway, CLDR with that reference list switching is a good argument.
Comment 5 Commit Notification 2024-04-24 20:41:06 UTC
Eike Rathke committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/92ea59703da91771ca78ade82ca796077c066da8

Resolves: tdf#159829 DecimalSeparator '.' period, ThousandSeparator ',' comma

It will be available in 24.8.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 6 Eike Rathke 2024-04-24 21:00:24 UTC
Added to release notes https://wiki.documentfoundation.org/ReleaseNotes/24.8#Changes_to_locale_data