ldap_control_paged_result deprecated

[Troopster19]

• LDAP Server Type: Active Directory
• PHP Version: 7.4.0

Description:

As per https://www.php.net/manual/en/migration74.deprecated.php:

ldap_control_paged_result_response() and ldap_control_paged_result() are deprecated. Pagination controls can be sent along with ldap_search() instead.

[stevebauman]

Hi @Troopster19, I've seen this - and it's going to be tricky to resolve in a backwards compatible manor, since PHP 7.2 does not support the serverctrls parameter in ldap_search.

https://www.php.net/manual/en/function.ldap-search

How do you think this should be tackled?

[Troopster19]

Sounds like there isn't going to be a very clean way to do it. Could a check be added for the PHP version number? Use ldap_search for >=7.3 and fall back to ldap_control_paged_result for <7.3?

[stevebauman]

I agree with that, I think that is the only way. I'll have to rebuild the runPaginate method in the query builder to determine the current PHP version, and then run a compatible method.

LdapRecord/src/Query/Builder.php

Line 507 in 3e024c4

protected function runPaginate($filter, $perPage, $isCritical)

[stevebauman]

Ok whew 😅 This one was a difficult to work out and implement - but it's done! Super excited about it.

A new release will be out in the next day or so with this fix.

[stevebauman]

Release is out! 🎉

[Troopster19]

Looks good so far! Thanks for all your great work :)