4

Tengo un esquema de usuarios por roles (medical, patient, physiotherapist) que deriva de la clase AbstractUser para utilizarlo en los request como se ve en los siguientes modelos:

#models.py

from __future__ import unicode_literals
from django.conf import settings
from django.contrib.auth.models import AbstractUser
from django.db import models
from django.contrib.auth import get_user_model
from django.dispatch import receiver
from django.db.models.signals import post_save

class User(AbstractUser):
    is_medical = models.BooleanField(default=False)
    is_physiotherapist = models.BooleanField(default=False)
    is_patient = models.BooleanField(default=False)
    slug = models.SlugField(max_length=100, blank=True)
    photo = models.ImageField(upload_to='avatars', null = True, blank = True)

    def get_medical_profile(self):
        medical_profile = None
        if hasattr(self, 'medicalprofile'):
            medical_profile=self.medicalprofile
        return medical_profile

    def get_patient_profile(self):
        patient_profile = None
        if hasattr(self, 'patientprofile'):
            patient_profile = self.patientprofile
        return patient_profile

    def get_physiotherapist_profile(self):
        physiotherapist_profile = None
        if hasattr(self, 'physiotherapistprofile'):
            physiotherapist_profile = self.physiotherapistprofile
        return physiotherapist_profile

    class Meta:
        db_table = 'auth_user'

class MedicalProfile(models.Model):
    user = models.OneToOneField(settings.AUTH_USER_MODEL, on_delete=models.CASCADE)
    name = models.CharField(max_length=64)

class PatientProfile(models.Model):
    user = models.OneToOneField(settings.AUTH_USER_MODEL, on_delete=models.CASCADE)
    name = models.CharField(max_length=64)

class PhysiotherapistProfile(models.Model):
    user = models.OneToOneField(settings.AUTH_USER_MODEL, on_delete=models.CASCADE)
    name = models.CharField(max_length=64)

Lo que deseo es que cuando se cree un nuevo usuario, de manera automática, se cree su perfil dependiendo de su campo si es médico, paciente, o fisioterapeuta (is_medical, is_patient, is_physiotherapist)

Para eso estoy utilizando el signal de post_save() de la siguiente manera en el mismo archivo models.py en donde estan mis modelos:

En el signal estoy enviándo tres parámetros mas el **kwargs:

  • sender, que es mi modelo de usuarios
  • created, un parámetro booleano que me indica que una instancia de mi modelo de usuario AUTH_USER_MODELha sido creada
  • instance, la instancia de usuario que se está creando

Para saber que un usuario esta a punto de crearse, de alguna manera debo indagar si en ese request que hago (crear el usuario), va ese usuario o esa instancia que esta por crearse, para lo cual hago user = self.request.user

    @receiver(post_save, sender=settings.AUTH_USER_MODEL)
    def create_profile_for_new_user(sender, created, instance, **kwargs):
        # Pregunto si en el request va el user, aqui va mi inquietud
        user = self.request.user
        #user = get_user_model()
        if created:
            if user.is_medical:
                profile=MedicalProfile(user=instance)
                profile.save()

Es una buena lógica, pero resulta que no he definido el self y obtengo (de manera obvia) este error

introducir la descripción de la imagen aquí

En el método en create_profile_for_new_user(...) al cual le estoy aplicando el signal de post_save() solo tengo cuatro atributos, por lo que si agrego self como atributo al inicio

@receiver(post_save, sender=settings.AUTH_USER_MODEL)
def create_profile_for_new_user(self,sender, created, instance, **kwargs):

obtendría este error:

TypeError: create_profile_for_new_user() missing 1 required positional argument: 'self'
127.0.0.1 - - [28/Dec/2015 22:57:04] "GET /admin/userprofile/user/add/?__debugger__=yes&cmd=resource&f=style.css HTTP/1.1" 200 -

Otra opción que estaba contemplando es obtener el usuario que se esta creando con la función get_user_model()como lo hago acá (importando previamente la función claro esta from django.contrib.auth import get_user_model):

