DjangoのPOST(投稿・更新)処理が機能しない

Question

viewのpostの処理と、更新時にユーザーフォームにすでに入れている状態を作り出すことがうまくいきません。
現在admin画面で一行だけ<Post_datail>モデルにデータ保存をしています。
そして、edit.htmlに遷移するときに

TypeError at /users/2/edit/
'QuerySet' object does not support item assignment
Request Method: GET
Request URL: http://127.0.0.1:8000/users/2/edit/
Django Version: 3.0.4
Exception Type: TypeError
Exception Value:
'QuerySet' object does not support item assignment
Exception Location: C:*****\views.py in UsersEdit, line 197

というエラーが出ています。
197行目はviews.pyのUsersEdit関数の

else:
data['form'] = UserForm()　←こちらの部分にあたります。

formにinstance= dataがセットされずに飛ばしてUserFormのところまで飛んでいます。

forms.py

from django import forms
from .models import Post_Detail

class UserForm(forms.ModelForm):
    class Meta:
        model = Post_Detail
        fields = ['name','like']

models.py

from django.contrib.auth.models import User
from django.db import models
from django.utils import timezone
from django.contrib.auth import get_user_model

class Post_Detail(models.Model):
    name = models.CharField('名前', max_length=100)
    like = models.CharField('好きなもの', max_length=100)
    regist_date = models.DateTimeField(default=timezone.now)    
    author = models.ForeignKey(get_user_model(), on_delete=models.CASCADE)

    def __str__(self):
        return self.author.username

urls.py

from django.urls import path
from . import views

urlpatterns = [
    path('', views.home, name='home'),
    path('users/<int:user_id>/', views.MyPage, name='mypage'),
    path('users/<int:user_id>/create/', views.create, name='create'),
    path('users/<int:user_id>/edit/', views.UsersEdit, name='edit'),

]

views.py

from django.contrib import auth
from django.conf import settings
from django.contrib.auth.decorators import login_required
from django.contrib.auth.models import User
from django.shortcuts import render, redirect,get_object_or_404
from .forms import UserForm,UserEditForm
from .models import Post_Detail


@login_required
def MyPage(request,user_id):
    user_id = request.user.id
    User.objects.get(pk = user_id)
    
    data = Post_Detail.objects.filter(author=user_id)
    params = {'message': 'mypage', 'data': data}
    return render(request, 'mypage.html', params)


@login_required
def create(request,user_id):
    User.objects.get(pk = user_id)
    params = {'message': '', 'form': None}
    if request.method == 'POST':
       form = UserForm(request.POST)
       if form.is_valid():
          instance = form.save(commit=False)
          instance.author = request.user
          instance.save()
          return redirect('mypage')
       else:
          params['message'] = '再入力して下さい'
          params['form'] = form
       else:
          params['form'] = UserForm()
       return render(request, 'create.html', params)

@login_required
def UsersEdit(request,user_id):

    User.objects.get(pk = user_id)
    data = get_object_or_404(Post_Detail, user_id)
    if request.method == 'POST':
       form = UserForm(request.POST,instance=data)
       if form.is_valid():
          instance = form.save(commit=False)
          instance.author = request.user
          instance.save()
          return redirect('edit')
       else:
          data['message'] = '再入力して下さい'
          data['form'] = form
       else:
           data['form'] = UserForm()
       return render(request, 'create.html', data)

mypage.html

{% extends 'base.html' %} 
{% load static %}
{% block title %}マイページ{% endblock title %}

{% block content %}
{% if request.user.is_authenticated %}
<div class="container">
    <h4 class="mt-4 border-bottom">マイページ</h4>
        <a href="#" class="btn btn-primary btn-sm my-3">変更する</a>
    <table class="table table-striped table-bordered">
      <thead>
        <tr>
          <th scope="col">ログイン中ユーザー名</th>
          <th scope="col">ユーザーアカウント名</th>
          <th scope="col">すきなもの</th>
        </tr>
      </thead>
      <tbody>
      {% for Post_Detail in data %}
        <tr>
            <td>{{ Post_Detail.author }}</td>
            <td>{{ Post_Detail.name }}</td>
            <td>{{ Post_Detail.like }}</td>
          </td>
        </tr>
        {% endfor %}
      </tbody>
    </table>
    </div>
{% endif %}
{% endblock content %}

