[REF] remove CRM_Core_Error check

[eileenmcnaughton]

Overview

Remove an IF that should always be true

Before

Handling for CRM_Core_Error when calling a function not expected to return a core error.

After

Handling removed, code easier to understand

Technical Details

I'm convinced there is no reason why find() should return a core_error & hence if it did we wouldn't want
to keep calm & carry on - smells a lot like copy & paste. Also makes code hard to understand

Comments

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[seamuslee001]

Change makes sense to me merging