Use an underscore to mask intentionally unused parameters
authorAlexander Ebert <ebert@woltlab.com>
Sat, 31 Oct 2020 12:46:11 +0000 (13:46 +0100)
committerAlexander Ebert <ebert@woltlab.com>
Sat, 31 Oct 2020 12:46:11 +0000 (13:46 +0100)
wcfsetup/install/files/js/WoltLabSuite/Core/Ui/User/Profile/Menu/Item/Abstract.js
wcfsetup/install/files/js/WoltLabSuite/Core/Upload.js
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/User/Profile/Menu/Item/Abstract.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Upload.ts

index c7b959a751a1ce08bbf332cc388dbcd5d3647b5a..5e3be39f4cacddcfeccd711f899b2be2b8200141 100644 (file)
@@ -76,8 +76,7 @@ define(["require", "exports", "tslib", "../../../../../Ajax"], function (require
         /**
          * Handles successful Ajax requests.
          */
-        // eslint-disable-next-line @typescript-eslint/no-unused-vars
-        _ajaxSuccess(data) {
+        _ajaxSuccess(_data) {
             // This should be an abstract method, but cannot be marked as such for backwards compatibility.
             throw new Error("Implement me!");
         }
index 8edf1c4a675cfd210e4bbf393af85d89ff3ab197..c83178089caf8ae2b9a408f5d8b693a94fc43ef5 100644 (file)
@@ -136,19 +136,14 @@ define(["require", "exports", "tslib", "./Ajax/Request", "./Core", "./Dom/Change
             Listener_1.default.trigger();
             return uploadId;
         }
-        // eslint-disable-next-line @typescript-eslint/no-unused-vars
-        _createFileTableRow(file) {
+        _createFileTableRow(_file) {
             // This should be an abstract method, but cannot be marked as such for backwards compatibility.
             throw new Error("Has to be implemented in subclass.");
         }
         /**
          * Handles a failed file upload.
          */
-        _failure(
-        /* eslint-disable @typescript-eslint/no-unused-vars */
-        uploadId, data, responseText, xhr, requestOptions
-        /* eslint-enable @typescript-eslint/no-unused-vars */
-        ) {
+        _failure(_uploadId, _data, _responseText, _xhr, _requestOptions) {
             // This should be an abstract method, but cannot be marked as such for backwards compatibility.
             return true;
         }
@@ -194,11 +189,7 @@ define(["require", "exports", "tslib", "./Ajax/Request", "./Core", "./Dom/Change
         /**
          * Handles a successful file upload.
          */
-        _success(
-        /* eslint-disable @typescript-eslint/no-unused-vars */
-        uploadId, data, responseText, xhr, requestOptions
-        /* eslint-enable @typescript-eslint/no-unused-vars */
-        ) {
+        _success(_uploadId, _data, _responseText, _xhr, _requestOptions) {
             // This should be an abstract method, but cannot be marked as such for backwards compatibility.
         }
         _upload(event, file, blob) {
@@ -255,8 +246,7 @@ define(["require", "exports", "tslib", "./Ajax/Request", "./Core", "./Dom/Change
          *
          * @since       5.2
          */
-        // eslint-disable-next-line @typescript-eslint/no-unused-vars
-        validateUpload(files) {
+        validateUpload(_files) {
             // This should be an abstract method, but cannot be marked as such for backwards compatibility.
             return true;
         }
index 5794b55945e6664da7f0d40df200eb80faf90d71..4a3b5bd846771cf22acbe332a73aaddfcbbba672 100644 (file)
@@ -90,8 +90,7 @@ abstract class UiUserProfileMenuItemAbstract implements AjaxCallbackObject {
   /**
    * Handles successful Ajax requests.
    */
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  _ajaxSuccess(data: ResponseData): void {
+  _ajaxSuccess(_data: ResponseData): void {
     // This should be an abstract method, but cannot be marked as such for backwards compatibility.
 
     throw new Error("Implement me!");
index b186f57decafbe6d1b4833d6a588508e7c3dc1ba..c61df61b0e02fc17de0a603b0bf1268fbe643cdc 100644 (file)
@@ -193,8 +193,7 @@ abstract class Upload {
     return uploadId;
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  protected _createFileTableRow(file: File | FileLikeObject): HTMLTableRowElement {
+  protected _createFileTableRow(_file: File | FileLikeObject): HTMLTableRowElement {
     // This should be an abstract method, but cannot be marked as such for backwards compatibility.
 
     throw new Error("Has to be implemented in subclass.");
@@ -204,15 +203,11 @@ abstract class Upload {
    * Handles a failed file upload.
    */
   protected _failure(
-    /* eslint-disable @typescript-eslint/no-unused-vars */
-
-    uploadId: number,
-    data: ResponseData,
-    responseText: string,
-    xhr: XMLHttpRequest,
-    requestOptions: RequestOptions
-
-    /* eslint-enable @typescript-eslint/no-unused-vars */
+    _uploadId: number,
+    _data: ResponseData,
+    _responseText: string,
+    _xhr: XMLHttpRequest,
+    _requestOptions: RequestOptions
   ): boolean {
     // This should be an abstract method, but cannot be marked as such for backwards compatibility.
 
@@ -267,15 +262,11 @@ abstract class Upload {
    * Handles a successful file upload.
    */
   protected _success(
-    /* eslint-disable @typescript-eslint/no-unused-vars */
-
-    uploadId: number,
-    data: ResponseData,
-    responseText: string,
-    xhr: XMLHttpRequestEventTarget,
-    requestOptions: RequestOptions
-
-    /* eslint-enable @typescript-eslint/no-unused-vars */
+    _uploadId: number,
+    _data: ResponseData,
+    _responseText: string,
+    _xhr: XMLHttpRequestEventTarget,
+    _requestOptions: RequestOptions
   ): void {
     // This should be an abstract method, but cannot be marked as such for backwards compatibility.
   }
@@ -342,8 +333,7 @@ abstract class Upload {
    *
    * @since       5.2
    */
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  protected validateUpload(files: FileCollection): boolean {
+  protected validateUpload(_files: FileCollection): boolean {
     // This should be an abstract method, but cannot be marked as such for backwards compatibility.
 
     return true;