close
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 <matthieu.gallien@nextcloud.com>
@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
BERJAYA 3 New Code Smells (required ≤ 0)
BERJAYA D Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

BERJAYA Catch issues before they fail your Quality Gate with our IDE extension BERJAYA 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