prevent removed permissions from crashing the app - disable the offen…

…ding service and inform the user
mathisdt · Jan 3, 2024 · 2809fb1 · 2809fb1
1 parent 954842f
commit 2809fb1
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/app/src/main/java/org/zephyrsoft/trackworktime/Basics.java b/app/src/main/java/org/zephyrsoft/trackworktime/Basics.java
@@ -523,7 +523,9 @@ private void checkLocationBasedTracking() {
     private void startLocationTrackerService(double latitude, double longitude, double tolerance, Boolean vibrate) {
         Set<String> missingPermissions = PermissionsUtil.missingPermissionsForTracking(context);
         if (!missingPermissions.isEmpty()) {
-            Logger.debug("not starting location tracker service because of missing permissions: {}", missingPermissions);
+            Logger.warn("not starting location tracker service because of missing permissions, disabling automatic tracking: {}", missingPermissions);
+            PreferencesUtil.disableAutomaticTracking(context);
+            Toast.makeText(context, context.getString(R.string.locationPermissionsRemoved), Toast.LENGTH_LONG).show();
             return;
         }
         try {
@@ -585,6 +587,13 @@ private void checkWifiBasedTracking() {
      * start the wifi-based tracking service by serviceIntent
      */
     private void startWifiTrackerService(String ssid, Boolean vibrate, Integer checkInterval) {
+        Set<String> missingPermissions = PermissionsUtil.missingPermissionsForTracking(context);
+        if (!missingPermissions.isEmpty()) {
+            Logger.warn("not starting WiFi tracker service because of missing permissions, disabling automatic tracking: {}", missingPermissions);
+            PreferencesUtil.disableAutomaticTracking(context);
+            Toast.makeText(context, context.getString(R.string.locationPermissionsRemoved), Toast.LENGTH_LONG).show();
+            return;
+        }
         try {
             Intent startIntent = buildWifiTrackerServiceIntent(ssid, vibrate, checkInterval);
             Logger.debug("try to start wifi-based tracking service");