From e98838ad7eda4908e79b38087750ba6f11993f70 Mon Sep 17 00:00:00 2001 From: XiangRongLin <41164160+XiangRongLin@users.noreply.github.com> Date: Sat, 30 Jan 2021 09:32:17 +0100 Subject: [PATCH] Invert usage of manager.isExpired() When it's expired it means, that the app should get the data. Meaning it should not abort prematurely by returning null. Co-authored-by: Tobias Groza --- app/src/main/java/org/schabi/newpipe/CheckForNewAppVersion.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/src/main/java/org/schabi/newpipe/CheckForNewAppVersion.java b/app/src/main/java/org/schabi/newpipe/CheckForNewAppVersion.java index 630fb01ff..8bea85b19 100644 --- a/app/src/main/java/org/schabi/newpipe/CheckForNewAppVersion.java +++ b/app/src/main/java/org/schabi/newpipe/CheckForNewAppVersion.java @@ -180,7 +180,7 @@ public final class CheckForNewAppVersion { } final long expiry = prefs.getLong(app.getString(R.string.update_expiry_key), 0); - if (manager.isExpired(expiry)) { + if (!manager.isExpired(expiry)) { return null; }