Simply loading of related objects in transition components
authoreileen <emcnaughton@wikimedia.org>
Sat, 5 Sep 2020 03:52:12 +0000 (15:52 +1200)
committereileen <emcnaughton@wikimedia.org>
Sat, 5 Sep 2020 03:53:06 +0000 (15:53 +1200)
commit863592c31644c445663bb8ad08b47aaed91ee24c
treef7b7c6a35033c688ad6a5ff2fa868b79bc37412a
parent30bd3f91efc7f6990f956efcef35e10e38e8380c
Simply loading of related objects in transition components

This alters transitionComponents to  bypass validateData to load the related objects

Stepping through validateData it's clear that validate data
1) loads the contribution - this is now in transitionComponents
2) loads the contact - that is not specifically necessary as it is not used in transitionComponents
3) does some validation that relates to the IPNyness of BaseIPN but not to an internally called function
4) calls 'loadObjects'

LoadObjects does some error handling, sets the processorID if passed in, which it isn't  and then calls
contribution->loadRelatedObjects( )

- we now do that last call directly in-function

OrderTest::testCancelWithParticipant passes directly through this code
CRM/Contribute/BAO/Contribution.php