diff --git a/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/linkHandler/YoutubeStreamLinkHandlerFactory.java b/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/linkHandler/YoutubeStreamLinkHandlerFactory.java index fca49585..5d07779c 100644 --- a/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/linkHandler/YoutubeStreamLinkHandlerFactory.java +++ b/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/linkHandler/YoutubeStreamLinkHandlerFactory.java @@ -60,73 +60,73 @@ public class YoutubeStreamLinkHandlerFactory extends LinkHandlerFactory { throw new IllegalArgumentException("The url parameter should not be empty"); } - String id; String lowercaseUrl = url.toLowerCase(); if (lowercaseUrl.contains("youtube")) { + if (lowercaseUrl.contains("list=")) { + throw new ParsingException("Error no suitable url: " + url); + } if (url.contains("attribution_link")) { try { String escapedQuery = Parser.matchGroup1("u=(.[^&|$]*)", url); String query = URLDecoder.decode(escapedQuery, "UTF-8"); - id = Parser.matchGroup1("v=" + ID_PATTERN, query); + return Parser.matchGroup1("v=" + ID_PATTERN, query); } catch (UnsupportedEncodingException uee) { throw new ParsingException("Could not parse attribution_link", uee); } - } else if (url.contains("vnd.youtube")) { - id = Parser.matchGroup1(ID_PATTERN, url); - } else if (url.contains("embed")) { - id = Parser.matchGroup1("embed/" + ID_PATTERN, url); - } else if (url.contains("googleads")) { + } + if (url.contains("vnd.youtube")) { + return Parser.matchGroup1(ID_PATTERN, url); + } + if (url.contains("embed")) { + return Parser.matchGroup1("embed/" + ID_PATTERN, url); + } + if (url.contains("googleads")) { throw new FoundAdException("Error found add: " + url); - } else { - id = Parser.matchGroup1("[?&]v=" + ID_PATTERN, url); } - } else if (lowercaseUrl.contains("youtu.be")) { - if (url.contains("v=")) { - id = Parser.matchGroup1("v=" + ID_PATTERN, url); - } else { - id = Parser.matchGroup1("[Yy][Oo][Uu][Tt][Uu]\\.[Bb][Ee]/" + ID_PATTERN, url); - } - } else if(lowercaseUrl.contains("hooktube")) { - if(lowercaseUrl.contains("&v=") - || lowercaseUrl.contains("?v=")) { - id = Parser.matchGroup1("[?&]v=" + ID_PATTERN, url); - } else if (url.contains("/embed/")) { - id = Parser.matchGroup1("embed/" + ID_PATTERN, url); - } else if (url.contains("/v/")) { - id = Parser.matchGroup1("v/" + ID_PATTERN, url); - } else if (url.contains("/watch/")) { - id = Parser.matchGroup1("watch/" + ID_PATTERN, url); - } else { + return Parser.matchGroup1("[?&]v=" + ID_PATTERN, url); + } + if (lowercaseUrl.contains("youtu.be")) { + if (lowercaseUrl.contains("list=")) { throw new ParsingException("Error no suitable url: " + url); } - } else { - throw new ParsingException("Error no suitable url: " + url); + if (url.contains("v=")) { + return Parser.matchGroup1("v=" + ID_PATTERN, url); + } + return Parser.matchGroup1("[Yy][Oo][Uu][Tt][Uu]\\.[Bb][Ee]/" + ID_PATTERN, url); } - - - if (!id.isEmpty()) { - return id; - } else { - throw new ParsingException("Error could not parse url: " + url); + if (lowercaseUrl.contains("hooktube")) { + if (lowercaseUrl.contains("&v=") + || lowercaseUrl.contains("?v=")) { + return Parser.matchGroup1("[?&]v=" + ID_PATTERN, url); + } + if (url.contains("/embed/")) { + return Parser.matchGroup1("embed/" + ID_PATTERN, url); + } + if (url.contains("/v/")) { + return Parser.matchGroup1("v/" + ID_PATTERN, url); + } + if (url.contains("/watch/")) { + return Parser.matchGroup1("watch/" + ID_PATTERN, url); + } } + throw new ParsingException("Error no suitable url: " + url); } @Override public boolean onAcceptUrl(final String url) throws FoundAdException { final String lowercaseUrl = url.toLowerCase(); - if (lowercaseUrl.contains("youtube") - || lowercaseUrl.contains("youtu.be") - || lowercaseUrl.contains("hooktube")) { - // bad programming I know - try { - getId(url); - return true; - } catch (FoundAdException fe) { - throw fe; - } catch (ParsingException e) { - return false; - } - } else { + if (!lowercaseUrl.contains("youtube") && + !lowercaseUrl.contains("youtu.be") && + !lowercaseUrl.contains("hooktube")) { + return false; + // bad programming I know <-- nice meme + } + try { + getId(url); + return true; + } catch (FoundAdException fe) { + throw fe; + } catch (ParsingException e) { return false; } }