Hi All, Please review patches attached.
Thanks, Abhijeet Kasurde
From c7f0621a561b4bdc558cafaa2e8ee182b571ebd8 Mon Sep 17 00:00:00 2001 From: Abhijeet Kasurde <[email protected]> Date: Mon, 28 Dec 2015 12:26:13 +0530 Subject: [PATCH] Fixed login error message box in LoginScreen page Fix added for showing error message returned from server to client browser. User is now notified with proper error messages returned by server. https://bugzilla.redhat.com/show_bug.cgi?id=1293870 Signed-off-by: Abhijeet Kasurde <[email protected]> --- install/ui/src/freeipa/widgets/LoginScreen.js | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/install/ui/src/freeipa/widgets/LoginScreen.js b/install/ui/src/freeipa/widgets/LoginScreen.js index eb95b9161f05eeac1ec9aed286c9730dada85d59..2dbd883c942acb2f6b61f5c4883bc977233d7495 100644 --- a/install/ui/src/freeipa/widgets/LoginScreen.js +++ b/install/ui/src/freeipa/widgets/LoginScreen.js @@ -272,12 +272,11 @@ define(['dojo/_base/declare', } this.set('view', 'login'); } else { + otp_f.set_value(''); + new_f.set_value(''); + ver_f.set_value(''); val_summary.add_error('login', result.message); } - - otp_f.set_value(''); - new_f.set_value(''); - ver_f.set_value(''); }, refresh: function() { @@ -426,4 +425,4 @@ define(['dojo/_base/declare', ]; return LoginScreen; -}); \ No newline at end of file +}); -- 2.4.3
From 3a0230d1446cbc09e38903decd9696d017e14913 Mon Sep 17 00:00:00 2001 From: Abhijeet Kasurde <[email protected]> Date: Mon, 28 Dec 2015 12:33:11 +0530 Subject: [PATCH] Fixed login error message box in LoginScreen page Fix added for showing error message returned from server to client browser. User is now notified with proper error messages returned by server. https://bugzilla.redhat.com/show_bug.cgi?id=1293870 Signed-off-by: Abhijeet Kasurde <[email protected]> --- install/ui/src/freeipa/widgets/LoginScreen.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/install/ui/src/freeipa/widgets/LoginScreen.js b/install/ui/src/freeipa/widgets/LoginScreen.js index eb95b9161f05eeac1ec9aed286c9730dada85d59..2c778b50cfb10bfa8eef25c5456c6ce913e02695 100644 --- a/install/ui/src/freeipa/widgets/LoginScreen.js +++ b/install/ui/src/freeipa/widgets/LoginScreen.js @@ -272,12 +272,12 @@ define(['dojo/_base/declare', } this.set('view', 'login'); } else { + otp_f.set_value(''); + new_f.set_value(''); + ver_f.set_value(''); val_summary.add_error('login', result.message); } - otp_f.set_value(''); - new_f.set_value(''); - ver_f.set_value(''); }, refresh: function() { @@ -426,4 +426,4 @@ define(['dojo/_base/declare', ]; return LoginScreen; -}); \ No newline at end of file +}); -- 2.4.3
-- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
