Autologin cookies on new user registration

[Migrated From Jforum.net]

--- src/net/jforum/view/forum/UserAction.java	(revision 2451)
+++ src/net/jforum/view/forum/UserAction.java	(working copy)
@@ -251,7 +251,12 @@
 					+ SystemGlobals.getValue(ConfigKeys.SERVLET_EXTENSION));
 		}
 		else {
-			this.logNewRegisteredUserIn(newUserId, u);
+			if (this.request.getParameter("autologin") != null
+					&& SystemGlobals.getBoolValue(ConfigKeys.AUTO_LOGIN_ENABLED)) {
+				this.logNewRegisteredUserIn(newUserId, u, true);
+			}else{
+				this.logNewRegisteredUserIn(newUserId, u, false);
+			}
 		}
 	}
 
@@ -269,7 +274,7 @@
 		if (isOk) {
 			// make account active
 			um.writeUserActive(userId);
-			this.logNewRegisteredUserIn(userId, u);
+			this.logNewRegisteredUserIn(userId, u, false);
 		} 
 		else {
 			message = I18n.getMessage("User.invalidActivationKey");
@@ -279,7 +284,7 @@
 
 	}
 
-	private void logNewRegisteredUserIn(int userId, User u) 
+	private void logNewRegisteredUserIn(int userId, User u, boolean shouldSetCookies) 
 	{
 		SessionFacade.setAttribute("logged", "1");
 
@@ -289,6 +294,14 @@
 		userSession.setUsername(u.getUsername());
 		userSession.setLastVisit(new Date(System.currentTimeMillis()));
 		userSession.setStartTime(new Date(System.currentTimeMillis()));
+		
+		if(shouldSetCookies){
+			ControllerUtils.addCookie(SystemGlobals.getValue(ConfigKeys.COOKIE_AUTO_LOGIN), "1");
+			ControllerUtils.addCookie(SystemGlobals.getValue(ConfigKeys.COOKIE_USER_HASH), 
+					MD5.crypt(SystemGlobals.getValue(ConfigKeys.USER_HASH_SEQUENCE) + userId));
+			ControllerUtils.addCookie(SystemGlobals.getValue(ConfigKeys.COOKIE_NAME_DATA), 
+					Integer.toString(userId));
+		}
 
 		SessionFacade.add(userSession);
 

To enable, either add a checkbox or a hidden field autologin to the form on templates/default/user_new.htm.

[originally posted on jforum.net by bcc]

[Migrated From Jforum.net]

Could you please add this to jira? We have frozen release 2.1.7 by now, so only bug fixes are being commited.

Thanks.

Rafael
[originally posted on jforum.net by Rafael Steil]

[Migrated From Jforum.net]

Is there a separate id/password there from the forum?
[originally posted on jforum.net by bcc]

[Migrated From Jforum.net]

bcc wrote:Is there a separate id/password there from the forum?

yes, you need to register
[originally posted on jforum.net by lazee]