| Summary: | Default format for dates should be dd/MM/yyyy for Italian locale | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Mauro Molinari <mauromol> |
| Component: | Localization | Assignee: | Eike Rathke <erack> |
| Status: | RESOLVED FIXED | ||
| Severity: | enhancement | CC: | elcico2001, erack, oliver.brinzing, valtermura |
| Priority: | medium | ||
| Version: | Inherited From OOo | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | target:7.0.0 | ||
| Crash report or crash signature: | Regression By: | ||
|
Description
Mauro Molinari
2020-02-10 14:50:51 UTC
https://en.wikipedia.org/wiki/Date_format_by_country seems to confirm that... I remember reporting it many years ago: Issue 25942 standard date format not taken from the system properties https://bz.apache.org/ooo/show_bug.cgi?id=25942 The request sounds legit to me. @Diego and @Valter: which is your take? I would work on this change moving the default from the year with two digits to the year with four digits. (In reply to Oliver Brinzing from comment #2) > Issue 25942 standard date format not taken from the system properties > https://bz.apache.org/ooo/show_bug.cgi?id=25942 Please note that that issue (tdf#73242) is unrelated to this one (which is about LibreOffice internal table of regional defaults). The list of locales that should be updated with the default from the year with two digits to the year with four digits should be: 1) it_CH 2) it_IT 3) fur_IT 4) lld_IT 5) sc_IT 6) vec_IT Any other opinion on this? Patch is ready. I'm just waiting for a feedback from the Italian l10n team (In reply to Marina Latini (SUSE) from comment #5) > The list of locales that should be updated with the default from the year > with two digits to the year with four digits should be: > 1) it_CH > 2) it_IT > 3) fur_IT > 4) lld_IT > 5) sc_IT > 6) vec_IT > > Any other opinion on this? > Patch is ready. I'm just waiting for a feedback from the Italian l10n team Hi, I agree with the change, Marina please proceed. @Marina: I assume you're simply swapping the default true/false of formatindex="20" and formatindex="21"? (In reply to Eike Rathke from comment #7) > @Marina: I assume you're simply swapping the default true/false of > formatindex="20" and formatindex="21"? @Eike: I was changing formatindex="18", formatindex="20", formatindex="21", formatindex="46" and formatindex="47". Three for the DATE usage and two for the DATE_TIME usage. If my assumptions are wrong I'm happy to edit my patch again. New amended patch submitted and verified by Jenkins. As suggested I reverted the changes to formatindex="18" and left only the ones for formatindex="20", formatindex="21". On formatindex="46" and formatindex="47" I reverted the change to the default waiting to have a better change including the proper support for the years with 4 digits. Happy to investigate it with you Eike if you think could be something doable for a newbie. Thanks for your reviews. Marina Marina Latini (SUSE) committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/618d1f1085cd7a9c672b851944aae106868266fa tdf#130563 Change the default dates format for Italian locale It will be available in 7.0.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. I tested the daily at /daily/master/Win-x86_64@tb77-TDF/2020-02-14_08.00.09/, just Calc. Now the default date format seems to be dd/MM/yyyy, which is what I asked for, thank you! However, when I enter a date/time value, like "14/02/2020 10:45:13", it gets changed to "14/02/20 10:45" for display purposes... Once again the format with two-digit year!! IMHO this is inconsistent. Yes, that is related to what I mentioned in the gerrit review with the DATE_TIME formats at https://gerrit.libreoffice.org/c/core/+/88462/2/i18npool/source/localedata/data/it_IT.xml#196 Entering a date+time will choose the default DATE_TIME format. We'll need a) dd/mm/yyyy hh:mm format support that can be set as default and possibly b) dd/mm/yyyy hh:mm:ss format to be set if the input includes seconds As is, there's the predefined formatindex="46" corresponding to the number formatter's NF_DATETIME_SYSTEM_SHORT_HHMM that is used at places to determine default and also mapping in file import, hence we can not simply change the number format code to include the century, or rather I'll have to check first if and how that would be possible or if we need to introduce a separate format. I can try to solve that minimally invasive in a follow-up change. (In reply to Eike Rathke from comment #13) > I can try to solve that minimally invasive in a follow-up change. Hi Eike, so should this be better reopened or is a new bug report better? Reopening and taking to myself. Eike Rathke committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/8453f6e5d1fd98700a37e30346a1af0daf8fb753 Resolves: tdf#130563 Add predefined 4-digit year date+time format It will be available in 7.0.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. Am I wrong or daily builds at /daily/master/Win-x86_64@tb77-TDF/current/ are still at 2020-02-14? tb77 builds have problems, for available builds see https://dev-builds.libreoffice.org/daily/master/current.html (In reply to Eike Rathke from comment #18) > tb77 builds have problems, for available builds see > https://dev-builds.libreoffice.org/daily/master/current.html Still, I can't see anything newer than 2020-02-14 07:59:56 for Windows. Yes. But please, the bug tracker is not the place to discuss current infrastructure problems. tb77 is/was the only available box to produce Windows master build full installation packages. (In reply to Eike Rathke from comment #20) > Yes. But please, the bug tracker is not the place to discuss current > infrastructure problems. tb77 is/was the only available box to produce > Windows master build full installation packages. I do not want to discuss about infrastructure problems, I would just like to test as you requested on comment #16!! > 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. Otherwise do not ask for testing ;-) (In reply to Mauro Molinari from comment #21) You could notice it's a message from "Commit Notification", not from Eike; it's an automated standard message. And yes, it's unfortunate that there's no builds available currently to test on Windows (but others could test on other platforms, so the request for testing is not absolutely wrong anyway ;-) ). |