diff options
| author | Marcel Klehr <mklehr@gmx.net> | 2025-01-24 10:12:56 +0100 |
|---|---|---|
| committer | Marcel Klehr <mklehr@gmx.net> | 2025-01-24 16:45:43 +0100 |
| commit | a6100021e33a7af44c996c58d10ad067939ddc02 (patch) | |
| tree | 6a937a5ec6453aa1bd5a19a2d38f848b4c92cd73 | |
| parent | c1db2eb0a97f0ebe8ce213887f623823b26a299a (diff) | |
| download | nextcloud-server-fix/perf/cache-avilable-taskt-types.tar.gz nextcloud-server-fix/perf/cache-avilable-taskt-types.zip | |
fix(TaskProcessing): Cache providersById in getPreferredProvidersfix/perf/cache-avilable-taskt-types
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
| -rw-r--r-- | lib/private/TaskProcessing/Manager.php | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/lib/private/TaskProcessing/Manager.php b/lib/private/TaskProcessing/Manager.php index 10224584b43..cb2434422aa 100644 --- a/lib/private/TaskProcessing/Manager.php +++ b/lib/private/TaskProcessing/Manager.php @@ -80,6 +80,7 @@ class Manager implements IManager { private IAppData $appData; private ?array $preferences = null; + private ?array $providersById = null; private ICache $cache; private ICache $distributedCache; @@ -744,9 +745,13 @@ class Manager implements IManager { $providers = $this->getProviders(); if (isset($this->preferences[$taskTypeId])) { - $provider = current(array_values(array_filter($providers, fn ($provider) => $provider->getId() === $this->preferences[$taskTypeId]))); - if ($provider !== false) { - return $provider; + $providersById = $this->providersById ?? array_reduce($providers, static function (array $carry, IProvider $provider) { + $carry[$provider->getId()] = $provider; + return $carry; + }, []); + $this->providersById = $providersById; + if (isset($providersById[$this->preferences[$taskTypeId]])) { + return $providersById[$this->preferences[$taskTypeId]]; } } // By default, use the first available provider |