From 3dbffdb417ab32695624563cb4a3d1fb79418723 Mon Sep 17 00:00:00 2001 From: Bruno Windels Date: Mon, 23 Aug 2021 19:28:43 +0200 Subject: [PATCH] don't requery the same homeserver we just did --- src/domain/login/LoginViewModel.js | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/domain/login/LoginViewModel.js b/src/domain/login/LoginViewModel.js index 72cde571..d629241d 100644 --- a/src/domain/login/LoginViewModel.js +++ b/src/domain/login/LoginViewModel.js @@ -36,6 +36,7 @@ export class LoginViewModel extends ViewModel { this._loadViewModel = null; this._loadViewModelSubscription = null; this._homeserver = defaultHomeserver; + this._queriedHomeserver = null; this._errorMessage = ""; this._hideHomeserver = false; this._isBusy = false; @@ -183,6 +184,11 @@ export class LoginViewModel extends ViewModel { } async queryHomeserver() { + // don't repeat a query we've just done + if (this._homeserver === this._queriedHomeserver || this._homeserver === "") { + return; + } + this._queriedHomeserver = this._homeserver; // given that setHomeserver already clears everything set here, // and that is the only way to change the homeserver, // we don't need to reset things again here.