[REF] Extract function to get locations to merge, rename 'operation' to is_replace'
authoreileen <emcnaughton@wikimedia.org>
Wed, 29 Jul 2020 06:53:51 +0000 (18:53 +1200)
committereileen <emcnaughton@wikimedia.org>
Sat, 22 Aug 2020 01:25:12 +0000 (13:25 +1200)
commit4c54e0bd41f31c5bf40855ff23e55e77d3d35115
tree3523acc6048d9c1ee940906261d8287e31a2002d
parentfde029fe1aed997370bc9fb5358fc973b370f65f
[REF] Extract function to get locations to merge, rename 'operation' to is_replace'

This is a preliminary refactor. I have a bug to fix in this code but cannot yet make sense of it. The mergeHandler class
is simply an object to refactor the functions onto. Much of this code is hard to work with as the use of static functions
necessitates a lot of compilation and compiling so the process of cleaning it up involves extracting functionns to this
new class (which may one day replace the Merger class). As it is a refactoring process the functions reflect the old code
more than the ideal code. However, it provides the change to give some documentation regarding the locBlock array
and also to rename the confusing field 'operation' to 'is_replace'
CRM/Dedupe/MergeHandler.php
CRM/Dedupe/Merger.php
tests/phpunit/api/v3/ContactTest.php