]> source.dussan.org Git - nextcloud-server.git/commitdiff
fix inGroup check, thus make integration tests succeed 15858/head
authorArthur Schiwon <blizzz@arthur-schiwon.de>
Thu, 13 Jun 2019 14:49:39 +0000 (16:49 +0200)
committerJohn Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Fri, 14 Jun 2019 04:23:58 +0000 (06:23 +0200)
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 ae2434056b1e2a7a6382ae0eacc84750ffaf7d0f..5b25979b2d26096876b7503d3bceec43d7a19c63 100644 (file)
@@ -130,7 +130,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;