Skip to content

Commit fb0eca5

Browse files
authored
fix: remove broken playlist support in TvHtml5Simply (#213)
1 parent a1d4a53 commit fb0eca5

2 files changed

Lines changed: 14 additions & 14 deletions

File tree

README.md

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -184,18 +184,18 @@ plugins:
184184
## Available Clients
185185
Currently, the following clients are available for use:
186186

187-
| Identifier | Opus Formats | OAuth | Age-restriction Support | Playback Support | Metadata Support | Additional Notes |
188-
|-------------------|--------------|-------|-------------------------|------------------|------------------------------|------------------------------------------------------|
189-
| `MUSIC` | No | No | No | No | Search | YouTube music search support via `ytmsearch:` prefix |
190-
| `WEB` | Yes | No | No | Yes + Livestream | Video, Search, Playlist, Mix | |
191-
| `MWEB` | Yes | No | No | Yes + Livestream | Video, Search, Playlist, Mix | |
192-
| `WEBEMBEDDED` | Yes | No | Limited | Yes + Livestream | Video | |
193-
| `ANDROID` | Yes | No | No | Yes + Livestream | Video, Search, Playlist, Mix | Heavily restricted, frequently dysfunctional |
194-
| `ANDROID_MUSIC` | Yes | No | No | Yes | Video, Search, Mix | |
195-
| `ANDROID_VR` | Yes | No | No | Yes + Livestream | Video, Search, Playlist, Mix | |
196-
| `IOS` | No | No | No | Yes + Livestream | Video, Search, Playlist, Mix | |
197-
| `TV` | Yes | Yes | With OAuth | Yes + Livestream | None | Playback requires sign-in |
198-
| `TVHTML5_SIMPLY` | Yes | No | No | Yes + Livestream | Video, Search, Playlist, Mix | |
187+
| Identifier | Opus Formats | OAuth | Age-restriction Support | Playback Support | Metadata Support | Additional Notes |
188+
|-------------------|--------------|-------|-------------------------|------------------|-------------------------------|------------------------------------------------------|
189+
| `MUSIC` | No | No | No | No | Search | YouTube music search support via `ytmsearch:` prefix |
190+
| `WEB` | Yes | No | No | Yes + Livestream | Video, Search, Playlist, Mix | |
191+
| `MWEB` | Yes | No | No | Yes + Livestream | Video, Search, Playlist, Mix | |
192+
| `WEBEMBEDDED` | Yes | No | Limited | Yes + Livestream | Video | |
193+
| `ANDROID` | Yes | No | No | Yes + Livestream | Video, Search, Playlist, Mix | Heavily restricted, frequently dysfunctional |
194+
| `ANDROID_MUSIC` | Yes | No | No | Yes | Video, Search, Mix | |
195+
| `ANDROID_VR` | Yes | No | No | Yes + Livestream | Video, Search, Playlist, Mix | |
196+
| `IOS` | No | No | No | Yes + Livestream | Video, Search, Playlist, Mix | |
197+
| `TV` | Yes | Yes | With OAuth | Yes + Livestream | None | Playback requires sign-in |
198+
| `TVHTML5_SIMPLY` | Yes | No | No | Yes + Livestream | Video, Search, Mix | |
199199

200200
> [!NOTE]
201201
> Clients that do not return Opus formats will require transcoding.

common/src/main/java/dev/lavalink/youtube/clients/TvHtml5Simply.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
import java.util.List;
1111

1212
public class TvHtml5Simply extends StreamingNonMusicClient {
13-
13+
1414
public static ClientConfig BASE_CONFIG = new ClientConfig()
1515
.withClientName("TVHTML5_SIMPLY")
1616
.withClientField("clientVersion", "1.0")
@@ -46,7 +46,7 @@ public ClientOptions getOptions() {
4646

4747
@Override
4848
public boolean canHandleRequest(@NotNull String identifier) {
49-
return super.canHandleRequest(identifier);
49+
return !identifier.contains("list=") && super.canHandleRequest(identifier);
5050
}
5151

5252
@Override

0 commit comments

Comments
 (0)