From 49a52126abe2ebca4f616fe6a2a125183305c88a Mon Sep 17 00:00:00 2001 From: Marcel Klehr Date: Tue, 4 Feb 2025 12:40:37 +0100 Subject: fix(TaskProcessing\Manager): Always use distributed cache and use PHP serialize Signed-off-by: Marcel Klehr --- lib/private/TaskProcessing/Manager.php | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) (limited to 'lib/private/TaskProcessing') diff --git a/lib/private/TaskProcessing/Manager.php b/lib/private/TaskProcessing/Manager.php index cb2434422aa..cdd9e909960 100644 --- a/lib/private/TaskProcessing/Manager.php +++ b/lib/private/TaskProcessing/Manager.php @@ -81,7 +81,6 @@ class Manager implements IManager { private IAppData $appData; private ?array $preferences = null; private ?array $providersById = null; - private ICache $cache; private ICache $distributedCache; public function __construct( @@ -101,7 +100,6 @@ class Manager implements IManager { ICacheFactory $cacheFactory, ) { $this->appData = $appDataFactory->get('core'); - $this->cache = $cacheFactory->createLocal('task_processing::'); $this->distributedCache = $cacheFactory->createDistributed('task_processing::'); } @@ -767,9 +765,8 @@ class Manager implements IManager { } public function getAvailableTaskTypes(bool $showDisabled = false): array { - if ($this->availableTaskTypes === null) { - // We use local cache only because distributed cache uses JSOn stringify which would botch our ShapeDescriptor objects - $this->availableTaskTypes = $this->cache->get('available_task_types'); + if ($this->availableTaskTypes === null && $this->distributedCache->get('available_task_types_v2') !== null) { + $this->availableTaskTypes = unserialize($this->distributedCache->get('available_task_types_v2')); } // Either we have no cache or showDisabled is turned on, which we don't want to cache, ever. if ($this->availableTaskTypes === null || $showDisabled) { @@ -812,7 +809,7 @@ class Manager implements IManager { } $this->availableTaskTypes = $availableTaskTypes; - $this->cache->set('available_task_types', $this->availableTaskTypes, 60); + $this->distributedCache->set('available_task_types_v2', serialize($this->availableTaskTypes), 60); } -- cgit v1.2.3