Bug 127363 - Add Sept as an abbreviation of September
Summary: Add Sept as an abbreviation of September
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: All All
: low enhancement
Assignee: Eike Rathke
URL: https://web.library.yale.edu/catalogi...
Whiteboard: target:6.4.0 target:7.2.0
Keywords:
Depends on:
Blocks: Number-Format
  Show dependency treegraph
 
Reported: 2019-09-05 10:31 UTC by Jonny Grant
Modified: 2021-08-13 12:30 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 Jonny Grant 2019-09-05 10:31:30 UTC
In English "Sept" is a common abbreviation of September. Unfortunately Calc doesn't recognise it. Calc only recognises "Sep" as a month date.

Please try and reproduce it, and avoid sending any generic reply without reproducing it.
Comment 1 Timur 2019-09-20 08:06:50 UTC
Please explain use case for that request/enhancement, how could it be recognized, as date format or just during typing, how exactly?
What would be date format on those cells? For "Sep" it's MMM.
What about other months?
Comment 2 Timur 2019-09-20 10:27:29 UTC
Enclosed URL shows really that Sept. is used in English as a common abbreviation of September. MSO also recognizes that. But we would need to take all languages in consideration.
Comment 3 Jonny Grant 2019-09-21 20:17:53 UTC
(In reply to Timur from comment #1)
> Please explain use case for that request/enhancement, how could it be
> recognized, as date format or just during typing, how exactly?
> What would be date format on those cells? For "Sep" it's MMM.
> What about other months?

Yes, just type in any cell

17 Sep <ENTER>

Did you find which line of code causes this?

This issue only impacts Sept, no other months.

Not necessary to take other languages into consideration on this ticket, you're welcome to create another PRs for other languages if you feel they may face the same issue.
Comment 4 QA Administrators 2019-09-22 03:00:11 UTC Comment hidden (obsolete)
Comment 5 Jonny Grant 2019-09-22 13:31:45 UTC Comment hidden (no-value)
Comment 6 Thomas Lendo 2019-09-25 20:46:41 UTC
I support this enhancement request.
When typing "17 Sept" and enter, Calc automatically should change to the date format of the chosen number format.

I can confirm that Sept is the normal abbreviation for September also in German and this month is the only with 4 characters. I assume all languages that have Sept[...] as month name behave identical.
Comment 7 Commit Notification 2019-09-26 23:24:22 UTC
Eike Rathke committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/f526a525758c8e4507e0bcc039330f4fc8a174b4

Resolves: tdf#127363 accept SEPT as SEP for September

It will be available in 6.4.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 8 How can I remove my account? 2019-09-27 08:06:45 UTC
Out of interest, was this bug report perhaps filed tongue-in-cheek, to make LibreOffice produce the same kind of errors in genetic research as Excel does? See https://www.sciencemag.org/news/2016/08/one-five-genetics-papers-contains-errors-thanks-microsoft-excel

(And yes, I know that those "errors" are caused by researchers not using their tools properly, it isn't Excel's (or LibreOffice's) fault that they use spreadsheets incorrectly.)
Comment 9 Eike Rathke 2019-09-27 12:12:29 UTC
Even with this change SEPT2 is still not a date for us..
Comment 10 Jonny Grant 2019-09-27 23:32:35 UTC
Great work Eike!
Comment 11 How can I remove my account? 2019-09-30 18:55:45 UTC
> Even with this change SEPT2 is still not a date for us

Oh good!
Comment 12 Commit Notification 2021-05-11 12:12:17 UTC
Xisco Fauli committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/38f3bd349586ecae867c74b746fd3db7c6104d70

tdf#127363: svl_qa_cppunit: Add unittest

It will be available in 7.2.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 13 Stéphane Guillou (stragu) 2021-08-13 12:30:58 UTC
Verified as fixed in:

Version: 7.0.6.2
Build ID: 00(Build:2)
CPU threads: 4; OS: Linux 5.4; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Ubuntu package version: 1:7.0.6-0ubuntu0.20.04.1_lo1
Calc: threaded