From c6d81dada837c701ffc3746c4dafae8b0b3a7b92 Mon Sep 17 00:00:00 2001 From: Damontecres Date: Wed, 12 Aug 2026 14:04:19 -0400 Subject: [PATCH] Fix playlist & collection random sorting on home page (#1820) ## Description Ensure that a playlist or collection's items are random sorted on the home page, if applicable. Previously, the request did sort by random then by sort name resulting in only sorting by sort name. However, there are still issues with random sorting in general, see #1819 ### Related issues Technically fixes #1806 Further bugs in #1819 ### Testing Emulator ## Screenshots N/A ## AI or LLM usage None --- .../wholphin/services/HomeSettingsService.kt | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/app/src/main/java/com/github/damontecres/wholphin/services/HomeSettingsService.kt b/app/src/main/java/com/github/damontecres/wholphin/services/HomeSettingsService.kt index 0ca1e89e..a2ddae15 100644 --- a/app/src/main/java/com/github/damontecres/wholphin/services/HomeSettingsService.kt +++ b/app/src/main/java/com/github/damontecres/wholphin/services/HomeSettingsService.kt @@ -859,18 +859,28 @@ class HomeSettingsService sortBy = row.sort?.let { buildList { - add(it.sort) - if (it.sort != ItemSortBy.SORT_NAME) { + if (it.sort == ItemSortBy.RANDOM) { add(ItemSortBy.SORT_NAME) + add(ItemSortBy.RANDOM) + } else { + add(it.sort) + if (it.sort != ItemSortBy.SORT_NAME) { + add(ItemSortBy.SORT_NAME) + } } } }, sortOrder = row.sort?.let { buildList { - add(it.direction) - if (it.sort != ItemSortBy.SORT_NAME) { + if (it.sort == ItemSortBy.RANDOM) { add(SortOrder.ASCENDING) + add(it.direction) + } else { + add(it.direction) + if (it.sort != ItemSortBy.SORT_NAME) { + add(SortOrder.ASCENDING) + } } } },