In-Portal Issue Tracker

Welcome to the In-Portal Open Source CMS Issue Tracker! This is a central management / tracking tool for all types of tasks / issues / bugs for the In-Portal Project. Before reporting any issues, please make sure to read the Guide into Issue Tracker and How to Properly Test and Report Bugs!

Viewing Issue Simple Details [ Jump to Notes ] [ Wiki ]

[ View Advanced ] [ Issue History ] [ Print ]

ID

Category

Type

Reproducibility

Date Submitted

Last Update

0000785

[In-Portal CMS] Front End

bug report

always

2010-06-25 13:16

2011-03-25 08:23

Reporter

alex

View Status

public

Project Name

In-Portal CMS

Assigned To

alex

Developer

Priority

normal

Resolution

fixed

Fixed in Version

5.1.1-B1

Status

closed

Product Version

5.1.0-RC1

Target Version

5.1.1

Time Estimate

No estimate

Summary

0000785: User isn't automatically logged-in, when "Email Activation" user registration is used

Description

We have "Email Activation" registration mode. This mode allows to validate user's email address and only then make user account active.

This works as follows:

1. user fills-in registration form -> user record is created in "Pending" status and email is sent to user
2. user opens received email and uses activation link in it -> user sees "you've been activated" page on site

I propose we should auto-login user, after 2nd step.

I think, that we should auto-login user, because we do so, when user registration mode is set to "Active".

Additional Information

Tags

No tags attached.

Reference

http://groups.google.com/group/in-portal-bugs/browse_thread/thread/5624dd29042241e0

Change Log Message

Estimate Points

0

login_user_after_email_activation.patch [^] (973 bytes) 2010-09-08 09:26 [Show Content]

Index: units/users/users_tag_processor.php
===================================================================
--- units/users/users_tag_processor.php	(revision 13919)
+++ units/users/users_tag_processor.php	(working copy)
@@ -98,7 +98,7 @@
 
 		function TestCodeIsValid($params)
 		{
-		   	$passed_key = trim($this->Application->GetVar('user_key'));
+			$passed_key = trim($this->Application->GetVar('user_key'));
 
 	     	// used for error reporting only -> rewrite code + theme (by Alex)
 	     	$user_current_object =& $this->Application->recallObject('u', null, Array('skip_autoload' => true)); // TODO: change theme too
@@ -327,6 +327,11 @@
 			$user->SetDBField('PwResetConfirm', '');
 			$user->SetDBField('PwRequestTime', 0);
 			$user->Update();
+
+			$user_helper =& $this->Application->recallObject('UserHelper');
+	    	/* @var $user_helper UserHelper */
+
+	    	$user_helper->loginUserById( $user->GetID() );
 		}
 
 	}
\ No newline at end of file

Relationships [ Relation Graph ] [ Dependency Graph ]

child of

closed (5.1.1)

Ability to login user without knowing their password

Notes
(0002796) Dmitry (manager) 2010-09-08 23:32	Tested good, please commit.

(0002801) alex (manager) 2010-09-10 02:52	Fix committed to 5.1.x branch. Commit Message: Fixes 0000785: User isn't automatically logged-in, when "Email Activation" user registration is used

(0003041) Dmitry (manager) 2010-11-30 12:48	Closing released 5.1.1 task.

Related Changesets
In-Portal CMS: 5.1.x r13927 Timestamp: 2010-09-10 02:52:35 Author: alex [ Details ] [ Diff ]	Fixes 0000785: User isn't automatically logged-in, when "Email Activation" user registration is used
	mod - /in-portal/branches/5.1.x/core/units/users/users_tag_processor.php		[ Diff ] [ File ]

Issue History
Date Modified	Username	Field	Change
2011-03-25 08:23	alex	Relationship added	child of 0000830
2010-11-30 12:48	Dmitry	Note Added: 0003041
2010-11-30 12:48	Dmitry	Status	resolved => closed
2010-09-10 02:52	alex	Note Added: 0002801
2010-09-10 02:52	alex	Status	reviewed and tested => resolved
2010-09-10 02:52	alex	Fixed in Version	=> 5.1.1-B1
2010-09-10 02:52	alex	Resolution	open => fixed
2010-09-10 02:52	alex	Changeset attached	5.1.x r13927
2010-09-08 23:32	Dmitry	Note Added: 0002796
2010-09-08 23:32	Dmitry	Assigned To	!COMMUNITY => alex
2010-09-08 23:32	Dmitry	Status	needs testing => reviewed and tested
2010-09-08 09:26	alex	Time Estimate Removed	1 =>
2010-09-08 09:26	alex	Assigned To	alex => !COMMUNITY
2010-09-08 09:26	alex	Developer	=> alex
2010-09-08 09:26	alex	Status	needs work => needs testing
2010-09-08 09:26	alex	File Added: login_user_after_email_activation.patch
2010-08-27 13:16	alex	Time Estimate Added	1
2010-07-31 14:28	Dmitry	Assigned To	=> alex
2010-07-31 14:28	Dmitry	Status	active => needs work
2010-07-31 14:28	Dmitry	Target Version	Icebox => 5.1.1
2010-06-25 13:16	alex	New Issue
2010-06-25 13:16	alex	Reference	=> http://groups.google.com/group/in-portal-bugs/browse_thread/thread/5624dd29042241e0

Web Development by Intechnic

In-Portal Open Source CMS

In-Portal Open Source CMS

Copyright © 2000 - 2009 MantisBT Group