• Do not register here on develop.twiki.org, login with your twiki.org account.
• Use View topic Item7848 for generic doc work for TWiki-6.1.1. Use View topic Item7851 for doc work on extensions that are not part of a release. More... Close
• Anything you create or change in standard webs (Main, TWiki, Sandbox etc) will be automatically reverted on every SVN update.
Does this site look broken?. Use the LitterTray web for test cases.

In Cairo there were two registration topics.

One called TWikiRegistration and one called TWikiRegistrationPub.

The Pub had no login name field and it asked for a password which TWiki stored in a password file.

The non-Pub had an extra login name field and it did not ask for password.

The idea was that if you had an Internet public site you would remove the TWikiRegistration topic and change the name of TWikiRegistrationPub to TWikiRegistration.

This was confusing. Dakar does this better by having only one TWikiRegistration topic with an IF that only shows the username when the configure parameter $cfg{Register}{AllowLoginName} is set.

Unfortunately the password fields are not conditionally shown.

In reality these fields should not be included when cfg{PasswordManager} is set to None. But this is not an available IF condition.

However - it is hard to imagine an installation where you have different login names and WikiNames and still let TWiki store a password in a .htpasswd file when you register. I would say that it would be very safe to assume that when you do have a login name TWiki should not prompt for a password to be set. NOTE that the password fields in the registration form are used to SET the password. Not to authenticate the registration.

Making TWiki.TWikiRegistration with an additional IF to add password field when $cfg{Register}{AllowLoginName} is not set will make in my view a well working solution - which gives the same function as we had with the two topics in Cairo. Some may later implement the code to make the IF linked to {PasswordManager} but as long as we only have the HtPasswdUser/!ApacheHtPasswdUser shipping with TWiki I see little reason for hurrying this now.

I have attached a diff that implements the conditional password fields and I have taken care not to alter any MAKETEXT.

KJL

Checked in your patch in SVN 8525

CC

ItemTemplate
Summary TWikiRegistration should not prompt for password when not used (with fix)
ReportedBy TWiki:Main.KennethLavrsen
Codebase

SVN Range Wed, 25 Jan 2006 build 8512
AppliesTo Engine
Component

Priority Urgent
CurrentState Closed
WaitingFor

Checkins 8525
Topic attachments
I Attachment History Action Size Date Who Comment
Unknown file formatdiff item1510.diff r1 manage 1.5 K 2006-01-26 - 02:17 KennethLavrsen Diff for TWikiRegistration password fields fix
Edit | Attach | Watch | Print version | History: r4 < r3 < r2 < r1 | Backlinks | Raw View | Raw edit | More topic actions
Topic revision: r4 - 2006-01-26 - CrawfordCurrie
 
This site is powered by the TWiki collaboration platform Powered by PerlCopyright © 2008-2018 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
Ideas, requests, problems regarding TWiki? Send feedback