Skip to content

[stable-3.16] Bugfix/do not crash when propagating changes with move or rename - #8199

Merged
mgallien merged 5 commits into
stable-3.16from
backport/8193/stable-3.16
Apr 24, 2025
Merged

[stable-3.16] Bugfix/do not crash when propagating changes with move or rename#8199
mgallien merged 5 commits into
stable-3.16from
backport/8193/stable-3.16

Conversation

@backportbot

@backportbot backportbot Bot commented Apr 24, 2025

Copy link
Copy Markdown

Backport of PR #8193

@backportbot backportbot Bot added this to the 3.16.4 milestone Apr 24, 2025
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
add original name
add renamed name
add target name

should make it easier to analyze rename operations

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
ensure we do not leak records and properly update them in client
database

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
when propagating a rename to the child items, we must not make child
items be done from propagator point of view

they will be part of their own propagator item and that will create
issues

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@mgallien
mgallien force-pushed the backport/8193/stable-3.16 branch from f65a2d2 to 2286255 Compare April 24, 2025 09:51
@mgallien
mgallien enabled auto-merge April 24, 2025 09:52
@github-actions

Copy link
Copy Markdown
Contributor

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

SHA256 checksum: 2b60f1326c484cd0b17c9d02eb16e4c116040aec857eef96ff9f0000b8a4e60e

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 merged commit 2ac7a9d into stable-3.16 Apr 24, 2025
@mgallien
mgallien deleted the backport/8193/stable-3.16 branch April 24, 2025 10:10
@sonarqubecloud

Copy link
Copy Markdown

Please retry analysis of this Pull-Request directly on SonarQube Cloud

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