diff options
author | Petr Vobornik <pvoborni@redhat.com> | 2016-01-20 18:33:21 +0100 |
---|---|---|
committer | Petr Vobornik <pvoborni@redhat.com> | 2016-02-23 10:18:35 +0100 |
commit | b9c27b672218c30d669d085b5a57045711542fb9 (patch) | |
tree | 0e1f36032935dcd0e807808977f45a469b25bff4 /install | |
parent | 31f42bc2e1e931b4c7dec9bf89eb94c844397ea2 (diff) | |
download | freeipa-b9c27b672218c30d669d085b5a57045711542fb9.tar.gz freeipa-b9c27b672218c30d669d085b5a57045711542fb9.tar.xz freeipa-b9c27b672218c30d669d085b5a57045711542fb9.zip |
webui: fixed showing of success message after password change on login
similar issue and cause as in https://fedorahosted.org/freeipa/ticket/5567
root cause is that binding triggers validation which clears messages in
validation summary. Maybe it could be refactored in a future to not use
the same validation summary field for API calls and fields.
Reviewed-By: Pavel Vomacka <pvomacka@redhat.com>
Diffstat (limited to 'install')
-rw-r--r-- | install/ui/src/freeipa/widgets/LoginScreen.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/install/ui/src/freeipa/widgets/LoginScreen.js b/install/ui/src/freeipa/widgets/LoginScreen.js index 2c778b50c..17f891e0e 100644 --- a/install/ui/src/freeipa/widgets/LoginScreen.js +++ b/install/ui/src/freeipa/widgets/LoginScreen.js @@ -262,7 +262,6 @@ define(['dojo/_base/declare', otp); if (result.status === 'ok') { - val_summary.add_success('login', this.password_change_complete); psw_f.set_value(''); psw_f2.set_value(''); // do not login if otp is used because it will fail (reuse of OTP) @@ -270,6 +269,7 @@ define(['dojo/_base/declare', psw_f.set_value(new_f.get_value()); this.login(); } + val_summary.add_success('login', this.password_change_complete); this.set('view', 'login'); } else { otp_f.set_value(''); |