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.
MIT License
903 stars 101 forks source link

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

Closed ryo1988 closed 1 year ago

ryo1988 commented 1 year ago

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;

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

runceel commented 1 year ago

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

runceel commented 1 year ago

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

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

ryo1988 commented 1 year ago

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

runceel commented 1 year ago

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