From 17bc99743b142aa04dc85d64ac0f606467800f51 Mon Sep 17 00:00:00 2001 From: Arthur Schiwon Date: Thu, 13 Jun 2019 16:49:39 +0200 Subject: [PATCH] fix inGroup check, thus make integration tests succeed 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 --- apps/user_ldap/lib/Group_LDAP.php | 1 - 1 file changed, 1 deletion(-) diff --git a/apps/user_ldap/lib/Group_LDAP.php b/apps/user_ldap/lib/Group_LDAP.php index cd4bd18cb4..0780e28f74 100644 --- a/apps/user_ldap/lib/Group_LDAP.php +++ b/apps/user_ldap/lib/Group_LDAP.php @@ -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;