Merge pull request #1020 from TeamNewPipe/fix/yt-subscriber-count

Stypox · web-flow · commit 76eeabac45de · 2023-01-29T09:44:22.000+01:00
[YouTube] Fix NPE in search when getting channel items without subscriber count
diff --git a/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/extractors/YoutubeChannelInfoItemExtractor.java b/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/extractors/YoutubeChannelInfoItemExtractor.java
@@ -96,8 +96,12 @@ public long getSubscriberCount() throws ParsingException {
             }
 
             if (withHandle) {
-                return Utils.mixedNumberWordToLong(getTextFromObject(
-                        channelInfoItem.getObject("videoCountText")));
+                if (channelInfoItem.has("videoCountText")) {
+                    return Utils.mixedNumberWordToLong(getTextFromObject(
+                            channelInfoItem.getObject("videoCountText")));
+                } else {
+                    return -1;
+                }
             }
 
             return Utils.mixedNumberWordToLong(getTextFromObject(

Original file line number	Diff line number	Diff line change
`@@ -96,8 +96,12 @@ public long getSubscriberCount() throws ParsingException {`
`96`	`96`	`}`
`97`	`97`
`98`	`98`	`if (withHandle) {`
`99`		`- return Utils.mixedNumberWordToLong(getTextFromObject(`
`100`		`- channelInfoItem.getObject("videoCountText")));`
	`99`	`+ if (channelInfoItem.has("videoCountText")) {`
	`100`	`+ return Utils.mixedNumberWordToLong(getTextFromObject(`
	`101`	`+ channelInfoItem.getObject("videoCountText")));`
	`102`	`+ } else {`
	`103`	`+ return -1;`
	`104`	`+ }`
`101`	`105`	`}`
`102`	`106`
`103`	`107`	`return Utils.mixedNumberWordToLong(getTextFromObject(`