comment réparer l'instance de modèle django

Sep 06 2020

messages d'erreur ValueError: Impossible d'attribuer "<User: [email protected]>": "Diagnoses.owner" doit être une instance "Patient"

J'obtiens l'erreur ci-dessus lorsque j'essaye de créer une nouvelle carte ou des diagnostics. La carte et les diagnostics sont censés être une instance du patient, tout comme la façon dont un patient a une carte et des diagnostics. La classe de patient a une clé prédéfinie référençant l'utilisateur. Voici le code

views.py

    from django.shortcuts import render
from rest_framework.generics import ListCreateAPIView, RetrieveUpdateDestroyAPIView
from .serializers import PatientsSerializer, PatientsCardSerializer, PatientsDiagnosesSerializer
from .models import Patient, Card, Diagnoses
from rest_framework import permissions
from .permissions import IsOwner


# Patient Views
class PatientListAPIView(ListCreateAPIView):
    serializer_class = PatientsSerializer
    queryset = Patient.objects.all()
    permission_classes = (permissions.IsAuthenticated, IsOwner,) 
    def perform_create(self, serializer):
        return serializer.save(owner=self.request.user)
    def get_queryset(self):
        return self.queryset.filter(owner=self.request.user)

class PatientDetailAPIView(RetrieveUpdateDestroyAPIView):
    serializer_class = PatientsSerializer
    permission_classes = (permissions.IsAuthenticated, IsOwner,)
    queryset = Patient.objects.all()
    lookup_field = "id"
    def get_queryset(self):
        return self.queryset.filter(owner=self.request.user)


class PatientCardListAPIView(ListCreateAPIView):
    serializer_class = PatientsCardSerializer
    queryset = Card.objects.all()
    permission_classes = (permissions.IsAuthenticated,) 
    def perform_create(self, serializer):
        return serializer.save(owner=self.request.user)
    def get_queryset(self):
        return self.queryset.filter(owner=self.request.user)

class PatientCardDetailAPIView(RetrieveUpdateDestroyAPIView):
    serializer_class = PatientsCardSerializer
    permission_classes = (permissions.IsAuthenticated, IsOwner,)
    queryset = Card.objects.all()
    lookup_field = "id"
    def get_queryset(self):
        return self.queryset.filter(owner=self.request.user)



class PatientDiagnosesListAPIView(ListCreateAPIView):
    serializer_class = PatientsDiagnosesSerializer
    queryset = Diagnoses.objects.all()
    permission_classes = (permissions.IsAuthenticated,) 
    def perform_create(self, serializer):
        return serializer.save(owner=self.request.user)
    def get_queryset(self):
        return self.queryset.filter(owner=self.request.user)

class PatientDiagnosesDetailAPIView(RetrieveUpdateDestroyAPIView):
    serializer_class = PatientsDiagnosesSerializer
    permission_classes = (permissions.IsAuthenticated, IsOwner,)
    queryset = Diagnoses.objects.all()
    lookup_field = "id"
    def get_queryset(self):
        return self.queryset.filter(owner=self.request.user)

Models.py

class Patient(models.Model):
    name = models.CharField(max_length=255, null=True)
    country = models.CharField(max_length=255, null=True)
    state = models.CharField(max_length=255, null=True)
    phone = models.CharField(max_length=255, null=True)
    email = models.CharField(max_length=255, null=True)
    owner = models.ForeignKey(to=User, null=True, on_delete=models.CASCADE)
    def __str__(self):
        return self.name
class Card(models.Model):    
    name = models.CharField(max_length=255, null=True)
    card_number = models.CharField(max_length=255, null=True)    
    owner = models.OneToOneField(Patient, null=True, blank=True, on_delete=models.CASCADE)
    def __str__(self):
        return (self.patient.name)+"'s card"
   
class Diagnoses(models.Model):
    sickness = models.CharField(max_length=255, null=True)
    note = models.TextField(max_length=255, null=True)   
    owner = models.ForeignKey(Patient, null=True, on_delete=models.SET_NULL)
    def __str__(self):
        return (self.patient.name)+"'s diagnoses"

Réponses

1 CodexLink Sep 07 2020 at 04:19

Jusqu'à présent, self.request.userc'est un Userobjet. Vous ne pouvez pas l'enregistrer dans l'API ou l'enregistrer dans le modèle à cause de cela. Et pour plus de clarté, ce n'est pas un Patientobjet.

En fonction de votre situation, je mettrais l' Patientobjet en file d'attente avec l' Userobjet (s'il s'agissait d'un objet de référence à sens unique) , je le mettrais dans l'argument et j'appliquerais la méthode save ().

class PatientCardListAPIView(ListCreateAPIView):
    serializer_class = PatientsCardSerializer
    queryset = Card.objects.all()
    permission_classes = (permissions.IsAuthenticated,) 

    def perform_create(self, serializer):
        getPatientFromUser = Patient.objects.get(owner=self.request.user)
        # getPatientFromUser returns `Patient` Instance. So you could bind it in `Card` Model.
        return serializer.save(owner=getPatientFromUser)

    def get_queryset(self):
        return self.queryset.filter(owner=self.request.user)

Remarques:

  1. C'est une solution pour une référence unidirectionnelle. ( Userétant appelé Patient)
  2. Si vous avez une relation de référence croisée entre Useret Patience(où l'utilisateur et la patience ont une clé étrangère l'un sur l'autre), alors déclarez votre Usermodèle.
  3. La solution interroge le patient auprès de l'utilisateur. En tant qu'utilisateur est incompatible avec cet objet étranger.