[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 Rubinsky <mrubinsk at horde.org>
  Summary            | Disabling Virtual Inbox in Preferences not working
  Queue              | IMP
  Version            | Git master
  Type               | Bug
-State              | Resolved
+State              | Assigned
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Michael Slusarz
------------------------------------------------------------------------------


Michael Rubinsky <mrubinsk at horde.org> (2011-03-04 15:55) 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?

Either way, I'll leave this for Michael to fix :)