Browse Source

Rename select-all helper and avoid shift tracking

pull/2738/head
Jon Crall 7 months ago
parent
commit
da8d52cf8f
  1. 4
      ErsatzTV.Core.Tests/Pages/MediaCardSelectionHelperTests.cs
  2. 5
      ErsatzTV/Pages/MultiSelectBase.cs

4
ErsatzTV.Core.Tests/Pages/MediaCardSelectionHelperTests.cs

@ -18,7 +18,7 @@ public class MediaCardSelectionHelperTests
var first = new MediaCardViewModel(2, "First", "Sub", "First", "", Core.Domain.MediaItemState.Normal, false); var first = new MediaCardViewModel(2, "First", "Sub", "First", "", Core.Domain.MediaItemState.Normal, false);
var second = new MediaCardViewModel(3, "Second", "Sub", "Second", "", Core.Domain.MediaItemState.Normal, false); var second = new MediaCardViewModel(3, "Second", "Sub", "Second", "", Core.Domain.MediaItemState.Normal, false);
MediaCardViewModel last = MultiSelectBase<Search>.SelectAllPageItems(selected, new[] { first, second }); MediaCardViewModel last = MultiSelectBase<Search>.ResetSelectionWithCards(selected, new[] { first, second });
selected.ShouldBe(new[] { first, second }, ignoreOrder: true); selected.ShouldBe(new[] { first, second }, ignoreOrder: true);
last.ShouldBe(second); last.ShouldBe(second);
@ -30,7 +30,7 @@ public class MediaCardSelectionHelperTests
var existingCard = new MediaCardViewModel(1, "Existing", "Sub", "Existing", "", Core.Domain.MediaItemState.Normal, false); var existingCard = new MediaCardViewModel(1, "Existing", "Sub", "Existing", "", Core.Domain.MediaItemState.Normal, false);
var selected = new HashSet<MediaCardViewModel> { existingCard }; var selected = new HashSet<MediaCardViewModel> { existingCard };
MediaCardViewModel last = MultiSelectBase<Search>.SelectAllPageItems(selected, []); MediaCardViewModel last = MultiSelectBase<Search>.ResetSelectionWithCards(selected, []);
selected.ShouldBeEmpty(); selected.ShouldBeEmpty();
last.ShouldBeNull(); last.ShouldBeNull();

5
ErsatzTV/Pages/MultiSelectBase.cs

@ -42,8 +42,7 @@ public class MultiSelectBase<T> : FragmentNavigationBase
protected void SelectAllPageItems(IEnumerable<MediaCardViewModel> cards) protected void SelectAllPageItems(IEnumerable<MediaCardViewModel> cards)
{ {
MediaCardViewModel lastSelected = SelectAllPageItems(SelectedItems, cards); ResetSelectionWithCards(SelectedItems, cards);
_recentlySelected = Optional(lastSelected);
StateHasChanged(); StateHasChanged();
} }
@ -249,7 +248,7 @@ public class MultiSelectBase<T> : FragmentNavigationBase
} }
} }
internal static MediaCardViewModel SelectAllPageItems( internal static MediaCardViewModel ResetSelectionWithCards(
ISet<MediaCardViewModel> selectedItems, ISet<MediaCardViewModel> selectedItems,
IEnumerable<MediaCardViewModel> cards) IEnumerable<MediaCardViewModel> cards)
{ {

Loading…
Cancel
Save