6.0.0-beta1
7/7/25

[#12346] unable to add IMAP acl settings
Summary unable to add IMAP acl settings
Queue IMP
Queue Version 6.1.1
Type Bug
State Resolved
Priority 1. Low
Owners slusarz (at) horde (dot) org
Requester weis (at) kip (dot) uni-heidelberg (dot) de
Created 06/12/2013 (4408 days ago)
Due
Updated 06/12/2013 (4408 days ago)
Assigned
Resolved 06/12/2013 (4408 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
06/12/2013 05:59:03 PM Michael Slusarz Comment #3
Assigned to Michael Slusarz
State ⇒ Resolved
Version ⇒ 6.1.1
Reply to this comment
IMP 6.1.2.
06/12/2013 05:58:58 PM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (master):

commit 6b2fad09ef995f5173fdc7333e80a63a502b47b5
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Wed Jun 12 11:58:47 2013 -0600

     [mms] Fixed regression preventing ACLs from being added (Bug #12346).

  imp/docs/CHANGES     |    1 +
  imp/lib/Imap/Acl.php |    2 +-
  imp/package.xml      |    4 ++--
  3 files changed, 4 insertions(+), 3 deletions(-)

http://git.horde.org/horde-git/-/commit/6b2fad09ef995f5173fdc7333e80a63a502b47b5
06/12/2013 11:13:18 AM weis (at) kip (dot) uni-heidelberg (dot) de Comment #1
Priority ⇒ 1. Low
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ IMP
Summary ⇒ unable to add IMAP acl settings
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
i was unable to add new acl values for shared folders.
To remove acl flags was no problem.
The problem could be tracked down to:

in imp/lib/Imap/Acl.php Line 90

         if (strlen($rights) || !$imp_imap->access(IMP_Imap::ACCESS_ACL)) {
             return;
         }

so 'addRights' is aborting when $rights are set.
It should be

         if ( ! strlen($rights) || !$imp_imap->access(IMP_Imap::ACCESS_ACL)) {
             return;
         }

Saved Queries