diff options
| author | Marcel Klehr <mklehr@gmx.net> | 2025-01-24 10:12:56 +0100 |
|---|---|---|
| committer | Andy Scherzinger <info@andy-scherzinger.de> | 2025-01-25 13:08:19 +0100 |
| commit | 35052624b9c499659d71686526836c77f4e888a2 (patch) | |
| tree | dcc67d9d78ea820b441161beb3613a52f9628db3 | |
| parent | 6e59f9a6930e56dafc2baaa1e6b0d4fc8a6c7fdc (diff) | |
| download | nextcloud-server-backport/50331/stable31.tar.gz nextcloud-server-backport/50331/stable31.zip | |
fix(TaskProcessing): Cache providersById in getPreferredProvidersbackport/50331/stable31
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 |