fix: add PHP 8.1 return types to Net_LDAP2_Search Iterator methods

[JohnVillalovos]

Add proper return types to the Iterator interface methods in Net_LDAP2_Search (current(), key(), next(), valid(), rewind()) to fix deprecation notices like:

Deprecated: Return type of Net_LDAP2_Search::current() should either
be compatible with Iterator::current(): mixed, or the
#[\ReturnTypeWillChange] attribute should be used to temporarily
suppress the notice

[Copilot]

Pull request overview

This PR updates Net_LDAP2_Search’s Iterator methods to add PHP 8.1-compatible return type declarations, addressing deprecation notices about missing/incorrect Iterator method return types.

Changes:

• Add mixed return types to current() and key().
• Add void return types to next() and rewind().
• Add bool return type to valid().

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

mixed return types require PHP 8.0+. The package metadata currently declares PHP >= 7.4 (see package.xml), so this change makes the library fail to parse on supported PHP versions. Either avoid using mixed (not really possible pre-8.0) or bump the minimum supported PHP version and update the package/composer constraints accordingly.

[JohnVillalovos]

Unsure if want to make the minimum required version 8.0?

Looking at: https://www.php.net/supported-versions.php

8.1 and earlier are end-of-life. But this change will work for >= 8.0.

[Copilot]

Same PHP compatibility issue as current(): mixed is PHP 8.0+, but the project metadata indicates PHP 7.4 support. If PHP 7.4 is still supported, this will be a parse error; otherwise, please update the declared PHP requirement to 8.0/8.1+ in the distribution metadata.