Fix bug that does not keep login

naver · Oct 15, 2014 · 004bc9b · 004bc9b
1 parent b8c0bf9
commit 004bc9b
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/app/controllers/UserApp.java b/app/controllers/UserApp.java
@@ -180,7 +180,7 @@ private static Result loginByFormRequest() {
         if (!authenticate.isAnonymous()) {
             addUserInfoToSession(authenticate);
 
-            if (sourceUser.rememberMe) {
+            if (authInfoForm.get().rememberMe) {
                 setupRememberMe(authenticate);
             }
 
@@ -236,7 +236,7 @@ private static Result loginByAjaxRequest() {
         if (!authenticate.isAnonymous()) {
             addUserInfoToSession(authenticate);
 
-            if (sourceUser.rememberMe) {
+            if (authInfoForm.get().rememberMe) {
                 setupRememberMe(authenticate);
             }
 

diff --git a/app/models/AuthInfo.java b/app/models/AuthInfo.java
@@ -28,4 +28,11 @@ public class AuthInfo {
     public String loginIdOrEmail;
     @Constraints.Required
     public String password;
+
+    /**
+     * used for enabling remember me feature
+     *
+     * remember me = keep a use logged in
+     */
+    public boolean rememberMe;
 }
diff --git a/public/javascripts/common/yobi.LoginDialog.js b/public/javascripts/common/yobi.LoginDialog.js
@@ -38,6 +38,7 @@ $(function(){
         htElement.welForm = htElement.welDialog.find("form");
         htElement.welInputId = htElement.welDialog.find("input[name='loginIdOrEmail']");
         htElement.welInputPw = htElement.welDialog.find("input[name='password']");
+        htElement.welInputRememberMe = htElement.welDialog.find("input[name='rememberMe']");
         htElement.welLoginError = htElement.welDialog.find(".error");
         htElement.welLoginErrorMsg = htElement.welLoginError.find(".error-message");
     }
@@ -74,7 +75,8 @@ $(function(){
             "dataType": "json",
             "data": {
                 "loginIdOrEmail" : htElement.welInputId.val(),
-                "password": htElement.welInputPw.val()
+                "password": htElement.welInputPw.val(),
+                "rememberMe": htElement.welInputRememberMe.is(":checked")
             }
         }).done(function(){
             document.location.reload();