Fix bug where in general, FindReplaceLogicTargets might accidentally
perform a replacement even though a prior search did not yield any
results.

The TextViewer's implementation fixes this bug intrinsically but in
doing so, it violates the specification of the interface it implements.

This PR adds a check that there really was a search-result before
performing any replacements
and unit-tests this Check.