Skip to content

Comments

[stable33] fix: Fix handling of deleting share from self#2385

Merged
miaulalala merged 2 commits intostable33from
backport/2319/stable33
Feb 19, 2026
Merged

[stable33] fix: Fix handling of deleting share from self#2385
miaulalala merged 2 commits intostable33from
backport/2319/stable33

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:06
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 #3335

Run Properties:  status check passed Passed #3335  •  git commit 99c4f0e8bc: [stable33] fix: Fix handling of deleting share from self
Project Activity
Branch Review backport/2319/stable33
Run status status check passed Passed #3335
Run duration 02m 39s
Commit git commit 99c4f0e8bc: [stable33] 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 1
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 9
View all changes introduced in this branch ↗︎

@miaulalala miaulalala merged commit bdebd88 into stable33 Feb 19, 2026
55 checks passed
@miaulalala miaulalala deleted the backport/2319/stable33 branch February 19, 2026 14:21
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