djangoのformsetでtemplateにform.fieldと書いた場合の保存方法

Question

追加
返信ありがとうございます。
https://blog.narito.ninja/detail/30
にあるお手本を用いて実装しようとしています。
この中のpost_formset.htmlの部分を以下のように変えたいです。

追加　
ありがとうございます。
参考にしたかたはgithubに出していらっしゃるのでそれをそのまま使っています。
https://github.com/naritotakizawa/django-modelformset-sample

djangoのformsetを使っています。
そのままのtemplateだと入力して送信をクリックすると、formset.save()は有効でデータベースに保存されます。そしてページを開きなおしても消えません。お手本どおりに動きます。

{% extends 'app/base.html' %}

{% block content %}
<form action="" method="post">
    <div class="row">
        {% for form in formset %}
            <div class="col-sm-4">
                {{ form.as_p }}
            </div>
        {% endfor %}
    </div>
    {{ formset.management_form }}
    {% csrf_token %}
    <button type="submit" class="btn btn-primary">送信</button>
</form>
{% endblock %}

しかし以下のように変更してみるとformset.save()が反映されていないようで、ページを開きなおすと消えてしまいます。

{% extends 'app/base.html' %}

{% block content %}
<form action="" method="post">
    <div class="row">
        {% for form in formset %}
            <div class="col-sm-4">
                {{ form.title }}
                {{ form.text }}
                {{ form.date }}
            </div>
        {% endfor %}
    </div>
    {{ formset.management_form }}
    {% csrf_token %}
    <button type="submit" class="btn btn-primary">送信</button>
</form>
{% endblock %}

view

from django.shortcuts import render, redirect
from .forms import PostCreateFormSet


def add(request):
    formset = PostCreateFormSet(request.POST or None)
    if request.method == 'POST' and formset.is_valid():
        formset.save()
        return redirect('app:index')

    context = {
        'formset': formset
    }

    return render(request, 'app/post_formset.html', context)

form

from django import forms
from .models import Post


class PostCreateForm(forms.ModelForm):

    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
        for field in self.fields.values():
            field.widget.attrs['class'] = 'form-control'

    class Meta:
        model = Post
        fields = '__all__'


# これがモデルフォームセット
PostCreateFormSet = forms.modelformset_factory(
    Post, form=PostCreateForm, extra=3
)

model

from django.db import models
from django.utils import timezone


class Post(models.Model):
    title = models.CharField('タイトル', max_length=200)
    text = models.TextField('本文')
    date = models.DateTimeField('日付', default=timezone.now)

    def __str__(self):
        return self.title

form.fieldとした場合の保存方法をご存知の方はいらっしゃいませんか。
よろしくお願いします。

Comments

[dameo]

ちょっと情報が足りなすぎてよく分かってませんが、ModelFormSetなら、form.fieldもModelなのでは？
Modelならsave()ってあった気がするけど、、、
質問するならもっと具体的に動くものをサンプルでも作って載せないと回答付かないと思います。
でなければ、聞きたい部分を絶対他の意味に取れないくらい詳細に説明して、部分ソースを載せるくらいかな。今回はソースの量も少ない上に説明がほぼないため、こういう問答が必要になることが予想されるので、敬遠されちゃったんだと思いますよ。

[dameo]

削除しました

[keraker]

返信ありがとうございます。
修正しました。

[dameo]

すみませんが、情報が全く足りていません。
「以下のようなtemplateだと」ではなくて、完全なソースコードを貼ってください。

また、有効かどうかの判断は具体的にどうやって判断していますか？(デバッガで何行目に止まったからとか、DBで具体的にこう調べたら具体的にこうだったとか、それ以外の方法でどうだったとか)
さらに「保存される」「formset.save()が反映」というのもどうやって判断したのか分かりません(DBを見てそう判断したのか、画面上何らかの方法で判断したのか、デバッガで追ったのかなど)。

保存方法もどこでどう保存しているのか全く書かれていません。
ようは、既存のものを変えたのであれば、私が全く同じソースを書けるくらいまで情報がないと、あなたが何を間違えたのかの情報にどうやっても辿り着かないのです。私はそれを推測しながら、小出しにされる情報から何度も何度も手探りで探したくないわけです。

私は回答側にも書いたとおり、誰がやっても同じ結果になるコードを書いています。同じコードを動かせるので、どこを間違えたのか手探りする必要もありません。言ってることが伝わるといいのですが。。。

[dameo]

ちなみに私の書いたコードだとindex.htmlが質問中最初のコードブロックに近く(ただしあなたのコードはそのままでは動きません)、index2.htmlが次のコードブロックに近いです(ただしあなたのコードはそのままでは動きません)。そして、ちゃんとどちらもviews.pyで保存されています。ご参考までに。

viewの書き方的には紹介頂いたサイトの書き方が一般的ですよ。ただ、initialを使った初期化コードはそのままでclean()が動くのかよく分かりません。あなたの情報だけだとそれ以外の未確定要因でいくらでも落ちれるので、見当も付きませんが。

[dameo]

そのまま使ってるんなら変えた部分のソースだけ全部貼ればいいでしょ...
質問する人がそんなに適当なのに、なんで回答する人が全部考えて要求しないと何もしないのか不思議です。

Answer 1

オブジェクトのidが送られていなかったからです。
今回のモデルにはprimary keyとなるfiledがなかったので、自動的にidというフィールドが追加されています。

https://docs.djangoproject.com/ja/3.1/topics/db/models/#automatic-primary-key-fields

以下のように最後に1行追加すると動くはずです。

app/templates/app/post_formset.html

{% extends 'app/base.html' %}

{% block content %}
<form action="" method="post">
    <div class="row">
        {% for form in formset %}
            <div class="col-sm-4">
                {{ form.title }}
                {{ form.text }}
                {{ form.date }}
                {{ form.id }} <!-- これが必要 -->
            </div>
        {% endfor %}
    </div>
    {{ formset.management_form }}
    {% csrf_token %}
    <button type="submit" class="btn btn-primary">送信</button>
</form>
{% endblock %}

Comments

[keraker]

ありがとうございます。
拝見したところ、 modelのほうのfieldのどれかにprimary_key=Trueを追加することでも解決できそうですね。
お手数おかけしました。

[keraker]

参考資料もありがとうございます。

[dameo]

> 拝見したところ、 modelのほうのfieldのどれかにprimary_key=Trueを追加することでも解決できそうですね。

これは機械的に考えればそうなんだけど、primary keyはuniqueなので、唯一の値でないといけない。
例えばtitleをprimary keyにしたとすると、同じタイトルの投稿ができなくなってしまうので、現実的には無理で、他も何かと都合が悪かったりします。まあ仮定の話なので、論理的には主キーにしても問題は解決しますね。

なお、このレベルの問題は普通に生成HTMLを見比べるだけで見つかるので、自分で解決してほしいです。

Answer 2

回答ではありませんが、インデントがないと書けないので、こちらに。
質問がよく分からなかったので、ドキュメントに書いてあるとおりのModelFormSetを使ったコードを書いてみました。コードそのものを載せるとdjangoが作った部分と混ざって嵩張るので、venv環境構築とパッチを行うシェルスクリプトになっています。基本的なコマンドが揃っているUnix系OSなら動くと思います。

スクリプトの使い方

create_env.shとpatch.diffを両方空のディレクトリに保存してください。
コンソールから、そのディレクトリに行き、

. create_env.sh

で以下の処理を行います。

1. venv仮想環境を構築
2. pipで仮想環境内にdjangoをインストール
3. djangoでmysiteプロジェクト作成
4. djangoでmyappアプリ作成
5. patch.diffパッチを当てます
6. djangoでmigrateを実施します
7. djangoでrunserverを実施します

あとはブラウザから確認するだけです。

http://localhost:8000/myapp/ ← index.htmlテンプレートを使ったレンダリング
http://localhost:8000/myapp/index2.html ← index2.htmlテンプレートを使ったレンダリング

※スクリプト内でgitで適当にcommitしたり、wgetで.gitignoreを取ってきたりもしています。

使い方の分からない人やUnix系の環境がない方は、パッチを見て、パッチを当てたコードを想像してください。

create_env.sh

#!/bin/bash
PYTHON=python3
if ! command -v $PYTHON; then
    PYTHON=python
fi

$PYTHON -m venv env
. env/bin/activate

pip install django==3.1
django-admin startproject mysite
cd mysite
django-admin startapp myapp

if command -v git; then
    git init
    wget -O .gitignore https://github.com/github/gitignore/raw/master/Python.gitignore
    git add .
    git commit -m 'initial commit'
fi

patch -p1 <../patch.diff

if command -v git; then
    git add .
    git commit -m 'patched'
fi

python manage.py makemigrations myapp
python manage.py migrate

if command -v git; then
    git add .
    git commit -m 'migrated'
fi

python manage.py runserver

patch.diff

diff --git a/myapp/forms.py b/myapp/forms.py
new file mode 100644
index 0000000..70a517f
--- /dev/null
+++ b/myapp/forms.py
@@ -0,0 +1,4 @@
+from django.forms import ModelForm, modelformset_factory, BaseModelFormSet
+from .models import Model1
+
+ModelFormSet1 = modelformset_factory(Model1, fields=('field1', 'field2'))
diff --git a/myapp/models.py b/myapp/models.py
index 71a8362..a2df697 100644
--- a/myapp/models.py
+++ b/myapp/models.py
@@ -1,3 +1,6 @@
 from django.db import models
 
-# Create your models here.
+class Model1(models.Model):
+    field1 = models.CharField(max_length=200)
+    field2 = models.CharField(max_length=200)
+
diff --git a/myapp/templates/index.html b/myapp/templates/index.html
new file mode 100644
index 0000000..b6dedea
--- /dev/null
+++ b/myapp/templates/index.html
@@ -0,0 +1,8 @@
+<h1>formsetのテスト</h1>
+<form method="post">
+    {% csrf_token %}
+    <table>
+    {{ formset }}
+    </table>
+    <button type="submit">submit</button>
+</form>
diff --git a/myapp/templates/index2.html b/myapp/templates/index2.html
new file mode 100644
index 0000000..1220bdc
--- /dev/null
+++ b/myapp/templates/index2.html
@@ -0,0 +1,14 @@
+<h1>formsetのテスト</h1>
+<form method="post">
+    {% csrf_token %}
+    <table>
+    {{ formset.management_form }}
+    {% for form in formset %}
+        {% for field in form %}
+            {{ field.label_tag }} {{ field }}
+        {% endfor %}
+    {% endfor %}
+    </table>
+    <button type="submit">submit</button>
+</form>
+
diff --git a/myapp/urls.py b/myapp/urls.py
new file mode 100644
index 0000000..0d74f3d
--- /dev/null
+++ b/myapp/urls.py
@@ -0,0 +1,8 @@
+from django.urls import path
+
+from . import views
+
+urlpatterns = [
+    path('', views.index, name='index'),
+    path('index2.html', views.index, name='index2'),
+]
diff --git a/myapp/views.py b/myapp/views.py
index 91ea44a..6f8b7d9 100644
--- a/myapp/views.py
+++ b/myapp/views.py
@@ -1,3 +1,18 @@
 from django.shortcuts import render
+from .forms import ModelFormSet1
+
+def index(request):
+    return index_common(request, 'index.html')
+
+def index2(request):
+    return index_common(request, 'index2.html')
+
+def index_common(request, template):
+    if request.method == 'POST':
+        formset = ModelFormSet1(request.POST)
+        if formset.is_valid():
+            formset.save()
+    else:
+        formset = ModelFormSet1()
+    return render(request, template, {'formset': formset})
 
-# Create your views here.
diff --git a/mysite/settings.py b/mysite/settings.py
index 1242d49..3506986 100644
--- a/mysite/settings.py
+++ b/mysite/settings.py
@@ -31,6 +31,7 @@ ALLOWED_HOSTS = []
 # Application definition
 
 INSTALLED_APPS = [
+    'myapp.apps.MyappConfig',
     'django.contrib.admin',
     'django.contrib.auth',
     'django.contrib.contenttypes',
diff --git a/mysite/urls.py b/mysite/urls.py
index e7f9f35..b41a8b1 100644
--- a/mysite/urls.py
+++ b/mysite/urls.py
@@ -14,8 +14,9 @@ Including another URLconf
     2. Add a URL to urlpatterns:  path('blog/', include('blog.urls'))
 """
 from django.contrib import admin
-from django.urls import path
+from django.urls import path, include
 
 urlpatterns = [
+    path('myapp/', include('myapp.urls')),
     path('admin/', admin.site.urls),
 ]