Skip to content

[stable-3.16] Bugfix/read only windows folders - #8196

Merged
mgallien merged 6 commits into
stable-3.16from
backport/8182/stable-3.16
Apr 24, 2025
Merged

[stable-3.16] Bugfix/read only windows folders#8196
mgallien merged 6 commits into
stable-3.16from
backport/8182/stable-3.16

Conversation

@backportbot

@backportbot backportbot Bot commented Apr 24, 2025

Copy link
Copy Markdown

Backport of PR #8182

@backportbot
backportbot Bot requested review from mgallien and nilsding April 24, 2025 08:33
@backportbot backportbot Bot added this to the 3.16.4 milestone Apr 24, 2025
@mgallien
mgallien enabled auto-merge April 24, 2025 08:34
@mgallien
mgallien disabled auto-merge April 24, 2025 09:01
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
needed to download a new file inside a read-only folder

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
the sync engine will remove invalid items inside read-only folders

not needed to remove them in tests and rather checks that they were
indeed removed

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
current QDir::rmdir API does not provide an error message when failing
to delete

retuse FileSystem::remove that may just works with folders

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@mgallien
mgallien force-pushed the backport/8182/stable-3.16 branch from db09d12 to 710e5fa Compare April 24, 2025 09:01
@mgallien
mgallien merged commit 2b606e5 into stable-3.16 Apr 24, 2025
@mgallien
mgallien deleted the backport/8182/stable-3.16 branch April 24, 2025 09:01
@github-actions

Copy link
Copy Markdown
Contributor

Artifact containing the AppImage: nextcloud-appimage-pr-8196.zip

SHA256 checksum: 3472145e6799c0306391b26df878e1b9a756812732e2d0daf058c1d6028ba363

To test this change/fix you can download the above artifact file, unzip it, and run it.

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

@sonarqubecloud

Copy link
Copy Markdown

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant