Remove getParameterByName global

.eslintrc.js

@@ -193,7 +193,6 @@ module.exports = {
                 'DlnaProfilePage': 'writable',
                 'DashboardPage': 'writable',
                 'Emby': 'readonly',
-                'getParameterByName': 'writable',
                 'Globalize': 'writable',
                 'Hls': 'writable',
                 'dfnshelper': 'writable',

src/components/favoriteitems.js

@@ -5,6 +5,7 @@ import { appHost } from './apphost';
 import imageLoader from './images/imageLoader';
 import globalize from '../scripts/globalize';
 import layoutManager from './layoutManager';
+import { getParameterByName } from '../utils/url.ts';
 import '../assets/css/scrollstyles.scss';
 import '../elements/emby-itemscontainer/emby-itemscontainer';
 

src/controllers/dashboard/devices/device.js

@@ -3,6 +3,7 @@ import dom from '../../../scripts/dom';
 import '../../../elements/emby-input/emby-input';
 import '../../../elements/emby-button/emby-button';
 import Dashboard from '../../../scripts/clientUtils';
+import { getParameterByName } from '../../../utils/url.ts';
 
 /* eslint-disable indent */
 

src/controllers/dashboard/dlna/profile.js

@@ -9,6 +9,7 @@ import '../../../elements/emby-checkbox/emby-checkbox';
 import '../../../components/listview/listview.scss';
 import Dashboard from '../../../scripts/clientUtils';
 import toast from '../../../components/toast/toast';
+import { getParameterByName } from '../../../utils/url.ts';
 
 /* eslint-disable indent */
 

src/controllers/dashboard/notifications/notification/index.js

@@ -2,6 +2,7 @@ import escapeHtml from 'escape-html';
 import 'jquery';
 import '../../../../elements/emby-checkbox/emby-checkbox';
 import Dashboard from '../../../../scripts/clientUtils';
+import { getParameterByName } from '../../../../utils/url.ts';
 
 function fillItems(elem, items, cssClass, idPrefix, currentList, isEnabledList) {
     let html = '<div class="checkboxList paperList" style="padding: .5em 1em;">';

src/controllers/dashboard/scheduledtasks/scheduledtask.js

@@ -7,6 +7,7 @@ import '../../../elements/emby-input/emby-input';
 import '../../../elements/emby-button/emby-button';
 import '../../../elements/emby-select/emby-select';
 import confirm from '../../../components/confirm/confirm';
+import { getParameterByName } from '../../../utils/url.ts';
 
 /* eslint-disable indent */
 

src/controllers/livetvguideprovider.js

@@ -2,6 +2,7 @@ import { Events } from 'jellyfin-apiclient';
 import loading from '../components/loading/loading';
 import globalize from '../scripts/globalize';
 import Dashboard, { pageIdOn } from '../scripts/clientUtils';
+import { getParameterByName } from '../utils/url.ts';
 
 function onListingsSubmitted() {
     Dashboard.navigate('livetvstatus.html');

src/controllers/livetvtuner.js

@@ -6,6 +6,7 @@ import '../elements/emby-button/emby-button';
 import '../elements/emby-checkbox/emby-checkbox';
 import '../elements/emby-select/emby-select';
 import Dashboard from '../scripts/clientUtils';
+import { getParameterByName } from '../utils/url.ts';
 
 function isM3uVariant(type) {
     return ['nextpvr'].indexOf(type || '') !== -1;

src/scripts/editorsidebar.js

@@ -3,6 +3,7 @@ import 'jquery';
 import globalize from './globalize';
 import 'material-design-icons-iconfont';
 import Dashboard from './clientUtils';
+import { getParameterByName } from '../utils/url.ts';
 
 /* eslint-disable indent */
 

src/scripts/libraryMenu.js

@@ -19,6 +19,7 @@ import '../assets/css/flexstyles.scss';
 import Dashboard, { pageClassOn } from './clientUtils';
 import ServerConnections from '../components/ServerConnections';
 import Headroom from 'headroom.js';
+import { getParameterByName } from '../utils/url.ts';
 
 /* eslint-disable indent */
 

src/scripts/site.js

@@ -26,7 +26,6 @@ import './routes';
 import '../components/themeMediaPlayer';
 import './autoBackdrops';
 import { pageClassOn, serverAddress } from './clientUtils';
-import { getWindowLocationSearch } from '../utils/url.ts';
 import './screensavermanager';
 import './serverNotifications';
 import '../components/playback/playerSelectionMenu';
@@ -43,18 +42,6 @@ import { currentSettings } from './settings/userSettings';
 import taskButton from './taskbutton';
 
 // TODO: Move this elsewhere
-window.getParameterByName = function(name, url) {
-    name = name.replace(/[[]/, '\\[').replace(/[\]]/, '\\]');
-    const regexS = '[\\?&]' + name + '=([^&#]*)';
-    const regex = new RegExp(regexS, 'i');
-    const results = regex.exec(url || getWindowLocationSearch());
-
-    if (results == null) {
-        return '';
-    }
-
-    return decodeURIComponent(results[1].replace(/\+/g, ' '));
-};
 
 function loadCoreDictionary() {
     const languages = ['af', 'ar', 'be-by', 'bg-bg', 'bn_bd', 'ca', 'cs', 'da', 'de', 'el', 'en-gb', 'en-us', 'eo', 'es', 'es-419', 'es-ar', 'es_do', 'es-mx', 'fa', 'fi', 'fil', 'fr', 'fr-ca', 'gl', 'gsw', 'he', 'hi-in', 'hr', 'hu', 'id', 'it', 'ja', 'kk', 'ko', 'lt-lt', 'mr', 'ms', 'nb', 'nl', 'pl', 'pr', 'pt', 'pt-br', 'pt-pt', 'ro', 'ru', 'sk', 'sl-si', 'sq', 'sv', 'ta', 'th', 'tr', 'uk', 'ur_pk', 'vi', 'zh-cn', 'zh-hk', 'zh-tw'];

src/utils/url.ts

@@ -1,4 +1,4 @@
-export const getWindowLocationSearch = (win: Window | null | undefined) => {
+export const getWindowLocationSearch = (win?: Window | null | undefined) => {
     let search = (win || window).location.search;
 
     if (!search) {
@@ -11,3 +11,16 @@ export const getWindowLocationSearch = (win: Window | null | undefined) => {
 
     return search || '';
 };
+
+export const getParameterByName = (name: string, url?: string | null | undefined) => {
+    name = name.replace(/[[]/, '\\[').replace(/[\]]/, '\\]');
+    const regexS = '[\\?&]' + name + '=([^&#]*)';
+    const regex = new RegExp(regexS, 'i');
+    const results = regex.exec(url || getWindowLocationSearch());
+
+    if (results == null) {
+        return '';
+    }
+
+    return decodeURIComponent(results[1].replace(/\+/g, ' '));
+};