]> source.dussan.org Git - nextcloud-server.git/commitdiff
fix inGroup check, thus make integration tests succeed 15984/head
authorArthur Schiwon <blizzz@arthur-schiwon.de>
Thu, 13 Jun 2019 14:49:39 +0000 (16:49 +0200)
committerBackportbot <backportbot-noreply@rullzer.com>
Mon, 17 Jun 2019 13:34:55 +0000 (13:34 +0000)
there is not such strange return mode. Having invalid user ids caused this
check to fail, and as side effect share limitation to groups to not work.

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
apps/user_ldap/lib/Group_LDAP.php

index cd4bd18cb4465629d9db98e2e35cb2d8ff7c6152..0780e28f7407297f3e1c005a78fdcf052c45d53f 100644 (file)
@@ -129,7 +129,6 @@ class Group_LDAP extends BackendUtility implements \OCP\GroupInterface, IGroupLD
 
                //usually, LDAP attributes are said to be case insensitive. But there are exceptions of course.
                $members = $this->_groupMembers($groupDN);
-               $members = array_keys($members); // uids are returned as keys
                if(!is_array($members) || count($members) === 0) {
                        $this->access->connection->writeToCache($cacheKey, false);
                        return false;