dev/financial#148 fully deprecate loadObjects function

[eileenmcnaughton]

Overview

dev/financial#148 fully deprecate loadObjects function

Before

Function still in use

After

Function fully deprecated

Technical Details

the check for $success is meaningful as it either returns success or fails along the way

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).

[monishdeb]

Agree with the changes, tested on local, works fine.

[eileenmcnaughton]

thanks @monishdeb