Alternate cleanup on failed - do not pass transaction, but use it

[eileenmcnaughton]

Overview

Alternative to #18020 focussing on failed - handles transaction fully within the failed function

Before

Transaction passed in (after no actions using it have been taken)

After

Transaction instantiated within failed function

Technical Details

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

[eileenmcnaughton]

Closing as @seamuslee001 preferred the other approach