Bug 1600954 - [autoconfig] Re-run config after password changed. r=jorgk
authorBen Bucksch <ben.bucksch@beonex.com>
Mon, 23 Dec 2019 22:04:15 +0100
changeset 28430 0ab86bf9ef19636c561cbb366224e45fec872254
parent 28429 77e6b123861641cfde5c9b7215e388afd5995cb0
child 28431 e108db02cb6d010d01e58cc5dc233b169592bd48
push id16837
push usermozilla@jorgk.com
push dateMon, 23 Dec 2019 21:13:24 +0000
treeherdercomm-central@0ab86bf9ef19 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjorgk
bugs1600954
Bug 1600954 - [autoconfig] Re-run config after password changed. r=jorgk
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() {