Add missing return types to Ajax callbacks
authorTim Düsterhus <duesterhus@woltlab.com>
Fri, 30 Oct 2020 10:36:13 +0000 (11:36 +0100)
committerTim Düsterhus <duesterhus@woltlab.com>
Fri, 30 Oct 2020 11:26:18 +0000 (12:26 +0100)
12 files changed:
wcfsetup/install/files/ts/WoltLabSuite/Core/BackgroundQueue.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Article/MarkAllAsRead.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Article/Search.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Dialog.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/File/Delete.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Page/Search.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Page/Search/Input.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Search/Input.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Suggestion.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/User/Editor.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/User/List.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/User/Trophy/List.ts

index c9dc8d074239d5f93c0c57051092e1e8ac91110b..7bf70a73a768e37a970cd590bb3ad2c5e6159e6a 100644 (file)
@@ -8,7 +8,7 @@
  */
 
 import * as Ajax from "./Ajax";
-import { AjaxCallbackObject, RequestOptions, ResponseData } from "./Ajax/Data";
+import { AjaxCallbackObject, CallbackSetup, ResponseData } from "./Ajax/Data";
 
 class BackgroundQueue implements AjaxCallbackObject {
   private _invocations = 0;
@@ -42,7 +42,7 @@ class BackgroundQueue implements AjaxCallbackObject {
     }
   }
 
