]> git.immae.eu Git - github/Chocobozzz/PeerTube.git/commitdiff
refactor(wt-plugin): create getVideoFile fn
authorkontrollanten <6680299+kontrollanten@users.noreply.github.com>
Mon, 7 Dec 2020 07:39:26 +0000 (08:39 +0100)
committerkontrollanten <6680299+kontrollanten@users.noreply.github.com>
Mon, 7 Dec 2020 07:43:57 +0000 (08:43 +0100)
client/src/assets/player/webtorrent/webtorrent-plugin.ts

index 287358ada93e5db5cf3f7c233c17791c24fba5e3..4eaaf51ecec01e67d862169b00e0526f3546a154 100644 (file)
@@ -122,8 +122,15 @@ class WebTorrentPlugin extends Plugin {
     return this.currentVideoFile ? this.currentVideoFile.resolution.id : -1
   }
 
+  private getVideoFile(): VideoFile {
+    const savedAverageBandwidth = getAverageBandwidthInStore()
+    return savedAverageBandwidth
+      ? this.getAppropriateFile(savedAverageBandwidth)
+      : this.pickAverageVideoFile()
+}
+
   updateVideoFile (
-    videoFile?: VideoFile,
+    videoFile: VideoFile,
     options: {
       forcePlay?: boolean,
       seek?: number,
@@ -131,14 +138,6 @@ class WebTorrentPlugin extends Plugin {
     } = {},
     done: () => void = () => { /* empty */ }
   ) {
-    // Automatically choose the adapted video file
-    if (videoFile === undefined) {
-      const savedAverageBandwidth = getAverageBandwidthInStore()
-      videoFile = savedAverageBandwidth
-        ? this.getAppropriateFile(savedAverageBandwidth)
-        : this.pickAverageVideoFile()
-    }
-
     if (videoFile === undefined) {
       throw Error(`Can't update video file since videoFile is undefined.`)
     }
@@ -427,7 +426,7 @@ class WebTorrentPlugin extends Plugin {
     if (this.autoplay) {
       this.player.posterImage.hide()
 
-      return this.updateVideoFile(undefined, { forcePlay: true, seek: this.startTime })
+      return this.updateVideoFile(this.getVideoFile(), { forcePlay: true, seek: this.startTime })
     }
 
     // Proxy first play
@@ -436,7 +435,7 @@ class WebTorrentPlugin extends Plugin {
       this.player.addClass('vjs-has-big-play-button-clicked')
       this.player.play = oldPlay
 
-      this.updateVideoFile(undefined, { forcePlay: true, seek: this.startTime })
+      this.updateVideoFile(this.getVideoFile(), { forcePlay: true, seek: this.startTime })
     }
   }