Skip to content

Commit 8db2d34

Browse files
committed
fix: can't load music playlists using local extraction
1 parent 87352f9 commit 8db2d34

2 files changed

Lines changed: 8 additions & 2 deletions

File tree

app/src/main/java/com/github/libretube/extensions/ToID.kt

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,18 @@
11
package com.github.libretube.extensions
22

33
import com.github.libretube.ui.dialogs.ShareDialog.Companion.YOUTUBE_FRONTEND_URL
4+
import com.github.libretube.ui.dialogs.ShareDialog.Companion.YOUTUBE_MUSIC_URL
5+
import com.github.libretube.ui.dialogs.ShareDialog.Companion.YOUTUBE_SHORT_URL
46

57
/**
6-
* format a Piped route to an ID
8+
* format a full YouTube url or a path to a video/channel/playlist ID
79
*/
810
fun String.toID(): String {
911
return this
10-
.replace(YOUTUBE_FRONTEND_URL, "")
12+
// remove any youtube origins from urls
13+
.removePrefix(YOUTUBE_FRONTEND_URL)
14+
.removePrefix(YOUTUBE_MUSIC_URL)
15+
.removePrefix(YOUTUBE_SHORT_URL)
1116
.replace("/watch?v=", "") // videos
1217
.replace("/channel/", "") // channels
1318
.replace("/playlist?list=", "") // playlists

app/src/main/java/com/github/libretube/ui/dialogs/ShareDialog.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,7 @@ class ShareDialog : DialogFragment() {
149149

150150
companion object {
151151
const val YOUTUBE_FRONTEND_URL = "https://www.youtube.com"
152+
const val YOUTUBE_MUSIC_URL = "https://music.youtube.com"
152153
const val YOUTUBE_SHORT_URL = "https://youtu.be"
153154
const val PIPED_FRONTEND_URL = "https://piped.video"
154155
}

0 commit comments

Comments
 (0)