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

Fix transfering ownership of a share to user with same id as receiver #10348

Conversation

danxuliu
Copy link
Member

When the ownership of a user share is transfered to the receiver the share is removed, as the receiver now owns the original file. However, due to a missing condition, any share with a group, link or remote with the same id as the user was removed, not only the user shares.

@danxuliu danxuliu added this to the Nextcloud 14 milestone Jul 23, 2018
@danxuliu danxuliu requested review from rullzer and MorrisJobke July 23, 2018 08:21
Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch!

@rullzer rullzer requested a review from schiessle July 24, 2018 12:01
When the ownership of a user share is transfered to the receiver the
share is removed, as the receiver now owns the original file. However,
due to a missing condition, any share with a group, link or remote with
the same id as the user was removed, not only the user shares.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@MorrisJobke MorrisJobke force-pushed the fix-transfering-ownership-of-a-share-to-user-with-same-id-as-receiver branch from ca404fe to b7474ed Compare July 24, 2018 13:05
@MorrisJobke
Copy link
Member

Rebased to trigger CI

@MorrisJobke MorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jul 24, 2018
@MorrisJobke MorrisJobke merged commit d67e18b into master Jul 24, 2018
@MorrisJobke MorrisJobke deleted the fix-transfering-ownership-of-a-share-to-user-with-same-id-as-receiver branch July 24, 2018 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants