Make everything @typescript-eslint/no-unused-vars clean
authorTim Düsterhus <duesterhus@woltlab.com>
Fri, 30 Oct 2020 10:28:28 +0000 (11:28 +0100)
committerTim Düsterhus <duesterhus@woltlab.com>
Fri, 30 Oct 2020 11:26:18 +0000 (12:26 +0100)
17 files changed:
.eslintrc.js
wcfsetup/install/files/js/WoltLabSuite/Core/CallbackList.js
wcfsetup/install/files/js/WoltLabSuite/Core/I18n/Plural.js
wcfsetup/install/files/js/WoltLabSuite/Core/Template.js
wcfsetup/install/files/js/WoltLabSuite/Core/Ui/Confirmation.js
wcfsetup/install/files/js/WoltLabSuite/Core/Ui/Dialog.js
wcfsetup/install/files/js/WoltLabSuite/Core/Ui/ItemList.js
wcfsetup/install/files/js/WoltLabSuite/Core/Ui/Page/Header/Fixed.js
wcfsetup/install/files/js/WoltLabSuite/Core/Ui/Screen.js
wcfsetup/install/files/ts/WoltLabSuite/Core/CallbackList.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/I18n/Plural.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Template.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Confirmation.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Dialog.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/ItemList.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Page/Header/Fixed.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Screen.ts

index 65070d272f9be5b2213f2ec8f4325d9a28c937b7..04aec1c84fa9ee6e670c9baa355843b8cc6695a1 100644 (file)
@@ -20,5 +20,11 @@ module.exports = {
     "@typescript-eslint/no-unsafe-call": 0,
     "@typescript-eslint/no-unsafe-member-access": 0,
     "@typescript-eslint/no-unsafe-return": 0,
+    "@typescript-eslint/no-unused-vars": [
+      "error",
+      {
+        "argsIgnorePattern": "^_"
+      }
+    ]
   }
 };
