fix: logic in VideoAdsPatch (#116)

epicsampler · web-flow · commit f47454109417 · 2022-08-03T21:07:31.000+02:00
diff --git a/app/src/main/java/app/revanced/integrations/patches/VideoAdsPatch.java b/app/src/main/java/app/revanced/integrations/patches/VideoAdsPatch.java
@@ -5,10 +5,10 @@
 
 public class VideoAdsPatch {
 
-    //Used by app.revanced.patches.youtube.ad.general.video.patch.VideoAdsPatch
-    //depends on Whitelist Patch. Still needs to be written
+    // Used by app.revanced.patches.youtube.ad.general.video.patch.VideoAdsPatch
+    // depends on Whitelist patch (still needs to be written)
     public static boolean shouldShowAds() {
-        return SettingsEnum.VIDEO_ADS_SHOWN.getBoolean() && Whitelist.shouldShowAds();
+        return SettingsEnum.VIDEO_ADS_SHOWN.getBoolean() || Whitelist.shouldShowAds();
     }
 
 }

Original file line number	Diff line number	Diff line change
`@@ -5,10 +5,10 @@`
`5`	`5`
`6`	`6`	`public class VideoAdsPatch {`
`7`	`7`
`8`		`- //Used by app.revanced.patches.youtube.ad.general.video.patch.VideoAdsPatch`
`9`		`- //depends on Whitelist Patch. Still needs to be written`
	`8`	`+ // Used by app.revanced.patches.youtube.ad.general.video.patch.VideoAdsPatch`
	`9`	`+ // depends on Whitelist patch (still needs to be written)`
`10`	`10`	`public static boolean shouldShowAds() {`
`11`		`- return SettingsEnum.VIDEO_ADS_SHOWN.getBoolean() && Whitelist.shouldShowAds();`
	`11`	`+ return SettingsEnum.VIDEO_ADS_SHOWN.getBoolean() \|\| Whitelist.shouldShowAds();`
`12`	`12`	`}`
`13`	`13`
`14`	`14`	`}`