Question

I’m trying to extend Django registration to include my own registration form. In principle this is fairly simple. I just have to write my own form (CustomRegistrationForm) which is a child of the original one (RegistrationForm). Then I can process my specific input by using the user_registered signal of django registration.

So here is what I did:

urls.py:

from django.conf.urls import patterns, include, url
from registration.views import register
from forms import CustomRegistrationForm
from django.contrib import admin
import regbackend
admin.autodiscover()

urlpatterns = patterns('',
    url(r'^register/$', register, {'backend': 'registration.backends.default.DefaultBackend', 'form_class': CustomRegistrationForm, 'template_name': 'custom_profile/registration_form.html'}, 
                        name='registration_register'),
)

regbackend.py:

from django import forms
from models import UserProfile
from forms import CustomRegistrationForm

def user_created(sender, user, request, **kwargs):
    form = CustomRegistrationForm(data=request.POST, files=request.FILES)
    if form.is_valid(): # HERE: always fails
        user_profile = UserProfile()
        user_profile.user = user
        user_profile.matriculation_number = form.cleaned_data['matriculation_number']
        user_profile.save()

from registration.signals import user_registered
user_registered.connect(user_created)

forms.py:

from models import UserProfile
from django import forms
from registration.forms import RegistrationForm
from django.utils.translation import ugettext_lazy as _

attrs_dict = {'class': 'required'}

class CustomRegistrationForm(RegistrationForm):
    matriculation_number = forms.CharField(widget=forms.TextInput(attrs=attrs_dict),
                                           label=_("Matriculation number"),
                                           max_length=12,
                                           error_messages={'invalid': _("This value has to be unique and may contain only numbers.")},
                                           initial=108)

    def clean_matriculation_number(self):
        print "Validating matriculation number."
        data = self.cleaned_data['matriculation_number']
        if len(data) != 12:
            raise forms.ValidationError(_("Matriculation number consists of exactly 12 digits."))
        return data

So the problem is the is_valid() function, because it always returns False. Even if there are no errors! So what is wrong? I spent hours on this one and I have no idea anymore 🙁

Any help is much appreciated!!

Answer 1

Ok I think I solved it (more or less).

I’m still not really sure, why the form did not validate. But as I said I was extending django-registration and the ‘register’ view already called is_valid() of the form, so I can assume that the form is valid when I process the posted data any futher. The view then calls the backend

backend.register(request, **form.cleaned_data)

with the request and the cleaned data (which is just username, email and password). So I can’t use it for registration because my additional information is missing. The backend then fires the signal that I am using and what I did is, is that I created the form again with the provided request. This form, however, will NOT validate (and I tried everything!!) I looked it up, I am doing the exact same thing as django-registration, but it’s not working in my code.

So I did not really solve the problem, because the form is still not validating. But I found peace with this, when I realized that the form was already validated by the ‘register’ view. So I am using form.data[..] instead of form.cleaned_data[..] now which shouldn’t be a problem…