mirror of
https://github.com/ppy/osu
synced 2025-02-18 19:36:58 +00:00
Fix back-to-front filter logic
Was copied across from a place which was checking for `match` and applied verbatim to a place that was `filter`. Which are polar opposites.
This commit is contained in:
parent
148e487c02
commit
1524623624
@ -129,12 +129,13 @@ namespace osu.Game.Screens.Select.Carousel
|
|||||||
public override void Filter(FilterCriteria criteria)
|
public override void Filter(FilterCriteria criteria)
|
||||||
{
|
{
|
||||||
base.Filter(criteria);
|
base.Filter(criteria);
|
||||||
bool match = Items.All(i => i.Filtered.Value);
|
|
||||||
|
|
||||||
match &= criteria.Sort != SortMode.DateRanked || BeatmapSet?.DateRanked != null;
|
bool filtered = Items.All(i => i.Filtered.Value);
|
||||||
match &= criteria.Sort != SortMode.DateSubmitted || BeatmapSet?.DateSubmitted != null;
|
|
||||||
|
|
||||||
Filtered.Value = match;
|
filtered |= criteria.Sort == SortMode.DateRanked && BeatmapSet?.DateRanked == null;
|
||||||
|
filtered |= criteria.Sort == SortMode.DateSubmitted && BeatmapSet?.DateSubmitted == null;
|
||||||
|
|
||||||
|
Filtered.Value = filtered;
|
||||||
}
|
}
|
||||||
|
|
||||||
public override string ToString() => BeatmapSet.ToString();
|
public override string ToString() => BeatmapSet.ToString();
|
||||||
|
Loading…
Reference in New Issue
Block a user