sascha
/
nextcloud-server
mirror of https://github.com/nextcloud/server.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix(TaskProcessing): Cache task types by user language 

fixes https://github.com/nextcloud/assistant/issues/357

Signed-off-by: Marcel Klehr <mklehr@gmx.net>
pull/55164/head
Marcel Klehr 2025-09-17 11:05:28 +07:00
parent cb2b3a6219
commit 62e63730df
2 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/private/TaskProcessing/Manager.php
Unescape Escape View File

@ -73,6 +73,8 @@ class Manager implements IManager {
public const LEGACY_PREFIX_TEXTTOIMAGE = 'legacy:TextToImage:';
public const LEGACY_PREFIX_SPEECHTOTEXT = 'legacy:SpeechToText:';
public const TASK_TYPES_CACHE_KEY = 'available_task_types_v3';
/** @var list<IProvider>|null */
private ?array $providers = null;
@ -108,6 +110,7 @@ class Manager implements IManager {
private IUserManager $userManager,
private IUserSession $userSession,
ICacheFactory $cacheFactory,
private IFactory $l10nFactory,
) {
$this->appData = $appDataFactory->get('core');
$this->distributedCache = $cacheFactory->createDistributed('task_processing::');
@ -813,12 +816,15 @@ class Manager implements IManager {
}
public function getAvailableTaskTypes(bool $showDisabled = false, ?string $userId = null): array {
// We cache by language, because some task type fields are translated
$cacheKey = self::TASK_TYPES_CACHE_KEY . ':' . $this->l10nFactory->findLanguage();
// userId will be obtained from the session if left to null
if (!$this->checkGuestAccess($userId)) {
return [];
}
if ($this->availableTaskTypes === null) {
$cachedValue = $this->distributedCache->get('available_task_types_v2');
$cachedValue = $this->distributedCache->get($cacheKey);
if ($cachedValue !== null) {
$this->availableTaskTypes = unserialize($cachedValue);
}
@ -864,7 +870,7 @@ class Manager implements IManager {
}
$this->availableTaskTypes = $availableTaskTypes;
$this->distributedCache->set('available_task_types_v2', serialize($this->availableTaskTypes), 60);
$this->distributedCache->set($cacheKey, serialize($this->availableTaskTypes), 60);
}

4
tests/lib/TaskProcessing/TaskProcessingTest.php
Unescape Escape View File

@ -1,4 +1,5 @@
<?php
/**
* SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
* SPDX-License-Identifier: AGPL-3.0-or-later
@ -28,6 +29,7 @@ use OCP\IServerContainer;
use OCP\IUser;
use OCP\IUserManager;
use OCP\IUserSession;
use OCP\L10N\IFactory;
use OCP\TaskProcessing\EShapeType;
use OCP\TaskProcessing\Events\GetTaskProcessingProvidersEvent;
use OCP\TaskProcessing\Events\TaskFailedEvent;
@ -612,6 +614,7 @@ class TaskProcessingTest extends \Test\TestCase {
$userManager,
\OC::$server->get(IUserSession::class),
\OC::$server->get(ICacheFactory::class),
\OC::$server->get(IFactory::class),
);
}
@ -1264,6 +1267,7 @@ class TaskProcessingTest extends \Test\TestCase {
\OC::$server->get(IUserManager::class),
\OC::$server->get(IUserSession::class),
\OC::$server->get(ICacheFactory::class),
\OC::$server->get(IFactory::class),
);
}