From fce324d1bc74bc0e6993909fc825ff36fa9c6b00 Mon Sep 17 00:00:00 2001 From: Christian Schabesberger Date: Mon, 12 Mar 2018 16:14:33 +0100 Subject: [PATCH] fix playlist banner foo --- .../youtube/YoutubePlaylistExtractor.java | 18 +++--------------- .../youtube/YoutubePlaylistExtractorTest.java | 1 + 2 files changed, 4 insertions(+), 15 deletions(-) diff --git a/src/main/java/org/schabi/newpipe/extractor/services/youtube/YoutubePlaylistExtractor.java b/src/main/java/org/schabi/newpipe/extractor/services/youtube/YoutubePlaylistExtractor.java index 27c92c7d4..a92dc3aad 100644 --- a/src/main/java/org/schabi/newpipe/extractor/services/youtube/YoutubePlaylistExtractor.java +++ b/src/main/java/org/schabi/newpipe/extractor/services/youtube/YoutubePlaylistExtractor.java @@ -74,21 +74,9 @@ public class YoutubePlaylistExtractor extends PlaylistExtractor { } @Override - public String getBannerUrl() throws ParsingException { - try { - Element el = doc.select("div[id=\"gh-banner\"] style").first(); - String cssContent = el.html(); - String url = "https:" + Parser.matchGroup1("url\\((.*)\\)", cssContent); - if (url.contains("s.ytimg.com")) { - return null; - } else { - return url.substring(0, url.indexOf(");")); - } - - - } catch (Exception e) { - throw new ParsingException("Could not get playlist Banner"); - } + public String getBannerUrl() { + return ""; // Banner can't be handled by frontend right now. + // Whoever is willing to implement this should also implement this in the fornt end } @Override diff --git a/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubePlaylistExtractorTest.java b/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubePlaylistExtractorTest.java index d472c5d6b..e22ca32b3 100644 --- a/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubePlaylistExtractorTest.java +++ b/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubePlaylistExtractorTest.java @@ -55,6 +55,7 @@ public class YoutubePlaylistExtractorTest { @Test public void testGetBannerUrl() throws Exception { + System.out.println(extractor.getBannerUrl()); assertTrue(extractor.getBannerUrl(), extractor.getBannerUrl().contains("yt")); }