prev
More Info
Revision 797025
(Back to Digest)
next


Commit Details
Bug Fixes in KDE-PIM
Thomas McGuire committed changes in /trunk/KDE/kdepim/kmail:
Fix totally broken renaming of online IMAP folders:

- emit the closed() signal before calling folderMgr()->contentsChanged()

This fixes an refcount assert when calculating the unread count.
The assert would trigger there because the folder is closed with refcount 0 but still selected. Now, the headerview will open the folder before that, making the refount 1.

- Don't show an error message when the old folder couldn't be unsubscribed. That error message doesn't make sense, since the old folder does not exist anymore, and therefore can't be unsubscribed anyway.

- Let the mainwidget re-open the folder after it is renamed. This fixes an assert when switching away from an renamed folder, which would close it despite not having called open() before.

- Also change the objectname when renaming, now renaming the same folder more than once works.

- tabs--
Bug 158861: crash on renaming open folder in imap

File Changes
Modified 6 files
/trunk/KDE/kdepim/kmail
/folderstorage.cpp
/imapaccountbase.cpp
/kmfoldermgr.cpp
/kmmainwidget.cpp
/kmmainwidget.h
/renamejob.cpp
Added 1 file
/trunk/KDE/kdepim/kmail
Deleted 1 file
/trunk/KDE/kdepim/kmail
8 files changed in total

KDE Commit-Digest by Danny Allen, 2006-2008
All issues in archive by Derek Kite