CRM-20750 follow up fix, reversal should be against original account.
authoreileen <emcnaughton@wikimedia.org>
Mon, 11 Sep 2017 02:37:50 +0000 (14:37 +1200)
committereileen <emcnaughton@wikimedia.org>
Mon, 11 Sep 2017 05:12:09 +0000 (17:12 +1200)
commitb0e806fac15b880e3bebd553151d1ce0503acb7f
tree376a1ed0fe289f1ae559ebc4a4f66dac54b0381d
parent5ca657dded2518f9b3613581cde3de696d5b52c4
CRM-20750 follow up fix, reversal should be against original account.

On trying to investigate why a test change was required in the original PR I came to the following conclusions:

1) the test function was causing problems by being hard to read. As a result it was too easy to see it as
'accounts magic' - I started a cleanup here
2) Once I understood what was being tested & failing I concluded the change in the PR was in fact wrong
as it was putting a negative entry against the new financial account rather than the original one.
CRM/Core/BAO/FinancialTrxn.php
tests/phpunit/api/v3/ContributionTest.php