Fixed #452. Player state is now accurate without interaction

.gitignore

@@ -7,11 +7,12 @@ icon*
 .vscode
 .sass-cache
 logs/
+.nova/
 
 # AmplitudeJS Specifics
 songs/*
 examples/testing/*
 src/PENDING
 examples/songs
 coverage/
-ui-tests/*
+ui-tests/

src/core/core.js

@@ -46,6 +46,12 @@ import Debug from "../utilities/debug.js";
  */
 import Visualizations from "../fx/visualizations.js";
 
+/**
+ * Import the Config State module.
+ * @module utilities/configState
+ */
+ import ConfigState from "../utilities/configState.js";
+
 /**
  * Interacts directly with native functions of the Audio element. Logic
  * leading up to these methods are handled by click handlers which call
@@ -102,6 +108,11 @@ let Core = (function() {
     }
     config.audio.play();
     config.audio.playbackRate = config.playback_speed;
+
+    /*
+      Sets the state of the player.
+    */
+    ConfigState.setPlayerState();
   }
 
   /**
@@ -123,14 +134,20 @@ let Core = (function() {
 			Flag that pause button was clicked.
 		*/
     config.paused = true;
-
+
+
     /*
 			If the song is live, we disconnect the stream so we aren't
 			saving it to memory.
 		*/
     if (config.active_metadata.live) {
       disconnectStream();
     }
+
+    /*
+      Sets the state of the player.
+    */
+    ConfigState.setPlayerState();
   }
 
   /**
@@ -164,6 +181,11 @@ let Core = (function() {
       disconnectStream();
     }
 
+    /*
+      Sets the state of the player.
+    */
+    ConfigState.setPlayerState();
+
     /*
 			Run the stop callback
 		*/

src/events/pause.js

@@ -82,8 +82,6 @@ let Pause = (function() {
       if (playlistAttribute != null && songIndexAttribute != null) {
         handleSongInPlaylistPause(playlistAttribute, songIndexAttribute);
       }
-
-      ConfigState.setPlayerState();
     }
   }
 

src/events/play.js

@@ -94,8 +94,6 @@ let Play = (function() {
       if (playlistAttribute != null && songIndexAttribute != null) {
         handleSongInPlaylistPlay(playlistAttribute, songIndexAttribute);
       }
-
-      ConfigState.setPlayerState();
     }
   }
 

src/events/playPause.js

@@ -91,8 +91,6 @@ let PlayPause = (function() {
       if (playlist != null && song != null) {
         handleSongInPlaylistPlayPause(playlist, song);
       }
-
-      ConfigState.setPlayerState();
     }
   }
 

src/events/stop.js

@@ -49,11 +49,6 @@ let Stop = (function() {
 				Stops the active song.
 			*/
       Core.stop();
-
-      /*
-        Set the state of the player.
-      */
-      ConfigState.setPlayerState();
     }
   }
 

src/index.js

@@ -761,10 +761,6 @@ let Amplitude = (function() {
 		*/
     TimeElements.resetDurationTimes();
 
-    /*
-      Sets the state of the player.
-    */
-    ConfigState.setPlayerState();
   }
 
   /**
@@ -802,11 +798,6 @@ let Amplitude = (function() {
 		*/
     Core.play();
 
-    /*
-      Sets the state of the player.
-    */
-    ConfigState.setPlayerState();
-
     /*
 			Sync all of the play pause buttons.
 		*/
@@ -858,11 +849,6 @@ let Amplitude = (function() {
 			Play the song
 		*/
     Core.play();
-
-    /*
-			Set the state of the player
-		*/
-    ConfigState.setPlayerState();
   }
 
   /**
@@ -876,8 +862,6 @@ let Amplitude = (function() {
    */
   function play() {
     Core.play();
-
-    ConfigState.setPlayerState();
   }
 
   /**
@@ -891,8 +875,6 @@ let Amplitude = (function() {
    */
   function pause() {
     Core.pause();
-
-    ConfigState.setPlayerState();
   }
 
   /**
@@ -905,8 +887,6 @@ let Amplitude = (function() {
    */
   function stop() {
     Core.stop();
-
-    ConfigState.setPlayerState();
   }
 
   /**