[#11930] Non-Youtube URLs should be ignored

This commit is contained in:
Thiago F. G. Albuquerque 2025-02-19 16:29:34 -03:00
parent e2b376ec99
commit a03fd7517f
2 changed files with 4 additions and 1 deletions

View file

@ -65,6 +65,7 @@ import org.schabi.newpipe.util.external_communication.ShareUtils;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.Collections; import java.util.Collections;
import java.util.List; import java.util.List;
import java.util.Objects;
import java.util.concurrent.atomic.AtomicBoolean; import java.util.concurrent.atomic.AtomicBoolean;
import java.util.stream.Collectors; import java.util.stream.Collectors;
import java.util.stream.Stream; import java.util.stream.Stream;
@ -461,6 +462,7 @@ public class LocalPlaylistFragment extends BaseLocalListFragment<List<PlaylistSt
final String videoIDs = entityStream final String videoIDs = entityStream
.map(entity -> getYouTubeId(entity.getUrl())) .map(entity -> getYouTubeId(entity.getUrl()))
.filter(Objects::nonNull)
.collect(Collectors.joining(",")); .collect(Collectors.joining(","));
return "http://www.youtube.com/watch_videos?video_ids=" + videoIDs; return "http://www.youtube.com/watch_videos?video_ids=" + videoIDs;

View file

@ -20,7 +20,8 @@ public class LocalPlaylistFragmentTest {
final Stream<StreamEntity> entityStream = asStreamEntityStream( final Stream<StreamEntity> entityStream = asStreamEntityStream(
"https://www.youtube.com/watch?v=1", "https://www.youtube.com/watch?v=1",
"https://www.youtube.com/watch?v=2", "https://soundcloud.com/cautious-clayofficial/cold-war-2", // non-Youtube URLs should be
"https://www.youtube.com/watch?v=2", // ignored
"https://www.youtube.com/watch?v=3" "https://www.youtube.com/watch?v=3"
); );