Unified the return types for `_ajaxSetup()` and `_dialogSetup()`
authorAlexander Ebert <ebert@woltlab.com>
Sat, 31 Oct 2020 15:23:13 +0000 (16:23 +0100)
committerAlexander Ebert <ebert@woltlab.com>
Sat, 31 Oct 2020 15:23:13 +0000 (16:23 +0100)
21 files changed:
wcfsetup/install/files/ts/WoltLabSuite/Core/Ajax/Data.ts
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/Confirmation.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Dialog.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Dialog/Data.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/File/Delete.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Page/JumpTo.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Page/Search.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/Page/Search/Handler.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/Activity/Recent.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/User/CoverPhoto/Delete.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/Profile/Menu/Item/Abstract.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/User/Profile/Menu/Item/Follow.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/User/Profile/Menu/Item/Ignore.ts
wcfsetup/install/files/ts/WoltLabSuite/Core/Ui/User/Trophy/List.ts

index 4deefb2ed90e918dfc5f7090f2594f9e28e0f935..3aa90655150209c4a46fe44f6d14983d9ea00a36 100644 (file)
@@ -43,7 +43,7 @@ export type CallbackSuccess = (
   requestData: RequestData
 ) => void;
 export type CallbackUploadProgress = (event: ProgressEvent) => void;