@receiver(post_save, sender=settings.AUTH_USER_MODEL)
    def create_profile_for_new_user(sender, created, instance, **kwargs):
        # Pregunto si en el request va el user, aqui va mi inquietud
        #user = self.request.user
        user = get_user_model()
        if created:
            if user.is_medical:
                profile=MedicalProfile(user=instance)
                profile.save()

Pero cuando lo hago obtengo este mensaje, y es lógico, pues mi objeto User (estoy es obteniendo una instancia del modelo User original de Django) no tiene el atributo por el que pregunto, en este caso is_medical

introducir la descripción de la imagen aquí

Acorde a lo anterior, no se como preguntar o indagar por el usuario para examinar sus atributos booleanos (is_patient, is_medical, is_physiotherapist) y crear su perfil respectivo acorde a ello.

De paso aprovecho y me surge otra inquietud: ¿Es posible utilizar el signal de post_save para más de un propósito?

En este caso, quiero utilizarlo tanto para crear un perfil de la instancia del usuario que se crea y para darle valor a un campo llamado slug en base a su atributo first_name. Es decir ¿lo podría aplicar a los métodos que necesite?

2
  • Interesante el color que tiene tu traceback, que tema o paquete es?
    – SalahAdDin
    Commented el 13 mar. 2016 a las 21:13
  • @SalahAdDin Está usando runserver_plus del paquete django-extensions. Instala django-extensions y ejecuta ./manage.py runserver_plus
    – matuuar
    Commented el 23 mar. 2017 a las 15:10

3 respuestas 3

3

Recuerda que create_profile_for_new_user es una función relacionada al modelo, no es una vista, por lo tanto no tienes acceso al request. Tampoco es una clase, por lo que no tienes acceso al self como quisieras.

El parámetro created no va a funcionar como quisieras ya que lo estás usando en la posición del parámetro incorrecta, el primer parámetro de las funciones que reciben una señal es el modelo que envía la señal (sender) y el segundo parámetro es la instancia de ese modelo (instance).

El parámetro instance es el que contiene el usuario que ha sido creado por lo que lo siguiente es válido:

@receiver(post_save, sender=settings.AUTH_USER_MODEL)
def create_profile_for_new_user(sender, instance, **kwargs):
    user = instance # 
    # ...

Ni siquiera eso es necesario, podrías hacerlo sin la necesidad de asignar la variable user:

@receiver(post_save, sender=settings.AUTH_USER_MODEL)
def create_profile_for_new_user(sender, instance, **kwargs):
    if instance.is_medical:
        # ...
    elif instance.is_physiotherapist:
        # ...
    elif instance.is_patient
        # ...

Si necesitas asignar los demás campos de tu modelo, también puedes hacerlo aunque pensé que esto ya lo habíamos resuelto:

@receiver(post_save, sender=settings.AUTH_USER_MODEL)
def create_profile_for_new_user(sender, instance, **kwargs):
    slug = slugify(instance.first_name)
    User.objects.filter(pk=instance.pk).update(slug=slug)
    # ...

Nota:

Observa que en ninguno de los ejemplos estoy usando el parámetro created que habías definido inialmente como parámetro.

Aunque es posible usar el paŕametro created del post_save, ten en cuenta que este es el tercer parámetro, no el segundo:

@receiver(post_save, sender=settings.AUTH_USER_MODEL)
def create_profile_for_new_user(sender, instance, created, **kwargs):
    if created:
        # ...

Actualización:

El problema que tienes sobre el IntegrityError que mencionas en tus comentarios se debe a que cuando te logueas, el sistema está tratando de guardar tu última fecha de logueo y al tratar de hacer esto se llama nuevamente al post_save y finalmente a tu función create_profile_for_new_user.

La solución es que uses siempre el parámetro created:

@receiver(post_save, sender=settings.AUTH_USER_MODEL)
def create_profile_for_new_user(sender, instance, created, **kwargs):
    if created:
        # ...

