From 2f3865d8cc8407c4a1adc27e7155ac6279338e5a Mon Sep 17 00:00:00 2001 From: RMidhunSuresh Date: Tue, 1 Feb 2022 18:23:41 +0530 Subject: [PATCH] firstStage should be a local variable --- src/matrix/registration/Registration.ts | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/matrix/registration/Registration.ts b/src/matrix/registration/Registration.ts index e9c0b512..c6e26604 100644 --- a/src/matrix/registration/Registration.ts +++ b/src/matrix/registration/Registration.ts @@ -33,7 +33,6 @@ export type RegistrationResponse = { export class Registration { private _hsApi: HomeServerApi; private _data: RegistrationParameters; - private _firstStage: BaseRegistrationStage; constructor(hsApi: HomeServerApi, data: RegistrationParameters) { this._hsApi = hsApi; @@ -47,13 +46,13 @@ export class Registration { this._initialDeviceDisplayName, undefined, this._inhibitLogin).response(); - this.parseStagesFromResponse(response); - return this._firstStage; + return this.parseStagesFromResponse(response); } - parseStagesFromResponse(response: RegistrationResponse) { + parseStagesFromResponse(response: RegistrationResponse): BaseRegistrationStage { const { session, params } = response; const flow = response.flows.pop(); + let firstStage: BaseRegistrationStage | undefined; let lastStage: BaseRegistrationStage; for (const stage of flow.stages) { const stageClass = registrationStageFromType(stage); @@ -61,14 +60,15 @@ export class Registration { throw new Error("Unknown stage"); } const registrationStage = new stageClass(this._hsApi, this._data, session, params?.[stage]); - if (!this._firstStage) { - this._firstStage = registrationStage; + if (!firstStage) { + firstStage = registrationStage; lastStage = registrationStage; } else { lastStage!.setNextStage(registrationStage); lastStage = registrationStage; } } + return firstStage!; } private get _username() { return this._data.username; }