From 27cd24f5fc994f026ca1f176811f48ba15795c71 Mon Sep 17 00:00:00 2001 From: sancodes Date: Fri, 8 Apr 2022 19:10:11 -0700 Subject: [PATCH] fix return statements --- src/components/groupedcards.js | 3 +- .../libraryoptionseditor.js | 3 +- src/components/tvproviders/schedulesdirect.js | 6 ++- src/controllers/itemDetails/index.js | 40 ++++++++++++------- src/controllers/list.js | 2 +- src/controllers/playback/video/index.js | 9 +++-- .../session/forgotPassword/index.js | 9 +++-- .../session/resetPassword/index.js | 3 +- src/scripts/libraryMenu.js | 6 ++- 9 files changed, 53 insertions(+), 28 deletions(-) diff --git a/src/components/groupedcards.js b/src/components/groupedcards.js index 25dd784305..4662648485 100644 --- a/src/components/groupedcards.js +++ b/src/components/groupedcards.js @@ -23,7 +23,8 @@ import ServerConnections from './ServerConnections'; if (!actionableParent || actionableParent.classList.contains('cardContent')) { apiClient.getJSON(apiClient.getUrl('Users/' + userId + '/Items/Latest', options)).then(function (items) { if (items.length === 1) { - return appRouter.showItem(items[0]); + appRouter.showItem(items[0]); + return; } const url = 'details?id=' + itemId + '&serverId=' + serverId; diff --git a/src/components/libraryoptionseditor/libraryoptionseditor.js b/src/components/libraryoptionseditor/libraryoptionseditor.js index 1f246c8a62..597d747840 100644 --- a/src/components/libraryoptionseditor/libraryoptionseditor.js +++ b/src/components/libraryoptionseditor/libraryoptionseditor.js @@ -323,7 +323,8 @@ import template from './libraryoptionseditor.template.html'; function onImageFetchersContainerClick(e) { const btnImageOptionsForType = dom.parentWithClass(e.target, 'btnImageOptionsForType'); if (btnImageOptionsForType) { - return showImageOptionsForType(dom.parentWithClass(btnImageOptionsForType, 'imageFetcher').getAttribute('data-type')); + showImageOptionsForType(dom.parentWithClass(btnImageOptionsForType, 'imageFetcher').getAttribute('data-type')); + return; } onSortableContainerClick.call(this, e); } diff --git a/src/components/tvproviders/schedulesdirect.js b/src/components/tvproviders/schedulesdirect.js index df33008aca..e9770e287f 100644 --- a/src/components/tvproviders/schedulesdirect.js +++ b/src/components/tvproviders/schedulesdirect.js @@ -122,9 +122,10 @@ export default function (page, providerId, options) { const selectedListingsId = $('#selectListing', page).val(); if (!selectedListingsId) { - return Dashboard.alert({ + Dashboard.alert({ message: globalize.translate('ErrorPleaseSelectLineup') }); + return; } loading.show(); @@ -168,7 +169,8 @@ export default function (page, providerId, options) { function refreshListings(value) { if (!value) { - return $('#selectListing', page).html(''); + $('#selectListing', page).html(''); + return; } loading.show(); diff --git a/src/controllers/itemDetails/index.js b/src/controllers/itemDetails/index.js index 48a3a57e6e..a84fbc63d4 100644 --- a/src/controllers/itemDetails/index.js +++ b/src/controllers/itemDetails/index.js @@ -140,7 +140,8 @@ function renderSeriesTimerSchedule(page, apiClient, seriesTimerId) { function renderTimerEditor(page, item, apiClient, user) { if (item.Type !== 'Recording' || !user.Policy.EnableLiveTvManagement || !item.TimerId || item.Status !== 'InProgress') { - return hideAll(page, 'btnCancelTimer'); + hideAll(page, 'btnCancelTimer'); + return; } hideAll(page, 'btnCancelTimer', true); @@ -148,7 +149,8 @@ function renderTimerEditor(page, item, apiClient, user) { function renderSeriesTimerEditor(page, item, apiClient, user) { if (item.Type !== 'SeriesTimer') { - return hideAll(page, 'btnCancelSeriesTimer'); + hideAll(page, 'btnCancelSeriesTimer'); + return; } if (user.Policy.EnableLiveTvManagement) { @@ -160,11 +162,13 @@ function renderSeriesTimerEditor(page, item, apiClient, user) { page.querySelector('.seriesTimerScheduleSection').classList.remove('hide'); hideAll(page, 'btnCancelSeriesTimer', true); - return renderSeriesTimerSchedule(page, apiClient, item.Id); + renderSeriesTimerSchedule(page, apiClient, item.Id); + return; } page.querySelector('.seriesTimerScheduleSection').classList.add('hide'); - return hideAll(page, 'btnCancelSeriesTimer'); + hideAll(page, 'btnCancelSeriesTimer'); + return; } function renderTrackSelections(page, instance, item, forceReload) { @@ -807,7 +811,7 @@ function renderNextUp(page, item, user) { if (item.Type != 'Series') { section.classList.add('hide'); - return undefined; + return; } ServerConnections.getApiClient(item.ServerId).getNextUpEpisodes({ @@ -1114,7 +1118,8 @@ function renderMoreFromSeason(view, item, apiClient) { if (section) { if (item.Type !== 'Episode' || !item.SeasonId || !item.SeriesId) { - return section.classList.add('hide'); + section.classList.add('hide'); + return; } const userId = apiClient.getCurrentUserId(); @@ -1124,7 +1129,8 @@ function renderMoreFromSeason(view, item, apiClient) { Fields: 'ItemCounts,PrimaryImageAspectRatio,BasicSyncInfo,CanDelete,MediaSourceCount' }).then(function (result) { if (result.Items.length < 2) { - return section.classList.add('hide'); + section.classList.add('hide'); + return; } section.classList.remove('hide'); @@ -1159,10 +1165,12 @@ function renderMoreFromArtist(view, item, apiClient) { if (section) { if (item.Type === 'MusicArtist') { if (!apiClient.isMinServerVersion('3.4.1.19')) { - return section.classList.add('hide'); + section.classList.add('hide'); + return; } } else if (item.Type !== 'MusicAlbum' || !item.AlbumArtists || !item.AlbumArtists.length) { - return section.classList.add('hide'); + section.classList.add('hide'); + return; } const query = { @@ -1181,7 +1189,8 @@ function renderMoreFromArtist(view, item, apiClient) { apiClient.getItems(apiClient.getCurrentUserId(), query).then(function (result) { if (!result.Items.length) { - return section.classList.add('hide'); + section.classList.add('hide'); + return; } section.classList.remove('hide'); @@ -1215,7 +1224,8 @@ function renderSimilarItems(page, item, context) { if (similarCollapsible) { if (item.Type != 'Movie' && item.Type != 'Trailer' && item.Type != 'Series' && item.Type != 'Program' && item.Type != 'Recording' && item.Type != 'MusicAlbum' && item.Type != 'MusicArtist' && item.Type != 'Playlist') { - return similarCollapsible.classList.add('hide'); + similarCollapsible.classList.add('hide'); + return; } similarCollapsible.classList.remove('hide'); @@ -1232,7 +1242,8 @@ function renderSimilarItems(page, item, context) { apiClient.getSimilarItems(item.Id, options).then(function (result) { if (!result.Items.length) { - return similarCollapsible.classList.add('hide'); + similarCollapsible.classList.add('hide'); + return; } similarCollapsible.classList.remove('hide'); @@ -1820,7 +1831,7 @@ function renderCast(page, item) { if (!people.length) { page.querySelector('#castCollapsible').classList.add('hide'); - return undefined; + return; } page.querySelector('#castCollapsible').classList.remove('hide'); @@ -1915,11 +1926,12 @@ export default function (view, params) { if (item.Type === 'Program') { const apiClient = ServerConnections.getApiClient(item.ServerId); - return apiClient.getLiveTvChannel(item.ChannelId, apiClient.getCurrentUserId()).then(function (channel) { + apiClient.getLiveTvChannel(item.ChannelId, apiClient.getCurrentUserId()).then(function (channel) { playbackManager.play({ items: [channel] }); }); + return; } playItem(item, item.UserData && mode === 'resume' ? item.UserData.PlaybackPositionTicks : 0); diff --git a/src/controllers/list.js b/src/controllers/list.js index f465d55b57..3a03a854dd 100644 --- a/src/controllers/list.js +++ b/src/controllers/list.js @@ -721,7 +721,7 @@ class ItemsView { return globalize.translate('Videos'); } - return undefined; + return; } function play() { diff --git a/src/controllers/playback/video/index.js b/src/controllers/playback/video/index.js index 134d30f413..1cb2f4cff7 100644 --- a/src/controllers/playback/video/index.js +++ b/src/controllers/playback/video/index.js @@ -54,14 +54,16 @@ import { appRouter } from '../../../components/appRouter'; recordingButtonManager = null; } - return view.querySelector('.btnRecord').classList.add('hide'); + view.querySelector('.btnRecord').classList.add('hide'); + return; } ServerConnections.getApiClient(item.ServerId).getCurrentUser().then(function (user) { if (user.Policy.EnableLiveTvManagement) { import('../../../components/recordingcreator/recordingbutton').then(({default: RecordingButton}) => { if (recordingButtonManager) { - return recordingButtonManager.refreshItem(item); + recordingButtonManager.refreshItem(item); + return; } recordingButtonManager = new RecordingButton({ @@ -1449,7 +1451,8 @@ import { appRouter } from '../../../components/appRouter'; /* eslint-disable-next-line compat/compat */ dom.addEventListener(view, window.PointerEvent ? 'pointerdown' : 'click', function (e) { if (dom.parentWithClass(e.target, ['videoOsdBottom', 'upNextContainer'])) { - return showOsd(); + showOsd(); + return; } const pointerType = e.pointerType || (layoutManager.mobile ? 'touch' : 'mouse'); diff --git a/src/controllers/session/forgotPassword/index.js b/src/controllers/session/forgotPassword/index.js index 9f02a62c1a..a0d2b7192e 100644 --- a/src/controllers/session/forgotPassword/index.js +++ b/src/controllers/session/forgotPassword/index.js @@ -5,17 +5,19 @@ import Dashboard from '../../../scripts/clientUtils'; function processForgotPasswordResult(result) { if (result.Action == 'ContactAdmin') { - return Dashboard.alert({ + Dashboard.alert({ message: globalize.translate('MessageContactAdminToResetPassword'), title: globalize.translate('ButtonForgotPassword') }); + return; } if (result.Action == 'InNetworkRequired') { - return Dashboard.alert({ + Dashboard.alert({ message: globalize.translate('MessageForgotPasswordInNetworkRequired'), title: globalize.translate('ButtonForgotPassword') }); + return; } if (result.Action == 'PinCode') { @@ -26,13 +28,14 @@ import Dashboard from '../../../scripts/clientUtils'; msg += '
'; msg += result.PinFile; msg += '
'; - return Dashboard.alert({ + Dashboard.alert({ message: msg, title: globalize.translate('ButtonForgotPassword'), callback: function () { Dashboard.navigate('forgotpasswordpin.html'); } }); + return; } } diff --git a/src/controllers/session/resetPassword/index.js b/src/controllers/session/resetPassword/index.js index 252798fd7e..fea8e9ca6d 100644 --- a/src/controllers/session/resetPassword/index.js +++ b/src/controllers/session/resetPassword/index.js @@ -9,13 +9,14 @@ import Dashboard from '../../../scripts/clientUtils'; msg += '
'; msg += '
'; msg += result.UsersReset.join('
'); - return Dashboard.alert({ + Dashboard.alert({ message: msg, title: globalize.translate('HeaderPasswordReset'), callback: function () { window.location.href = 'index.html'; } }); + return; } Dashboard.alert({ diff --git a/src/scripts/libraryMenu.js b/src/scripts/libraryMenu.js index c3b2400ba3..6603905982 100644 --- a/src/scripts/libraryMenu.js +++ b/src/scripts/libraryMenu.js @@ -645,7 +645,8 @@ import Headroom from 'headroom.js'; if (!user) { showBySelector('.libraryMenuDownloads', false); showBySelector('.lnkSyncToOtherDevices', false); - return showBySelector('.userMenuOptions', false); + showBySelector('.userMenuOptions', false); + return; } if (user.Policy.EnableContentDownloading) { @@ -957,7 +958,8 @@ import Headroom from 'headroom.js'; function setTitle (title) { if (title == null) { - return LibraryMenu.setDefaultTitle(); + LibraryMenu.setDefaultTitle(); + return; } if (title === '-') {