From 6bcc2ad86d476f8b90f3c664edf22751c8c9b27e Mon Sep 17 00:00:00 2001 From: "szabolcs.gyuris" Date: Tue, 5 Mar 2024 11:49:56 +0000 Subject: [PATCH] [backport 27]fix: Avoid clear cache with prefix [maybe needed for 28] --- .../Collaboration/Reference/ReferenceManager.php | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/lib/private/Collaboration/Reference/ReferenceManager.php b/lib/private/Collaboration/Reference/ReferenceManager.php index 2897410f5d613..40c08751f74e0 100644 --- a/lib/private/Collaboration/Reference/ReferenceManager.php +++ b/lib/private/Collaboration/Reference/ReferenceManager.php @@ -137,6 +137,11 @@ public function resolveReference(string $referenceId): ?IReference { $reference = $matchedProvider->resolveReference($referenceId); if ($reference) { + $cachePrefix = $matchedProvider->getCachePrefix($referenceId); + if ($cachePrefix !== '') { + // If a prefix is used we set an additional key to know when we need to delete by prefix during invalidateCache() + $this->cache->set('hasPrefix-' . md5($cachePrefix), true, self::CACHE_TTL); + } $this->cache->set($cacheKey, Reference::toCache($reference), self::CACHE_TTL); return $reference; } @@ -190,11 +195,15 @@ private function getFullCacheKey(IReferenceProvider $provider, string $reference */ public function invalidateCache(string $cachePrefix, ?string $cacheKey = null): void { if ($cacheKey === null) { - $this->cache->clear(md5($cachePrefix)); + // clear might be a heavy operation, so we only do it if there have actually been keys set + if ($this->cache->remove('hasPrefix-' . md5($cachePrefix))) { + $this->cache->clear(md5($cachePrefix)); + } + return; } - $this->cache->remove(md5($cachePrefix) . '-' . md5($cacheKey)); + $this->cache->remove(md5($cachePrefix) . '-' . md5($cacheKey ?? '')); } /**