Django 1.3 CreateView / ModelForm: проверка уникальности_все с одним полем, исключенным из формы - PullRequest
3 голосов
/ 01 августа 2011

Я ищу простой ответ на примере этой распространенной проблемы.Ответы, которые я нашел до сих пор, опускают критические точки для нас, новичков.

У меня есть приложение, в котором почти у каждой модели есть ForeignKey to User, и есть ограничение unique_together, где одно из полей всегда 'user'».

Например:

class SubscriberList(models.Model):

user = models.ForeignKey(User)
name = models.CharField(max_length=70)
date_created = models.DateTimeField(auto_now_add=True)

class Meta:
    unique_together = (
        ('user', 'name',),
        )   

def __unicode__(self):
    return self.name

SubscriberList всегда создается вошедшим в систему пользователем, и, таким образом, в форме для создания списка подписчиков я исключаю поле пользователя и присваиваю ему значениеself.request.user при сохранении формы, например, так:

class SubscriberListCreateView(AuthCreateView):
model = SubscriberList
template_name = "forms/app.html"
form_class = SubscriberListForm
success_url = "/app/lists/"

def form_valid(self, form):
    self.object = form.save(commit=False)
    self.object.user = self.request.user
    return super(SubscriberListCreateView, self).form_valid(form) 

А вот сопровождающая форма:

class SubscriberListForm(ModelForm):
class Meta:
    model = SubscriberList
    exclude = ('user')

С этим кодом допустимы правильные данные.Когда я отправляю данные, которые не unique_together, я получаю сообщение об ошибке целостности из базы данных.Причина для меня ясна - Django не проверяет unique_together, потому что исключено поле 'user'.

Как мне изменить существующий код, все еще используя CreateView, так что отправленные данные, которые не являются unique_together, выбрасываютошибка проверки формы, а не ошибка целостности из базы данных.

Ответы [ 3 ]

4 голосов
/ 02 августа 2011

Пример Yehonatan привел меня туда, но мне пришлось вызывать сообщения изнутри ValidationError for form_valid, а не отдельной функции form_invalid.

Это работает:

class SubscriberCreateView(AuthCreateView):
    model = Subscriber
    template_name = "forms/app.html"
    form_class = SubscriberForm
    success_url = "/app/subscribers/"

    def form_valid(self, form):
        self.object = form.save(commit=False)
        self.object.user = self.request.user

        try:
            self.object.full_clean()
        except ValidationError:
            #raise ValidationError("No can do, you have used this name before!")
            #return self.form_invalid(form)
            from django.forms.util import ErrorList
            form._errors["email"] = ErrorList([u"You already have an email with that name man."])
            return super(SubscriberCreateView, self).form_invalid(form)

        return super(SubscriberCreateView, self).form_valid(form) 
1 голос
/ 01 августа 2011

Принимая из документов по адресу: https://docs.djangoproject.com/en/dev/ref/models/instances/?from=olddocs#validating-objects

Вам нужно только вызвать метод full_clean () модели, если вы планируете самостоятельно обрабатывать ошибки проверки или если вы исключили поля изModelForm, требующие проверки.

Из документов из: https://docs.djangoproject.com/en/dev/ref/class-based-views/#formmixin

Представления, смешивающие FormMixin, должны обеспечивать реализацию form_valid () и form_invalid ().

Это означает, что для просмотра ошибки (которая не связана с формой) вам необходимо реализовать собственный form_invalid, добавить туда специальное сообщение об ошибке и вернуть его.

Таким образом, выполнение full_clean () на вашем объекте должно вызвать ошибку unique_together, чтобы ваш код мог выглядеть так:

def form_valid(self, form):
    self.object = form.save(commit=False)
    self.object.user = self.request.user
    # validate unique_together constraint
    try:
        self.object.full_clean()
    except ValidationError:
        # here you can return the same view with error messages
        # e.g.
        return self.form_invalid(form)
    return super(SubscriberListCreateView, self).form_valid(form)

def form_invalid(self, form):
    # using messages
    # from django.contrib import messages
    # messages.error('You already have a list with that name')
    # or adding a custom error
    from django.forms.util import ErrorList
    form._errors["name"] = ErrorList([u"You already have a list with that name"])
    return super(SubscriberListCreateView, self).form_invalid(form)

HTH

0 голосов
/ 05 октября 2011

добавление еще одного примера, который может быть немного проще для новичков.

forms.py

class GroupItemForm(ModelForm):
    def form_valid(self):
        self.object = self.save(commit=False)
        try:
            self.object.full_clean()
        except ValidationError:
            # here you can return the same view with error messages
            # e.g. field level error or...
            self._errors["sku"] = self.error_class([u"You already have an email with that name."])
            # ... form level error
            self.errors['__all__'] = self.error_class(["error msg"]
            return False
        return True

views.py

def add_stock_item_detail(request, item_id, form_class=GroupItemForm, template_name="myapp/mytemplate.html"):
    item = get_object_or_404(Item, pk=item_id)
    product = Product(item=item)
    if request.method == 'POST':
        form = form_class(request.POST, instance=product)
        if form.is_valid() and form.form_valid():
            form.save()
            return HttpResponseRedirect('someurl')
    else:
        form = form_class(instance=product)

    ctx.update({
        "form" : form,
     })

    return render_to_response(template_name, RequestContext(request, ctx))
...