[SOLUCIONADO] Extender modelo de usuarios

[kojicomics]

Necesito añadir unos campos extra para los usuarios que se registren en mi aplicación, por lo que he creado un modelo 'Profiles' relacionado con su tupla en la tabla User:

Código Python:

from django.contrib.auth.models import User
 
 
class Profiles(models.Model):
    user = models.OneToOneField(User)
    url = models.URLField()
    avatar = models.ImageField(upload_to='profile_avatar')
 
    def __str__(self):
        return self.user.username
 

Después he creado un formulario que pida el email, url y avatar en un fichero 'forms.py':

Código Python:

from django import forms
from django.contrib.auth.forms import UserCreationForm
 
 
class UserForm(UserCreationForm):
    email = forms.EmailField()
    url = forms.URLField()
    avatar = forms.ImageField()
 

Por último defino la clase SignIn que toma los valores del formulario y crea un usuario relacionado con el modelo Profiles:

Código Python:

from django.views.generic import FormView
from .forms import UserForm
from django.core.urlresolvers import reverse_lazy
from .models import Profiles
from django.contrib.auth.models import User
 
 
class SignIn(FormView):
    template_name = 'index/signin.html'
    form_class = UserForm
    success_url = reverse_lazy('signin')
 
    def form_valid(self, form):
        user = form.save()
        u = User.objects.get(username=str(user))
        u.email = form.cleaned_data['email']
        profile = Profiles()
        profile.user = user
        profile.url = form.cleaned_data['url']
        profile.avatar = form.cleaned_data['avatar']
        profile.save()
        return super(SignIn, self).form_valid(form)
 

Al ingresar los datos en el formulario sale el siguiente error: (1062, "Duplicate entry '' for key 'mail'")

Si elimino estas líneas de la vista, todo se crea correctamente pero no se asigna el campo email:
u = User.objects.get(username=str(user))
u.email = form.cleaned_data['email']

¿Cómo puedo ingresar el valor de "email" en el modelo User?

Gracias

[razpeitia]

Es porque no entiendes muchas cosas de Python + Django.

Vamos a examinar linea por linea.

Código Python:

user = form.save()
 

Ok, guardas el usuario. Pero esto tiene consecuencias. Al momento de salvarlo el usuario ya esta en la base de datos

Código Python:

u = User.objects.get(username=str(user))
 

WTF, acabas de guardar el usuario y ya tienes una referencia en la variable user. ¿Porque lo vuelves a obtener de la base de datos?

Código Python:

u.email = form.cleaned_data['email']
 

Estoy muy seguro que esto ya lo hace el form.

Código Python:

profile = Profiles()
profile.user = user
profile.url = form.cleaned_data['url']
profile.avatar = form.cleaned_data['avatar']
profile.save()
 

Esta es la parte buena.

Código Python:

return super(SignIn, self).form_valid(form)
 

Recuerdas el form.save() de alla arriba? Y recuerdas que la parte del email te dije que ya lo hacia el form? Pues sorpresa! cuando ejecutas form_valid. Trata de crear otro usuario, cosa que ya te adelantaste a hacer. Y cual es el resultado? Que ese email ya existe en la db. Y por eso te marca el error de email duplicado.

¿Entonces que debo de hacer?
Lo que debiste de hacer fue: hacer un override del metodo save del formulario. Usar el nuevo usuario que te genera para crear ahora si una instancia de Profile y guardarlo.

Aquí hay un ejemplo: http://stackoverflow.com/questions/5...tionform-basic

[kojicomics]

Muchas gracias

Después de eliminar las redundancias y crear el override, todo funciona correctamente

Nunca pensé que mi primera aplicación me fuese a aportar tanto... , cuando la termine tendré que hacer una maratón de lectura de libros sobre python, jajaja

Saludos!