Merge pull request #922 from unoplatform/dev/dr/bindListNullRef

fix: Fix possible null in Bindable init
This commit is contained in:
David 2022-11-16 18:18:32 -05:00 коммит произвёл GitHub
Родитель d056527959 96d296459d
Коммит 2d0a5e9d36
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
1 изменённых файлов: 3 добавлений и 1 удалений

Просмотреть файл

@ -41,7 +41,9 @@ public class BindableImmutableList<TItem, TBindableItem> : BindableEnumerable<II
/// <inheritdoc />
private protected override CollectionChangeSet<TItem> GetChanges(IImmutableList<TItem> previous, IImmutableList<TItem> current)
=> _analyzer.GetChanges(previous ?? ImmutableList<TItem>.Empty, current);
// '_analyzer' might be null when the base.ctor subscribe to the 'property' and invokes the 'OnOwnerUpdated'
// We can safely fallback on ListFeed<TItem>.DefaultAnalyzer as in that case the 'previous' will be null/empty anyway.
=> (_analyzer ?? ListFeed<TItem>.DefaultAnalyzer).GetChanges(previous ?? ImmutableList<TItem>.Empty, current);
private protected override IImmutableList<TItem> Replace(IImmutableList<TItem>? items, TItem oldItem, TItem newItem)
{