From eff386ffd813e7b7f48145fe874523a37c579939 Mon Sep 17 00:00:00 2001
From: Bill Thornton <thornbill@users.noreply.github.com>
Date: Mon, 15 Sep 2025 08:41:57 -0400
Subject: [PATCH] Remove unnecessary dynamic imports

---
 src/plugins/htmlVideoPlayer/plugin.js | 269 +++++++++++++-------------
 1 file changed, 132 insertions(+), 137 deletions(-)

diff --git a/src/plugins/htmlVideoPlayer/plugin.js b/src/plugins/htmlVideoPlayer/plugin.js
index 1a6704857f..0cb7bb419b 100644
--- a/src/plugins/htmlVideoPlayer/plugin.js
+++ b/src/plugins/htmlVideoPlayer/plugin.js
@@ -3,8 +3,10 @@ import debounce from 'lodash-es/debounce';
 import Screenfull from 'screenfull';
 
 import { useCustomSubtitles } from 'apps/stable/features/playback/utils/subtitleStyles';
+import subtitleAppearanceHelper from 'components/subtitlesettings/subtitleappearancehelper';
 import { AppFeature } from 'constants/appFeature';
 import { ServerConnections } from 'lib/jellyfin-apiclient';
+import { currentSettings as userSettings } from 'scripts/settings/userSettings';
 import { MediaError } from 'types/mediaError';
 
 import browser from '../../scripts/browser';
