[REF] [Merge custom fields] Copy complex function into merger class, ready to disentangle as little shared code is really used

[eileenmcnaughton]

Overview

Copy complext function into merger class, ready to disentangle as little shared code is really used

Before

Complex getTree function called - code does much work before & after to prepare to call this but as a shared function it can't be rationalised

After

Merger code has it's own copy - so we can clean up

Technical Details

This is VERY heavily tested

I had to make one function temporarily public but commented 'beware' and will try to change that as I clean it up

Comments

[civibot]

(Standard links)

• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[eileenmcnaughton]

thanks @colemanw