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.

Sign up for GitHub

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

Jump to bottom

[Bug] MAUI calls setter unnecessarily with CollectionView #3128

Closed
papafe opened this issue Dec 12, 2022 · 0 comments · Fixed by #3129
Closed

[Bug] MAUI calls setter unnecessarily with CollectionView #3128

papafe opened this issue Dec 12, 2022 · 0 comments · Fixed by #3129
Assignees
@papafe

Comments

@papafe
Copy link
Contributor

papafe commented Dec 12, 2022

MAUI is calling the setters of the properties unnecessarily when CollectionView/ListView is shown to screen. Even though they call the setter with the same value, this is a problem if it’s done on a RealmObject for which the current user doesn’t have permissions (compensating writes errors coming through). On iOS this also causes an error loop in some cases, probably due to cell recycling.
We need to modify WovenSetterMethodInfo to avoid calling the setter in case the new value is equal to the current value.

Reported in dotnet/maui#12001
@papafe papafe self-assigned this Dec 12, 2022
This was referenced Dec 12, 2022
Merged
Merged
MongoCaleb added a commit to mongodb-university/realm-template-apps that referenced this issue Dec 20, 2022
@papafe @cbush @MongoCaleb
MAUI template app (#117)
7c27387 
The code is ready for review, but there is a strange behaviour with sync
due to realm/realm-dotnet#3128. This can cause
some error loops on iOS in some specific situations

Co-authored-by: Ferdinando Papale <>
Co-authored-by: Chris Bush <chris.bush@10gen.com>
Co-authored-by: MongoCaleb <32645888+MongoCaleb@users.noreply.github.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@papafe papafe

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Avoid calling setter for bound properties for same value realm/realm-dotnet
1 participant
@papafe