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

Missing list notifications for updates to object in a list of mixed. #7264

Closed
rorbech opened this issue Jan 18, 2024 · 1 comment
Closed
Assignees

Comments

@rorbech
Copy link
Contributor

rorbech commented Jan 18, 2024

Expected results

When listening to changes for a list I'd expect to receive notifications when an object is updated, no matter if it is a normal list of objects or if it is in a list of mixed.

Actual Results

These Kotlin tests registers for list notifications; one showing the behavior for list of objects and one for list of mixed.

For list of objects we receive notifications of both the addition to the list but also for updates to the object in the list.

For list of mixed we only receive notification for the addition to the list, but not for updates to the object in the list.

Steps & Code to Reproduce

See https://github.com/realm/realm-kotlin/pull/1635/files

Core version

Core version: v13.24.0 and v14.0.0-beta.0

@jedelbo
Copy link
Contributor

jedelbo commented Jan 22, 2024

It would be rather costly to check for updates on objects linked through typed links (those that are used in Mixed). The reason is that that would mean that we should check every object mutation to see if the object is actually linked from an object/collection of interest. I think we should say that objects linked by typed links cannot be seen as part of the linking object and that you consequently will not receive notifications when mutated. So I propose not to fix this.

@sync-by-unito sync-by-unito bot closed this as completed Feb 6, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants