From aa48cdb14c6ce23e07199ab706ae97d11af8035b Mon Sep 17 00:00:00 2001
From: ByteHamster <info@bytehamster.com>
Date: Sat, 8 Feb 2020 00:12:33 +0100
Subject: Do not return true in handleKeycode if nothing was done

---
 .../antennapod/core/service/playback/PlaybackService.java     | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

(limited to 'core/src/main')

diff --git a/core/src/main/java/de/danoeh/antennapod/core/service/playback/PlaybackService.java b/core/src/main/java/de/danoeh/antennapod/core/service/playback/PlaybackService.java
index fbd8d65d8..bf59b0ffd 100644
--- a/core/src/main/java/de/danoeh/antennapod/core/service/playback/PlaybackService.java
+++ b/core/src/main/java/de/danoeh/antennapod/core/service/playback/PlaybackService.java
@@ -573,6 +573,8 @@ public class PlaybackService extends MediaBrowserServiceCompat {
                     mediaPlayer.prepare();
                 } else if (mediaPlayer.getPlayable() == null) {
                     startPlayingFromPreferences();
+                } else {
+                    return false;
                 }
                 taskManager.restartSleepTimer();
                 return true;
@@ -584,18 +586,19 @@ public class PlaybackService extends MediaBrowserServiceCompat {
                     mediaPlayer.prepare();
                 } else if (mediaPlayer.getPlayable() == null) {
                     startPlayingFromPreferences();
+                } else {
+                    return false;
                 }
                 taskManager.restartSleepTimer();
                 return true;
             case KeyEvent.KEYCODE_MEDIA_PAUSE:
                 if (status == PlayerStatus.PLAYING) {
                     mediaPlayer.pause(!UserPreferences.isPersistNotify(), false);
+                    return true;
                 }
-
-                return true;
+                return false;
             case KeyEvent.KEYCODE_MEDIA_NEXT:
-                if (notificationButton ||
-                        UserPreferences.shouldHardwareButtonSkip()) {
+                if (notificationButton || UserPreferences.shouldHardwareButtonSkip()) {
                     // assume the skip command comes from a notification or the lockscreen
                     // a >| skip button should actually skip
                     mediaPlayer.skip();
-- 
cgit v1.2.3