Remove return value on transitionComponents

[eileenmcnaughton]

Overview

Follow up cleanup from #18964

We no longer use the result from transitionComponents so this removes most of the code that calculates the updateResult value.

Before

Value returned but never used

After

poof

Technical Details

There are still instances coming back from the cancel & fail functions that I will remove as a follow up

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]

@seamuslee001 are you able to merge this? I kept it pretty small so it should be easy to see that it's not doing anything weird

[seamuslee001]

Yeh this seems to make sense merging