NOT NULL约束失败Django CreateView

时间:2017-08-12 20:03:26

标签: python django python-3.x django-class-based-views

我想在这个项目上实现两个目标。首先,我想将登录用户保存为审阅者。其次,我想将外键中的律师价值传递给审核表。 (通过这种方式,用户可以查看律师详细信息页面上的评论链接,并查看该律师,而不是其他任何人。)

我一直在抓住这个错误。

错误

IntegrityError at /lawyers/karlyn-rosemarie-hylton/review/
NOT NULL constraint failed: lawyers_review.reviewer_id

有人可以解释一下我做错了什么以及如何解决它?

第二个目标似乎与第一个目标非常相关。我没有收到任何错误。但我想确保我做得对。但是,在我通过第一个错误之前,我无法判断。

非常感谢任何获得帮助的帮助。

models.py

from django.db import models
from django.urls import reverse
from django.template.defaultfilters import slugify
from django.contrib.auth.models import User


class Lawyer(models.Model):
    name = models.CharField(max_length=55,
                            default='')
    picture = models.ImageField(upload_to='media/lawyers/pictures', 
                                default='')
    slug = models.SlugField()

    def average_rating(self):
        all_ratings = map(lambda x: x.rating, self.review_set.all())
        return np.mean(all_ratings)

    def save(self, *args, **kwargs):
        self.slug = (slugify(self.name))
        super(Lawyer, self).save(*args, **kwargs)

    def get_absolute_url(self):
        return reverse('lawyers:detail', kwargs={'slug': self.slug})

    def __str__(self):
        return self.name 


class Review(models.Model):
    RATING_CHOICES = (
        (1, '1'),
        (2, '2'),
        (3, '3'),
        (4, '4'),
        (5, '5'),
    )
    lawyer = models.ForeignKey(Lawyer, 
                                null=True)
    created = models.DateTimeField(auto_now_add=True)
    reviewer = models.ForeignKey(User)
    title = models.CharField(max_length=55,
                                default='')
    comment = models.TextField()
    rating = models.IntegerField(choices=RATING_CHOICES)

    def __str__(self):
        return self.lawyer.name

views.py

from django.shortcuts import render, get_object_or_404
from django.views.generic import (CreateView,
                                    DetailView,
                                    ListView)


from django.contrib.auth.models import User


from .models import Lawyer, Review


class AddLawyerView(CreateView):
    model = Lawyer 
    fields = ['name', 'picture']

    def post_valid(self, form):
        lawyer = form.save(Commit=False)
        lawyer.picture = form.cleaned_data['picture']
        return super(AddLawyerView, self).form_valid(form)


class LawyerDetail(DetailView):
    model = Lawyer 


class LawyerList(ListView):
    model = Lawyer 


class AddReviewView(CreateView):
    model = Review 
    fields = ['rating', 'title', 'comment']

    def post_valid(self, form):
        review = form.save(Commit=False)
        review.lawyer = get_object_or_404(Lawyer, 
                                            slug=self.kwargs.get('slug'))
        review.reviewer = User.objects.get(user=self.request.user)
        return super(AddReview, self).form_valid(form)

urls.py

from django.conf.urls import url


from .views import (AddLawyerView, 
                    LawyerDetail,
                    LawyerList,
                    AddReviewView)


urlpatterns = [
    url(r'^add-lawyer',
        AddLawyerView.as_view(),
        name = 'add-lawyer'),


    url(r'^(?P<slug>[\w-]+)/$',
        LawyerDetail.as_view(),
        name = 'detail'),


    url(r'^$',
        LawyerList.as_view(),
        name = 'list'),


    url(r'^(?P<slug>[\w-]+)/review/$',
        AddReviewView.as_view(),
        name = 'add-review'),
]

使用@Alasdair和@Sachin Kukreja的反馈,我回去做了一些修正。我改变了#34;承诺&#34; to&#34; commit&#34;和&#34; post_valid&#34; to&#34; form_valid&#34;。我还做了一个小调整来简化视图。已更正且正常工作的AddReviewView如下所示。有错误的人就是上面。

谢谢大家!

工作视图:

class AddReviewView(CreateView):
    model = Review 
    fields = ['rating', 'title', 'comment']

    def form_valid(self, form):
        review = form.save(commit=False)
        review.lawyer = get_object_or_404(Lawyer, 
                                            slug=self.kwargs.get('slug'))
        review.reviewer = self.request.user
        return super(AddReviewView, self).form_valid(form)

1 个答案:

答案 0 :(得分:2)

你有一个拼写错误,你压倒form_valid但是你写了post_valid

def form_valid(self, form):
    review = form.save(Commit=False)
    review.lawyer = get_object_or_404(Lawyer, 
                                        slug=self.kwargs.get('slug'))
    review.reviewer = User.objects.get(user=self.request.user)

    # save and return HttpResponse
    review.save()
    return HttpResponseRedirect(self.get_success_url())

但是,super调用仍会调用父类“form_valid,这仍会引发错误。我认为您可以在此覆盖方法中保存审阅并返回基类返回的内容,即return HttpResponseRedirect(self.get_success_url())

参考:form_valid in FormMixin classform_valid in ModelFormMixin class