diff --git a/src/main/java/org/thoughtcrime/securesms/ApplicationContext.java b/src/main/java/org/thoughtcrime/securesms/ApplicationContext.java index 91f032332..632db8b69 100644 --- a/src/main/java/org/thoughtcrime/securesms/ApplicationContext.java +++ b/src/main/java/org/thoughtcrime/securesms/ApplicationContext.java @@ -228,12 +228,8 @@ public class ApplicationContext extends MultiDexApplication { fetchWorkRequest); AppCompatDelegate.setCompatVectorFromResourcesEnabled(true); - if (BuildConfig.USE_PLAY_SERVICES) { - if (Prefs.isPushEnabled(this)) { - FcmReceiveService.register(this); - } else { - Log.i(TAG, "FCM disabled in user settings"); - } + if (Prefs.isPushEnabled(this)) { + FcmReceiveService.register(this); } else { Log.i(TAG, "FCM disabled at build time"); } diff --git a/src/main/java/org/thoughtcrime/securesms/preferences/NotificationsPreferenceFragment.java b/src/main/java/org/thoughtcrime/securesms/preferences/NotificationsPreferenceFragment.java index 782fecbd8..a4af08bea 100644 --- a/src/main/java/org/thoughtcrime/securesms/preferences/NotificationsPreferenceFragment.java +++ b/src/main/java/org/thoughtcrime/securesms/preferences/NotificationsPreferenceFragment.java @@ -73,27 +73,6 @@ public class NotificationsPreferenceFragment extends ListSummaryPreferenceFragme initializeRingtoneSummary(findPreference(Prefs.RINGTONE_PREF)); - CheckBoxPreference usePushService = this.findPreference("pref_push_enabled"); - usePushService.setChecked(Prefs.isPushEnabled(getContext())); - usePushService.setEnabled(BuildConfig.USE_PLAY_SERVICES); - usePushService.setOnPreferenceChangeListener((preference, newValue) -> { - final boolean enabled = (Boolean) newValue; - if (!enabled) { - new AlertDialog.Builder(getContext()) - .setMessage(R.string.pref_push_ask_disable) - .setPositiveButton(R.string.ok, (dialogInterface, i) -> { - ((CheckBoxPreference)preference).setChecked(false); - }) - .setNegativeButton(R.string.cancel, null) - .setNeutralButton(R.string.learn_more, (dialogInterface, i) -> { - DcHelper.openHelp(getActivity(), "#instant-delivery"); - }) - .show(); - return false; - } - return true; - }); - ignoreBattery = this.findPreference("pref_ignore_battery_optimizations"); ignoreBattery.setVisible(needsIgnoreBatteryOptimizations()); ignoreBattery.setOnPreferenceChangeListener((preference, newValue) -> { @@ -137,18 +116,6 @@ public class NotificationsPreferenceFragment extends ListSummaryPreferenceFragme notificationsEnabled.setChecked(!dcContext.isMuted()); } - @Override - public void onPause() { - super.onPause(); - - // we delay applying token changes to avoid changes and races if the user is just playing around - if (Prefs.isPushEnabled(getContext()) && FcmReceiveService.getToken() == null) { - FcmReceiveService.register(getContext()); - } else if(!Prefs.isPushEnabled(getContext()) && FcmReceiveService.getToken() != null) { - FcmReceiveService.deleteToken(); - } - } - @Override public void onActivityResult(int requestCode, int resultCode, Intent data) { super.onActivityResult(requestCode, resultCode, data); diff --git a/src/main/java/org/thoughtcrime/securesms/util/Prefs.java b/src/main/java/org/thoughtcrime/securesms/util/Prefs.java index f3031ae07..22dcae480 100644 --- a/src/main/java/org/thoughtcrime/securesms/util/Prefs.java +++ b/src/main/java/org/thoughtcrime/securesms/util/Prefs.java @@ -168,7 +168,7 @@ public class Prefs { } public static boolean isPushEnabled(Context context) { - return BuildConfig.USE_PLAY_SERVICES && getBooleanPreference(context, "pref_push_enabled", true); + return BuildConfig.USE_PLAY_SERVICES; } public static boolean isHardCompressionEnabled(Context context) { diff --git a/src/main/res/xml/preferences_notifications.xml b/src/main/res/xml/preferences_notifications.xml index af0009f93..db7b293a8 100644 --- a/src/main/res/xml/preferences_notifications.xml +++ b/src/main/res/xml/preferences_notifications.xml @@ -50,12 +50,6 @@ android:defaultValue="true" /> - -