I don't see any place where we still suffix the share ID with an owne…

…r ID. But spliting twice on colons breaks groups with contacts that contain colons in their ID, e.g. with some LDAP setups.
horde · Aug 4, 2014 · 5235615 · 5235615
1 parent 03dec20
commit 5235615
Showing 1 changed file with 0 additions and 6 deletions.
diff --git a/turba/lib/Object/Group.php b/turba/lib/Object/Group.php
@@ -167,17 +167,11 @@ public function listMembers($sort = null)
                 }
             } else {
                 list($sourceId, $contactId) = explode(':', $member, 2);
-                if (strpos($contactId, ':')) {
-                    list($owner, $contactId) = explode(':', $contactId, 2);
-                    $sourceId .= ':' . $owner;
-                }
-
                 try {
                     $driver = $GLOBALS['injector']->getInstance('Turba_Factory_Driver')->create($sourceId);
                 } catch (Turba_Exception $e) {
                     continue;
                 }
-
                 try {
                     $contact = $driver->getObject($contactId);
                 } catch (Horde_Exception_NotFound $e) {