Remove dead & duplicated code (#11405)

t3chguy · web-flow · commit 27d79458dabe · 2023-08-15T08:43:15.000Z
* Remove dead code

* Make dead code happier

* DRY pickle additional data calculation

* Iterate
diff --git a/src/AddThreepid.ts b/src/AddThreepid.ts
@@ -281,16 +281,15 @@ export default class AddThreepid {
     ): Promise<[success?: boolean, result?: IAuthData | Error | null] | undefined> {
         const authClient = new IdentityAuthClient();
 
-        let result: { success: boolean } | MatrixError;
         if (this.submitUrl) {
-            result = await this.matrixClient.submitMsisdnTokenOtherUrl(
+            await this.matrixClient.submitMsisdnTokenOtherUrl(
                 this.submitUrl,
                 this.sessionId!,
                 this.clientSecret,
                 msisdnToken,
             );
         } else if (this.bind) {
-            result = await this.matrixClient.submitMsisdnToken(
+            await this.matrixClient.submitMsisdnToken(
                 this.sessionId!,
                 this.clientSecret,
                 msisdnToken,
@@ -299,9 +298,6 @@ export default class AddThreepid {
         } else {
             throw new UserFriendlyError("The add / bind with MSISDN flow is misconfigured");
         }
-        if (result instanceof Error) {
-            throw result;
-        }
 
         if (this.bind) {
             await this.matrixClient.bindThreePid({
diff --git a/src/BasePlatform.ts b/src/BasePlatform.ts
@@ -67,7 +67,7 @@ export default abstract class BasePlatform {
     protected notificationCount = 0;
     protected errorDidOccur = false;
 
-    public constructor() {
+    protected constructor() {
         dis.register(this.onAction);
         this.startUpdateCheck = this.startUpdateCheck.bind(this);
     }
@@ -365,14 +365,7 @@ export default abstract class BasePlatform {
             return null;
         }
 
-        const additionalData = new Uint8Array(userId.length + deviceId.length + 1);
-        for (let i = 0; i < userId.length; i++) {
-            additionalData[i] = userId.charCodeAt(i);
-        }
-        additionalData[userId.length] = 124; // "|"
-        for (let i = 0; i < deviceId.length; i++) {
-            additionalData[userId.length + 1 + i] = deviceId.charCodeAt(i);
-        }
+        const additionalData = this.getPickleAdditionalData(userId, deviceId);
 
         try {
             const key = await crypto.subtle.decrypt(
@@ -387,6 +380,18 @@ export default abstract class BasePlatform {
         }
     }
 
+    private getPickleAdditionalData(userId: string, deviceId: string): Uint8Array {
+        const additionalData = new Uint8Array(userId.length + deviceId.length + 1);
+        for (let i = 0; i < userId.length; i++) {
+            additionalData[i] = userId.charCodeAt(i);
+        }
+        additionalData[userId.length] = 124; // "|"
+        for (let i = 0; i < deviceId.length; i++) {
+            additionalData[userId.length + 1 + i] = deviceId.charCodeAt(i);
+        }
+        return additionalData;
+    }
+
     /**
      * Create and store a pickle key for encrypting libolm objects.
      * @param {string} userId the user ID for the user that the pickle key is for.
@@ -408,15 +413,7 @@ export default abstract class BasePlatform {
         const iv = new Uint8Array(32);
         crypto.getRandomValues(iv);
 
-        const additionalData = new Uint8Array(userId.length + deviceId.length + 1);
-        for (let i = 0; i < userId.length; i++) {
-            additionalData[i] = userId.charCodeAt(i);
-        }
-        additionalData[userId.length] = 124; // "|"
-        for (let i = 0; i < deviceId.length; i++) {
-            additionalData[userId.length + 1 + i] = deviceId.charCodeAt(i);
-        }
-
+        const additionalData = this.getPickleAdditionalData(userId, deviceId);
         const encrypted = await crypto.subtle.encrypt({ name: "AES-GCM", iv, additionalData }, cryptoKey, randomArray);
 
         try {
diff --git a/src/IdentityAuthClient.tsx b/src/IdentityAuthClient.tsx
@@ -75,10 +75,6 @@ export default class IdentityAuthClient {
         return window.localStorage.getItem("mx_is_access_token");
     }
 
-    public hasCredentials(): boolean {
-        return Boolean(this.accessToken);
-    }
-
     // Returns a promise that resolves to the access_token string from the IS
     public async getAccessToken({ check = true } = {}): Promise<string | null> {
         if (!this.authEnabled) {
diff --git a/src/LegacyCallHandler.tsx b/src/LegacyCallHandler.tsx
@@ -429,15 +429,6 @@ export default class LegacyCallHandler extends EventEmitter {
         return this.calls.get(roomId) || null;
     }
 
-    public getAnyActiveCall(): MatrixCall | null {
-        for (const call of this.calls.values()) {
-            if (call.state !== CallState.Ended) {
-                return call;
-            }
-        }
-        return null;
-    }
-
     public getAllActiveCalls(): MatrixCall[] {
         const activeCalls: MatrixCall[] = [];
 
diff --git a/src/PasswordReset.ts b/src/PasswordReset.ts
@@ -47,41 +47,6 @@ export default class PasswordReset {
         this.clientSecret = this.client.generateClientSecret();
     }
 
-    /**
-     * Attempt to reset the user's password. This will trigger a side-effect of
-     * sending an email to the provided email address.
-     * @param {string} emailAddress The email address
-     * @param {string} newPassword The new password for the account.
-     * @param {boolean} logoutDevices Should all devices be signed out after the reset? Defaults to `true`.
-     * @return {Promise} Resolves when the email has been sent. Then call checkEmailLinkClicked().
-     */
-    public async resetPassword(
-        emailAddress: string,
-        newPassword: string,
-        logoutDevices = true,
-    ): Promise<IRequestTokenResponse> {
-        this.password = newPassword;
-        this.logoutDevices = logoutDevices;
-        this.sendAttempt++;
-
-        try {
-            const result = await this.client.requestPasswordEmailToken(
-                emailAddress,
-                this.clientSecret,
-                this.sendAttempt,
-            );
-            this.sessionId = result.sid;
-            return result;
-        } catch (err: any) {
-            if (err.errcode === "M_THREEPID_NOT_FOUND") {
-                err.message = _t("This email address was not found");
-            } else if (err.httpStatus) {
-                err.message = err.message + ` (Status ${err.httpStatus})`;
-            }
-            throw err;
-        }
-    }
-
     /**
      * Request a password reset token.
      * This will trigger a side-effect of sending an email to the provided email address.
diff --git a/src/RoomNotifs.ts b/src/RoomNotifs.ts
@@ -256,9 +256,8 @@ export function determineUnreadState(
         return { symbol: null, count: trueCount, color: NotificationColor.Grey };
     }
 
-    // We don't have any notified messages, but we might have unread messages. Let's
-    // find out.
-    let hasUnread = false;
+    // We don't have any notified messages, but we might have unread messages. Let's find out.
+    let hasUnread: boolean;
     if (threadId) hasUnread = doesRoomOrThreadHaveUnreadMessages(room.getThread(threadId)!);
     else hasUnread = doesRoomHaveUnreadMessages(room);
 
diff --git a/src/ScalarMessaging.ts b/src/ScalarMessaging.ts
@@ -679,7 +679,7 @@ function canSendEvent(event: MessageEvent<any>, roomId: string): void {
     }
     const me = client.credentials.userId!;
 
-    let canSend = false;
+    let canSend: boolean;
     if (isState) {
         canSend = room.currentState.maySendStateEvent(evType, me);
     } else {
diff --git a/src/components/views/dialogs/FeedbackDialog.tsx b/src/components/views/dialogs/FeedbackDialog.tsx
@@ -117,7 +117,7 @@ const FeedbackDialog: React.FC<IProps> = (props: IProps) => {
     return (
         <QuestionDialog
             className="mx_FeedbackDialog"
-            hasCancelButton={!!hasFeedback}
+            hasCancelButton={hasFeedback}
             title={_t("Feedback")}
             description={
                 <React.Fragment>
diff --git a/src/editor/model.ts b/src/editor/model.ts
@@ -461,7 +461,7 @@ export default class EditorModel {
      */
     public transform(callback: ManualTransformCallback): Promise<void> {
         const pos = callback();
-        let acPromise: Promise<void> | null = null;
+        let acPromise: Promise<void> | null;
         if (!(pos instanceof Range)) {
             acPromise = this.setActivePart(pos, true);
         } else {
diff --git a/src/indexing/BaseEventIndexManager.ts b/src/indexing/BaseEventIndexManager.ts
@@ -112,13 +112,6 @@ export default abstract class BaseEventIndexManager {
         throw new Error("Unimplemented");
     }
 
-    /**
-     * Check if our event index is empty.
-     */
-    public indexIsEmpty(): Promise<boolean> {
-        throw new Error("Unimplemented");
-    }
-
     /**
      * Check if the room with the given id is already indexed.
      *
diff --git a/src/integrations/IntegrationManagers.ts b/src/integrations/IntegrationManagers.ts
@@ -26,7 +26,6 @@ import IntegrationsImpossibleDialog from "../components/views/dialogs/Integratio
 import IntegrationsDisabledDialog from "../components/views/dialogs/IntegrationsDisabledDialog";
 import WidgetUtils from "../utils/WidgetUtils";
 import { MatrixClientPeg } from "../MatrixClientPeg";
-import { parseUrl } from "../utils/UrlUtils";
 
 const KIND_PREFERENCE = [
     // Ordered: first is most preferred, last is least preferred.
@@ -179,52 +178,6 @@ export class IntegrationManagers {
     public showDisabledDialog(): void {
         Modal.createDialog(IntegrationsDisabledDialog);
     }
-
-    /**
-     * Attempts to discover an integration manager using only its name. This will not validate that
-     * the integration manager is functional - that is the caller's responsibility.
-     * @param {string} domainName The domain name to look up.
-     * @returns {Promise<IntegrationManagerInstance>} Resolves to an integration manager instance,
-     * or null if none was found.
-     */
-    public async tryDiscoverManager(domainName: string): Promise<IntegrationManagerInstance | null> {
-        logger.log("Looking up integration manager via .well-known");
-        if (domainName.startsWith("http:") || domainName.startsWith("https:")) {
-            // trim off the scheme and just use the domain
-            domainName = parseUrl(domainName).host;
-        }
-
-        let wkConfig: IClientWellKnown;
-        try {
-            const result = await fetch(`https://${domainName}/.well-known/matrix/integrations`);
-            wkConfig = await result.json();
-        } catch (e) {
-            logger.error(e);
-            logger.warn("Failed to locate integration manager");
-            return null;
-        }
-
-        if (!wkConfig || !wkConfig["m.integrations_widget"]) {
-            logger.warn("Missing integrations widget on .well-known response");
-            return null;
-        }
-
-        const widget = wkConfig["m.integrations_widget"];
-        if (!widget["url"] || !widget["data"] || !widget["data"]["api_url"]) {
-            logger.warn("Malformed .well-known response for integrations widget");
-            return null;
-        }
-
-        // All discovered managers are per-user managers
-        const manager = new IntegrationManagerInstance(Kind.Account, widget["data"]["api_url"], widget["url"]);
-        logger.log("Got an integration manager (untested)");
-
-        // We don't test the manager because the caller may need to do extra
-        // checks or similar with it. For instance, they may need to deal with
-        // terms of service or want to call something particular.
-
-        return manager;
-    }
 }
 
 // For debugging
diff --git a/src/mjolnir/BanList.ts b/src/mjolnir/BanList.ts
@@ -63,10 +63,6 @@ export class BanList {
         return this._rules.filter((r) => r.kind === RULE_USER);
     }
 
-    public get roomRules(): ListRule[] {
-        return this._rules.filter((r) => r.kind === RULE_ROOM);
-    }
-
     public async banEntity(kind: string, entity: string, reason: string): Promise<any> {
         const type = ruleTypeToStable(kind);
         if (!type) return; // unknown rule type
diff --git a/src/models/Call.ts b/src/models/Call.ts
@@ -157,7 +157,7 @@ export abstract class Call extends TypedEventEmitter<CallEvent, CallEventHandler
         this.emit(CallEvent.Participants, value, prevValue);
     }
 
-    public constructor(
+    protected constructor(
         /**
          * The widget used to access this call.
          */
diff --git a/src/rageshake/submit-rageshake.ts b/src/rageshake/submit-rageshake.ts
@@ -117,7 +117,7 @@ async function collectBugReport(opts: IOpts = {}, gzipLogs = true): Promise<Form
             );
 
             body.append("secret_storage_ready", String(await client.isSecretStorageReady()));
-            body.append("secret_storage_key_in_account", String(!!(await secretStorage.hasKey())));
+            body.append("secret_storage_key_in_account", String(await secretStorage.hasKey()));
 
             body.append("session_backup_key_in_secret_storage", String(!!(await client.isKeyBackupKeyStored())));
             const sessionBackupKeyFromCache = await client.crypto.getSessionBackupPrivateKey();
diff --git a/src/sentry.ts b/src/sentry.ts
@@ -139,7 +139,7 @@ async function getCryptoContext(client: MatrixClient): Promise<CryptoContext> {
             !!(pkCache && (await pkCache.getCrossSigningKeyCache?.("user_signing"))),
         ),
         secret_storage_ready: String(await client.isSecretStorageReady()),
-        secret_storage_key_in_account: String(!!(await secretStorage.hasKey())),
+        secret_storage_key_in_account: String(await secretStorage.hasKey()),
         session_backup_key_in_secret_storage: String(!!(await client.isKeyBackupKeyStored())),
         session_backup_key_cached: String(!!sessionBackupKeyFromCache),
         session_backup_key_well_formed: String(sessionBackupKeyFromCache instanceof Uint8Array),
diff --git a/src/stores/room-list/RoomListStore.ts b/src/stores/room-list/RoomListStore.ts
@@ -577,10 +577,9 @@ export class RoomListStoreClass extends AsyncStoreWithClient<IState> implements
      * @param {IFilterCondition} filter The filter condition to add.
      */
     public async addFilter(filter: IFilterCondition): Promise<void> {
-        let promise = Promise.resolve();
         filter.on(FILTER_CHANGED, this.onPrefilterUpdated);
         this.prefilterConditions.push(filter);
-        promise = this.recalculatePrefiltering();
+        const promise = this.recalculatePrefiltering();
         promise.then(() => this.updateFn.trigger());
     }
 
diff --git a/src/stores/room-list/SlidingRoomListStore.ts b/src/stores/room-list/SlidingRoomListStore.ts
@@ -399,6 +399,4 @@ export class SlidingRoomListStoreClass extends AsyncStoreWithClient<IState> impl
     }
 
     protected async onAction(payload: ActionPayload): Promise<void> {}
-
-    protected async onDispatchAsync(payload: ActionPayload): Promise<void> {}
 }
diff --git a/src/stores/room-list/algorithms/Algorithm.ts b/src/stores/room-list/algorithms/Algorithm.ts
@@ -90,10 +90,6 @@ export class Algorithm extends EventEmitter {
         return this._stickyRoom ? this._stickyRoom.room : null;
     }
 
-    public get knownRooms(): Room[] {
-        return this.rooms;
-    }
-
     public get hasTagSortingMap(): boolean {
         return !!this.sortAlgorithms;
     }
diff --git a/src/stores/widgets/StopGapWidget.ts b/src/stores/widgets/StopGapWidget.ts
@@ -242,10 +242,6 @@ export class StopGapWidget extends EventEmitter {
         return parsed.toString().replace(/%24/g, "$");
     }
 
-    public get isManagedByManager(): boolean {
-        return !!this.scalarToken;
-    }
-
     public get started(): boolean {
         return !!this.messaging;
     }
diff --git a/src/stores/widgets/StopGapWidgetDriver.ts b/src/stores/widgets/StopGapWidgetDriver.ts
@@ -245,7 +245,7 @@ export class StopGapWidgetDriver extends WidgetDriver {
 
         if (!client || !roomId) throw new Error("Not in a room or not attached to a client");
 
-        let r: { event_id: string } | null = null; // eslint-disable-line camelcase
+        let r: { event_id: string } | null;
         if (stateKey !== null) {
             // state event
             r = await client.sendStateEvent(roomId, eventType, content, stateKey);
diff --git a/src/utils/WidgetUtils.ts b/src/utils/WidgetUtils.ts
diff --git a/src/utils/permalinks/PermalinkConstructor.ts b/src/utils/permalinks/PermalinkConstructor.ts
diff --git a/src/utils/permalinks/Permalinks.ts b/src/utils/permalinks/Permalinks.ts
diff --git a/test/utils/SearchInput-test.ts b/test/utils/SearchInput-test.ts
diff --git a/test/utils/permalinks/Permalinks-test.ts b/test/utils/permalinks/Permalinks-test.ts

Original file line number	Diff line number	Diff line change
`@@ -679,7 +679,7 @@ function canSendEvent(event: MessageEvent<any>, roomId: string): void {`
`679`	`679`	`}`
`680`	`680`	`const me = client.credentials.userId!;`
`681`	`681`
`682`		`- let canSend = false;`
	`682`	`+ let canSend: boolean;`
`683`	`683`	`if (isState) {`
`684`	`684`	`canSend = room.currentState.maySendStateEvent(evType, me);`
`685`	`685`	`} else {`
Original file line number	Diff line number	Diff line change
`@@ -112,13 +112,6 @@ export default abstract class BaseEventIndexManager {`
`112`	`112`	`throw new Error("Unimplemented");`
`113`	`113`	`}`
`114`	`114`
`115`		`- /**`
`116`		`- * Check if our event index is empty.`
`117`		`- */`
`118`		`- public indexIsEmpty(): Promise<boolean> {`
`119`		`- throw new Error("Unimplemented");`
`120`		`- }`
`121`		`-`
`122`	`115`	`/**`
`123`	`116`	`* Check if the room with the given id is already indexed.`
`124`	`117`	`*`