Add support for disabling user registration.

Review Request #760 — Created March 13, 2009 and submitted

Information

Review Board SVN (deprecated)
trunk

Reviewers

Add support for disabling user registration.

This provides a checkbox in the General Settings page for disabling user
registration. This only takes effect when using standard authentication,
and will hide the "Register" link and redirect requests made to
/account/register/ to the /account/login/ page instead.
Tested the /account/register/ path and the existence of the link with the following combinations:

* Registration enabled and standard auth
* Registration disabled and standard auth
* Registration enabled and LDAP
* Registration disabled and LDAP

All cases work. Note that LDAP is a good enough test for non-standard auth types, since the only auth-specific checks are against "builtin".
david
  1. 
      
  2. /trunk/reviewboard/admin/forms.py (Diff revision 2)
     
     
     
     
     
     
     
     
     
     
     
     
     
    If this only affects builtin auth, shouldn't this be part of an auth_builtin fieldset? That way it would appear/disappear based on what authentication backend you select.
    1. Originally I had made it part of the built-in auth only, but there's no reason it should be specific to built-in auth. For example, OpenID may need a register page. I don't want to limit it and make it specific to built-in auth for this reason.
    2. I suppose. I'm tempted to say let's make it builtin only for now and we can generalize later when multiple backends want to do it.
    3. Alright, we can do that. I suppose as long as we keep the settings name generic, it's fine.
  3. And you could get rid of this special case.
  4. 
      
chipx86
Review request changed
Change Summary:
Move "Enable registration" into "Basic Authentication Settings"
Diff:

Revision 3 (+24 -6)

Show changes

/trunk/reviewboard/accounts/views.py
/trunk/reviewboard/admin/forms.py
/trunk/reviewboard/admin/siteconfig.py
/trunk/reviewboard/templates/base.html
david
  1. Looks good!
  2. 
      
Loading...