runceel / ReactiveProperty

ReactiveProperty provides MVVM and asynchronous support features under Reactive Extensions. Target frameworks are .NET 6+, .NET Framework 4.7.2 and .NET Standard 2.0.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

The subscription to the Source.CollectionChanged event of FilteredReadOnlyObservableCollection is not being unsubscribed at the time of Dispose.

ryo1988 opened this issue · comments

English

https://github.com/runceel/ReactiveProperty/blob/a5590692978043ab3ab8e844e2294b25d4a08f50/Source/ReactiveProperty.NETStandard/Helpers/FilteredReadOnlyObservableCollection.cs#L351C55-L351C55

It is written as CollectionChanged -= Source_CollectionChanged;

However, it seems to need to be Source.CollectionChanged -= Source_CollectionChanged;

Japanese

https://github.com/runceel/ReactiveProperty/blob/a5590692978043ab3ab8e844e2294b25d4a08f50/Source/ReactiveProperty.NETStandard/Helpers/FilteredReadOnlyObservableCollection.cs#L351C55-L351C55

CollectionChanged -= Source_CollectionChanged;

となっていますが、

Source.CollectionChanged -= Source_CollectionChanged;

である必要がありそうです。

@ryo1988 ありがとうございます。マージしました。近いうちにリリースします。

プレビューパッケージとして公開しました。
特に問題なければ、このままリリース版として公開予定です。

https://www.nuget.org/packages/ReactiveProperty.WPF/9.3.4-pre202310290551

リリースありがとうございます。

@ryo1988
すいません。元記事に英文書き足しました。