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

[oC-Web] sharing broken in shared with others view #990

Closed
exalate-issue-sync bot opened this issue Nov 30, 2020 · 3 comments
Closed

[oC-Web] sharing broken in shared with others view #990

exalate-issue-sync bot opened this issue Nov 30, 2020 · 3 comments
Labels

Comments

@exalate-issue-sync
Copy link

exalate-issue-sync bot commented Nov 30, 2020

When opening the right sidebar for a resource in the shared with others view, both the People and the Links accordion items are "broken", as in:

  1. People shows a No permission to share this folder text at the same location where the button for adding people usually is. As this is the shared with others page, I'm clearly the owner of the resource, so I should be allowed to share it from here.
    2) Links shows the button for creating a new public link, but upon submitting the form there is a resource not found error message.

This happens on ocis only.

@exalate-issue-sync
Copy link
Author

Lukas Hirt commented: 1. Is because we are not checking for share permissions of the owner but of the user with whom the file is shared. So if we share a file with Marie and give her role Viewer, the webUI assumes that we have also only view permission on the file in "Shared with others" list. When we receive username instead of UUID as the shareowner, we can compare it with the currently logged in user and enable sharing.

  1. I didn't manage to reproduce this. Creating the link works just fine for me there. 🤷

@exalate-issue-sync
Copy link
Author

Benedikt Kulmann commented: Crossed out the second bug. Apparently that was happening on a share that was not removed upon folder deletion.

C0rby pushed a commit to owncloud/reva that referenced this issue Dec 9, 2020
A while ago we decided to use the users uuid only internally and to only display or return the users username. The ocs share api did still return the uuid.

Closes owncloud/ocis#990
C0rby pushed a commit to owncloud/reva that referenced this issue Dec 9, 2020
A while ago we decided to use the users uuid only internally and to only display or return the users username. The ocs share api did still return the uuid.

Closes owncloud/ocis#990
C0rby pushed a commit to owncloud/reva that referenced this issue Dec 9, 2020
A while ago we decided to use the users uuid only internally and to only display or return the users username. The ocs share api did still return the uuid.

Closes owncloud/ocis#990
C0rby pushed a commit to owncloud/reva that referenced this issue Dec 9, 2020
A while ago we decided to use the users uuid only internally and to only display or return the users username. The ocs share api did still return the uuid.

Closes owncloud/ocis#990
C0rby pushed a commit to owncloud/reva that referenced this issue Dec 9, 2020
A while ago we decided to use the users uuid only internally and to only display or return the users username. The ocs share api did still return the uuid.

Closes owncloud/ocis#990
@exalate-issue-sync
Copy link
Author

Benedikt Kulmann commented: Implemented in reva, needs to be merged and then updated in ocis.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

0 participants