Hi All,

Please review patches attached.

Thanks,
Abhijeet Kasurde
From c7f0621a561b4bdc558cafaa2e8ee182b571ebd8 Mon Sep 17 00:00:00 2001
From: Abhijeet Kasurde <akasu...@redhat.com>
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 <akasu...@redhat.com>
---
 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 <akasu...@redhat.com>
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 <akasu...@redhat.com>
---
 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

Reply via email to