Bug 89580

Summary:	EDITING: spellcheck and an abbreviation ending in a period
Product:	LibreOffice	Reporter:	Nick Levinson <Nick_Levinson>
Component:	Writer	Assignee:	Not Assigned <libreoffice-bugs>
Status:	RESOLVED WORKSFORME
Severity:	normal	CC:	ilmari.lauhakangas
Priority:	medium
Version:	4.2.8.2 release
Hardware:	x86-64 (AMD64)
OS:	Linux (All)
Whiteboard:
Crash report or crash signature:		Regression By:

Description Nick Levinson 2015-02-23 01:31:29 UTC

Running a spellcheck on a string ending in a period or dot causes the trailing period to be highlighted but adding it to the dictionary causes the trailing period to be omitted from the dictionary entry. Examples are "P.O." and "N.Y." (without quotation marks). I assume the problem is a coincidence of a word-ending period with a sentence-ending period. Two solutions are possible. Either:

Do not highlight the trailing period, so that highlighting will match what will be written into a dictionary.

Or do not mark a string as not being in a dictionary if the only difference between a string in a document and an entry in a dictionary is the trailing period in the dictionary.

My hardware description is a guess.

Comment 1 Buovjaga 2015-02-27 15:18:32 UTC

I tried it, but P.O. or N.Y. at the end of a sentence do not cause their trailing periods to be highlighted after spellcheck.

US spellcheck otherwise seems to be functioning for me, using aspell 0.60.7~20110707-1.1.

Can you test on 4.4.1 and include a screenshot of the problem?

Set to NEEDINFO.
Change back to UNCONFIRMED after you have provided more info.

You can install it separately: https://wiki.documentfoundation.org/Installing_in_parallel/Linux

Tested on:
Ubuntu 14.10 64-bit
Version: 4.4.1.2
Build ID: 40m0(Build:2)
Locale: en_US

Comment 2 Nick Levinson 2015-02-28 18:26:13 UTC

I'll assume it's been fixed in a version after mine. I don't want to upgrade before Fedora upgrades Linux generally, probably in a future major version, lest something clash. Thanks.

Comment 3 Buovjaga 2015-02-28 20:51:28 UTC

Ok, I'll mark this as WORKSFORME, as I don't know any commit with a fix.
If you test with a newer version and notice that it isn't actually fixed, you can set this back to UNCONFIRMED.

Parallel install for testing: https://wiki.documentfoundation.org/Installing_in_parallel/Linux