Skip to content

chore(files_versions): drop the deprecated \OCP\Versions rollback hook - #64402

Merged
susnux merged 1 commit into
masterfrom
worktree-remove-versions-rollback-hook
Sep 16, 2026
Merged

susnux merged 1 commit into
masterfrom
worktree-remove-versions-rollback-hook

Conversation

@susnux

@susnux susnux commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Summary

The listener only bridged VersionRestoredEvent to the legacy hook, was deprecated as "removed in the next version", and has no listeners left. Instead every consumer uses the typed event.

Checklist

AI (if applicable)

AI was used to verify no active users left (a hook problem because they can be called by string)

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

@susnux susnux added this to the Nextcloud 36 milestone Sep 15, 2026
@susnux
susnux requested a review from a team as a code owner September 15, 2026 21:59
@susnux susnux added 3. to review Waiting for reviews technical debt 🧱 🤔🚀 pending documentation This pull request needs an associated documentation update labels Sep 15, 2026
@susnux
susnux requested review from Altahrim, provokateurin and salmart-dev and removed request for a team September 15, 2026 21:59
@susnux susnux added the ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) label Sep 15, 2026
The listener only bridged VersionRestoredEvent to the legacy hook, was
deprecated in 32.0.0 as "removed in the next version", and has no
listeners left. The typed event keeps its consumers.

Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnux force-pushed the worktree-remove-versions-rollback-hook branch from ac08feb to 2f73fb2 Compare September 16, 2026 09:52
@susnux
susnux disabled auto-merge September 16, 2026 12:36
@susnux
susnux merged commit 9403d92 into master Sep 16, 2026
175 of 179 checks passed
@susnux
susnux deleted the worktree-remove-versions-rollback-hook branch September 16, 2026 12:36
@susnux

susnux commented Sep 16, 2026

Copy link
Copy Markdown
Contributor Author

documentation: nextcloud/documentation#15630

@susnux susnux removed the pending documentation This pull request needs an associated documentation update label Sep 16, 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 AI assisted ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants