From 4469d1130799220c2d070d4896e70ea4ff1c4507 Mon Sep 17 00:00:00 2001 From: Christian Schabesberger Date: Fri, 7 Sep 2018 21:40:36 +0200 Subject: [PATCH] fix channel links in description --- .../services/youtube/extractors/YoutubeStreamExtractor.java | 3 ++- .../services/youtube/YoutubePlaylistExtractorTest.java | 2 +- ...ractorDASHText.java => YoutubeStreamExtractorDASHTest.java} | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) rename extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/{YoutubeStreamExtractorDASHText.java => YoutubeStreamExtractorDASHTest.java} (97%) diff --git a/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/extractors/YoutubeStreamExtractor.java b/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/extractors/YoutubeStreamExtractor.java index f8d36c49..64e053a7 100644 --- a/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/extractors/YoutubeStreamExtractor.java +++ b/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/extractors/YoutubeStreamExtractor.java @@ -178,7 +178,8 @@ public class YoutubeStreamExtractor extends StreamExtractor { // They refer to the youtube search. We do not handle them. a.text(link); - } else if(redirectLink.toString().contains("watch?v=")) { + } else if(redirectLink.toString().contains("watch?v=") + || redirectLink.toString().contains("https://www.youtube.com/")) { // Another posibility is that this link is pointing to another video // we need to put the redirectLink in here explicitly in order to add the domain part to the link. a.text(redirectLink.toString()); diff --git a/extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubePlaylistExtractorTest.java b/extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubePlaylistExtractorTest.java index 7a7f5ace..e89db1ff 100644 --- a/extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubePlaylistExtractorTest.java +++ b/extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubePlaylistExtractorTest.java @@ -120,7 +120,7 @@ public class YoutubePlaylistExtractorTest { } } - public static class ImportantVideos implements BasePlaylistExtractorTest { + public static class HugePlaylist implements BasePlaylistExtractorTest { private static YoutubePlaylistExtractor extractor; @BeforeClass diff --git a/extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubeStreamExtractorDASHText.java b/extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubeStreamExtractorDASHTest.java similarity index 97% rename from extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubeStreamExtractorDASHText.java rename to extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubeStreamExtractorDASHTest.java index 8bb6453e..073df2a3 100644 --- a/extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubeStreamExtractorDASHText.java +++ b/extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubeStreamExtractorDASHTest.java @@ -35,7 +35,7 @@ import static org.schabi.newpipe.extractor.ServiceList.YouTube; /** * Test for {@link StreamExtractor} */ -public class YoutubeStreamExtractorDASHText { +public class YoutubeStreamExtractorDASHTest { private static StreamInfo info; @BeforeClass