Skip to content

Remove deprecated IServerContainer - #64095

Draft
come-nc wants to merge 11 commits into
masterfrom
fix/cleanup-server-container
Draft

Remove deprecated IServerContainer#64095
come-nc wants to merge 11 commits into
masterfrom
fix/cleanup-server-container

Conversation

@come-nc

@come-nc come-nc commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Remove deprecated IServerContainer interface and associated methods, cleanup resulting code a bit.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@come-nc come-nc self-assigned this Sep 7, 2026
@come-nc come-nc added the 2. developing Work in progress label Sep 7, 2026
@come-nc come-nc added this to the Nextcloud 36 milestone Sep 7, 2026
@CarlSchwan

Copy link
Copy Markdown
Member

Noice :)

@come-nc
come-nc force-pushed the fix/cleanup-server-container branch from cd259c3 to ac7091c Compare September 7, 2026 14:47
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
…asses

Switch query to protected and remove it from public API. Optimize a bit.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
…rvice

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
…ervice

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Supposedly this is slightly better for performances as the closure is
 not bound to the object.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc
come-nc force-pushed the fix/cleanup-server-container branch from ba8033c to 5e685dd Compare September 7, 2026 16:14
@come-nc come-nc added 3. to review Waiting for reviews technical debt 🧱 🤔🚀 performance 🚀 and removed 2. developing Work in progress labels Sep 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews performance 🚀 technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants