[REF] Move sendNotification out of recur, remove unused related_contact

[eileenmcnaughton]

Overview

[REF] Move sendNotification out of recur, remove unused related_contact

Before

Send notification in the recur function, needing extra ids to be passed into the function

After

Outside the function

Technical Details

We no longer use ids['related_contact'] & are attempting to simplify (& eventually remove
the ids variable). Note the surrounding if means it will only be called on specific trxn_types

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