-export type CallbackSetup = () => RequestOptions;
+export type AjaxCallbackSetup = () => RequestOptions;
 
 export interface AjaxCallbackObject {
   _ajaxFailure?: CallbackFailure;
@@ -51,7 +51,7 @@ export interface AjaxCallbackObject {
   _ajaxProgress?: CallbackProgress;
   _ajaxSuccess: CallbackSuccess;
   _ajaxUploadProgress?: CallbackUploadProgress;
-  _ajaxSetup: CallbackSetup;
+  _ajaxSetup: AjaxCallbackSetup;
 }
 
 export interface RequestOptions {
index 7bf70a73a768e37a970cd590bb3ad2c5e6159e6a..1199abb9e9905d1b7138719c283851aabff11f8b 100644 (file)
@@ -8,7 +8,7 @@
  */
 
 import * as Ajax from "./Ajax";
-import { AjaxCallbackObject, CallbackSetup, ResponseData } from "./Ajax/Data";
+import { AjaxCallbackObject, AjaxCallbackSetup, ResponseData } from "./Ajax/Data";
 
 class BackgroundQueue implements AjaxCallbackObject {
   private _invocations = 0;
@@ -42,7 +42,7 @@ class BackgroundQueue implements AjaxCallbackObject {
     }
   }
 
-  _ajaxSetup(): ReturnType<CallbackSetup> {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       url: this._url,
       ignoreError: true,
index ff81434f19ab9d24280707c8fac20dcd6c0dfc88..4a7c8add704e7b9dae5352bb412d301f50b0b382 100644 (file)
@@ -8,7 +8,7 @@
  */
 
 import * as Ajax from "../../Ajax";
-import { AjaxCallbackObject, CallbackSetup } from "../../Ajax/Data";
+import { AjaxCallbackObject, AjaxCallbackSetup } from "../../Ajax/Data";
 
 class UiArticleMarkAllAsRead implements AjaxCallbackObject {
   constructor() {
@@ -33,7 +33,7 @@ class UiArticleMarkAllAsRead implements AjaxCallbackObject {
     document.querySelectorAll(".articleList .newMessageBadge").forEach((el) => el.remove());
   }
 
-  _ajaxSetup(): ReturnType<CallbackSetup> {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       data: {
         actionName: "markAllAsRead",
index 49096a4e481bb122bd54a128ef23da489ffd1cfb..6fca7bb094bef36bd03d4ccac8a350f8764eee89 100644 (file)
@@ -1,6 +1,6 @@
 import * as Ajax from "../../Ajax";
-import { AjaxCallbackObject, CallbackSetup, DatabaseObjectActionResponse } from "../../Ajax/Data";
-import { DialogCallbackObject, CallbackSetup as DialogSetup } from "../Dialog/Data";
+import { AjaxCallbackObject, AjaxCallbackSetup, DatabaseObjectActionResponse } from "../../Ajax/Data";
+import { DialogCallbackObject, DialogCallbackSetup } from "../Dialog/Data";
 import DomUtil from "../../Dom/Util";
 import * as Language from "../../Language";
 import * as StringUtil from "../../StringUtil";
@@ -89,7 +89,7 @@ class UiArticleSearch implements AjaxCallbackObject, DialogCallbackObject {
     }
   }
 
-  _ajaxSetup(): ReturnType<CallbackSetup> {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       data: {
         actionName: "search",
@@ -98,7 +98,7 @@ class UiArticleSearch implements AjaxCallbackObject, DialogCallbackObject {
     };
   }
 
-  _dialogSetup(): ReturnType<DialogSetup> {
+  _dialogSetup(): ReturnType<DialogCallbackSetup> {
     return {
       id: "wcfUiArticleSearch",
       options: {
index 6bb30e04a4e188b5efea626d490c1787db784d52..e19a0eb49ef8a1f7ddc212958d9775904653a94b 100644 (file)
@@ -11,7 +11,7 @@
 import * as Core from "../Core";
 import * as Language from "../Language";
 import UiDialog from "./Dialog";
-import { DialogCallbackObject, CallbackSetup as DialogSetup } from "./Dialog/Data";
+import { DialogCallbackObject, DialogCallbackSetup } from "./Dialog/Data";
 
 class UiConfirmation implements DialogCallbackObject {
   private _active = false;
@@ -126,7 +126,7 @@ class UiConfirmation implements DialogCallbackObject {
     this.confirmButton.focus();
   }
 
-  _dialogSetup(): ReturnType<DialogSetup> {
+  _dialogSetup(): ReturnType<DialogCallbackSetup> {
     return {
       id: "wcfSystemConfirmation",
       options: {
index 8e48980a978cfeb43c20b1bef19c06f255b82d86..576c8974d235b5cc079cca3dc8246548e16a2710 100644 (file)
@@ -24,7 +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";
+import { AjaxCallbackSetup } from "../Ajax/Data";
 
 let _activeDialog: string | null = null;
 let _callbackFocus: (event: FocusEvent) => void;
@@ -889,7 +889,7 @@ const UiDialog = {
     return id.toString();
   },
 
-  _ajaxSetup(): ReturnType<CallbackSetup> {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {};
   },
 };
index 38be27a05720944340653e8d872dbef285039c31..02eb4032792d37f4f6742da15030e2ade70c8720 100644 (file)
@@ -2,11 +2,11 @@ import { RequestPayload, ResponseData } from "../../Ajax/Data";
 
 export type DialogHtml = DocumentFragment | string | null;
 
-export type CallbackSetup = () => DialogSettings;
+export type DialogCallbackSetup = () => DialogSettings;
 export type CallbackSubmit = () => void;
 
 export interface DialogCallbackObject {
-  _dialogSetup: CallbackSetup;
+  _dialogSetup: DialogCallbackSetup;
   _dialogSubmit?: CallbackSubmit;
 }
 
index 19a48e918cc348917246a22be428c1a9c6388872..f24d8def201b9efb371581b1ec18dc4becb127d8 100644 (file)
@@ -9,7 +9,7 @@
  */
 
 import * as Ajax from "../../Ajax";
-import { AjaxCallbackObject, CallbackSetup, DatabaseObjectActionResponse } from "../../Ajax/Data";
+import { AjaxCallbackObject, AjaxCallbackSetup, 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(): ReturnType<CallbackSetup> {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       url: "index.php?ajax-file-delete/&t=" + window.SECURITY_TOKEN,
     };
index bb6d31a8206d92c6623e2e288aba521063a07d40..44f4dda579b2ff4626167e909ad2f18fc2d5660a 100644 (file)
@@ -7,7 +7,7 @@
  * @module  WoltLabSuite/Core/Ui/Page/JumpTo
  */
 
-import { DialogCallbackObject, CallbackSetup as DialogSetup } from "../Dialog/Data";
+import { DialogCallbackObject, DialogCallbackSetup } from "../Dialog/Data";
 import * as Language from "../../Language";
 import UiDialog from "../Dialog";
 
@@ -88,7 +88,7 @@ class UiPageJumpTo implements DialogCallbackObject {
     UiDialog.close(this);
   }
 
-  _dialogSetup(): ReturnType<DialogSetup> {
+  _dialogSetup(): ReturnType<DialogCallbackSetup> {
     const source = `<dl>
         <dt><label for="jsPaginationPageNo">${Language.get("wcf.page.jumpTo")}</label></dt>
                 <dd>
index 611294a2fe1eecae9b6440e3ee4206ca883d2a62..cfc95d69f3517ffbcfd7cadaf74771a65ce2522c 100644 (file)
@@ -1,6 +1,6 @@
 import * as Ajax from "../../Ajax";
-import { AjaxCallbackObject, CallbackSetup, DatabaseObjectActionResponse } from "../../Ajax/Data";
-import { DialogCallbackObject, CallbackSetup as DialogSetup } from "../Dialog/Data";
+import { AjaxCallbackObject, AjaxCallbackSetup, DatabaseObjectActionResponse } from "../../Ajax/Data";
+import { DialogCallbackObject, DialogCallbackSetup } from "../Dialog/Data";
 import DomUtil from "../../Dom/Util";
 import * as Language from "../../Language";
 import * as StringUtil from "../../StringUtil";
@@ -89,7 +89,7 @@ class UiPageSearch implements AjaxCallbackObject, DialogCallbackObject {
     }
   }
 
-  _ajaxSetup(): ReturnType<CallbackSetup> {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       data: {
         actionName: "search",
@@ -98,7 +98,7 @@ class UiPageSearch implements AjaxCallbackObject, DialogCallbackObject {
     };
   }
 
-  _dialogSetup(): ReturnType<DialogSetup> {
+  _dialogSetup(): ReturnType<DialogCallbackSetup> {
     return {
       id: "wcfUiPageSearch",
       options: {
index 3e33ca0025a80ccaafc7db9996b1ed2e5d569c51..81d48de43a40ab27338df8923c90a20a10f435d7 100644 (file)
@@ -12,7 +12,7 @@ import * as Language from "../../../Language";
 import * as StringUtil from "../../../StringUtil";
 import DomUtil from "../../../Dom/Util";
 import UiDialog from "../../Dialog";
-import { DialogCallbackObject, CallbackSetup as DialogSetup } from "../../Dialog/Data";
+import { DialogCallbackObject, DialogCallbackSetup } from "../../Dialog/Data";
 import UiPageSearchInput from "./Input";
 import { DatabaseObjectActionResponse } from "../../../Ajax/Data";
 
@@ -137,7 +137,7 @@ class UiPageSearchHandler implements DialogCallbackObject {
     UiDialog.close(this);
   }
 
-  _dialogSetup(): ReturnType<DialogSetup> {
+  _dialogSetup(): ReturnType<DialogCallbackSetup> {
     return {
       id: "wcfUiPageSearchHandler",
       options: {
index 1ab9c20fcede03d0b6516e4cd95f88dd38f1fb82..c9ab493a9f4a23b68868ba1e2b425f6ccfaf207b 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 { CallbackSetup, DatabaseObjectActionPayload, DatabaseObjectActionResponse } from "../../Ajax/Data";
+import { AjaxCallbackSetup, 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(): ReturnType<CallbackSetup> {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       data: this.ajaxPayload,
     };
index 48b4d810e85739e9d12b564b4c0c73cb504fa8a9..ea120b3adafd48840fd5ebf3974d80d3a10ec0cf 100644 (file)
@@ -11,7 +11,7 @@ import * as Ajax from "../Ajax";
 import * as Core from "../Core";
 import {
   AjaxCallbackObject,
-  CallbackSetup,
+  AjaxCallbackSetup,
   DatabaseObjectActionPayload,
   DatabaseObjectActionResponse,
 } from "../Ajax/Data";
@@ -198,7 +198,7 @@ class UiSuggestion implements AjaxCallbackObject {
     });
   }
 
-  _ajaxSetup(): ReturnType<CallbackSetup> {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       data: this.ajaxPayload,
     };
index 690691d3daf2f7f83ad41f7413c1491662d5791b..01ac03263fa3aa43d7046bdb5d7da6e8293653a3 100644 (file)
@@ -1,5 +1,5 @@
 import * as Ajax from "../../../Ajax";
-import { AjaxCallbackObject, RequestOptions } from "../../../Ajax/Data";
+import { AjaxCallbackObject, AjaxCallbackSetup } from "../../../Ajax/Data";
 import * as Core from "../../../Core";
 import * as Language from "../../../Language";
 import DomUtil from "../../../Dom/Util";
@@ -92,7 +92,7 @@ class UiUserActivityRecent implements AjaxCallbackObject {
     }
   }
 
-  _ajaxSetup(): RequestOptions {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       data: {
         className: "wcf\\data\\user\\activity\\event\\UserActivityEventAction",
index b4ca4733a0fc5ce626ed809d674f08967a0cc25b..d3bd4b526012909a610ba4f2317b33b9961cef3c 100644 (file)
@@ -8,7 +8,7 @@
  */
 
 import * as Ajax from "../../../Ajax";
-import { AjaxCallbackObject, ResponseData } from "../../../Ajax/Data";
+import { AjaxCallbackObject, AjaxCallbackSetup, ResponseData } from "../../../Ajax/Data";
 import DomUtil from "../../../Dom/Util";
 import * as EventHandler from "../../../Event/Handler";
 import * as Language from "../../../Language";
@@ -61,7 +61,7 @@ class UiUserCoverPhotoDelete implements AjaxCallbackObject {
     UiNotification.show();
   }
 
-  _ajaxSetup() {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       data: {
         actionName: "deleteCoverPhoto",
index ac354aae7cf878267013b0ff5c082615c795fc24..a42ee29275e2e8c7355640e1b3bcff8db115493b 100644 (file)
@@ -8,9 +8,9 @@
  */
 
 import * as Ajax from "../../Ajax";
-import { AjaxCallbackObject, CallbackSetup } from "../../Ajax/Data";
+import { AjaxCallbackObject, AjaxCallbackSetup } from "../../Ajax/Data";
 import * as Core from "../../Core";
-import { DialogCallbackObject, CallbackSetup as DialogSetup } from "../Dialog/Data";
+import { DialogCallbackObject, DialogCallbackSetup } from "../Dialog/Data";
 import DomUtil from "../../Dom/Util";
 import * as Language from "../../Language";
 import * as StringUtil from "../../StringUtil";
@@ -183,7 +183,7 @@ class UserEditor implements AjaxCallbackObject, DialogCallbackObject {
     UiNotification.show();
   }
 
-  _ajaxSetup(): ReturnType<CallbackSetup> {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       data: {
         className: "wcf\\data\\user\\UserAction",
@@ -192,7 +192,7 @@ class UserEditor implements AjaxCallbackObject, DialogCallbackObject {
     };
   }
 
-  _dialogSetup(): ReturnType<DialogSetup> {
+  _dialogSetup(): ReturnType<DialogCallbackSetup> {
     return {
       id: "wcfUiUserEditor",
       options: {
index a6c5b564165b60fb707b091c88a1c55156e8a6cf..359a62aac3c7b5a4be01ecb7ee556da11aea0c72 100644 (file)
@@ -12,8 +12,8 @@ import * as Core from "../../Core";
 import DomUtil from "../../Dom/Util";
 import UiDialog from "../Dialog";
 import UiPagination from "../Pagination";
-import { AjaxCallbackObject, CallbackSetup, DatabaseObjectActionResponse } from "../../Ajax/Data";
-import { DialogCallbackObject, DialogData, CallbackSetup as DialogSetup } from "../Dialog/Data";
+import { AjaxCallbackObject, AjaxCallbackSetup, DatabaseObjectActionResponse } from "../../Ajax/Data";
+import { DialogCallbackObject, DialogData, DialogCallbackSetup } from "../Dialog/Data";
 
 /**
  * @constructor
@@ -98,7 +98,7 @@ class UiUserList implements AjaxCallbackObject, DialogCallbackObject {
     this.showPage();
   }
 
-  _ajaxSetup(): ReturnType<CallbackSetup> {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       data: {
         actionName: "getGroupedUserList",
@@ -108,7 +108,7 @@ class UiUserList implements AjaxCallbackObject, DialogCallbackObject {
     };
   }
 
-  _dialogSetup(): ReturnType<DialogSetup> {
+  _dialogSetup(): ReturnType<DialogCallbackSetup> {
     return {
       id: DomUtil.getUniqueId(),
       options: {
index 4a3b5bd846771cf22acbe332a73aaddfcbbba672..a2d7213f1fcea5c87eb8dcf8542a663f640e498b 100644 (file)
@@ -8,7 +8,7 @@
  */
 
 import * as Ajax from "../../../../../Ajax";
-import { AjaxCallbackObject, RequestOptions, ResponseData } from "../../../../../Ajax/Data";
+import { AjaxCallbackObject, AjaxCallbackSetup, ResponseData } from "../../../../../Ajax/Data";
 
 abstract class UiUserProfileMenuItemAbstract implements AjaxCallbackObject {
   protected readonly _button = document.createElement("a");
@@ -99,7 +99,7 @@ abstract class UiUserProfileMenuItemAbstract implements AjaxCallbackObject {
   /**
    * Returns the default Ajax request data
    */
-  _ajaxSetup(): RequestOptions {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     // This should be an abstract method, but cannot be marked as such for backwards compatibility.
 
     throw new Error("Implement me!");
index d836cba227b1500eed69ee59993c4b7499d641a7..633deaf9ab4c28f9e6dd00a26e00d85c435549df 100644 (file)
@@ -1,5 +1,5 @@
 import * as Language from "../../../../../Language";
-import { RequestOptions, ResponseData } from "../../../../../Ajax/Data";
+import { AjaxCallbackSetup, ResponseData } from "../../../../../Ajax/Data";
 import * as UiNotification from "../../../../Notification";
 import UiUserProfileMenuItemAbstract from "./Abstract";
 
@@ -29,7 +29,7 @@ class UiUserProfileMenuItemFollow extends UiUserProfileMenuItemAbstract {
     UiNotification.show();
   }
 
-  _ajaxSetup(): RequestOptions {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       data: {
         className: "wcf\\data\\user\\follow\\UserFollowAction",
index b376d01f1677f1e62087f7f375ff00ec1ba9bc81..c36e91ff4245771c1c61605f56e3a4ee0779a38e 100644 (file)
@@ -1,5 +1,5 @@
 import * as Language from "../../../../../Language";
-import { RequestOptions, ResponseData } from "../../../../../Ajax/Data";
+import { AjaxCallbackSetup, ResponseData } from "../../../../../Ajax/Data";
 import * as UiNotification from "../../../../Notification";
 import UiUserProfileMenuItemAbstract from "./Abstract";
 
@@ -29,7 +29,7 @@ class UiUserProfileMenuItemIgnore extends UiUserProfileMenuItemAbstract {
     UiNotification.show();
   }
 
-  _ajaxSetup(): RequestOptions {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       data: {
         className: "wcf\\data\\user\\ignore\\UserIgnoreAction",
index 51aba6ee188c32ef1cf61ac5a429219aeb2ea42e..9f407eaef1b48dee8cefe85f66fbc077a86c927c 100644 (file)
@@ -8,8 +8,8 @@
  */
 
 import * as Ajax from "../../../Ajax";
-import { AjaxCallbackObject, CallbackSetup, DatabaseObjectActionResponse } from "../../../Ajax/Data";
-import { DialogCallbackObject, DialogData, CallbackSetup as DialogSetup } from "../../Dialog/Data";
+import { AjaxCallbackObject, AjaxCallbackSetup, DatabaseObjectActionResponse } from "../../../Ajax/Data";
+import { DialogCallbackObject, DialogData, DialogCallbackSetup } from "../../Dialog/Data";
 import DomChangeListener from "../../../Dom/Change/Listener";
 import UiDialog from "../../Dialog";
 import UiPagination from "../../Pagination";
@@ -124,7 +124,7 @@ class UiUserTrophyList implements AjaxCallbackObject, DialogCallbackObject {
     this.showPage();
   }
 
-  _ajaxSetup(): ReturnType<CallbackSetup> {
+  _ajaxSetup(): ReturnType<AjaxCallbackSetup> {
     return {
       data: {
         actionName: "getGroupedUserTrophyList",
@@ -133,7 +133,7 @@ class UiUserTrophyList implements AjaxCallbackObject, DialogCallbackObject {
     };
   }
 
-  _dialogSetup(): ReturnType<DialogSetup> {
+  _dialogSetup(): ReturnType<DialogCallbackSetup> {
     return {
       id: "userTrophyListOverlay",
       options: {