Required parameter $flatten follows optional parameter $type in ldap_servers/ldap_servers.module

I am getting the following error after applying the patches.
Deprecated function: Creation of dynamic property LdapUserConf::$createLDAPAccounts is deprecated in LdapUserConf->load() (line 265 contrib/modules/ldap/ldap_user/LdapUserConf.class.php).

May be 8.2 specific.
Upgrading Drupal 7 on Debian 12 bookworm.

The last patch failed to apply. Therefore, the status is Needs work.

The latest patch is for a different problem. If #7 still applies, I would just stick with that and then make new issue(s) for the items mentioned in #8 and #10.

-function ldap_query_get_queries($qid = NULL, $type, $flatten = FALSE, $reset = FALSE) {
+function ldap_query_get_queries($qid = NULL, $type = 'all', $flatten = FALSE, $reset = FALSE) {

Truly, it is not necessary to make all the parameters optional. PHP handles a function declaration like ldap_query_get_queries($qid = NULL, $type, $flatten = FALSE, $reset = FALSE) as ldap_query_get_queries($qid, $type, $flatten = FALSE, $reset = FALSE) because it simply ignores the default value given for the optional parameters that comes before the required parameters. In previous PHP versions, that happened silently; the last versions opted for a deprecation message.

The correct change is the following one.

-function ldap_query_get_queries($qid = NULL, $type, $flatten = FALSE, $reset = FALSE) {
+function ldap_query_get_queries($qid, $type, $flatten = FALSE, $reset = FALSE) {

Applied patch #7, along with your recommended change in #13.

#8 doesn't apply because the change was already in dev.

#10 has been fixed in the MR, I think, since I added the properties to the class.

Beware, the patch ldap-php-8-compatibility-3302242-7 breaks LDAP pagination.

Instead of having:
```
ldap_search($this->connection, $ldap_query_params['base_dn'], $ldap_query_params['filter'], array("*"), 0, $this->searchPageSize, -1, LDAP_DEREF_NEVER, $servercontrols);
$result = $this->ldapQuery($ldap_query_params['scope'], $ldap_query_params);
```
we should just have :
```
$result = ldap_search($this->connection, $ldap_query_params['base_dn'], $ldap_query_params['filter'], array("*"), 0, $this->searchPageSize, -1, LDAP_DEREF_NEVER, $servercontrols);
```

The ldap_servers_php_supports_pagination has not been updated either. If patched as follows, it works again:
```
function ldap_servers_php_supports_pagination() {
if (version_compare(PHP_VERSION, '8.0.0') >= 0) {
return TRUE;
}
return (boolean)(function_exists('ldap_control_paged_result_response') && function_exists('ldap_control_paged_result'));
}
```

Despite these pagination issues, everything else works great with this patch.