jak naprawić instancję django Model

Question

komunikaty o błędach ValueError: Cannot przypisać „<User: [email protected]>”: „Diagnoses.owner” musi być instancją „Patient”

Otrzymuję powyższy błąd, gdy próbuję utworzyć nową kartę lub diagnozy. Karta i diagnozy mają być przykładem pacjenta, tak jak pacjent ma kartę, a także diagnozy. Klasa pacjentów ma klucz foregin odnoszący się do użytkownika. Oto kod

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"

Answer 1

Jak dotąd self.request.userjest to Userobiekt. Z tego powodu nie możesz zapisać go w API ani zapisać w modelu. I dla jasności, to nie jest Patientprzedmiot.

W oparciu o twoją sytuację umieściłbym Patientobiekt w kolejce z Userobiektem (jeśli byłby to jednostronny obiekt odniesienia) i umieściłbym to w argumencie i wykonałem metodę 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)

Uwagi:

1. To jest rozwiązanie dla jednostronnego odniesienia. ( Userokreślany jako Patient)
2. Jeśli masz relację odsyłacza między Useri Patience(gdzie Użytkownik i Cierpliwość mają na sobie klucz obcy), zadeklaruj swój UserModel.
3. Rozwiązanie pyta pacjenta od użytkownika. Jako użytkownik nie jest kompatybilny z tym obcym obiektem.