LibreOffice 3.5.6.2 Build-ID: 350m1(Build:2) Fedora 17 (x64) Steps to reproduce: 1. Activate Auto Correct for two capitalized letters under Extras->Settings 2. Type a word with a dash or a slash in the middle, then begin the next word with two capitalized letters, e.g. Test-TEst, Test/TEst 3. Hit space or enter Expected Result: "TEst" is replaced by "Test" Current Result: Word will not be corrected I think it is helpful, if the word is corrected in these situations, too, as it not to assume that the word is a neologism and must be capitalized this way.
Reproduced in 3.6.1.2
I will look into fixing this.
Still working on this.
I think I have a fix ready, but I'm going to test it a bit more before I submit it.
Patch submitted for review. To reproduce the problem: ------------------------- 1. Start a new text document in Writer. 2. Verify that AutoCorrect is enabled in Tools > AutoCorrect Options > Options. (Under "Correct TWo INitial CApitals", ensure "T" is selected). 3. Type a compound word separated by an AutoCorrect character (like the hyphen). For example, FAce-FAce will be AutoCorrected as Face-FAce (only the first word in the compound word will be corrected). What changes I made: -------------------- 1. In svxacorr.cxx SvxAutoCorrect::IsAutoCorrectChar, I added '-' as a character. 2. In svxacorr.cxx SvxAutoCorrect::FnCptlSttWrd, I added code to iterate through each AutoCorrectChar in a compound word to determine if capitalization needs fixing. To test: -------- 1. Start a new text document in Writer. 2. Verify that AutoCorrect is enabled in Tools > AutoCorrect Options > Options. (Under "Correct TWo INitial CApitals", ensure "T" is selected). 2. Ensure that non-compound words (example: FAce) are autocorrected. Note: Pressing Enter or Space should perform the AutoCorrect. 3. Ensure that compound words (FAce-FAce-FAce) are autocorrcted. Also try compound words with different AutoCorrect characters (example: FAce-FAce/Face). Note: As soon as you type an AutoCorrect character (a hyphen, for example), the word will be corrected. 4. Ensure that single letters, two letters, and three capital letters are not AutoCorrected when an AutoCorrect character is entered.
Christopher Copits committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=ea7d6d070247f6b676054a8256679ac2d2ae577c fdo#55693 Two initial capitalized letters corrected in compound words The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Caolan McNamara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=a526370737099b57b399f2ca8de43e74e24ab794 Related: fdo#55693 add regression tests for autocorrection The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Xisco Fauli committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/4dbaa38315605d2051a5127c96c555565e95444e tdf#55693: port test to CppunitTest_sw_autocorrect It will be available in 25.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.