Re: Review Request: Adds method to KBookmarkManager disable showing dialogs on errors.
On 2010-11-29 00:08:22, 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 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 --- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6004/#review9034 --- On 2010-11-28 21:00:30, Matthias Fuchs wrote: --- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6004/ --- (Updated 2010-11-28 21:00:30) Review request for kdelibs and David Faure. Summary --- 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 Diffs - /trunk/KDE/kdelibs/kio/bookmarks/kbookmarkmanager.h 1201809 /trunk/KDE/kdelibs/kio/bookmarks/kbookmarkmanager.cc 1201809 Diff: http://svn.reviewboard.kde.org/r/6004/diff Testing --- Thanks, Matthias
Re: Review Request: Adds method to KBookmarkManager disable showing dialogs on errors.
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6004/ --- (Updated 2010-11-30 20:57:03.986026) Review request for kdelibs and David Faure. Changes --- Messed up the last diff, this one is correct now. Summary --- 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 This addresses bug 207592. https://bugs.kde.org/show_bug.cgi?id=207592 Diffs (updated) - /trunk/KDE/kdelibs/kio/bookmarks/kbookmarkmanager.h 1202414 /trunk/KDE/kdelibs/kio/bookmarks/kbookmarkmanager.cc 1202414 Diff: http://svn.reviewboard.kde.org/r/6004/diff Testing --- Thanks, Matthias
Review Request: Adds method to KBookmarkManager disable showing dialogs on errors.
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6004/ --- Review request for kdelibs and David Faure. Summary --- 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 Diffs - /trunk/KDE/kdelibs/kio/bookmarks/kbookmarkmanager.h 1201809 /trunk/KDE/kdelibs/kio/bookmarks/kbookmarkmanager.cc 1201809 Diff: http://svn.reviewboard.kde.org/r/6004/diff Testing --- Thanks, Matthias
Re: Review Request: Adds method to KBookmarkManager disable showing dialogs on errors.
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6004/#review9021 --- one tiny thing: calling it setDialogsAllowed would be clearer than setDialogsActive. I'd check 'ship it' but this isn't my code at all :) - Chani On 2010-11-28 19:53:45, Matthias Fuchs wrote: --- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6004/ --- (Updated 2010-11-28 19:53:45) Review request for kdelibs and David Faure. Summary --- 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 Diffs - /trunk/KDE/kdelibs/kio/bookmarks/kbookmarkmanager.h 1201809 /trunk/KDE/kdelibs/kio/bookmarks/kbookmarkmanager.cc 1201809 Diff: http://svn.reviewboard.kde.org/r/6004/diff Testing --- Thanks, Matthias
Re: Review Request: Adds method to KBookmarkManager disable showing dialogs on errors.
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6004/ --- (Updated 2010-11-28 21:00:30.131444) Review request for kdelibs and David Faure. Changes --- Changed the name of the method to setDialogsAllowed. Summary --- 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 Diffs (updated) - /trunk/KDE/kdelibs/kio/bookmarks/kbookmarkmanager.h 1201809 /trunk/KDE/kdelibs/kio/bookmarks/kbookmarkmanager.cc 1201809 Diff: http://svn.reviewboard.kde.org/r/6004/diff Testing --- Thanks, Matthias