diff --git a/briar-android/src/main/java/org/briarproject/briar/android/navdrawer/NavDrawerActivity.java b/briar-android/src/main/java/org/briarproject/briar/android/navdrawer/NavDrawerActivity.java index 2a957d0ee5151dcfa9c0acee726fbc1ae90d1808..5376dc8f00cf0d103bb60a1dcaa721718c824ae9 100644 --- a/briar-android/src/main/java/org/briarproject/briar/android/navdrawer/NavDrawerActivity.java +++ b/briar-android/src/main/java/org/briarproject/briar/android/navdrawer/NavDrawerActivity.java @@ -71,6 +71,7 @@ import static org.briarproject.bramble.api.plugin.Plugin.State.ACTIVE; import static org.briarproject.bramble.api.plugin.Plugin.State.ENABLING; import static org.briarproject.bramble.api.plugin.Plugin.State.STARTING_STOPPING; import static org.briarproject.briar.android.BriarService.EXTRA_STARTUP_FAILED; +import static org.briarproject.briar.android.TestingConstants.IS_DEBUG_BUILD; import static org.briarproject.briar.android.activity.RequestCodes.REQUEST_PASSWORD; import static org.briarproject.briar.android.navdrawer.IntentRouter.handleExternalIntent; import static org.briarproject.briar.android.util.UiUtils.getDaysUntilExpiry; @@ -128,8 +129,10 @@ public class NavDrawerActivity extends BriarActivity implements navDrawerViewModel = provider.get(NavDrawerViewModel.class); pluginViewModel = provider.get(PluginViewModel.class); - navDrawerViewModel.showExpiryWarning() - .observe(this, this::showExpiryWarning); + if (IS_DEBUG_BUILD) { + navDrawerViewModel.showExpiryWarning() + .observe(this, this::showExpiryWarning); + } navDrawerViewModel.shouldAskForDozeWhitelisting().observe(this, ask -> { if (ask) showDozeDialog(getString(R.string.setup_doze_intro)); }); @@ -171,7 +174,7 @@ public class NavDrawerActivity extends BriarActivity implements public void onStart() { super.onStart(); lockManager.checkIfLockable(); - navDrawerViewModel.checkExpiryWarning(); + if (IS_DEBUG_BUILD) navDrawerViewModel.checkExpiryWarning(); } @Override @@ -350,7 +353,7 @@ public class NavDrawerActivity extends BriarActivity implements } private void showExpiryWarning(boolean show) { - int daysUntilExpiry = getDaysUntilExpiry(); + long daysUntilExpiry = getDaysUntilExpiry(); if (daysUntilExpiry < 0) { signOut(); return; @@ -362,7 +365,8 @@ public class NavDrawerActivity extends BriarActivity implements TextView expiryWarningText = expiryWarning.findViewById(R.id.expiryWarningText); String text = getResources().getQuantityString( - R.plurals.expiry_warning, daysUntilExpiry, daysUntilExpiry); + R.plurals.expiry_warning, (int) daysUntilExpiry, + (int) daysUntilExpiry); expiryWarningText.setText(text); // make close button functional ImageView expiryWarningClose = diff --git a/briar-android/src/main/java/org/briarproject/briar/android/navdrawer/NavDrawerViewModel.java b/briar-android/src/main/java/org/briarproject/briar/android/navdrawer/NavDrawerViewModel.java index 37f26b6e490f1d70a3dba561528fc2e69cc3367b..1d031d35d0ed120ebfceb1252831f08e373f5666 100644 --- a/briar-android/src/main/java/org/briarproject/briar/android/navdrawer/NavDrawerViewModel.java +++ b/briar-android/src/main/java/org/briarproject/briar/android/navdrawer/NavDrawerViewModel.java @@ -23,7 +23,6 @@ import static java.util.logging.Level.WARNING; import static java.util.logging.Logger.getLogger; import static org.briarproject.bramble.util.LogUtils.logException; import static org.briarproject.briar.android.TestingConstants.EXPIRY_DATE; -import static org.briarproject.briar.android.TestingConstants.IS_DEBUG_BUILD; import static org.briarproject.briar.android.controller.BriarControllerImpl.DOZE_ASK_AGAIN; import static org.briarproject.briar.android.settings.SettingsFragment.SETTINGS_NAMESPACE; import static org.briarproject.briar.android.util.UiUtils.needsDozeWhitelisting; @@ -59,10 +58,6 @@ public class NavDrawerViewModel extends AndroidViewModel { @UiThread void checkExpiryWarning() { - if (!IS_DEBUG_BUILD) { - showExpiryWarning.setValue(false); - return; - } dbExecutor.execute(() -> { try { Settings settings = diff --git a/briar-android/src/main/java/org/briarproject/briar/android/util/UiUtils.java b/briar-android/src/main/java/org/briarproject/briar/android/util/UiUtils.java index bf38052327c62c091d4f686a195064ee393e4f65..50dbd91ca5aa0c1b863347bf401ad04ad24a31b5 100644 --- a/briar-android/src/main/java/org/briarproject/briar/android/util/UiUtils.java +++ b/briar-android/src/main/java/org/briarproject/briar/android/util/UiUtils.java @@ -159,10 +159,9 @@ public class UiUtils { return DateUtils.formatDateTime(ctx, time, flags); } - public static int getDaysUntilExpiry() { + public static long getDaysUntilExpiry() { long now = System.currentTimeMillis(); - long daysBeforeExpiry = (EXPIRY_DATE - now) / DAYS.toMillis(1); - return (int) daysBeforeExpiry; + return (EXPIRY_DATE - now) / DAYS.toMillis(1); } public static SpannableStringBuilder getTeaser(Context ctx, Spanned text) {