From 375ece362dd971b2781e77eafad1aeb180a6dba0 Mon Sep 17 00:00:00 2001 From: Daniel Kesselberg Date: Sat, 24 Nov 2018 22:06:45 +0100 Subject: [PATCH 1/2] 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 d615a4241b..5501dd3d0e 100644 --- a/apps/user_ldap/lib/Access.php +++ b/apps/user_ldap/lib/Access.php @@ -880,7 +880,7 @@ class Access extends LDAPUtility implements IUserTools { }); } $this->batchApplyUserAttributes($recordsToUpdate); - return $this->fetchList($ldapRecords, count($attr) > 1); + return $this->fetchList($ldapRecords, $this->manyAttributes($attr)); } /** @@ -923,7 +923,7 @@ class Access extends LDAPUtility implements IUserTools { * @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)); } /** @@ -2016,4 +2016,17 @@ class Access extends LDAPUtility implements IUserTools { 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; + } + } From 5c10df6d2dc1c38691194a08688fd427c7427dbb Mon Sep 17 00:00:00 2001 From: Daniel Kesselberg Date: Sun, 25 Nov 2018 12:30:54 +0100 Subject: [PATCH 2/2] Add return type Signed-off-by: Daniel Kesselberg --- apps/user_ldap/lib/Access.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/user_ldap/lib/Access.php b/apps/user_ldap/lib/Access.php index 5501dd3d0e..48914e6428 100644 --- a/apps/user_ldap/lib/Access.php +++ b/apps/user_ldap/lib/Access.php @@ -2022,7 +2022,7 @@ class Access extends LDAPUtility implements IUserTools { * @param string|string[]|null $attr * @return bool */ - private function manyAttributes($attr) { + private function manyAttributes($attr): bool { if (\is_array($attr)) { return \count($attr) > 1; }