Skip to content

Conversation

@jotoeri
Copy link
Member

@jotoeri jotoeri commented Dec 14, 2021

Simple fix of a quite old bug i stumbled upon today. The Upload-options for folders are shown only if the user has permissions to do so. If permissions are not given, until now it fell back to the option for files instead of showing nothing.

Fixes #27962

Corresponding v-if:

<template v-if="isFolder && fileHasCreatePermission && config.isPublicUploadEnabled">

Showing the bug: files option on folder
grafik

Do not show the Upload-Option if it is not allowed to check.

Signed-off-by: Jonas Rittershofer <jotoeri@users.noreply.github.com>
@jotoeri jotoeri added 3. to review Waiting for reviews papercut Annoying recurring UX issue with possibly simple fix. labels Dec 14, 2021
@jotoeri
Copy link
Member Author

jotoeri commented Dec 14, 2021

/compile amend/

@jotoeri
Copy link
Member Author

jotoeri commented Dec 14, 2021

/backport to stable23

@jotoeri
Copy link
Member Author

jotoeri commented Dec 14, 2021

/backport to stable22

@jotoeri
Copy link
Member Author

jotoeri commented Dec 14, 2021

/backport to stable21

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

@artonge artonge merged commit 069fe1c into master Dec 14, 2021
@artonge artonge deleted the fix/allow_edit_folder branch December 14, 2021 12:06
@backportbot-nextcloud
Copy link

The backport to stable23 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

The backport to stable22 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

The backport to stable21 failed. Please do this backport manually.

@jotoeri
Copy link
Member Author

jotoeri commented Dec 14, 2021

The backport to stable21 failed. Please do this backport manually.

Somebody knows, why this happens? 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews papercut Annoying recurring UX issue with possibly simple fix.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow upload and editing on sharing results in read only

6 participants