Skip to content

[stable32] fix: Fix handling of deleting share from self#2387

Merged
miaulalala merged 2 commits intostable32from
backport/2319/stable32
Feb 19, 2026
Merged

[stable32] fix: Fix handling of deleting share from self#2387
miaulalala merged 2 commits intostable32from
backport/2319/stable32

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Feb 19, 2026

Backport of PR #2319

danxuliu and others added 2 commits February 19, 2026 14:07
If a share is deleted from self and it is not a group share it was
handled as a user share. However, other types of shares can be deleted
from self (for example, circle or room shares), so user shares need to
be explicitly checked.

Note that as those other share types are not explicitly handled no entry
is added to the activity for them, but that was already the case before
(this change just prevents an exception to be thrown).

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: Anna Larch <anna@nextcloud.com>
@miaulalala miaulalala enabled auto-merge February 19, 2026 14:08
@cypress
Copy link

cypress bot commented Feb 19, 2026

Activity    Run #3338

Run Properties:  status check passed Passed #3338  •  git commit dd17547e34: [stable32] fix: Fix handling of deleting share from self
Project Activity
Branch Review backport/2319/stable32
Run status status check passed Passed #3338
Run duration 04m 36s
Commit git commit dd17547e34: [stable32] fix: Fix handling of deleting share from self
Committer backportbot[bot]
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 10
View all changes introduced in this branch ↗︎

@miaulalala miaulalala merged commit 828a2a5 into stable32 Feb 19, 2026
54 of 56 checks passed
@miaulalala miaulalala deleted the backport/2319/stable32 branch February 19, 2026 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments