Djangoのフォームで、選択をフィルタリングする方法

Question

行き詰りました。

最初のプルダウンメニューで選択した項目を
次のプルダウンメニューに連動させたいです。
(admin管理サイトは使用しない）

#model.py

class Create_type(models.Model):
    name = models.CharField(max_length=20,unique=True)
    def __str__(self):
        return self.name

class Genre(models.Model):
    name = models.CharField(max_length=20,unique=True)
    create_type = models.ManyToManyField(Create_type)
    def __str__(self):
        return self.name

class Title_Post(models.Model):
    title = models.CharField(unique=True)
    create_type = models.ForeignKey(Create_type,blank=false)
    genre = models.ManyToManyField(Genre,blank=True)

以下のサイトを参考にForm.pyを作成しました。
https://stackoverflow.com/questions/291945/how-do-i-filter-foreignkey-choices-in-a-django-modelform

#form.py

class Title_Form(forms.ModelForm):
    class Meta:
        model = Title_Post
        fields = ('title','create_type','genre')
        
    def __init__(self, *args, **kwargs):
        super(Title_Form, self).__init__(*args, **kwargs)
        self.fields['genre'].queryset = Genre.objects.filter(
            type=self.instance.type)

#view.py
def title_new(request):
    if request.method == "POST":
        f = Title_Form(request.POST,request.FILES)
        if f.is_valid():
            post = f.save(commit=False)
            post.published_date = timezone.now()
            post.save()
            f.save_m2m()
            return redirect('app:title', pk=post.pk)
    else:
        f = Title_Form()
    return render(request, 'app/title_edit.html', {'form':f})

小説	映画	アニメ
SF	SF	SF
ホラー	ホラー	ホラー
	3D	3D

イメージ的には
Type(小説、映画、アニメ)があって、
Genre(ＳＦ、ホラー、3D)等のジャンルがあり、
フォーム上で小説を選択した場合、genreのリストでは「3D」を選択できないようにしたいです。
表示しないという動作はJavascriptで記述できたのですが、Javascriptを無効にした場合、不正な入力がされてしまうと思うので、Django側のModelやFormで入力をエラーとしたいです。

現状は、そもそもGenreに値が表示されません・・・

何卒、よろしくお願いいたします・・

Answer 1

とりえあえず、全部選択肢にでるように
formの__init__なしにしましょう。

それからTitle_Formのcleanをオーバーライドしましょう
cleanのoverride例
値がおかしかったらValidationErrorをraiseして、
あってたら特に何もしなくていいです。

Comments

[am120sec]

途中ですが、選択肢は表示されるようになりました。
次に値の検証ですが、ManyToManyでリレーションされたクラスのフィールドを取得する方法が解りません。
参考資料でも良いのでヒントを頂けないでしょうか。

class Title_Form(forms.ModelForm):

class Meta:
model = Title_Post
fields = ('title','create_type','genre')

def clean(self):
cleaned_date = super().clean()
genre = cleaned_date.get("genre")
create_type = cleaned_date.get("create_type")
genre_create_type = #ここです

if genre and create_type:
if genre.create_type != create_type:
raise form.balidationError(
"そのジャンルは使用できません"
)

[kokardy]

ちょっといま確認できませんけど、
cleaned_data.get("genre")ってやったらQuerySetが返ってきませんか？
forで回せるのではないかと。

[am120sec]

その前にcleaned_dataが上手く動いて無いようでした・・
簡単なcleaned_dataで作ってみてもダメです・・。
class Title_Form(forms.ModelForm):
volume = forms.IntegerField(required=False)
class Meta:
model = Title_Post
fields = ('title','create_type','genre','volume')

def clean_volume(self):
volume = self.cleaned_date["volume"]
if not 1 <= volume <= 10:
raise forms.ValidationError(u"1-10の範囲で入力してください")
return volume

これでフォームをSAVEすると、'Title_Form' object has no attribute 'cleaned_date'エラーが出てしまいます。

view.pyがおかしいのでしょうか・・。

[kokardy]

書くところ間違えました……
回答は消せないのかな

"cleaned_data"のタイプミスと思われます。

[am120sec]

私のほうからは削除できませんでした。

"cleaned_data"でいけました・・・・・・・・・！
中学生からやり直してきます(´;ω;｀)

[kokardy]

スマホからだと見つけられなかったんですが、
PCで開いたら削除リクエストの場所みつけました。
お騒がせしました。

Answer 2

やり方が正しいかどうかは置いておいて、とりあえず動く物ができましたので報告です。

genre_filter = genre.filter(create_type__name = create_type)

アンダースコアが2つ入っているのはcreate_typeのnameフィールドを指定するためらしいです。

ご指摘ありましたら是非ともお願いします。

#form.py
class Title_Form(forms.ModelForm):
    volume = forms.IntegerField(required=False)
    class Meta:
        model = Title_Post
        fields = ('title','create_type','genre')
        

    def clean(self):
        cleaned_date = super().clean()
        genre = self.cleaned_data.get("genre")
        create_type = self.cleaned_data.get("create_type")

        if genre and create_type:
            genre_filter = genre.filter(create_type__name = create_type)
            if len(genre_filter) == 0:
                raise forms.ValidationError(
                    "選択されたジャンルはタイプと関連性がないため使用できません"
                )