Skip to content

chore(Cache): Remove deprecated insert and update events#60797

Open
provokateurin wants to merge 1 commit into
masterfrom
chore/cache/remove-deprecated-events
Open

chore(Cache): Remove deprecated insert and update events#60797
provokateurin wants to merge 1 commit into
masterfrom
chore/cache/remove-deprecated-events

Conversation

@provokateurin
Copy link
Copy Markdown
Member

The emitting code was refactored by accident: #60102 (comment)

We can just remove them, because they have been deprecated in 21 already.

@provokateurin provokateurin added this to the Nextcloud 35 milestone May 28, 2026
@provokateurin provokateurin requested a review from a team as a code owner May 28, 2026 10:53
@provokateurin provokateurin added the 3. to review Waiting for reviews label May 28, 2026
@provokateurin provokateurin requested review from Altahrim and removed request for a team May 28, 2026 10:53
@provokateurin provokateurin added the technical debt 🧱 🤔🚀 label May 28, 2026
@CarlSchwan
Copy link
Copy Markdown
Member

autoloader needs an update

@CarlSchwan
Copy link
Copy Markdown
Member

@come-nc
Copy link
Copy Markdown
Contributor

come-nc commented May 28, 2026

this is used in a bunch of places https://github.com/search?q=org%3Anextcloud%20CacheEntryInsertedEvent&type=code

You are searching for the new one.
The removed one is CacheInsertEvent

Only collectives is using the old one it seems https://github.com/search?q=org%3Anextcloud+CacheInsertEvent&type=code

@come-nc come-nc added the pending documentation This pull request needs an associated documentation update label May 28, 2026
Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin force-pushed the chore/cache/remove-deprecated-events branch from 926d8ea to b1724db Compare May 28, 2026 12:02
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