reactivemarbles / DynamicData

Reactive collections based on Rx.Net
http://dynamic-data.org
MIT License
1.71k stars 182 forks source link

TransformMany of ReactiveList use add / remove rather than replace #70

Closed modplug closed 6 years ago

RolandPheasant commented 7 years ago

Although the PR appears to fix the issue, there was already an underlying issue with the way the operator handles Replaced items. It erroneously assumed that the original parent would contain the same items as the previous parent.

RolandPheasant commented 7 years ago

The original issue is now fixed. However I am unsure whether it will be possible deterministically apply a replace being as a parent replace does not necessarily mean that a child has been replace. @modplug I will leave this one for you to check out

RolandPheasant commented 6 years ago

This is an old one, transform many works. However dues to the nature of combining lists, replace is not possible