Skip to content

fix(read-only): make the parent folder writable before deleting a file - #9655

Merged
mgallien merged 1 commit into
masterfrom
bugfix/resolveConflictsReadOnlyFolders
Mar 20, 2026
Merged

fix(read-only): make the parent folder writable before deleting a file#9655
mgallien merged 1 commit into
masterfrom
bugfix/resolveConflictsReadOnlyFolders

Conversation

@mgallien

Copy link
Copy Markdown
Collaborator

to solve a conflict, we may need to delete a file inside a read-only folder

we need to ensure that temporary the parent folder can be modified

to solve a conflict, we may need to delete a file inside a read-only
folder

we need to ensure that temporary the parent folder can be modified

Signed-off-by: Matthieu Gallien <[email protected]>
@mgallien mgallien added this to the 34.0.0 milestone Mar 20, 2026
@github-actions

Copy link
Copy Markdown
Contributor

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

Digest: sha256:03b9b062a4872c15886dd6e936cc2d15bed7d2387a13ffb09ecfd26069d28eae

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.

@mgallien
mgallien enabled auto-merge March 20, 2026 09:29
@mgallien

Copy link
Copy Markdown
Collaborator Author

/backport to stable-4.0

@mgallien

Copy link
Copy Markdown
Collaborator Author

/backport to stable-33.0

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
3 New Code Smells (required ≤ 0)
D Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@mgallien
mgallien merged commit 715ba36 into master Mar 20, 2026
22 of 24 checks passed
@mgallien
mgallien deleted the bugfix/resolveConflictsReadOnlyFolders branch March 20, 2026 10:18
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.

2 participants