summaryrefslogtreecommitdiff
path: root/core/src/main/java/de
diff options
context:
space:
mode:
authorByteHamster <info@bytehamster.com>2020-02-08 00:12:33 +0100
committerByteHamster <info@bytehamster.com>2020-02-08 00:12:33 +0100
commitaa48cdb14c6ce23e07199ab706ae97d11af8035b (patch)
tree43c2aec85f4d20bde1cf634be8bd7bb667751302 /core/src/main/java/de
parentf790b78b1a6c2aaf0fb789be9542d09ed2da47d2 (diff)
downloadAntennaPod-aa48cdb14c6ce23e07199ab706ae97d11af8035b.zip
Do not return true in handleKeycode if nothing was done
Diffstat (limited to 'core/src/main/java/de')
-rw-r--r--core/src/main/java/de/danoeh/antennapod/core/service/playback/PlaybackService.java11
1 files changed, 7 insertions, 4 deletions
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();