Skip to content

fix: fix trashbin restore events #53752

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: fix trashbin restore events #53752

wants to merge 1 commit into from
+16 −21

Conversation

icewind1991
Copy link
Member

  • use the correct base folder for trying to get the target node (not in trash)
  • use the correct base folder when getting the NonExistingNode

@icewind1991 icewind1991 added this to the Nextcloud 32 milestone Jul 1, 2025
@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Jul 1, 2025
@icewind1991 icewind1991 requested a review from a team as a code owner July 1, 2025 15:03
@icewind1991 icewind1991 requested review from ArtificialOwl, provokateurin and come-nc and removed request for a team July 1, 2025 15:03
@icewind1991
Copy link
Member Author

/backport to stable31

@icewind1991 icewind1991 force-pushed the trasbin-event-fixes branch from a1a31ca to 8c924b9 Compare July 1, 2025 15:07
@icewind1991
fix: fix trashbin restore events
eeb3b81 
Signed-off-by: Robin Appelman <[email protected]>
@icewind1991 icewind1991 force-pushed the trasbin-event-fixes branch from 8c924b9 to eeb3b81 Compare July 2, 2025 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@come-nc come-nc come-nc approved these changes

@ArtificialOwl ArtificialOwl Awaiting requested review from ArtificialOwl ArtificialOwl is a code owner automatically assigned from nextcloud/server-backend

@provokateurin provokateurin Awaiting requested review from provokateurin provokateurin is a code owner automatically assigned from nextcloud/server-backend

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
3. to review Waiting for reviews backport-request
Projects
None yet
Milestone
Nextcloud 32
Development

Successfully merging this pull request may close these issues.
2 participants
@icewind1991 @come-nc