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

Bugfix/remove read only folders in read only folders #7020

Merged
merged 2 commits into from
Aug 23, 2024

Conversation

mgallien
Copy link
Collaborator

No description provided.

@mgallien
Copy link
Collaborator Author

/backport! to stable-3.13

sometime we need to create a folder on server because some local files
or folders modifications would be lost without that

in that case instead of locally deleting teh folder, we create it back
on server

when doing that, it is important to check the remote permissions such
that we do not try to create a folder in a read-only folder on server

Signed-off-by: Matthieu Gallien <[email protected]>
let unexpected new folders in read-only folders be read-write

would be done for folders that are inside a read-only folder and cannot
be uploaded because the parent folder is realy read-only

most probably those folders should never have been there

in case they would be read-only, users cannot delete them, in such
situation, letting them be read-write again should really help

Signed-off-by: Matthieu Gallien <[email protected]>
@mgallien mgallien force-pushed the bugfix/removeReadOnlyFoldersInReadOnlyFolders branch from ba8ab0a to 3e397e6 Compare August 23, 2024 07:56
@mgallien mgallien merged commit 82c9a52 into master Aug 23, 2024
11 of 14 checks passed
@mgallien mgallien deleted the bugfix/removeReadOnlyFoldersInReadOnlyFolders branch August 23, 2024 07:56
@nextcloud-desktop-bot
Copy link

AppImage file: nextcloud-PR-7020-3e397e664db079bc531037aba30aeb09337755b8-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

Copy link

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

Successfully merging this pull request may close these issues.

3 participants