Merge pull request #3516 from egsavage/reset-sleep-timer-on-play

Issue #2381 - restart sleep timer on play

core/src/main/java/de/danoeh/antennapod/core/service/playback/PlaybackService.java

@@ -827,6 +827,9 @@ public class PlaybackService extends MediaBrowserServiceCompat {
 
         @Override
         public void onPlaybackStart(@NonNull Playable playable, int position) {
+            if (taskManager.isSleepTimerActive()) {
+                taskManager.restartSleepTimer();
+            }
             taskManager.startWidgetUpdater();
             if (position != PlaybackServiceMediaPlayer.INVALID_TIME) {
                 playable.setPosition(position);

core/src/main/java/de/danoeh/antennapod/core/service/playback/PlaybackServiceTaskManager.java

@@ -248,6 +248,16 @@ public class PlaybackServiceTaskManager {
         }
     }
 
+    /**
+     * Restarts the sleep timer. If the sleep timer is not active, nothing will happen.
+     */
+    public synchronized void restartSleepTimer() {
+        if (isSleepTimerActive()) {
+            Log.d(TAG, "Restarting sleep timer");
+            sleepTimer.restart();
+        }
+    }
+
     /**
      * Returns the current sleep timer time or 0 if the sleep timer is not active.
      */
@@ -428,13 +438,15 @@ public class PlaybackServiceTaskManager {
             return timeLeft;
         }
 
-        public void onShake() {
+        public void restart() {
             postCallback(() -> {
                 setSleepTimer(waitingTime, shakeToReset, vibrate);
                 callback.onSleepTimerReset();
             });
-            shakeListener.pause();
-            shakeListener = null;
+            if (shakeListener != null) {
+                shakeListener.pause();
+                shakeListener = null;
+            }
         }
 
     }

core/src/main/java/de/danoeh/antennapod/core/service/playback/ShakeListener.java

@@ -52,7 +52,7 @@ class ShakeListener implements SensorEventListener
         double gForce = Math.sqrt(gX*gX + gY*gY + gZ*gZ);
         if (gForce > 2.25) {
             Log.d(TAG, "Detected shake " + gForce);
-            mSleepTimer.onShake();
+            mSleepTimer.restart();
         }
     }