create.html

{% extends 'base.html' %} 
{% load static %}

  <body>
  {% block content %}
    <form method="post">
        {% csrf_token %}
        {{ form.as_p }}
        <button type="submit" name="submit">Submit</button>
    </form>
{% endblock %}
    </div>
  </body>
</html>

edit.html

    {% extends 'base.html' %} 
    {% load static %}
    
      <body>

      {% block content %}
      <h2>ユーザ編集</h2>
      <form method="post">
            {% csrf_token %}
            {{ form.as_p }}
            <button type="submit" name="submit">Submit</button>
        </form>
    {% endblock %}
        </div>
      </body>
    </html>

Answer 1

UsersEditの変数dataはクエリセットであり辞書ではないでdata['form']でエラーが起きます。
なのでcreateと同じようにparamsのような返却ようの辞書を別で作成した方が良いかと思います。

@login_required
def UsersEdit(request,user_id):
    params = {'message': '', 'form': None}
    User.objects.get(pk = user_id)
    data = get_object_or_404(Post_Detail, pk=user_id)
    if request.method == 'POST':
       form = UserForm(request.POST,instance=data)
       if form.is_valid():
          instance = form.save(commit=False)
          instance.author = request.user
          instance.save()
          return redirect('edit')
       else:
          params['message'] = '再入力して下さい'
          params['form'] = form
    else:
       params['form'] = UserForm()
    return render(request, 'create.html', params)

Comments

[yayaya22]

ありがとうございます！別辞書を用意するということで解釈させていただきました。しかしエラーが出てしまいました。

TypeError at /users/2/edit/
cannot unpack non-iterable int object
Request Method: GET
Request URL: http://127.0.0.1:8000/users/2/edit/
Django Version: 3.0.4
Exception Type: TypeError

response = get_response(request) …
▼ Local vars
Variable Value
exc
TypeError('cannot unpack non-iterable int object')
get_response
<bound method BaseHandler._get_response of <django.core.handlers.wsgi.WSGIHandler object at 0x00000124D219D748>>
request
<WSGIRequest: GET '/users/2/edit/'>

※以下は太字の部分でした※
data = get_object_or_404(Post_Detail,user_id) …
▼ Local vars
Variable Value
params
{'form': None, 'message': ''}
request
<WSGIRequest: GET '/users/2/edit/'>
user_id
2

[__horito]

get_object_or_404(Post_Detail, user_id)
を
get_object_or_404(Post_Detail, pk=user_id)
に修正しました。
ご確認よろしくお願いします。
以下参考
https://docs.djangoproject.com/ja/3.2/intro/tutorial03/#a-shortcut-get-object-or-404

[yayaya22]

ありがとうございます！エラーが出てしまいました。

Page not found (404)
Request Method: GET
Request URL: http://127.0.0.1:8000/users/2/edit/
Raised by: subscriptions.views.UsersEdit

No Post_Detail matches the given query.

[__horito]

DBに指定のデータがないと404を返すような関数がget_object_or_404ですので
存在するデータを指定してみてください
以下参考
https://docs.djangoproject.com/ja/3.2/intro/tutorial03/#a-shortcut-get-object-or-404

[yayaya22]

ありがとうございます、
DBの存在は、
data = Post_Detail.objects.filter(author=user_id)
print(data)

としましたら
<QuerySet [<Post_Detail: Post_Detail object (41)>]>
とかえってきましたが、
この行に飛んでしまっているようです。
else:
params['form'] = UserForm()
return render(request, 'create.html',params )

[__horito]

リクエストメソッドがpostであるか確認してみてください

[yayaya22]

はい、postです。

if request.method == 'POST':
form = UserForm(request.POST,instance=data)
if form.is_valid():
instance = form.save(commit=False)
print(instance)
instance.author = request.user
instance.save()
return redirect('edit.html')

[__horito]

申し訳ありません。処理を考えてみましたがなぜそこに飛ぶのかは理解できませんでした。
if文のインデントとかは実際のでコード通りなのでしょうか？いま掲示させているのもだとインデントエラーが起きて再現できませんでした。