Si no usas esto, la función tratará de guardar nuevamente el perfil y generará un IntegrityError ya que el perfil se creó anteriormente cuando el usuario fue creado inicialmente.

15
  • Cesar, si. Tienes razón. Aunque si utilizo el parametro "created", el slug del post_save no me graba el perfil del usuario.
    – bgarcial
    Commented el 29 dic. 2015 a las 15:24
  • De otro lado solo que con respecto a cuando voy a iniciar sesión con un usuario creado el cual automaticamente se le ha creado su perfil de medicaluser con el signal, me pasa lo siguiente: cldup.com/DPXcPqDQHY.png
    – bgarcial
    Commented el 29 dic. 2015 a las 15:29
  • 1
    @bgarcial lo que te dice el error no es que la vista estará deprecada, lo que estará deprecado es la forma en la que pasas el parámetro de la vista, al parecer en la version 1.10 de Django ya no podrás pasar la vista como un string: 'django.contrib.auth.views.login'
    – César
    Commented el 29 dic. 2015 a las 16:01
  • 1
    @bgarcial mira mi actualizacion, la solucion es usar siempre el parametro created. Me comentas
    – César
    Commented el 29 dic. 2015 a las 16:10
  • 1
    @bgarcial es un usuario nuevo? Intenta debuggear el codigo usando pdb, solo debería crear el perfil cuando el usuario es creado, para los siguientes casos ya no debería crear nada
    – César
    Commented el 29 dic. 2015 a las 18:45
2

En el archivo settings.py tiene que especificar la clase que hereda de AbstractUser en la propiedad AUTH_USER_MODEL Por lo que quedaría

AUTH_USER_MODEL = "app.User"

Con ello tomará la instancia de tu clase, no la clase User por default (auth.User)

2
  • Hola Kristian, si en efecto estoy usando un modelo personalizado de usuarios y en mi settings ya lo tengo especificado, es por ello que en el signal el parámetro sender esta así: sender=settings.AUTH_USER_MODEL. Pero lo que realmente debo mirar es como obtener el perfil del usuario que estoy creando para preguntar si van en True el atributo o los atributos de si es usuario tipo paciente, o usuario tipo medico o usuario tipo fisioterapeuta y asi crear su respectivo perfil con sus campos especificos, que todo esto lo haga el signal. :)
    – bgarcial
    Commented el 29 dic. 2015 a las 3:25
  • @Kristian quizás te interese ¿Cómo etiquetamos las preguntas de Django?
    – Mariano
    Commented el 23 feb. 2018 a las 10:45
2

Finalmente, opté por trabajar sobreescribiendo el método save() de la clase AbstractUser en lugar de trabajar aplicando el signal post_save() a la función create_profile_for_new_user, asi que mis modelos User MedicalProfile, PatientProfile y PhisiotherapystProfile quedaron así:

from __future__ import unicode_literals
from django.conf import settings
from django.contrib.auth.models import AbstractUser
from django.db import models
from django.template.defaultfilters import slugify
from django.dispatch import receiver
from django.db.models.signals import post_save

