Remove calls & deprecate CRM_Core_BAO_PrevNextCache::setItem
authoreileen <emcnaughton@wikimedia.org>
Fri, 6 Mar 2020 00:36:23 +0000 (13:36 +1300)
committereileen <emcnaughton@wikimedia.org>
Fri, 6 Mar 2020 00:41:31 +0000 (13:41 +1300)
commit3bcde7f1f972a436d68fd2c995bbd84975291696
tree0a86a9d25ee3501023deede1d651a74c272aae1b
parent40cbd643a97e4a9c7ff63279e5e88a5a13f1be8b
Remove calls & deprecate CRM_Core_BAO_PrevNextCache::setItem

We have this function which

1) is mildly misleading - it appears to be generic to the prevnext table but is actually
only relevant to deduping as searches no longer use it
2) is about 50% deprecated and
3) the remainder is a single insert
4) is called from 2 places which use it a little differently

I think it's not really adding much value - I was going to remove the deprecated code
but I think in fact the goal should be to remove the whole function.

In general I think code on CRM_Core_BAO_PrevNextCache that is really dedupe-only code
should be on the dedupe classes. The history is that prevnext was created for searches
and kindof twisted to support dedupe as well but now search doesn't use much of what
is in the BAO class (if any)
CRM/Core/BAO/PrevNextCache.php
CRM/Dedupe/Finder.php
CRM/Dedupe/Merger.php
tests/phpunit/CRM/Core/BAO/PrevNextCacheTest.php