Django: модельная форма «объект не имеет атрибута cleaned_data»

86

Я пытаюсь сделать форму поиска для одного из моих классов. Модель формы:

from django import forms
from django.forms import CharField, ModelMultipleChoiceField, ModelChoiceField
from books.models import Book, Author, Category

class SearchForm(forms.ModelForm):
    authors = ModelMultipleChoiceField(queryset=Author.objects.all(),required=False)    
    category = ModelChoiceField (queryset=Category.objects.all(),required=False)
    class Meta:
        model = Book
        fields = ["title"]

Я использую следующее представление:

from django.shortcuts import render_to_response, redirect, get_object_or_404
from django.template import RequestContext
from books.models import Book,Author
from books.forms import BookForm, SearchForm
from users.models import User

def search_book(request):
    if request.method == "POST":
        form = SearchForm(request.POST)
        if form.is_valid():
            form = SearchForm(request.POST)
            stitle = form.cleaned_data['title']
            sauthor = form.cleaned_data['author']
            scategory = form.cleaned_data['category']
    else:
        form = SearchForm()
    return render_to_response("books/create.html", {
        "form": form,
    }, context_instance=RequestContext(request))

Форма отображается нормально, но когда я отправляю ее, я получаю сообщение об ошибке: 'SearchForm' object has no attribute 'cleaned_data'

Я не уверен, что происходит, может кто-нибудь мне помочь? Благодаря!

Джозеф
источник
8
Зачем звонить form = SearchForm(request.POST)дважды?
hughdbrown
@SafwanSamsudeen В моем комментарии говорится то же самое, что и в ответе, который был признан правильным 10 лет назад. Проблема в том, что код не должен вызывать SearchForm второй раз.
hughdbrown

Ответы:

184

По какой-то причине вы повторно создаете экземпляр формы после проверки is_valid(). Формы получают cleaned_dataатрибут только тогда, когда is_valid()он был вызван, а вы не вызывали его в этом новом, втором экземпляре.

Просто избавьтесь от второго, form = SearchForm(request.POST)и все будет хорошо.

Дэниел Розман
источник
7

Я бы написал такой код:

def search_book(request):
    form = SearchForm(request.POST or None)
    if request.method == "POST" and form.is_valid():
        stitle = form.cleaned_data['title']
        sauthor = form.cleaned_data['author']
        scategory = form.cleaned_data['category']
        return HttpResponseRedirect('/thanks/')
    return render_to_response("books/create.html", {
        "form": form,
    }, context_instance=RequestContext(request))

Очень похоже на документацию .

коричнево-коричневый
источник
Что ж, работает! Имеет ли значение определение местоположения формы?
Joseph
Я не знаю, в чем была ваша проблема, но думаю, что звонить SearchForm(request.POST)дважды не было необходимости. Остальное - просто витрина: мне нравится такой способ сворачивания аргументов конструкции формы, так что вам нужен только один вызов.
hughdbrown
Или именно то, что сказал @Daniel Roseman. На вашем месте я бы выбрал его в качестве предпочтительного ответа, потому что он точно определяет причину.
hughdbrown
2

Иногда, если мы забываем

return self.cleaned_data 

в чистой функции форм django у нас не будет никаких данных, хотя form.is_valid()они вернутся True.

субраманьям
источник