From f57262da6a8425ec567b0a4b7038bae1e7a4c7a5 Mon Sep 17 00:00:00 2001 From: Bnyro Date: Thu, 1 Dec 2022 17:11:31 +0100 Subject: [PATCH] fix opening public playlists --- .../main/java/com/github/libretube/api/PlaylistsHelper.kt | 6 +++--- .../java/com/github/libretube/ui/fragments/HomeFragment.kt | 2 +- .../com/github/libretube/ui/fragments/LibraryFragment.kt | 2 +- app/src/main/java/com/github/libretube/util/PlayingQueue.kt | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/app/src/main/java/com/github/libretube/api/PlaylistsHelper.kt b/app/src/main/java/com/github/libretube/api/PlaylistsHelper.kt index 736e83594..1231a0271 100644 --- a/app/src/main/java/com/github/libretube/api/PlaylistsHelper.kt +++ b/app/src/main/java/com/github/libretube/api/PlaylistsHelper.kt @@ -53,7 +53,7 @@ object PlaylistsHelper { suspend fun getPlaylist(playlistId: String): Playlist { // load locally stored playlists with the auth api - return when (getPlaylistType()) { + return when (getPrivatePlaylistType(playlistId)) { PlaylistType.PRIVATE -> RetrofitInstance.authApi.getPlaylist(playlistId) PlaylistType.PUBLIC -> RetrofitInstance.api.getPlaylist(playlistId) PlaylistType.LOCAL -> { @@ -227,11 +227,11 @@ object PlaylistsHelper { return importLists } - fun getPlaylistType(): PlaylistType { + fun getPrivatePlaylistType(): PlaylistType { return if (loggedIn()) PlaylistType.PRIVATE else PlaylistType.LOCAL } - fun getPlaylistType(playlistId: String): PlaylistType { + fun getPrivatePlaylistType(playlistId: String): PlaylistType { if (playlistId.all { it.isDigit() }) return PlaylistType.LOCAL if (playlistId.matches(pipedPlaylistRegex)) return PlaylistType.PRIVATE return PlaylistType.PUBLIC diff --git a/app/src/main/java/com/github/libretube/ui/fragments/HomeFragment.kt b/app/src/main/java/com/github/libretube/ui/fragments/HomeFragment.kt index f872acfeb..4fe5086c8 100644 --- a/app/src/main/java/com/github/libretube/ui/fragments/HomeFragment.kt +++ b/app/src/main/java/com/github/libretube/ui/fragments/HomeFragment.kt @@ -108,7 +108,7 @@ class HomeFragment : BaseFragment() { runOnUiThread { makeVisible(binding.playlistsRV, binding.playlistsTV) binding.playlistsRV.layoutManager = LinearLayoutManager(context) - binding.playlistsRV.adapter = PlaylistsAdapter(playlists.toMutableList(), PlaylistsHelper.getPlaylistType()) + binding.playlistsRV.adapter = PlaylistsAdapter(playlists.toMutableList(), PlaylistsHelper.getPrivatePlaylistType()) binding.playlistsRV.adapter?.registerAdapterDataObserver(object : RecyclerView.AdapterDataObserver() { override fun onItemRangeRemoved(positionStart: Int, itemCount: Int) { diff --git a/app/src/main/java/com/github/libretube/ui/fragments/LibraryFragment.kt b/app/src/main/java/com/github/libretube/ui/fragments/LibraryFragment.kt index ee8bce5ea..b1b1bb14c 100644 --- a/app/src/main/java/com/github/libretube/ui/fragments/LibraryFragment.kt +++ b/app/src/main/java/com/github/libretube/ui/fragments/LibraryFragment.kt @@ -123,7 +123,7 @@ class LibraryFragment : BaseFragment() { val playlistsAdapter = PlaylistsAdapter( playlists.toMutableList(), - PlaylistsHelper.getPlaylistType() + PlaylistsHelper.getPrivatePlaylistType() ) // listen for playlists to become deleted diff --git a/app/src/main/java/com/github/libretube/util/PlayingQueue.kt b/app/src/main/java/com/github/libretube/util/PlayingQueue.kt index 530ce68f2..fe64f85fd 100644 --- a/app/src/main/java/com/github/libretube/util/PlayingQueue.kt +++ b/app/src/main/java/com/github/libretube/util/PlayingQueue.kt @@ -110,7 +110,7 @@ object PlayingQueue { fun insertPlaylist(playlistId: String, newCurrentStream: StreamItem) { CoroutineScope(Dispatchers.IO).launch { try { - val playlistType = PlaylistsHelper.getPlaylistType(playlistId) + val playlistType = PlaylistsHelper.getPrivatePlaylistType(playlistId) val playlist = PlaylistsHelper.getPlaylist(playlistId) add( *playlist.relatedStreams