diff options
author | Cash Costello <cash.costello@gmail.com> | 2013-06-07 15:06:37 -0700 |
---|---|---|
committer | Cash Costello <cash.costello@gmail.com> | 2013-06-07 15:06:37 -0700 |
commit | ffb77c888d4016bb4d9e1499efe4d8ed363db18e (patch) | |
tree | f723f7c443dbbbbff5c204e631e81a46fbd4f712 /engine/lib | |
parent | 8dbdf2f72c9dccbbd471e805fbd112c6817cdcec (diff) | |
parent | 46ace645c75dbbaad4b2cdaeedffcd501487aa93 (diff) | |
download | elgg-ffb77c888d4016bb4d9e1499efe4d8ed363db18e.tar.gz elgg-ffb77c888d4016bb4d9e1499efe4d8ed363db18e.tar.bz2 |
Merge pull request #5610 from mrclay/5600
Fixes #5600: Entities are kept out of entity cache during save
Diffstat (limited to 'engine/lib')
-rw-r--r-- | engine/lib/entities.php | 46 |
1 files changed, 44 insertions, 2 deletions
diff --git a/engine/lib/entities.php b/engine/lib/entities.php index 5cfeca6f8..d0d97aa95 100644 --- a/engine/lib/entities.php +++ b/engine/lib/entities.php @@ -17,6 +17,15 @@ global $ENTITY_CACHE; $ENTITY_CACHE = array(); /** + * GUIDs of entities banned from the entity cache (during this request) + * + * @global array $ENTITY_CACHE_DISABLED_GUIDS + * @access private + */ +global $ENTITY_CACHE_DISABLED_GUIDS; +$ENTITY_CACHE_DISABLED_GUIDS = array(); + +/** * Cache subtypes and related class names. * * @global array|null $SUBTYPE_CACHE array once populated from DB, initially null @@ -26,6 +35,34 @@ global $SUBTYPE_CACHE; $SUBTYPE_CACHE = null; /** + * Remove this entity from the entity cache and make sure it is not re-added + * + * @param int $guid The entity guid + * + * @access private + * @todo this is a workaround until #5604 can be implemented + */ +function _elgg_disable_caching_for_entity($guid) { + global $ENTITY_CACHE_DISABLED_GUIDS; + + _elgg_invalidate_cache_for_entity($guid); + $ENTITY_CACHE_DISABLED_GUIDS[$guid] = true; +} + +/** + * Allow this entity to be stored in the entity cache + * + * @param int $guid The entity guid + * + * @access private + */ +function _elgg_enable_caching_for_entity($guid) { + global $ENTITY_CACHE_DISABLED_GUIDS; + + unset($ENTITY_CACHE_DISABLED_GUIDS[$guid]); +} + +/** * Invalidate this class's entry in the cache. * * @param int $guid The entity guid @@ -57,7 +94,7 @@ function _elgg_invalidate_cache_for_entity($guid) { * @todo Use an ElggCache object */ function _elgg_cache_entity(ElggEntity $entity) { - global $ENTITY_CACHE; + global $ENTITY_CACHE, $ENTITY_CACHE_DISABLED_GUIDS; // Don't cache non-plugin entities while access control is off, otherwise they could be // exposed to users who shouldn't see them when control is re-enabled. @@ -65,6 +102,11 @@ function _elgg_cache_entity(ElggEntity $entity) { return; } + $guid = $entity->getGUID(); + if (isset($ENTITY_CACHE_DISABLED_GUIDS[$guid])) { + return; + } + // Don't store too many or we'll have memory problems // @todo Pick a less arbitrary limit if (count($ENTITY_CACHE) > 256) { @@ -79,7 +121,7 @@ function _elgg_cache_entity(ElggEntity $entity) { elgg_get_metadata_cache()->clear($random_guid); } - $ENTITY_CACHE[$entity->guid] = $entity; + $ENTITY_CACHE[$guid] = $entity; } /** |