From 943f2576c578d556ee6e5fa77bfef16ab8efb515 Mon Sep 17 00:00:00 2001 From: Daniel Kesselberg Date: Sat, 24 Nov 2018 22:06:45 +0100 Subject: [PATCH] Fix count on string Signed-off-by: Daniel Kesselberg --- apps/user_ldap/lib/Access.php | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/apps/user_ldap/lib/Access.php b/apps/user_ldap/lib/Access.php index fb2582e8266..c5075a033b6 100644 --- a/apps/user_ldap/lib/Access.php +++ b/apps/user_ldap/lib/Access.php @@ -879,7 +879,7 @@ class Access extends LDAPUtility { }); } $this->batchApplyUserAttributes($recordsToUpdate); - return $this->fetchList($ldapRecords, count($attr) > 1); + return $this->fetchList($ldapRecords, $this->manyAttributes($attr)); } /** @@ -922,7 +922,7 @@ class Access extends LDAPUtility { * @return array */ public function fetchListOfGroups($filter, $attr, $limit = null, $offset = null) { - return $this->fetchList($this->searchGroups($filter, $attr, $limit, $offset), count($attr) > 1); + return $this->fetchList($this->searchGroups($filter, $attr, $limit, $offset), $this->manyAttributes($attr)); } /** @@ -2011,4 +2011,17 @@ class Access extends LDAPUtility { return $pagedSearchOK; } + /** + * Is more than one $attr used for search? + * + * @param string|string[]|null $attr + * @return bool + */ + private function manyAttributes($attr) { + if (\is_array($attr)) { + return \count($attr) > 1; + } + return false; + } + } -- 2.39.5