Remove unused code

mr3y-the-programmer · Oct 20, 2023 · d24691c · d24691c
1 parent a2f303e
commit d24691c
Show file tree

Hide file tree

Showing 5 changed files with 4 additions and 71 deletions.
diff --git a/shared/src/commonMain/kotlin/com/mr3y/ludi/shared/ui/presenter/DiscoverPagingFactory.kt b/shared/src/commonMain/kotlin/com/mr3y/ludi/shared/ui/presenter/DiscoverPagingFactory.kt
@@ -39,7 +39,7 @@ interface DiscoverPagingFactory {
 
     val soundtrackGamesPager: Pager<Int, Game>
 
-    fun createPagerForGamesWithQuery(searchQuery: String, filters: DiscoverFiltersState): Pager<Int, Game>
+    fun searchQueryBasedGamesPager(searchQuery: String, filters: DiscoverFiltersState): Pager<Int, Game>
 }
 
 @Inject
@@ -134,7 +134,7 @@ class DefaultDiscoverPagingFactory(
         )
     }
 
-    override fun createPagerForGamesWithQuery(
+    override fun searchQueryBasedGamesPager(
         searchQuery: String,
         filters: DiscoverFiltersState
     ): Pager<Int, Game> {

diff --git a/shared/src/commonMain/kotlin/com/mr3y/ludi/shared/ui/presenter/DiscoverViewModel.kt b/shared/src/commonMain/kotlin/com/mr3y/ludi/shared/ui/presenter/DiscoverViewModel.kt
@@ -91,7 +91,7 @@ class DiscoverViewModel(
             Initial.gamesState
         } else {
             DiscoverStateGames.SearchQueryBasedGames(
-                games = pagingFactory.createPagerForGamesWithQuery(searchText, filtersState).flow.cachedIn(coroutineScope)
+                games = searchQueryBasedGamesPager(searchText, filtersState).flow.cachedIn(coroutineScope)
             )
         }
     }.map {

diff --git a/shared/src/commonMain/kotlin/com/mr3y/ludi/shared/ui/presenter/model/DiscoverState.kt b/shared/src/commonMain/kotlin/com/mr3y/ludi/shared/ui/presenter/model/DiscoverState.kt
@@ -2,7 +2,6 @@ package com.mr3y.ludi.shared.ui.presenter.model
 
 import app.cash.paging.PagingData
 import com.mr3y.ludi.shared.core.model.Game
-import com.mr3y.ludi.shared.core.model.Result
 import kotlinx.coroutines.flow.Flow
 
 data class DiscoverState(
@@ -32,54 +31,6 @@ sealed interface DiscoverStateGames {
     ) : DiscoverStateGames
 }
 
-sealed interface TaggedGames {
-    val games: Flow<Result<List<Game>, Throwable>>
-
-    data class TrendingGames(
-        override val games: Flow<Result<List<Game>, Throwable>>
-    ) : TaggedGames
-
-    data class TopRatedGames(
-        override val games: Flow<Result<List<Game>, Throwable>>
-    ) : TaggedGames
-
-    data class BasedOnFavGenresGames(
-        override val games: Flow<Result<List<Game>, Throwable>>
-    ) : TaggedGames
-
-    data class MultiplayerGames(
-        override val games: Flow<Result<List<Game>, Throwable>>
-    ) : TaggedGames
-
-    data class FreeGames(
-        override val games: Flow<Result<List<Game>, Throwable>>
-    ) : TaggedGames
-
-    data class StoryGames(
-        override val games: Flow<Result<List<Game>, Throwable>>
-    ) : TaggedGames
-
-    data class BoardGames(
-        override val games: Flow<Result<List<Game>, Throwable>>
-    ) : TaggedGames
-
-    data class ESportsGames(
-        override val games: Flow<Result<List<Game>, Throwable>>
-    ) : TaggedGames
-
-    data class RaceGames(
-        override val games: Flow<Result<List<Game>, Throwable>>
-    ) : TaggedGames
-
-    data class PuzzleGames(
-        override val games: Flow<Result<List<Game>, Throwable>>
-    ) : TaggedGames
-
-    data class SoundtrackGames(
-        override val games: Flow<Result<List<Game>, Throwable>>
-    ) : TaggedGames
-}
-
 data class DiscoverFiltersState(
     val currentPage: Int,
     val allPlatforms: Set<Platform>,

diff --git a/shared/src/commonMain/kotlin/com/mr3y/ludi/shared/ui/screens/discover/DiscoverScreen.kt b/shared/src/commonMain/kotlin/com/mr3y/ludi/shared/ui/screens/discover/DiscoverScreen.kt
@@ -19,7 +19,6 @@ import androidx.compose.foundation.lazy.LazyRow
 import androidx.compose.foundation.lazy.grid.GridCells
 import androidx.compose.foundation.lazy.grid.GridItemSpan
 import androidx.compose.foundation.lazy.grid.LazyVerticalGrid
-import androidx.compose.foundation.lazy.items
 import androidx.compose.foundation.lazy.rememberLazyListState
 import androidx.compose.material.ExperimentalMaterialApi
 import androidx.compose.material.icons.Icons
@@ -68,7 +67,6 @@ import com.mr3y.ludi.shared.ui.presenter.model.DiscoverStateGames
 import com.mr3y.ludi.shared.ui.presenter.model.Platform
 import com.mr3y.ludi.shared.ui.presenter.model.Store
 import com.mr3y.ludi.shared.ui.presenter.model.Tag
-import com.mr3y.ludi.shared.ui.presenter.model.TaggedGames
 import kotlinx.coroutines.flow.distinctUntilChanged
 
 object DiscoverScreenTab : Screen, BottomBarTab {
@@ -429,22 +427,6 @@ fun SuggestedGamesPage(
     }
 }
 
-private fun getLabelFor(taggedGames: TaggedGames): String {
-    return when (taggedGames) {
-        is TaggedGames.TrendingGames -> "Trending"
-        is TaggedGames.TopRatedGames -> "Top rated"
-        is TaggedGames.MultiplayerGames -> "Multiplayer"
-        is TaggedGames.BasedOnFavGenresGames -> "You might also like"
-        is TaggedGames.FreeGames -> "Free to play"
-        is TaggedGames.StoryGames -> "Story based"
-        is TaggedGames.BoardGames -> "Board"
-        is TaggedGames.ESportsGames -> "ESports"
-        is TaggedGames.RaceGames -> "Race"
-        is TaggedGames.PuzzleGames -> "Puzzle"
-        is TaggedGames.SoundtrackGames -> "SoundTrack"
-    }
-}
-
 @Composable
 fun SearchQueryAndFilterPage(
     searchResultsGames: DiscoverStateGames.SearchQueryBasedGames,

diff --git a/shared/src/commonTest/kotlin/com/mr3y/ludi/shared/ui/presenter/DiscoverViewModelTest.kt b/shared/src/commonTest/kotlin/com/mr3y/ludi/shared/ui/presenter/DiscoverViewModelTest.kt
@@ -57,7 +57,7 @@ class DiscoverViewModelTest {
                 override val soundtrackGamesPager: Pager<Int, Game>
                     get() = TODO("Not yet implemented")
 
-                override fun createPagerForGamesWithQuery(
+                override fun searchQueryBasedGamesPager(
                     searchQuery: String,
                     filters: DiscoverFiltersState
                 ): Pager<Int, Game> {