Skip to content

Remove unused legacy methods - #65493

Merged
come-nc merged 6 commits into
masterfrom
chore/remove-unused-legacy-methods
Oct 15, 2026
Merged

come-nc merged 6 commits into
masterfrom
chore/remove-unused-legacy-methods

Conversation

@come-nc

@come-nc come-nc commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Summary

I used https://aneba.sitegui.dev/35/ to identify legacy methods which are not used anymore by apps from the appstore.
I removed those that were not used by core either.

I turned a few methods private when the last uses were in the same class.

Checklist

AI (if applicable)

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

These methods are not used by core or any app in the appstore.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
These methods are not used by core or any app in the appstore.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
These methods are not used by core or any app in the appstore.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc added this to the Nextcloud 36 milestone Oct 14, 2026
@come-nc come-nc self-assigned this Oct 14, 2026
@come-nc
come-nc requested a review from a team as a code owner October 14, 2026 09:22
@come-nc come-nc added the 3. to review Waiting for reviews label Oct 14, 2026
@come-nc
come-nc requested review from icewind1991 and removed request for a team October 14, 2026 09:22
@come-nc come-nc added the technical debt 🧱 🤔🚀 label Oct 14, 2026
These methods are not used by core or any app in the appstore.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
These methods are not used by core or any app in the appstore.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc
come-nc force-pushed the chore/remove-unused-legacy-methods branch from 9924962 to f423d25 Compare October 14, 2026 10:26
@susnux susnux added the pending documentation This pull request needs an associated documentation update label Oct 14, 2026
@susnux

susnux commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Even if unused - once merged please put it into docs for tracking:
https://docs.nextcloud.com/server/latest/developer_manual/release_notes/critical_changes.html#removed-back-end-apis

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc
come-nc force-pushed the chore/remove-unused-legacy-methods branch from 39e06a2 to 86466e7 Compare October 15, 2026 10:38
@come-nc
come-nc merged commit 15acdd7 into master Oct 15, 2026
155 checks passed
@come-nc
come-nc deleted the chore/remove-unused-legacy-methods branch October 15, 2026 16:06
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 pending documentation This pull request needs an associated documentation update technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants