chore: update Promise.reject()(s) across codebase to throw Error("message"), instead of regular string

src/components/accessSchedule/accessSchedule.js

@@ -74,7 +74,7 @@ export function show(options) {
             if (dlg.submitted) {
                 resolve(options.schedule);
             } else {
-                reject();
+                reject(new Error('AccessSchedule closed without resolving'));
             }
         });
         dlg.querySelector('.btnCancel').addEventListener('click', () => {

src/components/collectionEditor/collectionEditor.js

@@ -258,7 +258,7 @@ class CollectionEditor {
                 return Promise.resolve();
             }
 
-            return Promise.reject();
+            return Promise.reject(new Error('CollectionEditorError'));
         });
     }
 }

src/components/dialog/dialog.js

@@ -114,7 +114,7 @@ function showDialog(options = { dialogOptions: {}, buttons: [] }) {
         if (dialogResult) {
             return dialogResult;
         } else {
-            return Promise.reject();
+            return Promise.reject(new Error('ShowDialogError'));
         }
     });
 }

src/components/directorybrowser/directorybrowser.js

@@ -165,15 +165,15 @@ function validatePath(path, validateWriteable, apiClient) {
         if (response) {
             if (response.status === 404) {
                 alertText(globalize.translate('PathNotFound'));
-                return Promise.reject();
+                return Promise.reject(new Error('PathNotFoundError'));
             }
             if (response.status === 500) {
                 if (validateWriteable) {
                     alertText(globalize.translate('WriteAccessRequired'));
                 } else {
                     alertText(globalize.translate('PathNotFound'));
                 }
-                return Promise.reject();
+                return Promise.reject(new Error('ValidatePathError'));
             }
         }
         return Promise.resolve();

src/components/guide/guide-settings.js

@@ -125,7 +125,7 @@ function showEditor(options) {
             if (settingsChanged) {
                 resolve();
             } else {
-                reject();
+                reject(new Error('ShowEditorError'));
             }
         });
 

src/components/htmlMediaHelper.js

@@ -102,7 +102,7 @@ export function handleHlsJsMediaError(instance, reject) {
         console.error('cannot recover, last media error recovery failed ...');
 
         if (reject) {
-            reject();
+            reject(new Error('HlsJsMediaError'));
         } else {
             onErrorInternal(instance, MediaError.FATAL_HLS_ERROR);
         }
@@ -214,7 +214,7 @@ export function playWithPromise(elem, onErrorFn) {
             });
     } catch (err) {
         console.error('error calling video.play: ' + err);
-        return Promise.reject();
+        return Promise.reject(new Error('VideoPlayError'));
     }
 }
 
@@ -263,7 +263,7 @@ export function bindEventsToHlsPlayer(instance, hls, elem, onErrorFn, resolve, r
     hls.on(Hls.Events.MANIFEST_PARSED, function () {
         playWithPromise(elem, onErrorFn).then(resolve, function () {
             if (reject) {
-                reject();
+                reject(new Error('BindEventsToHlsPlayerError'));
                 reject = null;
             }
         });
@@ -328,7 +328,7 @@ export function bindEventsToHlsPlayer(instance, hls, elem, onErrorFn, resolve, r
                     hls.destroy();
 
                     if (reject) {
-                        reject();
+                        reject(new Error('HlsError'));
                         reject = null;
                     } else {
                         onErrorInternal(instance, MediaError.FATAL_HLS_ERROR);

src/components/imageDownloader/imageDownloader.js

@@ -369,7 +369,7 @@ function onDialogClosed() {
     if (hasChanges) {
         currentResolve();
     } else {
-        currentReject();
+        currentReject(new Error('OnDialogClosedError'));
     }
 }
 

src/components/imageeditor/imageeditor.js

@@ -439,7 +439,7 @@ function showEditor(options, resolve, reject) {
             if (hasChanges) {
                 resolve();
             } else {
-                reject();
+                reject(new Error('ShowEditorError'));
             }
         });
 

src/components/itemContextMenu.js

@@ -633,7 +633,7 @@ function executeCommand(item, id, options) {
                 deleteSeriesTimer(apiClient, item, resolve, id);
                 break;
             default:
-                reject();
+                reject(new Error('ExecuteCommandError'));
                 break;
         }
     });

src/components/itemidentifier/itemidentifier.js

@@ -397,7 +397,7 @@ function onDialogClosed() {
     if (hasChanges) {
         currentResolve();
     } else {
-        currentReject();
+        currentReject(new Error('OnDialogClosedError'));
     }
 }
 

src/components/lyricseditor/lyricseditor.js

@@ -371,7 +371,7 @@ function showEditorInternal(itemId, serverId) {
                 if (hasChanges) {
                     resolve();
                 } else {
-                    reject();
+                    reject(new Error('ShowEditorInternalError'));
                 }
             });
 

src/components/metadataEditor/personEditor.js

@@ -58,7 +58,7 @@ function show(person) {
             if (submitted) {
                 resolve(person);
             } else {
-                reject();
+                reject(new Error('PersonEditorError'));
             }
         });
 

