[jan] Fix parsing GETACL responses from RFC 4314 capable IMAP servers…

… (Bug #14466).
horde · Sep 15, 2016 · 1f19690 · 1f19690
1 parent 130f087
commit 1f19690
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 5 deletions.
diff --git a/framework/Imap_Client/lib/Horde/Imap/Client/Data/Acl.php b/framework/Imap_Client/lib/Horde/Imap/Client/Data/Acl.php
@@ -78,12 +78,20 @@ protected function _normalize()
          * to handle these rights when dealing with RFC 2086 servers since
          * we are abstracting out use of ACL_CREATE/ACL_DELETE to their
          * component RFC 4314 rights. */
+        foreach ($this->_virtual as $key => $val) {
+            foreach ($val as $right) {
+                if ($this[$right]) {
+                    foreach (array_keys($this->_virtual) as $virtual) {
+                        unset($this[$virtual]);
+                    }
+                    return;
+                }
+            }
+        }
         foreach ($this->_virtual as $key => $val) {
             if ($this[$key]) {
                 unset($this[$key]);
-                if (!$this[reset($val)]) {
-                    $this->_rights = array_unique(array_merge($this->_rights, $val));
-                }
+                $this->_rights = array_unique(array_merge($this->_rights, $val));
             }
         }
     }

diff --git a/framework/Imap_Client/package.xml b/framework/Imap_Client/package.xml
@@ -21,7 +21,7 @@
  </stability>
  <license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
  <notes>
-* 
+* [jan] Fix parsing GETACL responses from RFC 4314 capable IMAP servers (Bug #14466).
  </notes>
  <contents>
   <dir baseinstalldir="/" name="/">
@@ -3050,7 +3050,7 @@
    <date>2016-09-01</date>
    <license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
    <notes>
-* 
+* [jan] Fix parsing GETACL responses from RFC 4314 capable IMAP servers (Bug #14466).
    </notes>
   </release>
  </changelog>