Is Returning a value and also have an escaping parameter in the same function considered bad practice?

Question

Is this code considered bad practice? Is there any benefit or downside to having both a return value and a completion handler in the same function?

For example:

<