Skip to content
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.

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

Transfer not possible if a reshare to a group/room exists where the user is no member #22803

Closed
juliushaertl opened this issue Sep 11, 2020 · 1 comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug needs info

Comments

@juliushaertl
Copy link
Member

From #22761

While doing that I have noticed that currently it is only possible to transfer ownership of a reshare if it is a reshare with a group (or room) and the reshare is transferred to a user in the group (or room).

When transferring ownership of a reshare with another user or with a group to a user not in the group restoring the share fails (but the command succeeds, it only fails for the specific files that are reshares).

When transferring ownership of a path that is a reshare the command fails (as when a specific path is provided the path tries to move the file, it does not take into account reshares).

Note that the above is not related to #22648 (that pull request fixed the transferring ownership of folder reshared with group to a user in the group scenario). The failure when restoring a reshare is caused by getMountPoint() being called on a null object. However, even before #21489 it was failing too (in that case it complained that it could not increase permissions).

@rullzer @juliushaertl Feel free to either work on the issues or change the pull request state to To review if not being able to transfer reshares is not an issue (or if you prefer to merge this now and work on them later ;-) ).

@juliushaertl juliushaertl added bug 1. to develop Accepted and waiting to be taken care of labels Sep 11, 2020
@juliushaertl juliushaertl self-assigned this Sep 11, 2020
@juliushaertl juliushaertl removed their assignment Jul 26, 2022
@szaimen
Copy link
Contributor

szaimen commented Jan 23, 2023

Hi, please update to 24.0.9 or better 25.0.3 and report back if it fixes the issue. Thank you!

My goal is to add a label like e.g. 25-feedback to this ticket of an up-to-date major Nextcloud version where the bug could be reproduced. However this is not going to work without your help. So thanks for all your effort!

If you don't manage to reproduce the issue in time and the issue gets closed but you can reproduce the issue afterwards, feel free to create a new bug report with up-to-date information by following this link: https://github.com/nextcloud/server/issues/new?assignees=&labels=bug%2C0.+Needs+triage&template=BUG_REPORT.yml&title=%5BBug%5D%3A+

@szaimen szaimen added needs info 0. Needs triage Pending check for reproducibility or if it fits our roadmap and removed 1. to develop Accepted and waiting to be taken care of labels Jan 23, 2023
@szaimen szaimen closed this as completed Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug needs info
Projects
None yet
Development

No branches or pull requests

2 participants