Skip to content

Comments

[stable30] fix(TaskProcessing): Refactor TextToImage fallback#58427

Open
backportbot[bot] wants to merge 4 commits intostable30from
backport/58112/stable30
Open

[stable30] fix(TaskProcessing): Refactor TextToImage fallback#58427
backportbot[bot] wants to merge 4 commits intostable30from
backport/58112/stable30

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Feb 19, 2026

Backport of PR #58112

@backportbot backportbot bot self-assigned this Feb 19, 2026
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
instead of UserFacingProcessing Exception which is not in nc < 33

Signed-off-by: Marcel Klehr <mklehr@gmx.net>
@github-actions github-actions bot changed the title [stable30] [stable32] fix(TaskProcessing): Refactor TextToImage fallback [stable30] fix(TaskProcessing): Refactor TextToImage fallback Feb 19, 2026
@miaulalala
Copy link
Contributor

@marcelklehr the backport had some issues

Signed-off-by: Marcel Klehr <mklehr@gmx.net>
@marcelklehr marcelklehr force-pushed the backport/58112/stable30 branch from ca686ac to b877312 Compare February 19, 2026 14:17
@marcelklehr
Copy link
Member

Seems fixed now, although some CI checks still fail :/

$newProvider = new class($oldProvider, $this->appData, $this->l10nFactory, $userManager) implements IProvider, ISynchronousProvider {
private \OCP\TextToImage\IProvider $provider;
private IAppData $appData;
private IFactory $l10nFactory;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unused, remove?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried to keep the code as similar to the master version as possible, that's why I left it in there. but we can remove it, sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants