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

share dialog checkboxes misbehave #12250

Closed
schiessle opened this issue Nov 4, 2018 · 6 comments
Closed

share dialog checkboxes misbehave #12250

schiessle opened this issue Nov 4, 2018 · 6 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug feature: sharing high

Comments

@schiessle
Copy link
Member

I just noticed that the checkboxes of the share dialog misbehave on master.

If I create a new group share this is how it looks (same happens for user shares):

image

"Can edit" and "Can reshare" is checked. Now I uncheck "Can reshare" and this is what happens:

image

"Can edit" is also unchecked! It is not just a front-end issue. If I reload the page, "Can edit" is still unckeched. So it seems we really remove all permission when the user just removes "can reshare"

If I remove "Can edit" instead the dialog behaves correctly and the "can reshare" permissions stays.

This only happens for file shares, for folders the behaviour seems to be correct. My assumption is that this is the case because for folders we have additional "edit permissions" in the '...' menu while for file shares we just have the reshare permission.

Maybe some of our @nextcloud/javascript experts can have a look.

cc @MariusBluem because we discovered and discussed it last week

@nextcloud-bot
Copy link
Member

GitMate.io thinks possibly related issues are #2679 (Share permissions misbehaving), #3918 (share dialog broken), #2658 (Sharing and permissions), #2368 (share dialog avatar flickers on permission change), and #1094 (Sharing dialog: Names sorting is case sensitive ).

@MariusBluem
Copy link
Member

Reproduced with Firefox and Safari on macOS yesterday - you were faster @schiessle 😅

@MorrisJobke
Copy link
Member

@skjnldsv @danxuliu @ChristophWurst Any idea?

@MorrisJobke
Copy link
Member

cc @nextcloud/javascript

@skjnldsv
Copy link
Member

No idea. How is the backend? Are we sure this is only a javascript issue?

@skjnldsv skjnldsv added the 0. Needs triage Pending check for reproducibility or if it fits our roadmap label Jun 12, 2019
@MorrisJobke MorrisJobke removed this from the Nextcloud 15.0.9 milestone Jul 3, 2019
@skjnldsv skjnldsv added this to Backlog in New sidebar standard via automation Aug 7, 2019
@skjnldsv skjnldsv moved this from Backlog to Review in progress in New sidebar standard Aug 7, 2019
@skjnldsv
Copy link
Member

Fixed with #15719

New sidebar standard automation moved this from Review in progress to Done Oct 29, 2019
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 feature: sharing high
Projects
No open projects
Development

No branches or pull requests

6 participants