extern/egroupware_official
1
0
Fork 0
mirror of https://github.com/EGroupware/egroupware.git synced 2024-11-25 01:13:25 +01:00
Code Issues Packages Projects Releases Wiki Activity

some Windows AD seem to get confused by using ldap_get_entries together with ldap_(first|next)_entry

Browse Source 
check if we got the requested number of entries ($start[1]) back and only continue in that case
This commit is contained in:
ralf 2024-05-03 15:17:35 +02:00
parent fc44e90b80
commit 3cb0779e62
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
api/src/Contacts/Ldap.php
View File

@ -1388,9 +1388,12 @@ class Ldap
}
// if we have a non-empty cookie from paged results, continue reading from the server
while (is_array($start) && count($start) === 3 && $start[0] === false && $start[2] !== '')
while (is_array($start) && count($start) === 3 && $start[0] === false && $start[2] !== '' &&
// some Windows AD seem to get confused by using ldap_get_entries together with ldap_(first|next)_entry
// check if we got the requested number of entries ($start[1]) back and only continue in that case
count($entries)-1 === $start[1])
{
foreach($this->_searchLDAP($_ldapContext, $_filter, $_attributes, $_addressbooktype, $_skipPlugins, $order_by, $start) as $contact)
foreach($this->_searchLDAP($_ldapContext, $_filter, $_attributes, $_addressbooktype, $_skipPlugins, $order_by, $start, $read_photo) as $contact)
{
$contacts[] = $contact;
}