Merge pull request #6865 from Bnyro/master

refactor: directly add loaded videos to playlist to improve UX
libre-tube · Dec 7, 2024 · 589bd45 · 589bd45
2 parents 84a79a1 + ade0732
commit 589bd45
Showing 1 changed file with 5 additions and 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
@@ -184,16 +184,15 @@ object PlaylistsHelper {
                 } else {
                     // if not logged in, all video information needs to become fetched manually
                     // Only do so with `MAX_CONCURRENT_IMPORT_CALLS` videos at once to prevent performance issues
-                    val streams = playlist.videos.chunked(
-                        MAX_CONCURRENT_IMPORT_CALLS
-                    ).map { videos ->
-                        videos.parallelMap {
+                    for (videoIdList in playlist.videos.chunked(MAX_CONCURRENT_IMPORT_CALLS)) {
+                        val streams = videoIdList.parallelMap {
                             runCatching { StreamsExtractor.extractStreams(it) }
                                 .getOrNull()
                                 ?.toStreamItem(it)
                         }.filterNotNull()
-                    }.flatten()
-                    addToPlaylist(playlistId, *streams.toTypedArray())
+
+                        addToPlaylist(playlistId, *streams.toTypedArray())
+                    }
                 }
             }
         }