From f99da9d898f0a711e0bfcca69d9f64eaef092b28 Mon Sep 17 00:00:00 2001 From: Jon Crall Date: Tue, 30 Dec 2025 16:08:12 -0500 Subject: [PATCH] Tighten select-all helper semantics --- ErsatzTV/Pages/MediaCardSelectionHelper.cs | 12 ++++++++---- ErsatzTV/Pages/MultiSelectBase.cs | 2 -- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/ErsatzTV/Pages/MediaCardSelectionHelper.cs b/ErsatzTV/Pages/MediaCardSelectionHelper.cs index d94b11db0..68e930102 100644 --- a/ErsatzTV/Pages/MediaCardSelectionHelper.cs +++ b/ErsatzTV/Pages/MediaCardSelectionHelper.cs @@ -10,11 +10,15 @@ public static class MediaCardSelectionHelper ISet selectedItems, IEnumerable cards) { - List cardList = (cards ?? Enumerable.Empty()).ToList(); - selectedItems.Clear(); - selectedItems.UnionWith(cardList); - return cardList.LastOrDefault(); + MediaCardViewModel last = default; + foreach (MediaCardViewModel card in cards ?? Enumerable.Empty()) + { + last = card; + selectedItems.Add(card); + } + + return last; } } diff --git a/ErsatzTV/Pages/MultiSelectBase.cs b/ErsatzTV/Pages/MultiSelectBase.cs index 42fdb54f5..75f1ffe26 100644 --- a/ErsatzTV/Pages/MultiSelectBase.cs +++ b/ErsatzTV/Pages/MultiSelectBase.cs @@ -44,8 +44,6 @@ public class MultiSelectBase : FragmentNavigationBase { MediaCardViewModel lastSelected = MediaCardSelectionHelper.SelectAllPageItems(SelectedItems, cards); _recentlySelected = Optional(lastSelected); - - StateHasChanged(); } protected void ClearSelection()