From e7feedb61eb2b464219134e86ca1a97f6014f199 Mon Sep 17 00:00:00 2001
From: coderPaddyS <urulu@student.kit.edu>
Date: Fri, 15 Nov 2024 11:55:30 +0100
Subject: [PATCH] [fix] the exercise gets started automatically if the timer
 finishes with active app.

---
 app/src/main/AndroidManifest.xml                       |  4 +++-
 .../secuso/aktivpause/activities/TimerActivity.java    | 10 +++-------
 .../org/secuso/aktivpause/service/TimerService.java    | 10 +++++++---
 3 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index 9707c46..88e8f33 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -11,6 +11,7 @@
     <uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED" />
     <uses-permission android:name="android.permission.WAKE_LOCK" />
     <uses-permission android:name="android.permission.FOREGROUND_SERVICE" />
+    <uses-permission android:name="android.permission.FOREGROUND_SERVICE_SPECIAL_USE" />
 
     <application
         android:name=".PFAktivpause"
@@ -119,7 +120,8 @@
         <service
             android:name="org.secuso.aktivpause.service.TimerService"
             android:enabled="true"
-            android:exported="false" />
+            android:exported="false"
+            android:foregroundServiceType="specialUse"/>
         <service
             android:name=".backup.PFABackupService"
             android:enabled="true"
diff --git a/app/src/main/java/org/secuso/aktivpause/activities/TimerActivity.java b/app/src/main/java/org/secuso/aktivpause/activities/TimerActivity.java
index 6d28457..0f3c4ce 100644
--- a/app/src/main/java/org/secuso/aktivpause/activities/TimerActivity.java
+++ b/app/src/main/java/org/secuso/aktivpause/activities/TimerActivity.java
@@ -12,7 +12,6 @@ import android.graphics.drawable.Drawable;
 import android.net.Uri;
 import android.os.IBinder;
 import android.preference.PreferenceManager;
-import androidx.annotation.NonNull;
 import androidx.constraintlayout.widget.ConstraintLayout;
 import androidx.constraintlayout.widget.ConstraintSet;
 import androidx.core.app.ActivityCompat;
@@ -117,12 +116,9 @@ public class TimerActivity extends BaseActivity implements LoaderManager.LoaderC
             boolean isRunning = intent.getBooleanExtra("isRunning", false);
             boolean isPaused = intent.getBooleanExtra("isPaused", false);
 
-            //if(intent.getBooleanExtra("done" ,false)) {
-                // TODO: show a dialog here to start the exercise?
-            //}
-
-            //Log.d(TAG, millisUntilDone + "/" + initialDuration + " (" + (isRunning ? "Running" : "") + (isPaused ? "Paused" : "") + (!isRunning && !isPaused ?  "Stopped" : "") + ")");
-
+            if (millisUntilDone <= 0) {
+                startActivity(new Intent(TimerActivity.this, ExerciseActivity.class));
+            }
             updateUI(isRunning, isPaused, initialDuration, millisUntilDone);
         }
     };
diff --git a/app/src/main/java/org/secuso/aktivpause/service/TimerService.java b/app/src/main/java/org/secuso/aktivpause/service/TimerService.java
index f42aa08..8b74948 100644
--- a/app/src/main/java/org/secuso/aktivpause/service/TimerService.java
+++ b/app/src/main/java/org/secuso/aktivpause/service/TimerService.java
@@ -9,12 +9,14 @@ import android.content.Context;
 import android.content.Intent;
 import android.content.IntentFilter;
 import android.content.SharedPreferences;
+import android.content.pm.ServiceInfo;
 import android.os.Binder;
 import android.os.CountDownTimer;
 import android.os.IBinder;
 import android.preference.PreferenceManager;
 import android.provider.Settings;
 import androidx.core.app.NotificationCompat;
+import androidx.core.app.ServiceCompat;
 import androidx.core.content.ContextCompat;
 
 import org.secuso.aktivpause.R;
@@ -33,6 +35,7 @@ import static org.secuso.aktivpause.activities.tutorial.FirstLaunchManager.WORK_
 import static org.secuso.aktivpause.receivers.NotificationCancelReceiver.ACTION_NOTIFICATION_CANCELED;
 import static org.secuso.aktivpause.receivers.NotificationDeletedReceiver.ACTION_NOTIFICATION_DELETED;
 
+import kotlin.Unit;
 /**
  * The main timer service. It handles the work timer and sends updates to the notification and the {@link TimerActivity}.
  * When the work time is up, an alarm will fire to start the exercises.
@@ -139,9 +142,10 @@ public class TimerService extends Service {
     public void onCreate() {
         super.onCreate();
 
+
         registerReceiver(timerReceiver, new IntentFilter(TIMER_BROADCAST));
-        registerReceiver(notificationDeletedReceiver, new IntentFilter(ACTION_NOTIFICATION_DELETED));
-        registerReceiver(notificationPreferenceChangedReceiver, new IntentFilter(ACTION_NOTIFICATION_CANCELED));
+        ContextCompat.registerReceiver(this, notificationDeletedReceiver, new IntentFilter(ACTION_NOTIFICATION_DELETED), ContextCompat.RECEIVER_NOT_EXPORTED);
+        ContextCompat.registerReceiver(this, notificationPreferenceChangedReceiver, new IntentFilter(ACTION_NOTIFICATION_CANCELED),  ContextCompat.RECEIVER_NOT_EXPORTED);
     }
 
     @Override
@@ -363,7 +367,7 @@ public class TimerService extends Service {
 
     private void updateNotification() {
         if(isRunning() || isPaused()) {
-            startForeground(NOTIFICATION_ID, buildNotification());
+            ServiceCompat.startForeground(this, NOTIFICATION_ID, buildNotification(), ServiceInfo.FOREGROUND_SERVICE_TYPE_SPECIAL_USE);
         } else {
             stopForeground(true);
         }