@@ -331,8 +333,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     incrementFetchQueue() {
         if (this.#fetchQueue <= 0) {
             this.isFetching = true;
@@ -343,8 +345,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     decrementFetchQueue() {
         this.#fetchQueue--;
 
@@ -355,8 +357,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     updateVideoUrl(streamInfo) {
         const mediaSource = streamInfo.mediaSource;
         const item = streamInfo.item;
@@ -407,8 +409,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     setSrcWithFlvJs(elem, options, url) {
         return import('flv.js').then(({ default: flvjs }) => {
             const flvPlayer = flvjs.createPlayer({
@@ -433,8 +435,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     setSrcWithHlsJs(elem, options, url) {
         return new Promise((resolve, reject) => {
             requireHlsPlayer(async () => {
@@ -474,8 +476,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     async setCurrentSrc(elem, options) {
         elem.removeEventListener('error', this.onError);
 
@@ -578,8 +580,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     getTextTracks() {
         const videoElement = this.#mediaElement;
         if (videoElement) {
@@ -596,8 +598,8 @@ export class HtmlVideoPlayer {
     setSubtitleOffset = debounce(this._setSubtitleOffset, 100);
 
     /**
-         * @private
-         */
+     * @private
+     */
     _setSubtitleOffset(offset) {
         const offsetValue = parseFloat(offset);
 
@@ -625,8 +627,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     updateCurrentTrackOffset(offsetValue, currentTrackIndex = PRIMARY_TEXT_TRACK_INDEX) {
         let offsetToCompare = this.#currentTrackOffset;
         if (this.isSecondaryTrack(currentTrackIndex)) {
@@ -651,19 +653,19 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         * These browsers will not clear the existing active cue when setting an offset
-         * for native TextTracks.
-         * Any previous text tracks that are on the screen when the offset changes will remain next
-         * to the new tracks until they reach the end time of the new offset's instance of the track.
-         */
+     * @private
+     * These browsers will not clear the existing active cue when setting an offset
+     * for native TextTracks.
+     * Any previous text tracks that are on the screen when the offset changes will remain next
+     * to the new tracks until they reach the end time of the new offset's instance of the track.
+     */
     requiresHidingActiveCuesOnOffsetChange() {
         return !!browser.firefox;
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     hideTextTrackWithActiveCues(currentTrack) {
         if (currentTrack.activeCues) {
             currentTrack.mode = 'hidden';
@@ -671,11 +673,11 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * Forces the active cue to clear by disabling then re-enabling the track.
-         * The track mode is reverted inside of a 0ms timeout to free up the track
-         * and allow it to disable and clear the active cue.
-         * @private
-         */
+     * Forces the active cue to clear by disabling then re-enabling the track.
+     * The track mode is reverted inside of a 0ms timeout to free up the track
+     * and allow it to disable and clear the active cue.
+     * @private
+     */
     forceClearTextTrackActiveCues(currentTrack) {
         if (currentTrack.activeCues) {
             currentTrack.mode = 'disabled';
@@ -686,8 +688,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     setTextTrackSubtitleOffset(currentTrack, offsetValue, currentTrackIndex) {
         if (currentTrack.cues) {
             offsetValue = this.updateCurrentTrackOffset(offsetValue, currentTrackIndex);
@@ -713,8 +715,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     setTrackEventsSubtitleOffset(trackEvents, offsetValue, currentTrackIndex) {
         if (Array.isArray(trackEvents)) {
             offsetValue = this.updateCurrentTrackOffset(offsetValue, currentTrackIndex) * 1e7; // ticks
@@ -741,8 +743,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     isAudioStreamSupported(stream, deviceProfile, container) {
         const codec = (stream.Codec || '').toLowerCase();
 
@@ -765,8 +767,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     getSupportedAudioStreams() {
         const profile = this.#lastProfile;
 
@@ -808,8 +810,8 @@ export class HtmlVideoPlayer {
         // https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/audioTracks
 
         /**
-             * @type {ArrayLike<any>|any[]}
-             */
+         * @type {ArrayLike<any>|any[]}
+         */
         const elemAudioTracks = elem.audioTracks || [];
         console.debug(`found ${elemAudioTracks.length} audio tracks`);
 
@@ -891,26 +893,26 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         * @param e {Event} The event received from the `<video>` element
-         */
+     * @private
+     * @param e {Event} The event received from the `<video>` element
+     */
     onEnded = (e) => {
         /**
-             * @type {HTMLMediaElement}
-             */
+         * @type {HTMLMediaElement}
+         */
         const elem = e.target;
         this.destroyCustomTrack(elem);
         onEndedInternal(this, elem, this.onError);
     };
 
     /**
-         * @private
-         * @param e {Event} The event received from the `<video>` element
-         */
+     * @private
+     * @param e {Event} The event received from the `<video>` element
+     */
     onTimeUpdate = (e) => {
         /**
-             * @type {HTMLMediaElement}
-             */
+         * @type {HTMLMediaElement}
+         */
         const elem = e.target;
         // get the player position and the transcoding offset
         const time = elem.currentTime;
@@ -934,21 +936,21 @@ export class HtmlVideoPlayer {
     };
 
     /**
-         * @private
-         * @param e {Event} The event received from the `<video>` element
-         */
+     * @private
+     * @param e {Event} The event received from the `<video>` element
+     */
     onVolumeChange = (e) => {
         /**
-             * @type {HTMLMediaElement}
-             */
+         * @type {HTMLMediaElement}
+         */
         const elem = e.target;
         saveVolume(elem.volume);
         Events.trigger(this, 'volumechange');
     };
 
     /**
-         * @private
-         */
+     * @private
+     */
     onNavigatedToOsd = () => {
         const dlg = this.#videoDialog;
         if (dlg) {
@@ -959,8 +961,8 @@ export class HtmlVideoPlayer {
     };
 
     /**
-         * @private
-         */
+     * @private
+     */
     onStartedAndNavigatedToOsd() {
         // If this causes a failure during navigation we end up in an awkward UI state
         this.setCurrentTrackElement(this.#subtitleTrackIndexToSetOnPlaying);
@@ -971,23 +973,23 @@ export class HtmlVideoPlayer {
 
         if (this.#secondarySubtitleTrackIndexToSetOnPlaying != null && this.#secondarySubtitleTrackIndexToSetOnPlaying >= 0) {
             /**
-                 * Using a 0ms timeout to set the secondary subtitles because of some weird race condition when
-                 * setting both primary and secondary tracks at the same time.
-                 * The `TextTrack` content and cues will somehow get mixed up and each track will play a mix of both languages.
-                 * Putting this in a timeout fixes it completely.
-                 */
+             * Using a 0ms timeout to set the secondary subtitles because of some weird race condition when
+             * setting both primary and secondary tracks at the same time.
+             * The `TextTrack` content and cues will somehow get mixed up and each track will play a mix of both languages.
+             * Putting this in a timeout fixes it completely.
+             */
             setTimeout(() => this.setSecondarySubtitleStreamIndex(this.#secondarySubtitleTrackIndexToSetOnPlaying), 0);
         }
     }
 
     /**
-         * @private
-         * @param e {Event} The event received from the `<video>` element
-         */
+     * @private
+     * @param e {Event} The event received from the `<video>` element
+     */
     onPlaying = (e) => {
         /**
-             * @type {HTMLMediaElement}
-             */
+         * @type {HTMLMediaElement}
+         */
         const elem = e.target;
         if (!this.#started) {
             this.#started = true;
@@ -1016,15 +1018,15 @@ export class HtmlVideoPlayer {
     };
 
     /**
-         * @private
-         */
+     * @private
+     */
     onPlay = () => {
         Events.trigger(this, 'unpause');
     };
 
     /**
-         * @private
-         */
+     * @private
+     */
     ensureValidVideo(elem) {
         if (elem !== this.#mediaElement) {
             return;
@@ -1042,22 +1044,22 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     onClick = () => {
         Events.trigger(this, 'click');
     };
 
     /**
-         * @private
-         */
+     * @private
+     */
     onDblClick = () => {
         Events.trigger(this, 'dblclick');
     };
 
     /**
-         * @private
-         */
+     * @private
+     */
     onPause = () => {
         Events.trigger(this, 'pause');
     };
@@ -1067,13 +1069,13 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         * @param e {Event} The event received from the `<video>` element
-         */
+     * @private
+     * @param e {Event} The event received from the `<video>` element
+     */
     onError = (e) => {
         /**
-             * @type {HTMLMediaElement}
-             */
+         * @type {HTMLMediaElement}
+         */
         const elem = e.target;
         const errorCode = elem.error ? (elem.error.code || 0) : 0;
         const errorMessage = elem.error ? (elem.error.message || '') : '';
@@ -1113,8 +1115,8 @@ export class HtmlVideoPlayer {
     };
 
     /**
-         * @private
-         */
+     * @private
+     */
     destroyCustomRenderedTrackElements(targetTrackIndex) {
         if (this.isPrimaryTrack(targetTrackIndex)) {
             if (this.#videoSubtitlesElem) {
@@ -1138,8 +1140,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     destroyNativeTracks(videoElement, targetTrackIndex) {
         if (videoElement) {
             const destroySingleTrack = typeof targetTrackIndex === 'number';
@@ -1158,8 +1160,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     destroyStoredTrackInfo(targetTrackIndex) {
         if (this.isPrimaryTrack(targetTrackIndex)) {
             this.#customTrackIndex = -1;
@@ -1176,8 +1178,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     destroyCustomTrack(videoElement, targetTrackIndex) {
         this.destroyCustomRenderedTrackElements(targetTrackIndex);
         this.destroyNativeTracks(videoElement, targetTrackIndex);
@@ -1197,8 +1199,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     fetchSubtitlesUwp(track) {
         return Windows.Storage.StorageFile.getFileFromPathAsync(track.Path).then(function (storageFile) {
             return Windows.Storage.FileIO.readTextAsync(storageFile);
@@ -1208,8 +1210,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     async fetchSubtitles(track, item) {
         if (window.Windows && itemHelper.isLocalItem(item)) {
             return this.fetchSubtitlesUwp(track, item);
@@ -1230,8 +1232,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     setTrackForDisplay(videoElement, track, targetTextTrackIndex = PRIMARY_TEXT_TRACK_INDEX) {
         if (!track) {
             // Destroy all tracks by passing undefined if there is no valid primary track
@@ -1263,8 +1265,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     renderSsaAss(videoElement, track, item) {
         const supportedFonts = ['application/vnd.ms-opentype', 'application/x-truetype-font', 'font/otf', 'font/ttf', 'font/woff', 'font/woff2'];
         const availableFonts = [];
@@ -1359,11 +1361,10 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     renderSubtitlesWithCustomElement(videoElement, track, item, targetTextTrackIndex) {
-        Promise.all([import('../../scripts/settings/userSettings'), this.fetchSubtitles(track, item)]).then((results) => {
-            const [userSettings, subtitleData] = results;
+        this.fetchSubtitles(track, item).then((subtitleData) => {
             const subtitleAppearance = userSettings.getSubtitleAppearanceSettings();
             const subtitleVerticalPosition = parseInt(subtitleAppearance.verticalPosition, 10);
 
@@ -1399,20 +1400,18 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     setSubtitleAppearance(elem, innerElem) {
-        Promise.all([import('../../scripts/settings/userSettings'), import('../../components/subtitlesettings/subtitleappearancehelper')]).then(([userSettings, subtitleAppearanceHelper]) => {
-            subtitleAppearanceHelper.applyStyles({
-                text: innerElem,
-                window: elem
-            }, userSettings.getSubtitleAppearanceSettings());
-        });
+        subtitleAppearanceHelper.applyStyles({
+            text: innerElem,
+            window: elem
+        }, userSettings.getSubtitleAppearanceSettings());
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     getCueCss(appearance, selector) {
         return `${selector}::cue {
                 ${appearance.text.map((s) => s.value !== undefined && s.value !== '' ? `${s.name}:${s.value}!important;` : '').join('')}
@@ -1420,29 +1419,25 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     setCueAppearance() {
-        Promise.all([import('../../scripts/settings/userSettings'), import('../../components/subtitlesettings/subtitleappearancehelper')]).then(([userSettings, subtitleAppearanceHelper]) => {
-            const elementId = `${this.id}-cuestyle`;
+        const elementId = `${this.id}-cuestyle`;
 
-            let styleElem = document.querySelector(`#${elementId}`);
-            if (!styleElem) {
-                styleElem = document.createElement('style');
-                styleElem.id = elementId;
-                document.getElementsByTagName('head')[0].appendChild(styleElem);
-            }
+        let styleElem = document.querySelector(`#${elementId}`);
+        if (!styleElem) {
+            styleElem = document.createElement('style');
+            styleElem.id = elementId;
+            document.getElementsByTagName('head')[0].appendChild(styleElem);
+        }
 
-            styleElem.innerHTML = this.getCueCss(subtitleAppearanceHelper.getStyles(userSettings.getSubtitleAppearanceSettings()), '.htmlvideoplayer');
-        });
+        styleElem.innerHTML = this.getCueCss(subtitleAppearanceHelper.getStyles(userSettings.getSubtitleAppearanceSettings()), '.htmlvideoplayer');
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     async renderTracksEvents(videoElement, track, item, targetTextTrackIndex = PRIMARY_TEXT_TRACK_INDEX) {
-        const { currentSettings: userSettings } = await import('../../scripts/settings/userSettings');
-
         if (!itemHelper.isLocalItem(item) || track.IsExternal) {
             const format = (track.Codec || '').toLowerCase();
             if (format === 'ssa' || format === 'ass') {
@@ -1513,8 +1508,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     updateSubtitleText(timeMs) {
         const allTrackEvents = [this.#currentTrackEvents, this.#currentSecondaryTrackEvents];
         const subtitleTextElements = [this.#videoSubtitlesElem, this.#videoSecondarySubtitlesElem];
@@ -1545,8 +1540,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     setCurrentTrackElement(streamIndex, targetTextTrackIndex) {
         console.debug(`setting new text track index to: ${streamIndex}`);
 
@@ -1606,8 +1601,8 @@ export class HtmlVideoPlayer {
     }
 
     /**
-         * @private
-         */
+     * @private
+     */
     createMediaElement(options) {
         const dlg = document.querySelector('.videoPlayerContainer');