Chris,
I think its fixed. Could you please cvs checkout and test it again.
I tested with both velocity and jsp.
We could really use some test cases for this feature...
Thanks for all your testing and debugging on this.
Have you ever used Cactus?...
-------------------------------------
David Sean Taylor
[EMAIL PROTECTED]
-------------------------------------
http://jakarta.apache.org/jetspeed
-------------------------------------
> -----Original Message-----
> From: Chris Kimpton [mailto:[EMAIL PROTECTED]]
> Sent: Tuesday, July 03, 2001 6:32 AM
> To: [EMAIL PROTECTED]
> Subject: Confirming a user is broken - suggested fix
>
>
> Hi,
>
> I am having problems with the JLoginUser action, it does not confirm
> new users.
>
> The problem seems to be due to the bit where, its worked out that we
> have a secret key and thus are trying to confirm a user, it then does
> a data.getUser() -but nothing has put it into rundata prior to this -
> so this always returns null and so hits the return statement... this
> should probably give an error.
>
> So, to fix this, I would change user = data.getUser() to
>
> user = JetspeedSecurty.getUser(username) - where username is in the
> parameters.
>
> A further problem now comes when it tries to do the Turbine LoginUser
> action. This expects to find a "password" parameter in the request
> too - but this is not in the confirmation registration form and thus
> sends the user to the login form...
>
> Is this to be expected - my initial thoughts would be to log the user
> in... but thinking further, perhaps a message saying you have been
> confirmed, please login to continue would be ok.
>
> Thoughts/comments?
>
> Chris
>
> =====
> Need somewhere to Live in London - http://freeflats.com
>
> __________________________________________________
> Do You Yahoo!?
> Get personalized email addresses from Yahoo! Mail
> http://personal.mail.yahoo.com/
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
