Skip to content

Fixed that .Edit() operations would not defer notifications for new subscriptions occurring during the edit.#1133

Open
JakenVeina wants to merge 1 commit into
mainfrom
issues/1129
Open

Fixed that .Edit() operations would not defer notifications for new subscriptions occurring during the edit.#1133
JakenVeina wants to merge 1 commit into
mainfrom
issues/1129

Conversation

@JakenVeina

Copy link
Copy Markdown
Collaborator

Fixed that subscriptions occurring during an .Edit() operation upon a SourceCache<> or SourceList<> would still publish an initial notification to the subscriber. This resulted in the potential for notifications to be duplicated, for items added or manipulated during the edit.

Instead, such subscriptions now have their notifications deferred until the .Edit() is complete.

Resolves #1129.

… a `SourceCache<>` or `SourceList<>` would still publish an initial notification to the subscriber. This resulted in the potential for notifications to be duplicated, for items added or manipulated during the edit.

Instead, such subscriptions now have their notifications deferred until the `.Edit()` is complete.

Resolves #1129.
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.

[Bug]: ArgumentException: An item with the same key was already added with Filter on 9.4.31+

1 participant