class User(AbstractUser):
    is_medical = models.BooleanField(default=False)
    is_physiotherapist = models.BooleanField(default=False)
    is_patient = models.BooleanField(default=False)
    slug = models.SlugField(max_length=100, blank=True)
    photo = models.ImageField(upload_to='avatars', null = True, blank = True)

    def save(self, *args, **kwargs):
        user = super(User, self).save( *args, **kwargs)

        # Creating and user with medical, patient and physiotherapist profiles
        if self.is_medical and not MedicalProfile.objects.filter(user=self).exists()\
                and self.is_patient and not PatientProfile.objects.filter(user=self).exists()\
                and self.is_physiotherapist and not PhysiotherapistProfile.objects.filter(user=self).exists():

            medical_profile=MedicalProfile(user=self).save()
            patient_profile=PatientProfile(user=self).save()
            physiotherapist_profile=PhysiotherapistProfile(user=self).save()
            #profile.save()

        # Creating and user with medical and patient profiles
        elif self.is_medical and not MedicalProfile.objects.filter(user=self).exists()\
            and self.is_patient and not PatientProfile.objects.filter(user=self).exists():

            medical_profile=MedicalProfile(user=self).save()
            patient_profile=PatientProfile(user=self).save()

        # Creating and user with medical and physiotherapist profiles
        elif self.is_medical and not MedicalProfile.objects.filter(user=self).exists()\
            and self.is_physiotherapist and not PhysiotherapistProfile.objects.filter(user=self).exists():

            medical_profile=MedicalProfile(user=self).save()
            physiotherapist_profile=PhysiotherapistProfile(user=self).save()

        # Creating and user with physiotherapist and patient profiles
        elif self.is_physiotherapist and not PhysiotherapistProfile.objects.filter(user=self).exists()\
            and self.is_patient and not PatientProfile.objects.filter(user=self).exists():

            physiotherapist_profile = PhysiotherapistProfile(user=self).save()
            patient_profile = PatientProfile(user=self).save()

        # Creating and user with medical profile
        elif self.is_medical and not MedicalProfile.objects.filter(user=self).exists():
            profile = MedicalProfile(user=self)
            profile.save()

        # Creating and user with patient profile
        elif self.is_patient and not PatientProfile.objects.filter(user=self).exists():
            profile = PatientProfile(user=self)
            profile.save()

        # Creating and user with physiotherapist profiles
        elif self.is_physiotherapist and not PhysiotherapistProfile.objects.filter(user=self).exists():
            profile = PhysiotherapistProfile(user=self)
            profile.save()



    # We get the profiles user according with their type
    def get_medical_profile(self):
        medical_profile = None
        if hasattr(self, 'medicalprofile'):
            medical_profile=self.medicalprofile
        return medical_profile

    def get_patient_profile(self):
        patient_profile = None
        if hasattr(self, 'patientprofile'):
            patient_profile = self.patientprofile
        return patient_profile

    def get_physiotherapist_profile(self):
        physiotherapist_profile = None
        if hasattr(self, 'physiotherapistprofile'):
            physiotherapist_profile = self.physiotherapistprofile
        return physiotherapist_profile

    # We redefine the attributes (create db_table attribute) in class Meta to say to Django
    # that users will save in the same table that the Django default user model
    # https://github.com/django/django/blob/master/django/contrib/auth/models.py#L343
    class Meta:

        db_table = 'auth_user'

class MedicalProfile(models.Model):
    user = models.OneToOneField(settings.AUTH_USER_MODEL, on_delete=models.CASCADE)
    #active = models.BooleanField(default=True)
    name = models.CharField(max_length=64)


class PatientProfile(models.Model):
    user = models.OneToOneField(settings.AUTH_USER_MODEL, on_delete=models.CASCADE)
    #active = models.BooleanField(default=True)
    name = models.CharField(max_length=64)


class PhysiotherapistProfile(models.Model):
    user = models.OneToOneField(settings.AUTH_USER_MODEL, on_delete=models.CASCADE)
    #active = models.BooleanField(default=True)
    name = models.CharField(max_length=64)

# Enter the username as slug field
@receiver(post_save, sender = settings.AUTH_USER_MODEL)
def post_save_user(sender, instance, **kwargs):
    slug = slugify(instance.username)
    User.objects.filter(pk=instance.pk).update(slug=slug)

De esta manera, se puede crear un usuario que tenga todas las combinaciones de roles (medico, paciente, fisioterapeuta) posibles.

De todas maneras se sigue utilizando un signal post_save() aplicado en la función post_save_user() para ingresar un valor de slug a mi campo slug en el modelo User. Este valor es basado en el atributo username

Muchas gracias por sus constantes orientaciones. :)

1
  • Normalmente es muy desaconcejable editar o modificar el usuario Django, siempre recomiendan hacer un perfil añadido con un enlace Uno a Uno para evitar problemas al modificar el modelo User. Muy buena respuesta, no olvides marcar la respuesta aceptada.
    – SalahAdDin
    Commented el 13 mar. 2016 a las 21:15

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.