Only use fallback methods for clientVersion when clientVersion hasn't been successfully extracted yet

Author: wb9688

extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/YoutubeParsingHelper.java

@@ -216,7 +216,7 @@ public static boolean isHardcodedClientVersionValid() throws IOException, Extrac
         return response.length() > 50; // ensure to have a valid response
     }
 
-    private static void getClientVersionAndKey() throws IOException, ExtractionException {
+    private static void extractClientVersionAndKey() throws IOException, ExtractionException {
         final String url = "https://www.youtube.com/results?search_query=test";
         final String html = getDownloader().get(url).responseBody();
         final JsonObject initialData = getInitialData(html);
@@ -259,11 +259,12 @@ private static void getClientVersionAndKey() throws IOException, ExtractionExcep
                 contextClientVersion = Parser.matchGroup1(pattern, html);
                 if (!isNullOrEmpty(contextClientVersion)) {
                     clientVersion = contextClientVersion;
+                    break;
                 }
             } catch (Parser.RegexException ignored) { }
         }
 
-        if (shortClientVersion != null) {
+        if (!isNullOrEmpty(clientVersion) && !isNullOrEmpty(shortClientVersion)) {
             clientVersion = shortClientVersion;
         }
 
@@ -283,7 +284,7 @@ public static String getClientVersion() throws IOException, ExtractionException
         if (!isNullOrEmpty(clientVersion)) return clientVersion;
         if (isHardcodedClientVersionValid()) return clientVersion = HARDCODED_CLIENT_VERSION;
 
-        getClientVersionAndKey();
+        extractClientVersionAndKey();
         if (isNullOrEmpty(key)) throw new ParsingException("Could not extract client version");
         return clientVersion;
     }
@@ -294,7 +295,7 @@ public static String getClientVersion() throws IOException, ExtractionException
     public static String getKey() throws IOException, ExtractionException {
         if (!isNullOrEmpty(key)) return key;
 
-        getClientVersionAndKey();
+        extractClientVersionAndKey();
         if (isNullOrEmpty(key)) throw new ParsingException("Could not extract key");
         return key;
     }