Skip to content

[stable-33.0] fix(cfapi): avoid creation of db records if CfExecute failed #9914

Merged
nilsding merged 2 commits intostable-33.0from
backport/9912/stable-33.0
Apr 27, 2026
Merged

[stable-33.0] fix(cfapi): avoid creation of db records if CfExecute failed #9914
nilsding merged 2 commits intostable-33.0from
backport/9912/stable-33.0

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented Apr 27, 2026

Backport of PR #9912

when the call to CfExecute fails (e.g. due to "The cloud operation was
canceled by user.") consider all new placeholders as invalid.
VfsCfApi::finalizeNewPlaceholders won't create a new metadata record for
new, errored items which would cause random deletions in a future sync due
to the combination of {db: exists, local: absent, remote: exists}
=> (placeholder) item was apparently removed locally; even though only the
creation of the item failed ...

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
…in discovery

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
@nilsding nilsding enabled auto-merge April 27, 2026 09:38
@github-actions
Copy link
Copy Markdown

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

Digest: sha256:f0a5c06db19f0da4e06b6f4e5e5fb36f5c49572b1f49e784e7c7e216b0e25b95

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.

@nilsding nilsding merged commit b32b09d into stable-33.0 Apr 27, 2026
12 of 16 checks passed
@nilsding nilsding deleted the backport/9912/stable-33.0 branch April 27, 2026 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

1 participant