Bug 243318

Summary:	Inbox gets NoSelect set when having server side subscription enabled
Product:	[Frameworks and Libraries] Akonadi	Reporter:	Volker Krause <vkrause>
Component:	IMAP resource	Assignee:	Kevin Ottens <ervin>
Status:	RESOLVED FIXED
Severity:	major	CC:	amantia, bjoern.ricks, kdepim-bugs, ludwig.reiter, vkrause
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Volker Krause 2010-07-01 08:58:38 UTC

Version:           unspecified (using Devel) 
OS:                Linux

With server side subscription enabled some servers (including Cyrus) change the folder listing order and no longer do a strict top-down listing (which is what the IMAP resource currently expects). This causes the IMAP resource to fill in the missing hierarchy with structural folders (NoContent flag set), which then prevents content listing of those folders.

Reproducible: Sometimes

Steps to Reproduce:
Create a new IMAP account for a Cyrus/Kolab server and enable server-side subscription.

Actual Results:  
Inbox (and potentially other folders) have the NoContent attribute set to 1, preventing subsequent folder listing.


see the isDummy == true case in ImapResource::onMailBoxesReceived(), possible solution would be to not use streaming for collection listing and build up the full tree in memory first. Alternatively, and much more complicated, we could extend CollectionSync to accept updates of already received collections later on in streaming mode.

Comment 1 András Manţia 2010-07-28 13:00:07 UTC

I tried and cannot reproduce with trunk and the demo kolab server. Might have been fixed with rev1152264 ?

Comment 2 Volker Krause 2010-07-30 09:06:10 UTC

Still reproduceable with revision 1152264, closer inspection of that commit shows it's unrelated anyway.

Comment 3 Volker Krause 2010-08-02 08:59:08 UTC

*** Bug 240602 has been marked as a duplicate of this bug. ***

Comment 4 Kevin Ottens 2010-08-02 11:31:15 UTC

SVN commit 1158203 by ervin:

Merged revisions 1156785,1157161,1157175 via svnmerge from 
svn+ssh://ervin@svn.kde.org/home/kde/trunk/KDE/kdepim/runtime/resources/imap

(r1157161 actually fixes BR 243318)
BUG: 243318

................
  r1156785 | tokoe | 2010-07-29 19:29:40 +0200 (Thu, 29 Jul 2010) | 13 lines
  
  Merged revisions 1156016,1156751 via svnmerge from 
  svn+ssh://tokoe@svn.kde.org/home/kde/branches/KDE/4.5/kdepim
  
  ........
    r1156016 | tokoe | 2010-07-28 14:46:42 +0200 (Wed, 28 Jul 2010) | 2 lines
    
    Do not ask for password when authentication mode is GSSAPI
  ........
    r1156751 | tokoe | 2010-07-29 18:35:13 +0200 (Thu, 29 Jul 2010) | 2 lines
    
    Some more check for not asking for kwallet when using GSSAPI authentication
  ........
................
  r1157161 | amantia | 2010-07-30 13:22:41 +0200 (Fri, 30 Jul 2010) | 4 lines
  
  Accept bottom-up listing order from the IMAP server (if we receive a folder that was created as a dummy parent, make it non-dummy, including being selectable, listable).
  
  CCBUG: 242686
................
  r1157175 | vkrause | 2010-07-30 14:02:27 +0200 (Fri, 30 Jul 2010) | 3 lines
  
  Make sure the D-Bus settings interface is available early enough, the account
  wizard fails otherwise.
................


 _M            . (directory)  
 M  +2 -0      imapresource.cpp  
 U             imapresource.h  
 M  +21 -1     retrievecollectionstask.cpp  
 M  +1 -0      retrievecollectionstask.h  
 M  +8 -0      sessionpool.cpp  
 M  +8 -2      settings.cpp  
 M  +3 -1      settingspasswordrequester.cpp  


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

Comment 5 Volker Krause 2010-08-17 11:02:10 UTC

*** Bug 246157 has been marked as a duplicate of this bug. ***