-  _ajaxSetup(): RequestOptions {
+  _ajaxSetup(): ReturnType<CallbackSetup> {
     return {
       url: this._url,
       ignoreError: true,
index f0a561f8da6458e4b0101432868def7b817c0789..ff81434f19ab9d24280707c8fac20dcd6c0dfc88 100644 (file)
@@ -8,7 +8,7 @@
  */
 
 import * as Ajax from "../../Ajax";
-import { AjaxCallbackObject } from "../../Ajax/Data";
+import { AjaxCallbackObject, CallbackSetup } from "../../Ajax/Data";
 
 class UiArticleMarkAllAsRead implements AjaxCallbackObject {
   constructor() {
@@ -23,7 +23,7 @@ class UiArticleMarkAllAsRead implements AjaxCallbackObject {
     Ajax.api(this);
   }
 
-  _ajaxSuccess() {
+  _ajaxSuccess(): void {
     /* remove obsolete badges */
     // main menu
     const badge = document.querySelector(".mainMenu .active .badge");
@@ -33,7 +33,7 @@ class UiArticleMarkAllAsRead implements AjaxCallbackObject {
     document.querySelectorAll(".articleList .newMessageBadge").forEach((el) => el.remove());
   }
 
-  _ajaxSetup() {
+  _ajaxSetup(): ReturnType<CallbackSetup> {
     return {
       data: {
         actionName: "markAllAsRead",
index 4ff65ee804e4ebba786a3e897d821351e638f852..99ba488e3ea16ec3c06b9b9eb8ddcb272a93def2 100644 (file)
@@ -1,5 +1,5 @@
 import * as Ajax from "../../Ajax";
-import { AjaxCallbackObject, DatabaseObjectActionResponse } from "../../Ajax/Data";
+import { AjaxCallbackObject, CallbackSetup, DatabaseObjectActionResponse } from "../../Ajax/Data";
 import { DialogCallbackObject } from "../Dialog/Data";
 import DomUtil from "../../Dom/Util";
 import * as Language from "../../Language";
@@ -85,7 +85,7 @@ class UiArticleSearch implements AjaxCallbackObject, DialogCallbackObject {
     }
   }
 
-  _ajaxSetup() {
+  _ajaxSetup(): ReturnType<CallbackSetup> {
     return {
       data: {
         actionName: "search",
index 241a7ff5b6d78414bd4dafc9631809f2b80a6dc5..8f10490c43a8f0b608d06a01654c84cfe343df22 100644 (file)
@@ -24,6 +24,7 @@ import * as Language from "../Language";
 import * as Environment from "../Environment";
 import * as EventHandler from "../Event/Handler";
 import UiDropdownSimple from "./Dropdown/Simple";
+import { CallbackSetup } from "../Ajax/Data";
 
 let _activeDialog: string | null = null;
 let _callbackFocus: (event: FocusEvent) => void;
@@ -886,7 +887,7 @@ const UiDialog = {
     return id.toString();
   },
 
-  _ajaxSetup() {
+  _ajaxSetup(): ReturnType<CallbackSetup> {
     return {};
   },
 };
index c62417475eb9f13b246c1472659808062d3021c2..19a48e918cc348917246a22be428c1a9c6388872 100644 (file)
@@ -9,7 +9,7 @@
  */
 
 import * as Ajax from "../../Ajax";
-import { AjaxCallbackObject, DatabaseObjectActionResponse } from "../../Ajax/Data";
+import { AjaxCallbackObject, CallbackSetup, DatabaseObjectActionResponse } from "../../Ajax/Data";
 import * as Core from "../../Core";
 import DomChangeListener from "../../Dom/Change/Listener";
 import * as Language from "../../Language";
@@ -163,7 +163,7 @@ class UiFileDelete implements AjaxCallbackObject {
     Core.triggerEvent(this.target, "change");
   }
 
-  _ajaxSetup() {
+  _ajaxSetup(): ReturnType<CallbackSetup> {
     return {
       url: "index.php?ajax-file-delete/&t=" + window.SECURITY_TOKEN,
     };
index c2a3054b351bde2db680b386099ab39624b541e6..b0ed37d9f2028de4556c26392d71aa641c5e5e96 100644 (file)
@@ -1,5 +1,5 @@
 import * as Ajax from "../../Ajax";
-import { AjaxCallbackObject, DatabaseObjectActionResponse } from "../../Ajax/Data";
+import { AjaxCallbackObject, CallbackSetup, DatabaseObjectActionResponse } from "../../Ajax/Data";
 import { DialogCallbackObject } from "../Dialog/Data";
 import DomUtil from "../../Dom/Util";
 import * as Language from "../../Language";
@@ -89,7 +89,7 @@ class UiPageSearch implements AjaxCallbackObject, DialogCallbackObject {
     }
   }
 
-  _ajaxSetup() {
+  _ajaxSetup(): ReturnType<CallbackSetup> {
     return {
       data: {
         actionName: "search",
index f75e4551133c1dd5bfed6e12d90882307d0dc756..f027992a8a66832b837abe28ec0c26abaaa11428 100644 (file)
@@ -58,7 +58,7 @@ class UiPageSearchInput extends UiSearchInput {
     return data;
   }
 
-  _ajaxSuccess(data) {
+  _ajaxSuccess(data): void {
     this.callbackSuccess(data);
   }
 }
index d92d2a23214e82227a73e9e20178d48f1595936b..1ab9c20fcede03d0b6516e4cd95f88dd38f1fb82 100644 (file)
@@ -11,7 +11,7 @@ import * as Ajax from "../../Ajax";
 import * as Core from "../../Core";
 import DomUtil from "../../Dom/Util";
 import UiDropdownSimple from "../Dropdown/Simple";
-import { DatabaseObjectActionPayload, DatabaseObjectActionResponse } from "../../Ajax/Data";
+import { CallbackSetup, DatabaseObjectActionPayload, DatabaseObjectActionResponse } from "../../Ajax/Data";
 import AjaxRequest from "../../Ajax/Request";
 import { CallbackDropdownInit, CallbackSelect, SearchInputOptions } from "./Data";
 
@@ -359,7 +359,7 @@ class UiSearchInput {
     return listItem;
   }
 
-  _ajaxSetup() {
+  _ajaxSetup(): ReturnType<CallbackSetup> {
     return {
       data: this.ajaxPayload,
     };
index a1e9752ae0033e00da54adf61fdee7b3cdc95232..28887bdae2a72b5f29fed3c1c71532e0a730bc64 100644 (file)
@@ -11,9 +11,9 @@ import * as Ajax from "../Ajax";
 import * as Core from "../Core";
 import {
   AjaxCallbackObject,
+  CallbackSetup,
   DatabaseObjectActionPayload,
   DatabaseObjectActionResponse,
-  RequestPayload,
 } from "../Ajax/Data";
 import UiDropdownSimple from "./Dropdown/Simple";
 
@@ -187,7 +187,7 @@ class UiSuggestion implements AjaxCallbackObject {
     });
   }
 
-  _ajaxSetup(): RequestPayload {
+  _ajaxSetup(): ReturnType<CallbackSetup> {
     return {
       data: this.ajaxPayload,
     };
index e11794dce39bcfbd41747aa9b1f2e17fa70df7f5..b4ea7e318f95dee32a7292bbb5d354e27d3dd8d7 100644 (file)
@@ -8,7 +8,7 @@
  */
 
 import * as Ajax from "../../Ajax";
-import { AjaxCallbackObject } from "../../Ajax/Data";
+import { AjaxCallbackObject, CallbackSetup } from "../../Ajax/Data";
 import * as Core from "../../Core";
 import { DialogCallbackObject, DialogSettings } from "../Dialog/Data";
 import DomUtil from "../../Dom/Util";
@@ -119,7 +119,7 @@ class UserEditor implements AjaxCallbackObject, DialogCallbackObject {
     });
   }
 
-  _ajaxSuccess(data) {
+  _ajaxSuccess(data): void {
     let button: HTMLElement;
     switch (data.actionName) {
       case "ban":
@@ -183,7 +183,7 @@ class UserEditor implements AjaxCallbackObject, DialogCallbackObject {
     UiNotification.show();
   }
 
-  _ajaxSetup() {
+  _ajaxSetup(): ReturnType<CallbackSetup> {
     return {
       data: {
         className: "wcf\\data\\user\\UserAction",
index 3113ffb40558bdee23565211c1e9c74d88f9b22c..bba9ccc98d1324c46d48d0211134687da92ce6db 100644 (file)
@@ -12,7 +12,7 @@ import * as Core from "../../Core";
 import DomUtil from "../../Dom/Util";
 import UiDialog from "../Dialog";
 import UiPagination from "../Pagination";
-import { AjaxCallbackObject, DatabaseObjectActionResponse, RequestOptions } from "../../Ajax/Data";
+import { AjaxCallbackObject, CallbackSetup, DatabaseObjectActionResponse } from "../../Ajax/Data";
 import { DialogCallbackObject, DialogData, DialogSettings } from "../Dialog/Data";
 
 /**
@@ -98,7 +98,7 @@ class UiUserList implements AjaxCallbackObject, DialogCallbackObject {
     this.showPage();
   }
 
-  _ajaxSetup(): RequestOptions {
+  _ajaxSetup(): ReturnType<CallbackSetup> {
     return {
       data: {
         actionName: "getGroupedUserList",
index 73458a611573d953760670e8a9803fc9452bb4a6..b48c48d79c508260fbeb68dc049f5239de794e18 100644 (file)
@@ -8,7 +8,7 @@
  */
 
 import * as Ajax from "../../../Ajax";
-import { AjaxCallbackObject, DatabaseObjectActionResponse } from "../../../Ajax/Data";
+import { AjaxCallbackObject, CallbackSetup, DatabaseObjectActionResponse } from "../../../Ajax/Data";
 import { DialogCallbackObject, DialogData } from "../../Dialog/Data";
 import DomChangeListener from "../../../Dom/Change/Listener";
 import UiDialog from "../../Dialog";
@@ -124,7 +124,7 @@ class UiUserTrophyList implements AjaxCallbackObject, DialogCallbackObject {
     this.showPage();
   }
 
-  _ajaxSetup() {
+  _ajaxSetup(): ReturnType<CallbackSetup> {
     return {
       data: {
         actionName: "getGroupedUserTrophyList",