On November 29th, 2010, 12:08 a.m., Albert Astals Cid wrote:

Why did you change the if/else logic? Also you added another kError when there was an existing one already.

Also as commented on IRC maybe it would make sense to emit a signal to warn whoever might be interested that an error happened

On November 29th, 2010, 9:35 a.m., Matthias Fuchs wrote:

I changed that to always have a useful debug output. The second one is there to make it easier if there are bug reports, as it is hard to work with translated messages, especially if they are quite important. So not the whole message is translated, probably I should us an error code there, as file.errorString() is translated.

Btw. on the signal.

Should it just be an "void error()" signal or a "void error(translatedErrorMessage)" one?

- Matthias

On November 28th, 2010, 9 p.m., Matthias Fuchs wrote:

Adds method to KBookmarkManager disable showing dialogs on errors.
This way it is possible to deactivate those messages if KBoomarkManager is not run in the gui thread.
BUG:207592