src/components/playback/playbackmanager.js

@@ -506,7 +506,7 @@ function getOptimalMediaSource(apiClient, item, versions) {
     });
 
     if (!promises.length) {
-        return Promise.reject();
+        return Promise.reject(new Error('GetOptimalMediaSourceError'));
     }
 
     return Promise.all(promises).then(function (results) {
@@ -2286,7 +2286,7 @@ export class PlaybackManager {
             // If it's still null then there's nothing to play
             if (!firstItem) {
                 showPlaybackInfoErrorMessage(self, `PlaybackError.${MediaError.NO_MEDIA_ERROR}`);
-                return Promise.reject();
+                return Promise.reject(new Error(MediaError.NO_MEDIA_ERROR));
             }
 
             if (firstItem.MediaType === 'Photo' || firstItem.MediaType === 'Book') {
@@ -2335,7 +2335,7 @@ export class PlaybackManager {
             if (item.IsPlaceHolder) {
                 loading.hide();
                 showPlaybackInfoErrorMessage(self, 'PlaybackErrorPlaceHolder');
-                return Promise.reject();
+                return Promise.reject(new Error('PlaybackInfoError'));
             }
 
             // Normalize defaults to simplfy checks throughout the process
@@ -2384,7 +2384,7 @@ export class PlaybackManager {
         function onInterceptorRejection() {
             cancelPlayback();
 
-            return Promise.reject();
+            return Promise.reject(new Error('InterceptorRejectionError'));
         }
 
         function onPlaybackRejection(e) {
@@ -2402,7 +2402,7 @@ export class PlaybackManager {
 
             showPlaybackInfoErrorMessage(self, displayErrorCode);
 
-            return Promise.reject();
+            return Promise.reject(new Error('PlaybackRejectionError'));
         }
 
         function destroyPlayer(player) {
@@ -2952,11 +2952,11 @@ export class PlaybackManager {
                             }
                         } else {
                             showPlaybackInfoErrorMessage(self, `PlaybackError.${MediaError.NO_MEDIA_ERROR}`);
-                            return Promise.reject();
+                            return Promise.reject(new Error('GetOptimalMediaSourceError'));
                         }
                     });
                 } else {
-                    return Promise.reject();
+                    return Promise.reject(new Error('GetPlaybackInfoError'));
                 }
             });
         }
@@ -3737,7 +3737,7 @@ export class PlaybackManager {
         const nextItem = this._playQueueManager.getNextItemInfo();
 
         if (!nextItem?.item) {
-            return Promise.reject();
+            return Promise.reject(new Error('GetNextItemError'));
         }
 
         const apiClient = ServerConnections.getApiClient(nextItem.item.ServerId);
@@ -3907,7 +3907,7 @@ export class PlaybackManager {
             });
         }
 
-        return Promise.reject();
+        return Promise.reject(new Error('PlayTrailersError'));
     }
 
     getSubtitleUrl(textStream, serverId) {

src/components/playback/playersettingsmenu.js

@@ -145,7 +145,7 @@ function showAspectRatioMenu(player, btn) {
             return Promise.resolve();
         }
 
-        return Promise.reject();
+        return Promise.reject(new Error('ShowAspectRatioMenuError'));
     });
 }
 
@@ -167,7 +167,7 @@ function showPlaybackRateMenu(player, btn) {
             return Promise.resolve();
         }
 
-        return Promise.reject();
+        return Promise.reject(new Error('ShowPlaybackRateMenuError'));
     });
 }
 
@@ -282,7 +282,7 @@ function handleSelectedOption(id, options, player) {
             break;
     }
 
-    return Promise.reject();
+    return Promise.reject(new Error('HandleSelectedOptionError'));
 }
 
 export default {

src/components/playlisteditor/playlisteditor.ts

@@ -429,7 +429,7 @@ export class PlaylistEditor {
                 return Promise.resolve();
             }
 
-            return Promise.reject(new Error());
+            return Promise.reject(new Error('ShowPlaylistEditorError'));
         });
     }
 }

src/components/prompt/prompt.js

@@ -87,7 +87,7 @@ export default (() => {
             if (submitValue) {
                 return submitValue;
             } else {
-                return Promise.reject();
+                return Promise.reject(new Error('PromptError'));
             }
         });
     }

src/components/recordingcreator/recordingcreator.js

