From 350f3951d548b9cc3e592deb288617447af7952b Mon Sep 17 00:00:00 2001 From: Ryan Heise Date: Sun, 29 Dec 2019 21:15:43 +1100 Subject: [PATCH] ensureStopped in setUrl. Add missing success()'s. --- .../src/main/java/com/ryanheise/just_audio/AudioPlayer.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/android/src/main/java/com/ryanheise/just_audio/AudioPlayer.java b/android/src/main/java/com/ryanheise/just_audio/AudioPlayer.java index 36fabf8..f352029 100644 --- a/android/src/main/java/com/ryanheise/just_audio/AudioPlayer.java +++ b/android/src/main/java/com/ryanheise/just_audio/AudioPlayer.java @@ -203,6 +203,7 @@ public class AudioPlayer implements MethodCallHandler { if (state != PlaybackState.none && state != PlaybackState.stopped && state != PlaybackState.completed) { throw new IllegalStateException("Can call setUrl only from none/stopped/completed states"); } + ensureStopped(); transition(PlaybackState.connecting); this.url = url; if (extractor != null) { @@ -277,9 +278,11 @@ public class AudioPlayer implements MethodCallHandler { public void stop(final Result result) { switch (state) { case stopped: + result.success(null); break; case completed: transition(PlaybackState.stopped); + result.success(null); break; // TODO: Allow stopping from buffered state. case playing: