Apache OpenOffice (AOO) Bugzilla – Issue 126447
When using LanguageTools, toggling the checkbox "check grammar" in the spell checker removes content
Last modified: 2019-04-27 16:28:57 UTC
Created attachment 84859 [details] Fix for the bug that removes the content when "check grammar" is toggled This was reported before, in https://bugs.documentfoundation.org/show_bug.cgi?id=39348 Steps to reproduce: 1. install LanguageTool (http://extensions.openoffice.org/de/project/languagetool) 2. write some faulty text 3. open the spell checker 4. toggle the "check grammar" checkbox -> content in the text gets removed Check out this video: https://videobin.org/+9it/cp6.ogg I attached a patch that fixed said bug.
@Stefan: Thanks for the patch. Seems to be an easy fix. @All: I suggest to accept this issue and fix for the 4.1.2 release. Reasons: - Preventing data loss - Easy fix - Low risk - Support the work from AOO contributors
Status and issue type changed
It's lower case "true" of course.
I took a quick look at the fix for apparently the same issue referenced in https://bugs.documentfoundation.org/show_bug.cgi?id=39348 Oddly, it seems the fix applied to deal with this same problem by LO http://cgit.freedesktop.org/libreoffice/core/commit/?id=63eef1c1cf203aa75093b479a1e8dd24593546e7 is basically what we have now I think. That is, it is the reverse of the patch submitted here. Could this behavior be caused by changes in the LanguageTools extension?
We can surely include this in 4.1.2, but I'd appreciate to see an answer by Stefan to the previous question by Kay, to make sure that we not make code changes that are specific to LanguageTool or even to a certain version of it.
Analysis: - I confirm the issue with OpenOffice 4.1.1 - Stefan's patch itself does not apply. - "before" and "after" in the patch are reversed, and this may address the perplexity by Kay too. - I attach a version of Stefan's patch that does apply (in short: I swapped the "before" and "after" lines and I changed "True" into "true"). I built trunk (not AOO410, i.e., OpenOffice 4.1.2) with this new version of the patch, and this fixes the bug. So the approach I will follow is: - Patch is committed to trunk now - Buildbots will produce a new build from trunk - We give people the opportunity to double-check this issue before merging to AOO410 for OpenOffice 4.1.2.
Created attachment 84897 [details] Patch by Stefan Heinemann, modified version
"pescetti" committed SVN revision 1700126 into trunk: #i126447# Fix text deletion when toggling "Check grammar" in spell checker.
Now merged to AOO410 for OpenOffice 4.1.2.
To verify: Repeat the "Steps to reproduce" described in the first post of this issue and check if the bug still appears.
Verified fixed in OpenOffice 4.1.2-RC2 with latest LanguageTool (3.1) and instructions from the original post.