Campos opcionales con UserChangeForm

[kojicomics]

Hola a todos

Tengo un modelo de usuarios extendido mediante 'AbstractUser':

models.py

Código Python:

from django.db import models
from django.contrib.auth.models import AbstractUser
 
 
class Profiles(AbstractUser):
    url = models.URLField()
    avatar = models.ImageField(upload_to='profile_avatar')
 

Tengo también un formulario para modificar los usuarios basado en 'UserChangeForm':

forms.py

Código Python:

from django import forms
from django.contrib.auth.forms import UserChangeForm
from django.utils.translation import ugettext as _
from .models import Profiles
 
 
class UserEditForm(UserChangeForm):
 
    class Meta:
        model = Profiles
        fields = ('username', 'first_name', 'last_name', 'email', 'url',
                  'avatar',)
        exclude = ('password',)
 
    def clean_username(self):
        username = self.cleaned_data['username']
        try:
            Profiles.objects.get(username=username)
        except Profiles.DoesNotExist:
            return username
        raise forms.ValidationError(_('Duplicate username'))
 
    def clean_avatar(self):
        avatar = self.cleaned_data['avatar']
        if avatar:
            if avatar._size > 1024 * 1024:
                raise forms.ValidationError(_('Image file too large ( > 1mb )'))
            return avatar
        else:
            raise forms.ValidationError(_("Couldn't read uploaded image"))
 
    def clean_password(self):
        return ""
 

El formulario lo ejecuto mediante esta vista:

views.py

Código Python:

from django.views.generic import UpdateView
from .models import Profiles
from .forms import UserEditForm
 
 
class UserEdit(UpdateView):
    model = Profiles
    form_class = UserEditForm
    success_url = reverse_lazy('profile')
    template_name = 'profile/edit.html'
 
    def get_object(self, queryset=None):
        return self.request.user
 

El formulario de modificación funciona correctamente en la aplicación, pero tengo dos problemas:

El primero es que el formulario requiere que el usuario inserte obligatoriamente los campos adicionales (los de mi modelo extendido) para poder enviar el formulario.

Lo he solucionado redefiniendo los campos dentro del formulario junto con el atributo "required=False" pero me gustaría saber si hay algún método de aplicar ésto a todos los campos sin tener que modificarlos uno a uno.

El segundo problema, y el más importante, es que los campos que el usuario no rellene en el formulario, en vez de ignorarlos, elimina su valor anterior.

Una posible solución es generar el valor de las etiquetas dinámicamente mediante:

Código:

value={{user.fieldname}}

Pero en caso de eliminar el valor de un campo volveríamos al mismo problema.

Un saludo, Gracias

[razpeitia]

En tu modelo pones en los campos que quieres que sean opcionales blank=True.

No ocupas checar que el username sea único, en el modelo base, username es un indice único y ademas ese tipo de validaciones se hace a nivel instancia, no a nivel campo.

[kojicomics]

Iniciado por razpeitia

En tu modelo pones en los campos que quieres que sean opcionales blank=True.

El problema de especificar si son opcionales o no en el modelos, es que por ejemplo me interesa que el email sea obligatorio en el formulario de registro pero opcional en el formulario de modificación.

Iniciado por razpeitia

No ocupas checar que el username sea único, en el modelo base, username es un indice único y ademas ese tipo de validaciones se hace a nivel instancia, no a nivel campo.

Probé a validar los datos a nivel de instancia pero no muestra el mensaje de error en el template, mientras que al validar desde el formulario sí lo hace.

Éste es un ejemplo:

Código Python:

class Profiles(AbstractUser):
    avatar = models.ImageField(upload_to='profile_avatar', blank=True)
 
    def clean(self):
        avatar = self.avatar
        if avatar.file.size > 1024*1024:
            raise ValidationError(_('Image file too large ( > 1mb )'))
 

También probé con forms.ValidationError pero tampoco muestra nada

De todas formas, como dije en el post, mi principal duda es cómo evitar que si un usuario deja en blanco un campo al modificar su perfil, éste se actualice con el valor vacío, en vez de conservar el anterior

Un saludo

[razpeitia]

Iniciado por kojicomics

El problema de especificar si son opcionales o no en el modelos, es que por ejemplo me interesa que el email sea obligatorio en el formulario de registro pero opcional en el formulario de modificación.

Eso no tiene sentido. Imagina que me registro con un email y luego al actualizarlo lo quito. No habría diferencia hay si lo tuvieras opcional desde el inicio.

Iniciado por kojicomics

Probé a validar los datos a nivel de instancia pero no muestra el mensaje de error en el template, mientras que al validar desde el formulario sí lo hace.

Que no muestre el error es porque simplemente no lo muestras, no se como estés renderizando el formulario. Pero estoy seguro que no estas mostrando los forms.non_field_errors.

[kojicomics]

Al final he conseguido solucionar el problema de los valores en blanco. Compruebo el tamaño del valor, de forma que si es igual a 0, utiliza el valor anterior

forms.py

Código Python:

class UserEditForm(UserChangeForm):
 
    class Meta:
        model = Profiles
        fields = ('username', 'first_name', 'last_name', 'email', 'url',
                  'avatar',)
        exclude = ('password',)
 
    def __init__(self, user, *args, **kwargs):
        super(UserEditForm, self).__init__(*args, **kwargs)
        self.user = user
 
    def clean(self):
        cleaned_data = self.cleaned_data
        for field, value in list(cleaned_data.items()):
            old_value = getattr(self.user, field)
            if len(value) > 0:
                cleaned_data[field] = value
            else:
                cleaned_data[field] = old_value
 
        return cleaned_data
 
    def clean_password(self):
        return ""
 

views.py

Código Python:

class UserEdit(UpdateView):
    model = Profiles
    form_class = UserEditForm
    success_url = reverse_lazy('profile')
    template_name = 'profile/edit.html'
 
    def get_form_kwargs(self):
        kwargs = super(UserEdit, self).get_form_kwargs()
        kwargs['user'] = self.request.user
        return kwargs
 
    def get_object(self, queryset=None):
        return self.request.user