Skip to content

Fix NC 33.0.0 compatibility, deprecated execute() method was removed#282

Closed
michael-dev wants to merge 1 commit intonextcloud:masterfrom
michael-dev:fix/nextcloud-33.0.0
Closed

Fix NC 33.0.0 compatibility, deprecated execute() method was removed#282
michael-dev wants to merge 1 commit intonextcloud:masterfrom
michael-dev:fix/nextcloud-33.0.0

Conversation

@michael-dev
Copy link
Copy Markdown
Contributor

NC 33.0.0 removed the OC\DB\QueryBuilder\QueryBuilder::execute() method that was long deprecated.

So this pull request fixes the resulting
Call to undefined method OC\\DB\\QueryBuilder\\QueryBuilder::execute()
errors and restores compatibility with NC 33.0.0

This fixes

Call to undefined method OC\\DB\\QueryBuilder\\QueryBuilder::execute()

Signed-off-by: M. Braun <michael-dev@fami-braun.de>
@yds
Copy link
Copy Markdown

yds commented Feb 26, 2026

works for me, thx for the patch..

sooner this gets merged into a release the better. this is an essential plugin for those of us who use it, and a NC upgrade showstopper when this plugin is broken.

@thohai
Copy link
Copy Markdown

thohai commented Feb 27, 2026

Hey guys! My nextcloud is currently broken because of this issue. Is there any timeline, when this will make it into a release? Is there anything I can contribute to make the process faster?

@stefangweichinger
Copy link
Copy Markdown

Applied patch manually, thanks.
Works with NC-33.0.0 here.

@jensb
Copy link
Copy Markdown
Contributor

jensb commented Mar 13, 2026

This patch seems to be trivial enough and there are several success reports. Does this also fix the IMAP login method mentioned in #266? What else needs to be done to get it approved and a new version to be released?
I am willing to help since my Nextcloud instance cannot be upgraded as well due to this incompatibility.
Thanks!

@Rubeanie
Copy link
Copy Markdown

Manually patched my instance with no issues, excited for this merge (NC-33.0.0 ix-application)

@romanf
Copy link
Copy Markdown

romanf commented Mar 18, 2026

I think user_extern is no longer maintained.

@miaulalala
Copy link
Copy Markdown
Collaborator

superseded by #283

@miaulalala miaulalala closed this Mar 20, 2026
@System-End
Copy link
Copy Markdown

in pr 283,l max version is 32. tho it appears it works with NC33

@stefangweichinger
Copy link
Copy Markdown

Had to apply this one again manually after upgrading my docker-container to NC-33.0.1

In only edited Base.php, unsure if I have to edit IMAP.php again as well (I only use the imap-backend).

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants