Fixed that .Bind() operators for ISortedChangeSet<TObject, TKey> streams were not properly recognizing the ResetOnFirstTimeLoad option. The option was only being used if the initial changeset also exceeded the ResetThreshold setting. (reactivemarbles#935)

JakenVeina · web-flow · commit b6e851e28888 · 2024-09-03T01:46:24.000-05:00
diff --git a/src/DynamicData/Binding/SortedObservableCollectionAdaptor.cs b/src/DynamicData/Binding/SortedObservableCollectionAdaptor.cs
@@ -68,7 +68,7 @@ public void Adapt(ISortedChangeSet<TObject, TKey> changes, IObservableCollection
                 break;
 
             case SortReason.InitialLoad:
-                if (resetOnFirstTimeLoad && (changes.Count - changes.Refreshes > refreshThreshold))
+                if (resetOnFirstTimeLoad || (changes.Count - changes.Refreshes > refreshThreshold))
                 {
                     using (collection.SuspendNotifications())
                     {

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ public void Adapt(ISortedChangeSet<TObject, TKey> changes, IObservableCollection`
`68`	`68`	`break;`
`69`	`69`
`70`	`70`	`case SortReason.InitialLoad:`
`71`		`- if (resetOnFirstTimeLoad && (changes.Count - changes.Refreshes > refreshThreshold))`
	`71`	`+ if (resetOnFirstTimeLoad \|\| (changes.Count - changes.Refreshes > refreshThreshold))`
`72`	`72`	`{`
`73`	`73`	`using (collection.SuspendNotifications())`
`74`	`74`	`{`