Issue 89379 - Spellcheck: "Ignore All" does not work via dialog and dict is read-only
Summary: Spellcheck: "Ignore All" does not work via dialog and dict is read-only
Status: CLOSED FIXED
Alias: None
Product: Writer
Classification: Application
Component: code (show other issues)
Version: DEV300m10
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: stefan.baltzer
QA Contact: issues@sw
URL:
Keywords: regression
: 89188 91809 92000 (view as issue list)
Depends on:
Blocks:
 
Reported: 2008-05-13 10:23 UTC by stefan.baltzer
Modified: 2013-08-07 14:43 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description stefan.baltzer 2008-05-13 10:23:28 UTC 
- Have a Writer document with several identical spelling errors (i.e. "Hello
worldd worldd worldd")
 - Launch Spellcheck Dialog
 - "worldd" should get selected first
 - Click "Ignore All"
 -> The next occurrence is marked as wrong.

Extra: 
 - Look at Tools-Options-Language Settings-Writing Aids
 - Select the "Ignore All List" dictionary, Click "Edit"
 - See that the list is empty (it should contain "worldd")
 - Insert a word, click "New"
-> Dialog "Dict is read-only" pops up
Comment 1 thomas.lange 2008-05-13 11:11:38 UTC 
.
Comment 2 thomas.lange 2008-05-13 11:14:36 UTC 
Well the read-only is probably the source it does not work.
This probably glitched in unnoticed with CWS tl41 when the DictionaryPath
setting was reworked to be a multi-path. 

It looks like the IgnoreAllList (or likely all non-persistent dictionaries) need
just a little bit of special handling.
Comment 3 stefan.baltzer 2008-05-13 11:33:13 UTC 
Set keyword "regression".
Comment 4 rjbutler 2008-06-02 06:34:17 UTC 
This appears to be a duplicate of 89188
Comment 5 thomas.lange 2008-06-02 14:29:59 UTC 
*** Issue 89188 has been marked as a duplicate of this issue. ***
Comment 6 thomas.lange 2008-06-02 14:30:28 UTC 
Yes it is a duplicate of that issue. Thanks! Keeping this one as original...
Comment 7 thomas.lange 2008-06-09 11:00:18 UTC 
Fixed in CWS tl53.

TL->CMC: Thanks for the patch!
Comment 8 thomas.lange 2008-06-09 12:32:26 UTC 
Ooops wrong issue for that comment. ^^°
Comment 9 thomas.lange 2008-06-09 12:36:44 UTC 
.
Comment 10 thomas.lange 2008-06-09 13:22:08 UTC 
Fixed in CWS tl53.
IgnoreAllList is writable again.
Thus the state of affairs is the same as it was before this specific bug was
introduced with OOo 2.4. That includes that "Ignore All" in the dialog does
still not work correctly if the same word is repeatedly in the very same
sentence. Everything is fine for all following sentences.


Files changed:
- linguistic/source/dicimp.cxx
Comment 11 thomas.lange 2008-06-30 08:04:49 UTC 
.
Comment 12 stefan.baltzer 2008-07-03 16:55:52 UTC 
SBA: Verified in CWS tl53.
Comment 13 sgautier.ooo 2008-09-09 17:26:38 UTC 
Verified in OOO300m5 - Closing - Sophie
Comment 14 stefan.baltzer 2009-04-24 10:34:55 UTC 
*** Issue 92000 has been marked as a duplicate of this issue. ***
Comment 15 stefan.baltzer 2009-04-24 10:48:08 UTC 
*** Issue 91809 has been marked as a duplicate of this issue. ***