[Tickets #13177] Re: folders in getMailboxes() with disable IMAP mailbox subscriptions

[noreply at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/13177
------------------------------------------------------------------------------
  Ticket             | 13177
  Updated By         | Michael Rubinsky <mrubinsk at horde.org>
  Summary            | folders in getMailboxes() with disable IMAP mailbox
                     | subscriptions
  Queue              | Synchronization
  Version            | Git master
  Type               | Bug
-State              | Assigned
+State              | Resolved
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Michael Rubinsky
------------------------------------------------------------------------------


Michael Rubinsky <mrubinsk at horde.org> (2014-05-12 22:48) wrote:

> This isn't correct - correct fix is to not leak search mailboxes  
> from IMP API calls.
>
> Not sure what you need to revert so reassigning.

The search mailboxes were not involved in this bug (though I might  
have added some comments related to the search mailboxes in the code  
that was committed). That check was added much earlier. I didn't  
consider removing it from the returned value at the time since I  
wasn't sure when it was added and didn't want to break BC by removing  
it.

*This* bug and commit had to do with not returning any folders other  
than the INBOX if IMAP subscriptions were not in use (since the  
subscribed flag would not be set on any mailbox in this case).