Do not autoplay in mobile in setMusic

ericdrobinson · ericdrobinson · commit 17c80e4083a2 · 2017-12-12T13:40:39.000-05:00
This change appears to have been made to support playlist auto-advance in mobile. This was a workaround to handle the autoplay guard lower in the function which was removed in an earlier commit on this branch. The play call is now handled external to the `setMusic` call, which allows external logic to determine whether play should be called on mobile or not.
diff --git a/src/APlayer.js b/src/APlayer.js
@@ -478,7 +478,6 @@ class APlayer {
         // get this audio object
         if (this.isMobile && this.audio) {
             this.audio.src = this.music.url;
-            this.play();
         }
         else if (!this.isMobile && this.audios[indexMusic]) {
             this.audio = this.audios[indexMusic];

Original file line number	Diff line number	Diff line change
`@@ -478,7 +478,6 @@ class APlayer {`
`478`	`478`	`// get this audio object`
`479`	`479`	`if (this.isMobile && this.audio) {`
`480`	`480`	`this.audio.src = this.music.url;`
`481`		`- this.play();`
`482`	`481`	`}`
`483`	`482`	`else if (!this.isMobile && this.audios[indexMusic]) {`
`484`	`483`	`this.audio = this.audios[indexMusic];`