John McNally wrote:
> I would like the default to remain that the session is cleared. I think it
> should be possible to move the code that is in Turbine into the LoginUser
> action. That way it can be easily changed to fit a particular circumstance.
> Someone tell me the problem with the following approach.
I think it would be OK. The suggestion was, aside to moving the code to LoingUser,
to make clearing session variables EXCEPT user and acl optional, beacuse this was
what Hervé was asking for.
> Move the code from Turbine that clears the session into the beginning of the
> LoginUser action. This code can be removed from LoginUser:
>
> if ( data.getUserFromSession() != null )
> {
> return;
> }
>
> Its function can be duplicated by adding the following to the end of
> LoginUser:
>
> data.setAction(null);
+1, but remeber to add a comment there.
> I do not like LoginUser relying on behavior of LogoutUser.
Right. It shouldn't depend on that. I meant that as long as these two
are used together, the situation with continuing other users session
won't happen.
Rafal
------------------------------------------------------------
To subscribe: [EMAIL PROTECTED]
To unsubscribe: [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?: [EMAIL PROTECTED]