projects / civicrm-core.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 0ab5c4c) | patch
Do not treat an empty array as not set in caches.
authoreileen <emcnaughton@wikimedia.org>
Thu, 6 Sep 2018 02:44:55 +0000 (14:44 +1200)
committereileen <emcnaughton@wikimedia.org>
Thu, 6 Sep 2018 02:45:46 +0000 (14:45 +1200)
commit22263d41db90acee0220259d37e16f1b36ab76ab
treef0d02787cce956d5228e849251975f1e37c474d4tree | snapshot (zip tar.gz)
parent0ab5c4c7b5ab3dd58e886404d0da0de8e7f57df5commit | diff
Do not treat an empty array as not set in caches.

On testing Redis we found that even after Redis cache had been
built the set function was still being called. It turned out the reason is that
when a value was set to an empty array that was being treated as FALSEY.

In fact the cache function returns NULL if nothing is set & no default is
specified & we should only treat NULL as a cache miss
CRM/Core/BAO/Cache.php diff | blob | blame | history