From 9379e60fc0ee268fb6c99ae70e58028439822b61 Mon Sep 17 00:00:00 2001
From: Yusuf <Joseph@tinychan.org>
Date: Thu, 2 May 2024 11:11:11 +0300
Subject: [PATCH] fix: sort tv shows by date played (#741)

* Fix wrong "Date Played" sort for TV shows

* fix: change sortby dateplayed to seriesdateplayed right before retrieving items

---------

Co-authored-by: Jarne Demeulemeester <jarnedemeulemeester@gmail.com>
---
 .../java/dev/jdtech/jellyfin/viewmodels/LibraryViewModel.kt     | 2 +-
 data/src/main/java/dev/jdtech/jellyfin/models/SortBy.kt         | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/core/src/main/java/dev/jdtech/jellyfin/viewmodels/LibraryViewModel.kt b/core/src/main/java/dev/jdtech/jellyfin/viewmodels/LibraryViewModel.kt
index b44950f8..5ec4e2f8 100644
--- a/core/src/main/java/dev/jdtech/jellyfin/viewmodels/LibraryViewModel.kt
+++ b/core/src/main/java/dev/jdtech/jellyfin/viewmodels/LibraryViewModel.kt
@@ -61,7 +61,7 @@ constructor(
                     parentId = parentId,
                     includeTypes = itemType,
                     recursive = recursive,
-                    sortBy = sortBy,
+                    sortBy = if (libraryType == CollectionType.TvShows && sortBy == SortBy.DATE_PLAYED) SortBy.SERIES_DATE_PLAYED else sortBy, // Jellyfin uses a different enum for sorting series by data played
                     sortOrder = sortOrder,
                 ).cachedIn(viewModelScope)
                 _uiState.emit(UiState.Normal(items))
diff --git a/data/src/main/java/dev/jdtech/jellyfin/models/SortBy.kt b/data/src/main/java/dev/jdtech/jellyfin/models/SortBy.kt
index 5cedc8d1..4d2f2161 100644
--- a/data/src/main/java/dev/jdtech/jellyfin/models/SortBy.kt
+++ b/data/src/main/java/dev/jdtech/jellyfin/models/SortBy.kt
@@ -7,6 +7,7 @@ enum class SortBy(val sortString: String) {
     DATE_ADDED("DateCreated"),
     DATE_PLAYED("DatePlayed"),
     RELEASE_DATE("PremiereDate"),
+    SERIES_DATE_PLAYED("SeriesDatePlayed"),
     ;
 
     companion object {