6.0.0-beta1
7/14/25

[#13177] folders in getMailboxes() with disable IMAP mailbox subscriptions
Summary folders in getMailboxes() with disable IMAP mailbox subscriptions
Queue Synchronization
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners mrubinsk (at) horde (dot) org
Requester samuel.wolf (at) wolf-maschinenbau (dot) de
Created 05/10/2014 (4083 days ago)
Due
Updated 05/12/2014 (4081 days ago)
Assigned 05/12/2014 (4081 days ago)
Resolved 05/12/2014 (4081 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
05/12/2014 10:48:05 PM Michael Rubinsky Comment #6
State ⇒ Resolved
Reply to this comment
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).
05/12/2014 10:22:41 PM Michael Slusarz Comment #5
State ⇒ Assigned
Reply to this comment
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.
05/12/2014 10:22:33 PM Git Commit Comment #4 Reply to this comment
Changes have been made in Git (master):

commit 0908a774af5a7359b74ccd6273dc9208ac22c1ee
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Mon May 12 16:20:15 2014 -0600

     Don't return search mailboxes in mailboxList API call

     search mailboxes have no meaning outside of IMP

     Ticket #13177

  imp/lib/Api.php |    3 ++-
  1 files changed, 2 insertions(+), 1 deletions(-)

http://github.com/horde/horde/commit/0908a774af5a7359b74ccd6273dc9208ac22c1ee
05/10/2014 05:06:47 PM Michael Rubinsky Version ⇒ Git master
Queue ⇒ Synchronization
 
05/10/2014 05:06:27 PM Michael Rubinsky Comment #3
State ⇒ Resolved
Assigned to Michael Rubinsky
Reply to this comment
Horde_Core 2.12.0alpha2
05/10/2014 05:05:02 PM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (master):

commit 82ca72ea0e5512ca6f78b050bfbb8b54aad7eaa5
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Sat May 10 12:53:18 2014 -0400

     Fix returning all mailboxes when not using subscriptions when 
using IMP >= 6.2.

     Bug: 13177

  .../lib/Horde/Core/ActiveSync/Imap/Factory.php     |   13 +++++++++++--
  1 files changed, 11 insertions(+), 2 deletions(-)

http://github.com/horde/horde/commit/82ca72ea0e5512ca6f78b050bfbb8b54aad7eaa5
05/10/2014 09:57:53 AM samuel (dot) wolf (at) wolf-maschinenbau (dot) de Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ folders in getMailboxes() with disable IMAP mailbox subscriptions
Queue ⇒ IMP
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
Reply to this comment
With disable IMAP mailbox subscriptions 
Horde_Core_ActiveSync_Imap_Factory::getMailboxes() must show all 
folders.

$_prefs['subscribe']['value'] = 0;


http://marc.info/?t=139946751200002&r=1&w=2

Saved Queries