Skip to content

Conversation

@provokateurin
Copy link
Member

@provokateurin provokateurin commented Apr 28, 2025

Summary

Shares were only filtered if the current user is the owner, but in a reshare they are not the owner.

Checklist

@provokateurin provokateurin added bug 3. to review Waiting for reviews labels Apr 28, 2025
@provokateurin provokateurin added this to the Nextcloud 32 milestone Apr 28, 2025
@provokateurin provokateurin requested a review from a team as a code owner April 28, 2025 12:43
@provokateurin provokateurin requested review from Altahrim, skjnldsv and yemkareems and removed request for a team April 28, 2025 12:43
@provokateurin
Copy link
Member Author

/backport to stable31

@provokateurin
Copy link
Member Author

/backport to stable30

@provokateurin provokateurin requested review from a team and come-nc May 5, 2025 13:48
This was referenced May 6, 2025
@susnux
Copy link
Contributor

susnux commented May 6, 2025

(tests would be nice to test the difference between before and after)

@provokateurin provokateurin force-pushed the fix/files_sharing/filter-own-reshared-shares branch from 2b402dc to 9269a8b Compare May 6, 2025 11:25
@provokateurin provokateurin requested a review from come-nc May 6, 2025 11:26
@blizzz
Copy link
Member

blizzz commented May 6, 2025

The same tests fails three times in a row:

Screenshot_20250506_174147

Not sure whether related, but could be.

@provokateurin
Copy link
Member Author

It's related, I'm working on it.

Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin force-pushed the fix/files_sharing/filter-own-reshared-shares branch from 9269a8b to 7c6602c Compare May 19, 2025 06:18
@provokateurin provokateurin enabled auto-merge May 19, 2025 06:18
@provokateurin provokateurin merged commit 9fa0477 into master May 19, 2025
208 of 216 checks passed
@provokateurin provokateurin deleted the fix/files_sharing/filter-own-reshared-shares branch May 19, 2025 08:20
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 bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Reshare with (own) group ends up as double share for sharee

7 participants