Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolution for #1946 was incorrect. This commit finally solves it. #2096

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

dfahlander
Copy link
Collaborator

@dfahlander dfahlander commented Nov 15, 2024

Bugfix: Resolution for #1946 was incorrect. This commit should solve it for good.

The problem: When using a compound index where one of the properties is the primary key and the table uses auto-incremented primary keys, live queries on that compund index aren't properly notified when new items are added that affects the query.

The previous fix had the correct thought but a bug in the implementation. The unit tests happened to succeed despite of this (which was pure luck). It added primary key number instead of the array containing the primary key number and the other parts of the compound index.

@dfahlander dfahlander merged commit 527ac54 into master Nov 15, 2024
5 checks passed
@dfahlander dfahlander deleted the issue1946-II branch November 15, 2024 15:52
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.

1 participant