Move function to delete merged contacts to the Merger class and include all instances...
authoreileen <emcnaughton@wikimedia.org>
Tue, 28 Jul 2020 00:21:05 +0000 (12:21 +1200)
committereileen <emcnaughton@wikimedia.org>
Tue, 28 Jul 2020 00:30:52 +0000 (12:30 +1200)
commiteec58f8a91308cff16614b5a2bdf2a4e043c7d8e
tree87da96e67248383ad0e91cb7461c6b121637ec80
parent6010fe35306be8f1a3f6546cc2244636d6f7322d
Move function to delete merged contacts to the Merger class and include all instances of the pair

The PrevNextCache class is a bit of a mess. It has 2 roles
1) be a cache
2) provide functionality specific to the Merger class

I am working to move this second functionality to the merger class and also to remove the cacheKey
filter on it. The cacheKey pertains to a particular search set. However if 2 people are deduping different search
sets and one person merges a contact pair they should not remain available in the other person's result set.

(Likewise, and to follow, if 2 contacts have a conflict we should update all search sets with that information,
not just the one that was actively being deduped)
CRM/Core/BAO/PrevNextCache.php
CRM/Dedupe/Merger.php