CRM_Core_BAO_Cache - Deprecate getItems(), getItem(), setItem(), deleteGroup()
authorTim Otten <totten@civicrm.org>
Thu, 24 Jan 2019 00:11:42 +0000 (16:11 -0800)
committerTim Otten <totten@civicrm.org>
Wed, 30 Jan 2019 02:53:11 +0000 (18:53 -0800)
commit920fa38fa645f1fb6da180061bdcd389038f645b
tree319583b0b75b94c9e44427bd7bf0f1d3a7ad7c08
parent5a302bbc2f3bd75b28401667f48966d65a4a0b36
CRM_Core_BAO_Cache - Deprecate getItems(), getItem(), setItem(), deleteGroup()

These interfaces predate PSR-16 -- which is more flexible and complete.
PSR-16 supports TTLs, default-values, multi-key operations, etc.  PSR-16
drivers can be extended/decorated/replaced.  There are third-party
implementations of PSR-16.  And (personally) I find the code which consumes
PSR-16 to be more readable+writeable (e.g. `$cache->get($key)` vs
`CRM_Core_BAO_Cache::getItem($group, $item)`).

However, `CRM_Core_BAO_Cache` has been around forever.  I currently count
ten distinct cache-groups which rely on it (5 from `civicrm-core` and 5 from
`universe`). So we shouldn't remove it outright.
CRM/Core/BAO/Cache.php