Remove another use of loadRelatedObjects() function

[mattwire]

Overview

We don't need to call loadRelatedObjects() until we get to composeMessageArray() function. I've changed $loadAll to TRUE because that's how the parent function was calling it so this ensures the same data is available.

Before

More calls to deprecated functions.

After

Less calls to deprecated functions.

Technical Details

Comments

@eileenmcnaughton This probably makes most sense to you :-)

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

[eileenmcnaughton]

@mattwire yep - I checked & it seems fine. composeMessageArray is on called from this one place (& I don't think it's params need to be pass-by-ref)

[seamuslee001]

Merging as per @eileenmcnaughton 's review