Skip to content

[stable19] Fix share permission checkboxes enabled when permissions can not be set - #21574

Merged
MorrisJobke merged 5 commits into
stable19from
backport/21379/stable19
Jul 6, 2020
Merged

[stable19] Fix share permission checkboxes enabled when permissions can not be set#21574
MorrisJobke merged 5 commits into
stable19from
backport/21379/stable19

Conversation

@backportbot-nextcloud

@backportbot-nextcloudbackportbot-nextcloudBot commented Jun 24, 2020

Copy link
Copy Markdown

⚠️ This backport had conflicts and is incomplete ⚠️ (Conflicts happened only in the compiled files of last commit)

backport of #21379

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@danxuliu

Copy link
Copy Markdown
Member

/compile amend /

A sharee can reshare a file and set the edit, create, delete and share
permissions of the reshare only if the received share has edit, create,
delete and share permissions, or if they were revoked in the received
share after being set in the reshare. Therefore, the permission
checkboxes in the share menu should be enabled only if the user can set
them (otherwise trying to check them will lead to an error).
Note that "sharePermissions" has all the permissions if the file is not
a reshare but a file owned by the user.
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
@rullzerrullzer added the 3. to review Waiting for reviews label Jun 30, 2020
@rullzerrullzer mentioned this pull request Jul 6, 2020
3 tasks

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Code looks good 👍

@MorrisJobke
MorrisJobke merged commit 62f71a0 into stable19Jul 6, 2020
@MorrisJobke
MorrisJobke deleted the backport/21379/stable19 branch July 6, 2020 20:59
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@danxuliu@rullzer@MorrisJobke