Bug 187041

Summary:	JJ: Inconsistency in folder search
Product:	[Unmaintained] kmail	Reporter:	Ruchir Brahmbhatt <ruchir.brahmbhatt>
Component:	new folder tree	Assignee:	Szymon Stefanek <pragma>
Status:	RESOLVED FIXED
Severity:	normal	CC:	jtamate, vincent.touffi
Priority:	NOR
Version First Reported In:	1.11.1
Target Milestone:	---
Platform:	openSUSE
OS:	Unspecified
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	a patch

Description Ruchir Brahmbhatt 2009-03-13 12:29:14 UTC

Version:           1.11.1 (using KDE 4.2.1)
Installed from:    SuSE RPMs

When we filter the folders and quit kmail, kmail shows filtered folders only when restarted and to see all folders, we have to type any keyword and clear it.

Steps to reproduce:
1. Open kmail.
2. Type keyword in folder filter box.
3. This will show folders matching in account. 
4. Quit kmail.
5. Open kmail again.

Actual result:
It shows filtered folders only. Filter keyword is not shown so in order to see all folders again, we have to enter any keyword and then clear it.

Expected result:
Either it should clear filter and show full mailbox or it should show keywords also with option to clear which brings back all folders in mailbox.

Reproducible: Always

Comment 1 Jaime Torres 2009-03-13 17:41:06 UTC

Confirmed in kde 4.2.65

Comment 2 Vincent Dupont 2009-05-20 23:13:03 UTC

Created attachment 33877 [details]
a patch

This patch applies to kmail/kmmainwidget.cpp

Comment 3 Thomas McGuire 2009-05-21 14:06:28 UTC

Hi Vincent,

thanks for the patch. While it fixes the issue, I'm not sure if that is the best way to do it. I haven't tested this yet, but doesn't it flicker when saving the tree, or slow down? Another approach might be to not save the visible/hidden state of items in FolderView::saveItemStates(), I see no reason why this should be useful.

Could you please use reviewboard for the next iteration of the patch? See http://techbase.kde.org/Contribute/Send_Patches#Reviewboard

Comment 4 Thomas McGuire 2009-05-22 12:46:27 UTC

SVN commit 971322 by tmcguire:

Don't hide some folders when restarting KMail after it was quit with having a quick search active.

Patch by Vincent Dupont <vincent.touffi at gmail dot com>, thank you!

BUG: 187041
http://reviewboard.kde.org/r/743/


 M  +0 -2      folderview.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=971322

Comment 5 Thomas McGuire 2009-05-25 21:03:22 UTC

SVN commit 972824 by tmcguire:

Backport r971322 by tmcguire from trunk to the 4.2 branch:

Don't hide some folders when restarting KMail after it was quit with having a quick search active.

Patch by Vincent Dupont <vincent.touffi at gmail dot com>, thank you!

CCBUG: 187041
http://reviewboard.kde.org/r/743/



 M  +0 -2      folderview.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=972824