Handle YouTube URL change during SDK load

Fixes cookpete/react-player#20
philip-luther · Nov 22, 2024 · 3bafcc7 · 3bafcc7
1 parent bb7d450
commit 3bafcc7
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/src/players/Base.js b/src/players/Base.js
@@ -38,7 +38,11 @@ export default class Base extends Component {
     if (this.props.playing || this.preloading) {
       this.preloading = false
       this.isReady = true
-      this.play()
+      if (this.loadOnReady) {
+        this.load(this.loadOnReady, this.props.playing)
+      } else {
+        this.play()
+      }
     }
   }
 }
diff --git a/src/players/YouTube.js b/src/players/YouTube.js
@@ -56,6 +56,11 @@ export default class YouTube extends Base {
       }
       return
     }
+    if (this.loadingSDK) {
+      this.loadOnReady = url
+      return
+    }
+    this.loadingSDK = true
     this.getSDK().then(YT => {
       this.player = new YT.Player(this.playerId, {
         width: '100%',
@@ -66,7 +71,10 @@ export default class YouTube extends Base {
           ...this.props.youtubeConfig.playerVars
         },
         events: {
-          onReady: this.onReady,
+          onReady: () => {
+            this.loadingSDK = false
+            this.onReady()
+          },
           onStateChange: this.onStateChange,
           onError: this.props.onError
         }