index 865d610d1f28b54926fdc16761d893d535793034..0d7795c8bfebb1277e099aa573506f6a6fd0e89d 100644 (file)
@@ -37,7 +37,7 @@ define(["require", "exports"], function (require, exports) {
         forEach(identifier, callback) {
             var _a;
             if (identifier === null) {
-                this._callbacks.forEach((callbacks, identifier) => {
+                this._callbacks.forEach((callbacks, _identifier) => {
                     callbacks.forEach(callback);
                 });
             }
index 82843bf9f30036690ffdac457776de00a00f8131..f57471651e245cf1cba14bbcbf8144b8c98f719a 100644 (file)
@@ -141,7 +141,7 @@ define(["require", "exports", "tslib", "../StringUtil"], function (require, expo
                 return PLURAL_ONE;
         },
         // Tibetan
-        bo(n) { },
+        bo(_n) { },
         // Bosnian
         bs(n) {
             const v = Plural.getV(n);
@@ -270,7 +270,7 @@ define(["require", "exports", "tslib", "../StringUtil"], function (require, expo
                 return PLURAL_ONE;
         },
         // Indonesian
-        id(n) { },
+        id(_n) { },
         // Icelandic
         is(n) {
             const f = Plural.getF(n);
@@ -278,9 +278,9 @@ define(["require", "exports", "tslib", "../StringUtil"], function (require, expo
                 return PLURAL_ONE;
         },
         // Japanese
-        ja(n) { },
+        ja(_n) { },
         // Javanese
-        jv(n) { },
+        jv(_n) { },
         // Georgian
         ka(n) {
             if (n == 1)
@@ -292,14 +292,14 @@ define(["require", "exports", "tslib", "../StringUtil"], function (require, expo
                 return PLURAL_ONE;
         },
         // Khmer
-        km(n) { },
+        km(_n) { },
         // Kannada
         kn(n) {
             if (n >= 0 && n <= 1)
                 return PLURAL_ONE;
         },
         // Korean
-        ko(n) { },
+        ko(_n) { },
         // Kurdish
         ku(n) {
             if (n == 1)
@@ -316,7 +316,7 @@ define(["require", "exports", "tslib", "../StringUtil"], function (require, expo
                 return PLURAL_ONE;
         },
         // Lao
-        lo(n) { },
+        lo(_n) { },
         // Lithuanian
         lt(n) {
             const mod10 = n % 10;
@@ -361,7 +361,7 @@ define(["require", "exports", "tslib", "../StringUtil"], function (require, expo
                 return PLURAL_ONE;
         },
         // Malay
-        ms(n) { },
+        ms(_n) { },
         // Maltese
         mt(n) {
             const mod100 = n % 100;
@@ -373,7 +373,7 @@ define(["require", "exports", "tslib", "../StringUtil"], function (require, expo
                 return PLURAL_MANY;
         },
         // Burmese
-        my(n) { },
+        my(_n) { },
         // Norwegian
         no(n) {
             if (n == 1)
@@ -486,9 +486,9 @@ define(["require", "exports", "tslib", "../StringUtil"], function (require, expo
                 return PLURAL_ONE;
         },
         // Tajik
-        tg(n) { },
+        tg(_n) { },
         // Thai
-        th(n) { },
+        th(_n) { },
         // Turkmen
         tk(n) {
             if (n == 1)
@@ -515,9 +515,9 @@ define(["require", "exports", "tslib", "../StringUtil"], function (require, expo
                 return PLURAL_ONE;
         },
         // Vietnamese
-        vi(n) { },
+        vi(_n) { },
         // Chinese
-        zh(n) { },
+        zh(_n) { },
     };
     return Plural;
 });
index 16d68aa2c0b6756d4948ec60a0ed8d4685111777..92f9572e8ba32db7695708f9149163e88aa63280 100644 (file)
@@ -55,7 +55,7 @@ define(["require", "exports", "tslib", "./Template.grammar", "./StringUtil", "./
          *
          * @param  {object}  v  Parameters to pass to the template.
          */
-        fetch(v) {
+        fetch(_v) {
             // this will be replaced in the init function
             throw new Error("This Template is not initialized.");
         }
@@ -66,7 +66,7 @@ define(["require", "exports", "tslib", "./Template.grammar", "./StringUtil", "./
         get: function () {
             throw new Error("WCF.Template.callbacks is no longer supported");
         },
-        set: function (value) {
+        set: function (_value) {
             throw new Error("WCF.Template.callbacks is no longer supported");
         },
     });
index 256677aacbaa17a44cbb886b165039d40430f90e..e5ffacd24b448f5f6676c7279694e3a4f39ed5a4 100644 (file)
@@ -31,7 +31,7 @@ define(["require", "exports", "tslib", "../Core", "../Language", "./Dialog"], fu
             this.confirmButton = document.createElement("button");
             this.confirmButton.classList.add("buttonPrimary");
             this.confirmButton.textContent = Language.get("wcf.global.confirmation.confirm");
-            this.confirmButton.addEventListener("click", (ev) => this._confirm());
+            this.confirmButton.addEventListener("click", (_ev) => this._confirm());
             formSubmit.appendChild(this.confirmButton);
             const cancelButton = document.createElement("button");
             cancelButton.textContent = Language.get("wcf.global.confirmation.cancel");
index 7d3bee5ba3a73734e0b8de4394d68e31c955afda..b4b22c1dd9a0934a63a34bcd612cb75733f8ee0a 100644 (file)
@@ -24,7 +24,6 @@ define(["require", "exports", "tslib", "../Core", "../Dom/Change/Listener", "./S
     let _dialogFullHeight = false;
     const _dialogObjects = new WeakMap();
     const _dialogToObject = new Map();
-    let _focusedBeforeDialog;
     let _keyupListener;
     const _validCallbacks = ["onBeforeClose", "onClose", "onShow"];
     // list of supported `input[type]` values for dialog submit
@@ -85,7 +84,6 @@ define(["require", "exports", "tslib", "../Core", "../Dom/Change/Listener", "./S
             Listener_1.default.add("Ui/Dialog", () => {
                 this._initStaticDialogs();
             });
-            UiScreen.setDialogContainer(_container);
             window.addEventListener("resize", () => {
                 _dialogs.forEach((dialog) => {
                     if (!Core.stringToBool(dialog.dialog.getAttribute("aria-hidden"))) {
@@ -432,8 +430,6 @@ define(["require", "exports", "tslib", "../Core", "../Dom/Change/Listener", "./S
                 _container.setAttribute("aria-hidden", "false");
                 _container.setAttribute("close-on-click", data.backdropCloseOnClick ? "true" : "false");
                 _activeDialog = id;
-                // Keep a reference to the currently focused element to be able to restore it later.
-                _focusedBeforeDialog = document.activeElement;
                 // Set the focus to the first focusable child of the dialog element.
                 const closeButton = data.header.querySelector(".dialogCloseButton");
                 if (closeButton)
index 7260b2c7d5b4f55603771cd1d3ae5faa027ed33b..2cd0c4d941d35f40f87ab3766e9e355e0b18a301 100644 (file)
@@ -16,7 +16,6 @@ define(["require", "exports", "tslib", "../Core", "../Dom/Traverse", "../Languag
     Suggestion_1 = tslib_1.__importDefault(Suggestion_1);
     Simple_1 = tslib_1.__importDefault(Simple_1);
     Util_1 = tslib_1.__importDefault(Util_1);
-    let _activeId = "";
     const _data = new Map();
     /**
      * Creates the DOM structure for target element. If `element` is a `<textarea>`
@@ -120,7 +119,6 @@ define(["require", "exports", "tslib", "../Core", "../Dom/Traverse", "../Languag
      */
     function keyDown(event) {
         const input = event.currentTarget;
-        _activeId = input.id;
         const lastItem = input.parentElement.previousElementSibling;
         if (event.key === "Backspace") {
             if (input.value.length === 0) {
index 6ce133f916582f19fe16f38ffadd8d1bf7891b16..b4fe87d3b335f2665aa8814481dee0c8a3a8835a 100644 (file)
@@ -17,7 +17,6 @@ define(["require", "exports", "tslib", "../../../Event/Handler", "../../Alignmen
     UiScreen = tslib_1.__importStar(UiScreen);
     let _isMobile = false;
     let _pageHeader;
-    let _pageHeaderContainer;
     let _pageHeaderPanel;
     let _pageHeaderSearch;
     let _searchInput;
@@ -94,7 +93,6 @@ define(["require", "exports", "tslib", "../../../Event/Handler", "../../Alignmen
      */
     function init() {
         _pageHeader = document.getElementById("pageHeader");
-        _pageHeaderContainer = document.getElementById("pageHeaderContainer");
         initSearchBar();
         UiScreen.on("screen-md-down", {
             match() {
index 2a3f61c936c3c53c9c61286b6d7f93e049dc48ff..00ac334de005838b7cba0893f150f0d4e87ffa06 100644 (file)
@@ -14,7 +14,6 @@ define(["require", "exports", "tslib", "../Core", "../Environment"], function (r
     Core = tslib_1.__importStar(Core);
     Environment = tslib_1.__importStar(Environment);
     const _mql = new Map();
-    let _dialogContainer;
     let _scrollDisableCounter = 0;
     let _scrollOffsetFrom;
     let _scrollTop = 0;
@@ -163,13 +162,9 @@ define(["require", "exports", "tslib", "../Core", "../Environment"], function (r
     }
     exports.pageOverlayIsActive = pageOverlayIsActive;
     /**
-     * Sets the dialog container element. This method is used to
-     * circumvent a possible circular dependency, due to `Ui/Dialog`
-     * requiring the `Ui/Screen` module itself.
+     * @deprecated 5.4 - This method is a noop.
      */
-    function setDialogContainer(container) {
-        _dialogContainer = container;
-    }
+    function setDialogContainer(_container) { }
     exports.setDialogContainer = setDialogContainer;
     function _getQueryObject(query) {
         if (typeof query !== "string" || query.trim() === "") {
index 988d2b8242d13969af2b89f5e5cdabd4f617fa1f..99e93e558d2bac8aa742ec5d0f464bb7cdbc43e1 100644 (file)
@@ -38,7 +38,7 @@ class CallbackList {
    */
   forEach(identifier: string | null, callback: (cb: Callback) => unknown): void {
     if (identifier === null) {
-      this._callbacks.forEach((callbacks, identifier) => {
+      this._callbacks.forEach((callbacks, _identifier) => {
         callbacks.forEach(callback);
       });
     } else {
index eef3242a3edd1b39e68af5b80617674a686d9bc2..445b3a3a2964e80020bab16a589a4a6b2826d707 100644 (file)
@@ -151,7 +151,7 @@ const Plural = {
   },
 
   // Tibetan
-  bo(n: number) {},
+  bo(_n: number) {},
 
   // Bosnian
   bs(n: number): string | undefined {
@@ -276,7 +276,7 @@ const Plural = {
   },
 
   // Indonesian
-  id(n: number) {},
+  id(_n: number) {},
 
   // Icelandic
   is(n: number): string | undefined {
@@ -286,10 +286,10 @@ const Plural = {
   },
 
   // Japanese
-  ja(n: number) {},
+  ja(_n: number) {},
 
   // Javanese
-  jv(n: number) {},
+  jv(_n: number) {},
 
   // Georgian
   ka(n: number): string | undefined {
@@ -302,7 +302,7 @@ const Plural = {
   },
 
   // Khmer
-  km(n: number) {},
+  km(_n: number) {},
 
   // Kannada
   kn(n: number): string | undefined {
@@ -310,7 +310,7 @@ const Plural = {
   },
 
   // Korean
-  ko(n: number) {},
+  ko(_n: number) {},
 
   // Kurdish
   ku(n: number): string | undefined {
@@ -328,7 +328,7 @@ const Plural = {
   },
 
   // Lao
-  lo(n: number) {},
+  lo(_n: number) {},
 
   // Lithuanian
   lt(n: number): string | undefined {
@@ -375,7 +375,7 @@ const Plural = {
   },
 
   // Malay
-  ms(n: number) {},
+  ms(_n: number) {},
 
   // Maltese
   mt(n: number): string | undefined {
@@ -387,7 +387,7 @@ const Plural = {
   },
 
   // Burmese
-  my(n: number) {},
+  my(_n: number) {},
 
   // Norwegian
   no(n: number): string | undefined {
@@ -503,10 +503,10 @@ const Plural = {
   },
 
   // Tajik
-  tg(n: number) {},
+  tg(_n: number) {},
 
   // Thai
-  th(n: number) {},
+  th(_n: number) {},
 
   // Turkmen
   tk(n: number): string | undefined {
@@ -535,10 +535,10 @@ const Plural = {
   },
 
   // Vietnamese
-  vi(n: number) {},
+  vi(_n: number) {},
 
   // Chinese
-  zh(n: number) {},
+  zh(_n: number) {},
 };
 
 export = Plural;
index 47693603282a0bfd2013bebba512a1291c6c0e13..11edf783d449a50ec538721d30ed27a8c42c34b5 100644 (file)
@@ -63,7 +63,7 @@ class Template {
    *
    * @param  {object}  v  Parameters to pass to the template.
    */
-  fetch(v: object): string {
+  fetch(_v: object): string {
     // this will be replaced in the init function
     throw new Error("This Template is not initialized.");
   }
@@ -75,7 +75,7 @@ Object.defineProperty(Template, "callbacks", {
   get: function () {
     throw new Error("WCF.Template.callbacks is no longer supported");
   },
-  set: function (value) {
+  set: function (_value) {
     throw new Error("WCF.Template.callbacks is no longer supported");
   },
 });
index 0da77a351640dacfdf4097950d1408bf82a558f1..cde445dc4005dd08c7c3d2c891adc2a8ca18bc74 100644 (file)
@@ -44,7 +44,7 @@ class UiConfirmation implements DialogCallbackObject {
     this.confirmButton = document.createElement("button");
     this.confirmButton.classList.add("buttonPrimary");
     this.confirmButton.textContent = Language.get("wcf.global.confirmation.confirm");
-    this.confirmButton.addEventListener("click", (ev) => this._confirm());
+    this.confirmButton.addEventListener("click", (_ev) => this._confirm());
     formSubmit.appendChild(this.confirmButton);
 
     const cancelButton = document.createElement("button");
index f74b1f630284c2c3c735bbcd338b246c265396fc..241a7ff5b6d78414bd4dafc9631809f2b80a6dc5 100644 (file)
@@ -32,7 +32,6 @@ const _dialogs = new Map<ElementId, DialogData>();
 let _dialogFullHeight = false;
 const _dialogObjects = new WeakMap<DialogCallbackObject, DialogInternalData>();
 const _dialogToObject = new Map<ElementId, DialogCallbackObject>();
-let _focusedBeforeDialog: Element | null;
 let _keyupListener: (event: KeyboardEvent) => boolean;
 const _validCallbacks = ["onBeforeClose", "onClose", "onShow"];
 
@@ -107,8 +106,6 @@ const UiDialog = {
       this._initStaticDialogs();
     });
 
-    UiScreen.setDialogContainer(_container);
-
     window.addEventListener("resize", () => {
       _dialogs.forEach((dialog) => {
         if (!Core.stringToBool(dialog.dialog.getAttribute("aria-hidden"))) {
@@ -518,9 +515,6 @@ const UiDialog = {
       _container.setAttribute("close-on-click", data.backdropCloseOnClick ? "true" : "false");
       _activeDialog = id;
 
-      // Keep a reference to the currently focused element to be able to restore it later.
-      _focusedBeforeDialog = document.activeElement;
-
       // Set the focus to the first focusable child of the dialog element.
       const closeButton = data.header.querySelector(".dialogCloseButton");
       if (closeButton) closeButton.setAttribute("inert", "true");
index 16828591a3f5a68b1b805fabd0046f668830650f..c41e37423c1c6716436b37c1ab37ea8383757e1d 100644 (file)
@@ -15,7 +15,6 @@ import UiDropdownSimple from "./Dropdown/Simple";
 import { DatabaseObjectActionPayload } from "../Ajax/Data";
 import DomUtil from "../Dom/Util";
 
-let _activeId = "";
 const _data = new Map<string, ElementData>();
 
 /**
@@ -134,7 +133,6 @@ function handleLimit(elementId: string): void {
  */
 function keyDown(event: KeyboardEvent): void {
   const input = event.currentTarget as HTMLInputElement;
-  _activeId = input.id;
 
   const lastItem = input.parentElement!.previousElementSibling as HTMLElement | null;
   if (event.key === "Backspace") {
index a2efebc8c33fc36e0b2ae8726498ba88029ff098..db261f881a095931f507bf741abe9bcea1f38f24 100644 (file)
@@ -16,7 +16,6 @@ import * as UiScreen from "../../Screen";
 let _isMobile = false;
 
 let _pageHeader: HTMLElement;
-let _pageHeaderContainer: HTMLElement;
 let _pageHeaderPanel: HTMLElement;
 let _pageHeaderSearch: HTMLElement;
 let _searchInput: HTMLInputElement;
@@ -110,7 +109,6 @@ function closeSearchBar(): void {
  */
 export function init(): void {
   _pageHeader = document.getElementById("pageHeader")!;
-  _pageHeaderContainer = document.getElementById("pageHeaderContainer")!;
 
   initSearchBar();
 
index f5005127f19e8594d21866d03c1fc2e7d3dea259..ea076172b51cacb386d04ef997da5089bdb88c2b 100644 (file)
@@ -13,7 +13,6 @@ import * as Environment from "../Environment";
 
 const _mql = new Map<string, MediaQueryData>();
 
-let _dialogContainer: Element;
 let _scrollDisableCounter = 0;
 let _scrollOffsetFrom: string;
 let _scrollTop = 0;
@@ -179,13 +178,9 @@ export function pageOverlayIsActive(): boolean {
 }
 
 /**
- * Sets the dialog container element. This method is used to
- * circumvent a possible circular dependency, due to `Ui/Dialog`
- * requiring the `Ui/Screen` module itself.
+ * @deprecated 5.4 - This method is a noop.
  */
-export function setDialogContainer(container: Element): void {
-  _dialogContainer = container;
-}
+export function setDialogContainer(_container: Element): void {}
 
 function _getQueryObject(query: string): MediaQueryData {
   if (typeof (query as any) !== "string" || query.trim() === "") {