Bug 1600954 - [autoconfig] Re-run config after password changed. r=jorgk a=mkmelin
authorBen Bucksch <ben.bucksch@beonex.com>
Mon, 23 Dec 2019 22:04:15 +0100
changeset 36051 c139962d61d6d14d69e2e38b3e2aa61866c325a2
parent 36050 b713c06a547123bc7b28d926839cf3ffcb9adba6
child 36052 8dfff3c0f5a6b10bcfa207323da5f91a78382e21
push id71
push usergeoff@darktrojan.net
push dateTue, 07 Jan 2020 21:16:26 +0000
treeherdercomm-esr68@939b668210b1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjorgk, mkmelin
bugs1600954
Bug 1600954 - [autoconfig] Re-run config after password changed. r=jorgk a=mkmelin
mail/components/accountcreation/content/emailWizard.js
--- a/mail/components/accountcreation/content/emailWizard.js
+++ b/mail/components/accountcreation/content/emailWizard.js
@@ -449,16 +449,17 @@ EmailConfigWizard.prototype = {
    * Start from beginning with possibly new email address.
    */
   onStartOver() {
     this._currentConfig = null;
     if (this._abortable) {
       this.onStop();
     }
     this.switchToMode("start");
+    this.checkStartDone();
   },
 
   getConcreteConfig() {
     var result = this._currentConfig.copy();
     replaceVariables(result, this._realname, this._email, this._password);
     result.rememberPassword =
       e("remember_password").checked && !!this._password;
     if (result.incoming.addonAccountType) {
@@ -496,32 +497,31 @@ EmailConfigWizard.prototype = {
    * e-mail address and non-blank realname to start with.
    *
    * A change to the email address also automatically restarts the
    * whole process.
    */
   onInputEmail() {
     this._email = e("email").value;
     this.onStartOver();
-    this.checkStartDone();
   },
 
   onInputRealname() {
     this._realname = e("realname").value;
     this.checkStartDone();
   },
 
   onInputUsername() {
     this._exchangeUsername = e("usernameEx").value;
-    this.checkStartDone();
+    this.onStartOver();
   },
 
   onInputPassword() {
     this._password = e("password").value;
-    this.checkStartDone();
+    this.onStartOver();
   },
 
   /**
    * This does very little other than to check that a name was entered at all
    * Since this is such an insignificant test we should be using a very light
    * or even jovial warning.
    */
   onBlurRealname() {