[Tickets #9600] Re: Disabling Virtual Inbox in Preferences not working

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/9600
------------------------------------------------------------------------------
  Ticket             | 9600
  Updated By         | Michael Slusarz <slusarz at horde.org>
  Summary            | Disabling Virtual Inbox in Preferences not working
  Queue              | IMP
  Version            | Git master
  Type               | Bug
-State              | Assigned
+State              | Resolved
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Michael Slusarz
------------------------------------------------------------------------------


Michael Slusarz <slusarz at horde.org> (2011-03-04 17:22) wrote:

> I can confirm the issue, as well as the fact that this change fixes  
> it. Though if this fix is correct, I don't think we even need the  
> conditional to begin with. I admit I don't fully understand how we  
> *don't* get infinite recursion here, but my guess is that it's  
> because the IMP_Imap_Tree object is already cached by the injector  
> even though IMP_Imap_Tree::init() hasn't completed, so when  
> IMP_Search::setVFolders() is called, the  
> IMP_Imap_Tree::updateVFolders() call succeeds?

The problem came with my addition of the session key check for treeob.  
  I misread the code and thought that this key was not set until after  
the first session access.  However, it is created before we even  
initialize the imaptree object the first time.  This has been fixed  
(although strangely enough, this did not affect me).