- remove check_symbol_exists(), check_function_exists() should be enough
- fix check for HAVE_LDAP_EXTENDED_OPERATION_S before ldap_extended_operation_s
  is used

This way we get a build time warning if the var isn't defined at all, e.g.
a missing check_include_files() CMake call.

Remove unused var, HAVE_LDAP_H.

This code crashes in the destructor of LdapClient on FreeBSD.
The crash is visible to uses who
- start KOrganizer
- click the "New Event" button

The underlying issue is a lot like the one described in
	https://blogs.kde.org/2021/02/20/uniqueptr-difference-between-libstdc-and-libc-crashes-your-application
There is a convoluted call-chain on destruction of LdapClient:
- ~LdapClient
- ~std::unique_ptr<LdapClientPrivate>
- ~LdapClientPrivate
- LdapClient::cancelQuery
- (accesses to members of LdapClientPrivate `d`)

With libcxx, the pointer in `d` is already set to nullptr and
SEGV happens. I'm not sure it isn't UB, anyway, since the
destructor body for LdapClient has already run.

The fix moves the implementation of `cancelQuery()` into
the private class. This means that the LdapClient class does
a little less poking-and-prodding in the private class,
but also cuts out the call-from-private-back-to-destroyed-
owning-LdapClient, fixing the SEGV and possible UB.

"static Q_REQUIRED_RESULT foo bar()" -> "Q_REQUIRED_RESULT static foo bar()"