[LdapServer] Fix the SearchResultDone result code if an operation exception occurs.

[ChadSikorra]

This resolves the issues brought up in this PR / bug report: #58

The issue is that we are sending the OperationException result code, during server search handler implementations, as an out of order message after the SearchResultDone is already sent. This corrects the logic so that the result code and any diagnostic message is now sent properly in the SearchResultDone message.

I will likely change the interface for handler implementations as part of the 1.0.0 release so that the new ServerSearchResult object needs to be returned by handler implementations instead of just the entries object.

[codecov]

Codecov Report

Merging #63 (8f7a4cb) into master (51935ed) will increase coverage by 0.96%.
The diff coverage is 98.86%.

❗ Current head 8f7a4cb differs from pull request most recent head f2081a6. Consider uploading reports for the commit f2081a6 to get more accurate results

@@             Coverage Diff              @@
##             master      #63      +/-   ##
============================================
+ Coverage     85.10%   86.07%   +0.96%     
+ Complexity     3660     1844    -1816     
============================================
  Files           256      129     -127     
  Lines          9472     4739    -4733     
============================================
- Hits           8061     4079    -3982     
+ Misses         1411      660     -751     

Impacted Files	Coverage Δ
...tocol/ServerProtocolHandler/ServerSearchResult.php	96.29% <96.29%> (ø)
...ocol/ServerProtocolHandler/ServerPagingHandler.php	92.45% <100.00%> (+1.34%)	⬆️
...ProtocolHandler/ServerPagingUnsupportedHandler.php	100.00% <100.00%> (ø)
...ocol/ServerProtocolHandler/ServerSearchHandler.php	100.00% <100.00%> (ø)
...otocol/ServerProtocolHandler/ServerSearchTrait.php	78.12% <100.00%> (+5.04%)	⬆️

... and 129 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more