From f69620196c5c10bf12ce6bd89885a4a5a12c90c7 Mon Sep 17 00:00:00 2001 From: Marcel Klehr Date: Mon, 12 Jan 2026 13:21:09 +0100 Subject: [PATCH 1/3] fix(TaskProcessing): Refactor TextToImage fallback Signed-off-by: Marcel Klehr --- lib/private/TaskProcessing/Manager.php | 24 ++++++++++++++++++++++-- 1 file changed, 22 insertions(+), 2 deletions(-) diff --git a/lib/private/TaskProcessing/Manager.php b/lib/private/TaskProcessing/Manager.php index 35bd10270868a..eb93496654fc4 100644 --- a/lib/private/TaskProcessing/Manager.php +++ b/lib/private/TaskProcessing/Manager.php @@ -303,13 +303,18 @@ private function _getTextToImageProviders(): array { $oldProviders = $this->textToImageManager->getProviders(); $newProviders = []; foreach ($oldProviders as $oldProvider) { - $newProvider = new class($oldProvider, $this->appData) implements IProvider, ISynchronousProvider { + $newProvider = new class($oldProvider, $this->appData, $this->l10nFactory, $this->userManager) implements IProvider, ISynchronousProvider { private \OCP\TextToImage\IProvider $provider; private IAppData $appData; + private IFactory $l10nFactory; - public function __construct(\OCP\TextToImage\IProvider $provider, IAppData $appData) { + private IUserManager $userManager; + + public function __construct(\OCP\TextToImage\IProvider $provider, IAppData $appData, IFactory $l10nFactory, IUserManager $userManager) { $this->provider = $provider; $this->appData = $appData; + $this->l10nFactory = $l10nFactory; + $this->userManager = $userManager; } public function getId(): string { @@ -342,6 +347,21 @@ public function process(?string $userId, array $input, callable $reportProgress) } catch (\OCP\Files\NotFoundException) { $folder = $this->appData->newFolder('text2image'); } + if ($input['numberOfImages'] > 12) { + throw new UserFacingProcessingException( + 'numberOfImages cannot be greater than 12', + userFacingMessage: + $this->l10nFactory->get('core', $this->l10nFactory->getUserLanguage($this->userManager->get($userId))) + ->t('Cannot generate more than 12 images') + ); + } + if ($input['numberOfImages'] < 1) { + throw new UserFacingProcessingException( + 'numberOfImages must be greater than 0', + userFacingMessage: + $this->l10nFactory->get('core', $this->l10nFactory->getUserLanguage($this->userManager->get($userId))) + ->t('Cannot generate less than 1 image')); + } $resources = []; $files = []; for ($i = 0; $i < $input['numberOfImages']; $i++) { From 3c99524ff516e61f195f9b7abd0686aa2d298f36 Mon Sep 17 00:00:00 2001 From: Marcel Klehr Date: Tue, 3 Feb 2026 09:17:32 +0100 Subject: [PATCH 2/3] Fix: Run cs:fix --- lib/private/TaskProcessing/Manager.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/private/TaskProcessing/Manager.php b/lib/private/TaskProcessing/Manager.php index eb93496654fc4..961758e98bd99 100644 --- a/lib/private/TaskProcessing/Manager.php +++ b/lib/private/TaskProcessing/Manager.php @@ -352,7 +352,7 @@ public function process(?string $userId, array $input, callable $reportProgress) 'numberOfImages cannot be greater than 12', userFacingMessage: $this->l10nFactory->get('core', $this->l10nFactory->getUserLanguage($this->userManager->get($userId))) - ->t('Cannot generate more than 12 images') + ->t('Cannot generate more than 12 images') ); } if ($input['numberOfImages'] < 1) { From c3e36871b15c868e0cd200662e8fe76d00c4c9ec Mon Sep 17 00:00:00 2001 From: Marcel Klehr Date: Thu, 5 Feb 2026 17:31:17 +0100 Subject: [PATCH 3/3] fix: Use ProcessingException instead of UserFacingProcessing Exception which is not in nc < 33 Signed-off-by: Marcel Klehr --- lib/private/TaskProcessing/Manager.php | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/lib/private/TaskProcessing/Manager.php b/lib/private/TaskProcessing/Manager.php index 961758e98bd99..084bcc4c49158 100644 --- a/lib/private/TaskProcessing/Manager.php +++ b/lib/private/TaskProcessing/Manager.php @@ -348,19 +348,14 @@ public function process(?string $userId, array $input, callable $reportProgress) $folder = $this->appData->newFolder('text2image'); } if ($input['numberOfImages'] > 12) { - throw new UserFacingProcessingException( - 'numberOfImages cannot be greater than 12', - userFacingMessage: - $this->l10nFactory->get('core', $this->l10nFactory->getUserLanguage($this->userManager->get($userId))) - ->t('Cannot generate more than 12 images') + throw new ProcessingException( + 'numberOfImages cannot be greater than 12' ); } if ($input['numberOfImages'] < 1) { - throw new UserFacingProcessingException( - 'numberOfImages must be greater than 0', - userFacingMessage: - $this->l10nFactory->get('core', $this->l10nFactory->getUserLanguage($this->userManager->get($userId))) - ->t('Cannot generate less than 1 image')); + throw new ProcessingException( + 'numberOfImages must be greater than 0' + ); } $resources = []; $files = [];