0

我有一个表单 ReviewForm 允许一个用户审查另一个用户(审查的来源和审查的目标)。我想检查用户是否没有审查自己,我认为这样做的地方应该在 ReviewForm 本身。但是,这两个用户(源和目标)不是表单的一部分,因为他们之前是通过 UI 选择的。这样做最干净的方法是什么?现在我正在视图中进行验证,但我觉得不太好。

class ReviewForm(forms.ModelForm):
    class Meta:
        model = models.Review
        fields=["title", "content"]


class Review(models.Model):
    title=models.CharField(max_length=50,blank=True)
    content=models.TextField(blank=True)
    source=models.ForeignKey(Profile,related_name="reviews_by")
    target=models.ForeignKey(Profile,related_name="reviews_of")


def new_review(request,profile_id):
    form=ReviewForm()
    if request.method=='POST':
        review=models.Review()
        review.source=request.user.profile
        review.target=models.Profile.objects.get(id=profile_id)
        form = ReviewForm(request.POST, instance = review)
        if request.user.profile.id == int(profile_id):
            form.errors["__all__"]= "Don't review yourself!"
        if form.is_valid():
            form.save()
            return redirect(reverse("profile"))
        else:
            return render(request,"reviews/new.html",{"form":form})
    return render(request,"reviews/new.html",{"form":form})

谢谢你。

4

1 回答 1

0

试试这个:

class ReviewForm(forms.ModelForm):
    def clean(self):
        if self.instance.source == self.instance.target:
            raise forms.ValidationError("You cannot review your entity")
        return super(ReviewForm, self).clean()

    class Meta:
        model = models.Review
        fields=["title", "content"]
于 2013-06-20T05:50:45.627 回答