Skip to content

[stable-34.0] fix(discovery): move file lock state off the GUI thread. - #10514

Merged
nilsding merged 2 commits into
stable-34.0from
backport/10500/stable-34.0
Jul 31, 2026
Merged

[stable-34.0] fix(discovery): move file lock state off the GUI thread.#10514
nilsding merged 2 commits into
stable-34.0from
backport/10500/stable-34.0

Conversation

@backportbot

@backportbot backportbot Bot commented Jul 31, 2026

Copy link
Copy Markdown

Backport of PR #10500

Fix for #10464.

isFileLocked opened files synchronously on the GUI thread during
discovery. A blocking open froze the UI and, through the shell
extension, Explorer too. Compute the lock state on the local
listing worker and carry it in LocalInfo, so handleExcluded
reads a precomputed flag. Skip symlinks, virtual placeholders
and directories.

Assisted-by: Claude Code:claude-opus-4-8
Signed-off-by: Camila Ayres <[email protected]>
@backportbot
backportbot Bot requested a review from camilasan as a code owner July 31, 2026 13:18
@backportbot
backportbot Bot requested a review from claucambra July 31, 2026 13:18
@backportbot
backportbot Bot requested review from i2h3 and mgallien as code owners July 31, 2026 13:18
@backportbot
backportbot Bot requested a review from nilsding July 31, 2026 13:18
@backportbot backportbot Bot added this to the 34.0.1 milestone Jul 31, 2026
@github-actions

Copy link
Copy Markdown
Contributor

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

Digest: sha256:e13b08455001169139dabc04b87e37fe5f5ec38f12ea02a3aab9e1623898fe54

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

@nilsding
nilsding merged commit f30bd9d into stable-34.0 Jul 31, 2026
21 checks passed
@nilsding
nilsding deleted the backport/10500/stable-34.0 branch July 31, 2026 14:33
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.

3 participants