fix regression on notification before first play

android/src/main/java/co/broadcastapp/muckabout/RemoteStreamerPlugin.java

@@ -84,8 +84,6 @@ public void load() {
         dataSourceFactory = new DefaultDataSource.Factory(context);
         mediaSession = new MediaSessionCompat(context, "wnyc");
 
-        startMediaService();
-
         AudioAttributes audioAttributes = new AudioAttributes.Builder()
                 .setUsage(AudioAttributes.USAGE_MEDIA)
                 .setContentType(AudioAttributes.CONTENT_TYPE_MUSIC)
@@ -112,7 +110,18 @@ public void play(PluginCall call) {
             call.reject("URL is required");
             return;
         }
-
+
+        if (service == null) {
+            startMediaService();
+            while (service == null) {
+                try {
+                    Thread.sleep(100);
+                } catch (InterruptedException e) {
+                    e.printStackTrace();
+                }
+            }
+        }
+
         handler.post(() -> {
             releasePlayer();
             player = new ExoPlayer.Builder(getContext()).build();
@@ -162,8 +171,7 @@ public void onPlaybackStateChanged(int state) {
                         break;
                     case Player.STATE_ENDED:
                         stopUpdatingTime();
-                        service.setPlaybackState(PlaybackStateCompat.STATE_NONE);
-                        service.update();
+                        stop();
                         notifyListeners("stop", new JSObject());
                         break;
                 }
@@ -249,11 +257,17 @@ private void seekTo(Long position) {
 
     @PluginMethod
     public void stop(PluginCall call) {
+        stop();
+        call.resolve();
+    }
+
+    public void stop() {
         releasePlayer();
-        service.setPlaybackState(PlaybackStateCompat.STATE_NONE);
-        service.update();
+        if (service != null) {
+            // stop may be called before the service is started
+            service.destroy();
+        }
         notifyListeners("stop", new JSObject());
-        call.resolve();
     }
 
     @PluginMethod

android/src/main/java/co/broadcastapp/muckabout/RemoteStreamerService.java

@@ -17,6 +17,7 @@
 import android.support.v4.media.session.PlaybackStateCompat;
 
 import androidx.core.app.NotificationCompat;
+import androidx.core.content.ContextCompat;
 import androidx.media.session.MediaButtonReceiver;
 import androidx.media.app.NotificationCompat.MediaStyle;
 
@@ -146,6 +147,8 @@ public void connectAndInitialize(RemoteStreamerPlugin plugin, Intent intent) {
 
         public void destroy() {
             stopForeground(true);
+            //mediaSession.setActive(false);
+            notificationManager.cancel(NOTIFICATION_ID);
             stopSelf();
         }
 
@@ -218,11 +221,6 @@ public void setPlaybackSpeed(float playbackSpeed) {
         @SuppressLint("RestrictedApi")
         public void update() {
             if (possibleActionsUpdate) {
-                if (playbackState == PlaybackStateCompat.STATE_NONE) {
-                    mediaSession.setActive(false);
-                    notificationManager.cancel(NOTIFICATION_ID);
-                    return;
-                }
                 if (notificationBuilder != null) {
                     notificationBuilder.mActions.clear();
                 }

package.json

@@ -1,7 +1,7 @@
 {
   "name": "@nypublicradio/capacitor-remote-streamer",
   "repository": "git://github.com/nypublicradio/capacitor-remote-streamer.git",
-  "version": "0.2.5",
+  "version": "0.2.6",
   "description": "Stream remote HLS and MP3 streams on iOS and Android. Capacitor 6.",
   "main": "dist/plugin.cjs.js",
   "repository": {