Bug 181607

Summary:	wrong 'ignore all' action on spell checker
Product:	[Unmaintained] kdelibs	Reporter:	Cristóvão Sousa <crisjss>
Component:	kspell	Assignee:	Zack Rusin <zack>
Status:	RESOLVED UNMAINTAINED
Severity:	normal	CC:	andresbajotierra
Priority:	NOR
Version First Reported In:	4.1
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Cristóvão Sousa 2009-01-22 16:44:52 UTC

Version:            (using KDE 4.1.3)
OS:                Linux
Installed from:    Unlisted Binary Package

Bug tested in spell checkers at Kate 3.1.3, Konqueror 4.1.3 and Kile 2.0.80 :

Spell checker action 'Ignore All' do just the same as the 'Add to Dictionary' action:
the misspelled word will be added to '.aspell.en.pws' and '.config/enchant/en.dic' dictionaries, and will no more (at any application) be considered a wrong word.


How to Reproduce:

- type a misspelled word, like 'qwertyuiop', in Kate;
- run spell checker and click 'Ignore all' when prompted about that word;
- close and reopen Kate;
- type the same misspelled word;
- run spell checker.

The misspelled word will now be a valid word.
Check the dictionary files:
$ grep qwertyuiop ~/.aspell.en.pws
$ grep qwertyuiop ~/.config/enchant/en.dic


Expected Behaviour:

The 'ignored all' misspelled word should not be added to the dictionary, it should be ignored only in the current spell check run.

Comment 1 Dario Andres 2009-06-15 22:55:40 UTC

Here using:

Qt: 4.5.1 (qt-copy  971295)
KDE: 4.2.91 (KDE 4.2.91 (KDE 4.3 >= 20090609))
kdelibs svn rev. 982117 / kdebase svn rev. 982118
aspell 0.60.6, enchant 1.4.2
on ArchLinux i686 - Kernel 2.6.29.4

I can confirm this bug

Comment 2 Dario Andres 2009-06-15 22:59:02 UTC

Indeed:

void Dialog::slotSkipAll()
{
    //### do we want that or should we have a d->ignoreAll list?
    Speller speller = d->checker->speller();
    speller.addToPersonal(d->currentWord.word);
    d->checker->setSpeller(speller);
    d->checker->continueChecking();
}

Comment 3 Christoph Cullmann 2024-09-14 17:06:48 UTC

Hi,

kdelibs (version 4 and earlier) is no longer maintained since a few years.

KDE Frameworks 5 or 6 might already have resolved this bug.

If not, please re-open against the matching framework if feasible or against the application that shows the issue.

We then can still dispatch it to the right Bugzilla product or component.

Greetings
Christoph Cullmann