@@ -174,7 +174,7 @@ function showEditor(itemId, serverId) {
             if (currentRecordingFields?.hasChanged()) {
                 resolve();
             } else {
-                reject();
+                reject(new Error('CurrentRecordingFieldsError'));
             }
         });
 

src/components/recordingcreator/recordinghelper.js

@@ -162,7 +162,7 @@ function toggleRecording(serverId, programId, timerId, timerStatus, seriesTimerI
         // schedule recording
         return createRecording(apiClient, programId);
     } else {
-        return Promise.reject();
+        return Promise.reject(new Error('ToggleRecordingError'));
     }
 }
 

src/components/recordingcreator/seriesrecordingeditor.js

@@ -236,7 +236,7 @@ function showEditor(itemId, serverId, options) {
                     deleted: recordingDeleted
                 });
             } else {
-                reject();
+                reject(new Error('RecordingUpdatedError'));
             }
         });
 

src/components/sortmenu/sortmenu.js

@@ -100,7 +100,7 @@ class SortMenu {
                     return;
                 }
 
-                reject();
+                reject(new Error('SortMenuError'));
             });
         });
     }

src/components/subtitleeditor/subtitleeditor.js

@@ -468,7 +468,7 @@ function showEditorInternal(itemId, serverId) {
                 if (hasChanges) {
                     resolve();
                 } else {
-                    reject();
+                    reject(new Error('SubtitleEditorError'));
                 }
             });
 

src/components/tunerPicker.js

@@ -175,7 +175,7 @@ function TunerPicker() {
                 return Promise.resolve(deviceResult);
             }
 
-            return Promise.reject();
+            return Promise.reject(new Error('TunerPickerError'));
         });
     };
 }

src/components/viewContainer.js

@@ -223,7 +223,7 @@ export function tryRestoreView(options) {
         }
     }
 
-    return Promise.reject();
+    return Promise.reject(new Error('TryRestoreViewError'));
 }
 
 function triggerDestroy(view) {

src/plugins/bookPlayer/plugin.js

@@ -360,7 +360,7 @@ export class BookPlayer {
                     this.bindEvents();
 
                     return this.rendition.book.locations.generate(1024).then(async () => {
-                        if (this.cancellationToken) reject();
+                        if (this.cancellationToken) reject(new Error('GenerateLocationsError'));
 
                         const percentageTicks = options.startPositionTicks / 10000000;
                         if (percentageTicks !== 0.0) {
@@ -380,7 +380,7 @@ export class BookPlayer {
                     });
                 }, () => {
                     console.error('failed to display epub');
-                    return reject();
+                    return reject(new Error('SetCurrentSrcError'));
                 });
             });
         });

src/plugins/chromecastPlayer/plugin.js

@@ -31,7 +31,7 @@ function sendConnectionResult(isOk) {
             resolve();
         }
     } else if (reject) {
-        reject();
+        reject(new Error('SendConnectionResultError'));
     } else {
         playbackManager.removeActivePlayer(PlayerName);
     }

src/plugins/htmlAudioPlayer/plugin.js

@@ -62,7 +62,7 @@ function requireHlsPlayer(callback) {
 
 function enableHlsPlayer(url, item, mediaSource, mediaType) {
     if (!htmlMediaHelper.enableHlsJsPlayer(mediaSource.RunTimeTicks, mediaType)) {
-        return Promise.reject();
+        return Promise.reject(new Error('EnableHlsPlayerError'));
     }
 
     if (url.indexOf('.m3u8') !== -1) {
@@ -80,7 +80,7 @@ function enableHlsPlayer(url, item, mediaSource, mediaType) {
                 if (contentType === 'application/vnd.apple.mpegurl' || contentType === 'application/x-mpegurl') {
                     resolve();
                 } else {
-                    reject();
+                    reject(new Error('HlsPlayerContentTypeError'));
                 }
             }, reject);
         });

src/plugins/playAccessValidation/plugin.js

@@ -32,11 +32,11 @@ class PlayAccessValidation {
 
             // reject but don't show an error message
             if (!options.fullscreen) {
-                return Promise.reject();
+                return Promise.reject(new Error('PlayAccessValidationInterceptError'));
             }
 
             return showErrorMessage()
-                .finally(() => Promise.reject());
+                .finally(() => Promise.reject(new Error('ShowPlayAccessValidationInterceptError')));
         });
     }
 }

src/plugins/syncPlay/core/QueueCore.js

@@ -203,7 +203,7 @@ class QueueCore {
     startPlayback(apiClient) {
         if (!this.manager.isFollowingGroupPlayback()) {
             console.debug('SyncPlay startPlayback: ignoring, not following playback.');
-            return Promise.reject();
+            return Promise.reject(new Error('IsNotFollowingGroupPlaybackError'));
         }
 
         if (this.isPlaylistEmpty()) {