Как проверить поле формы Django на поле FK - PullRequest
0 голосов
/ 29 октября 2010

Я пытаюсь создать правило проверки, которое передает ошибку, если (целевой этап) объекты target_date больше, чем проекты target_date. Проблема в том, что ни одна из ошибок не возникает.

Для простоты,

models.py

class Project(models.Model):
    target_date = models.DateField()

class Milestone(models.Model):
    project = models.ForeignKey(Project)
    target_date = models.DateField()

Forms.py

class MilestoneAddForm(forms.ModelForm):

    class Meta:
        model = Milestone

    def clean_target_date(self):
        tdate = self.cleaned_data['target_date']
        if tdate > self.instance.project.target_date
            raise forms.ValidationError("Target Date is outside or project target date.")
        return tdate

views.py (удален несвязанный код)

def MilestoneManage(request, project_id):
    qs = Project.objects.none()
    if request.method == 'POST':
        formset = MilestoneFormSet(request.POST, prefix='new')
        if formset.is_valid():
            newMiles = formset.save(commit=False)
            for new in newMiles:
                new.project_id = project.id
                new.save()
            return HttpResponseRedirect(reverse('project.views.detail', args=(project.id,)))
    else:
        formset = MilestoneFormSet2(queryset=qs, prefix='new')
    return render_to_response('project/manageMilestones.html', {
        'formset': formset,
    }, context_instance=RequestContext(request))

1 Ответ

1 голос
/ 29 октября 2010

Это должно работать.См. Встроенные комментарии для получения дополнительной информации о том, что было изменено.

def clean_target_date(self):
    #                 V-- typo (cleand_data => cleaned_data)
    tdate = self.cleaned_data['target_date']
    project = self.instance.project
    # project allows for nulls, so check that first.
    if project is None:
        raise forms.ValidationError("There's no project associated with this milestone.")
    #                   V-- Do an attribute lookup, don't use the query lookup syntax
    if tdate > project.target_date:
        raise forms.ValidationError("Target Date is outside or project target date.")
    return tdate
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...