Skip to content

fix(sync): apply the search filter as a restriction - #13683

Open
kesselb wants to merge 1 commit into
mainfrom
bug/13673/apply-filter
Open

kesselb wants to merge 1 commit into
mainfrom
bug/13673/apply-filter

Conversation

@kesselb

@kesselb kesselb commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Fix #13673

SyncService filtered its candidates with a parameter that widens the search instead of narrowing it. With a subject term in the query the filter stopped restricting, so a search returned every thread newer than the oldest match. Add a separate parameter that always narrows.

Assisted-by: ClaudeCode:claude-opus-5

馃 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (N/A)

@nextcloud-command nextcloud-command added the AI assisted This PR contains AI-assisted commits label Sep 15, 2026
SyncService filtered its candidates with a parameter that widens the
search instead of narrowing it. With a subject term in the query the
filter stopped restricting, so a search returned every thread newer
than the oldest match. Add a separate parameter that always narrows.

Assisted-by: ClaudeCode:claude-opus-5

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb
kesselb force-pushed the bug/13673/apply-filter branch from d33fc21 to e420ce0 Compare September 21, 2026 10:33
@kesselb kesselb self-assigned this Sep 21, 2026
@kesselb
kesselb requested a review from madbob September 21, 2026 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing AI assisted This PR contains AI-assisted commits

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Search in a large mailbox freezes the browser: sync ignores the filter when the query has a subject term

2 participants