I just tried the modification above, it works fine. I don't know cvs yet but the changes are minor but useful I think.
If anyone can apply it to jetspeed... ? Just a change of one line, to be sure to do the login when creating new user. Thx, Aurelien > -----Message d'origine----- > De : Aur�lien Pernoud [mailto:apernoud@;sopragroup.com] > Envoy� : mardi 15 octobre 2002 15:40 > � : 'Jetspeed Developers List' > Objet : Creating new user skips the Login class => bug, enhancement ? > > > > Hi there, > > I've extended the JLoginUser for my needs, but I'm stuck with > a little issue > : > When I'm creating a new user (without confirming by email), I > directly go to > the portal with defaults portlets, but it doesn't go through the Login > part... So some of my initializing is missing, so portlets > have errors. I > have to logout first, and then log in to have portlets get > their content. > > I see I can modify the CreateNewUserAndConfirm action to > perform the login > after, but there's no line allowing to change the default action for > creation in js.pr (did I miss it ???). > > Shouldn't the fonctionnement be : > "Create New User" => if no confirm wanted => Login, cause now > as I saw in > the source it only does : > (line 207) > if (!enableMail && !newUserApproval) // > (That's my case, and > it's the default case) > { > user.setHasLoggedIn(new Boolean (true)); > user.setLastLogin(new Date(0)); > } > > And this skips the default login that may have been changed. > > I'm not quite sure, but I think we could change this with : > > if (!enableMail && !newUserApproval) > { > ActionLoader.getInstance().exec(data, > JetspeedResources.getString("action.login")); > } > > I can't test cause I don't want to change Jetspeed src for my > needs, but it > must work. > > Thanks for considering request, > > Aur�lien Pernoud > -- To unsubscribe, e-mail: <mailto:jetspeed-dev-unsubscribe@;jakarta.apache.org> For additional commands, e-mail: <mailto:jetspeed-dev-help@;jakarta.apache.org>
