Skip to content

Fixed: Run search inline to prevent Kodi race condition#1941

Merged
basrieter merged 1 commit intoretrospect-addon:masterfrom
oliv3r:search_race
Mar 7, 2026
Merged

Fixed: Run search inline to prevent Kodi race condition#1941
basrieter merged 1 commit intoretrospect-addon:masterfrom
oliv3r:search_race

Conversation

@oliv3r
Copy link
Contributor

@oliv3r oliv3r commented Mar 5, 2026

Container.Update triggers a separate plugin instance that races with endOfDirectory. Slow API responses cause Kodi to render the stale search-history page instead of the actual results.

Run the search directly in the keyboard branch so results are rendered in the same plugin instance.

Known issue: empty needle is passed through, so refreshing search results triggers the keyboard pop-up instead of re-running the query.

Container.Update triggers a separate plugin instance that races with
endOfDirectory. Slow API responses cause Kodi to render the stale
search-history page instead of the actual results.

Run the search directly in the keyboard branch so results are rendered
in the same plugin instance.

Known issue: empty needle is passed through, so refreshing search
results triggers the keyboard pop-up instead of re-running the query.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 7, 2026

@basrieter
Copy link
Collaborator

There was something with searching and caching within Retrospect. But let's just try this.

@basrieter basrieter merged commit 9c653f6 into retrospect-addon:master Mar 7, 2026
5 of 9 checks passed
basrieter added a commit that referenced this pull request Mar 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants