Skip to content

Hide newFile menu if quota is set to 0B - #15767

Merged
MorrisJobke merged 1 commit into
masterfrom
bugfix/0b-quota-create
May 28, 2019
Merged

Hide newFile menu if quota is set to 0B#15767
MorrisJobke merged 1 commit into
masterfrom
bugfix/0b-quota-create

Conversation

@juliusknorr

Copy link
Copy Markdown
Member

Hide the add new file button if 0B quota is configured for a user.

The button will be shown again, once the user enters a folder that has a different quota on the mount, e.g. inside of shared folders with write permissions.

image

Signed-off-by: Julius Härtl <jus@bitgrid.net>
@ChristophWurst

Copy link
Copy Markdown
Member

Isn't the text misleading? Not enough quote is not the same as no permissions. You could still have permission to create, but the quote doesn't allow it.

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

Isn't the text misleading? Not enough quote is not the same as no permissions. You could still have permission to create, but the quote doesn't allow it.

Yes, but on the other hand the 0B quota workaround is always used to block users from uploading files to their home, so the permission-wording made sense from my perspective. I tried to keep the changes as minimal as possible so we can backport this easier.

@ChristophWurstChristophWurst 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.

Looks good!

@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.

Tested and works 👍

@MorrisJobke
MorrisJobke merged commit 64fb083 into masterMay 28, 2019
@MorrisJobke
MorrisJobke deleted the bugfix/0b-quota-create branch May 28, 2019 12:27
@juliusknorr

Copy link
Copy Markdown
MemberAuthor

/backport to stable16

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

/backport to stable15

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable15 in #15855

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

/backport to stable16

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable16 in #15856

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

@juliusknorr@ChristophWurst@MorrisJobke