modelの更新がうまくできない

Question

#やっていること
現在、在庫管理アプリを個人的に作成しています。

#問題
primarykeyを設定しているコード以外のものをAdminサイトより更新することは出来ますが、
アプリより更新すると、PrimaryKeyと同じ値が既に存在しますと言われてしまいます。

Forms.pyにてPrimaryKeyの編集が出来ないよう、”readonly”の設定をしているため触れないのですが、
なぜか追加？という形で更新がかかってしまっているようです。

Primarykeyを外してみると普通に更新は行えており、更新したデータが2重で登録されるなどの事象はおきていません。
※PrimaryKeyを設定しているのはkamoku_codeです

#ソース

from django.core.files.base import endswith_cr
from django.http.response import Http404, HttpResponse
from django.shortcuts import redirect, render, get_object_or_404
from django.http import HttpResponse
from .forms import KamokuForm, Kamoku_editForm
from .models import Kamoku

# Create your views here.

def main(request):
    return render(request, 'main.html')


def kamoku(request):
    kamokuForm = KamokuForm(request.POST)
    if kamokuForm.is_valid():
        Kamoku.objects.create(**kamokuForm.cleaned_data)
        return redirect('kamoku')
    context = {
        'kamokuForm': kamokuForm,
        'kamokus': Kamoku.objects.all(),
    }
    return render(request, 'kamoku.html', context)


def kamoku_edit(request, id):
    kamoku = get_object_or_404(Kamoku, pk=id)
    kamokuForm = Kamoku_editForm(instance=kamoku)
    if request.method == 'POST':
        kamokuForm = Kamoku_editForm(data=request.POST)
        if kamokuForm.is_valid():
            kamoku.kamoku_name = kamokuForm.cleaned_data['kamoku_name']
            kamokuForm.save()
            return redirect('kamoku')
    
    context = {
        'kamokus': Kamoku.objects.all(),
        'kamokuForm': kamokuForm,
    }
    return render(request, 'kamoku_edit.html', context)

from django import forms
from django.contrib.admin.widgets import AdminDateWidget
from .models import Kamoku

class KamokuForm(forms.ModelForm):
    class Meta:
        model = Kamoku
        fields = ('kamoku_code', 'kamoku_name')
        
class Kamoku_editForm(forms.ModelForm):
    kamoku_code = forms.IntegerField(
        widget=forms.TextInput(
            attrs={
                'readonly': 'disabled'
            }
        ),
    )
    class Meta:
        model = Kamoku
        fields = ('kamoku_code', 'kamoku_name')

from django.db import models

class Kamoku(models.Model):
    kamoku_code = models.IntegerField(blank=False, null=False, primary_key=True, verbose_name="科目コード")
    kamoku_name = models.CharField(max_length=255,verbose_name="科目名称")
    created_at = models.DateTimeField(auto_now_add=True)
    updated_at = models.DateTimeField(auto_now=True)
    
    def __str__(self):
        return self.kamoku_name

#実際の画面

#最後に
更新処理は正しく動いているように思いますが、もしかするとそこがおかしいのかもしれません。
PrimaryKeyを外した場合は更新されることを確認しています。

PrimaryKeyをモデルに設定したことがないため、このような動きが正常なのか、自分の更新処理がおかしいのか判断がつきません。

わかる方が見えましたらご教授ください。

#追記
乗せた画像がものすごくわかりずらいですが、左側の一覧より100で始まるレコードを選択し、更新処理内でPKを指定しているため、右側の画面に情報が出ています。
科目名称という部分のみを変更して更新ボタンを押すと画像のようになります。

Answer 1

if request.method == 'POST':
        kamokuForm = Kamoku_editForm(data=request.POST)
```↓
```Python
if request.method == 'POST':
        kamokuForm = Kamoku_editForm(request.POST, instance=kamoku)

Comments

[setuna0531]

ForestSeo様
ご回答ありがとうございます！

確認したところ無事動かすことが出来ました！
とても助かりました！

また、今回の回答の質問なのですが、POSTで受け取ったときにデータを渡していないから追加扱いになっていた感じでしょうか？

[ForestSeo]

そうですね。
POST送信時もformにinstanceを渡さないとダメです

[setuna0531]

なるほどですね・・・！
IFに入る前で呼び出してるからいけるだろうと思ってました(;^ω^)

ありがとうございました！

Answer 2

if kamokuForm.is_valid():
kamoku.kamoku_name =　kamokuForm.cleaned_data['kamoku_name']
kamokuForm.save()
return redirect('kamoku')

formをセーブするのではなく、モデルインスタンスをセーブすれば良いのではないでしょうか。

追記
言葉足らずでした。フォームをセーブしてしまうと新しいレコードが作成されてしまうと思います。だからモデルを更新したいのならモデルインスタンスにフォームの値を入れてモデルインスタンスをセーブすれば良いと思います。

def kamoku_edit(request, id):
    kamoku = get_object_or_404(Kamoku, pk=id) #これを変更したいなら
    kamokuForm = Kamoku_editForm(instance=kamoku)
    if request.method == 'POST':
        kamokuForm = Kamoku_editForm(data=request.POST)
        if kamokuForm.is_valid():
            kamoku.kamoku_name = kamokuForm.cleaned_data['kamoku_name']
            kamokuForm.save()                 #kamoku.save()とする
            return redirect('kamoku')

Comments

[setuna0531]

yaha4967さん
コメントありがとうございます。

＞formをセーブするのではなく、モデルインスタンスをセーブすれば良いのではないでしょうか。
→formでPrimaryKeyがついているデータを更新することは不可能という意味でしょうか？
　できればFormを利用して保存等をしたいというのはあります。

[setuna0531]

yaha4967さん

追記ありがとうございます！
Formをsaveするのではなく、モデルインスタンスをsaveしたのですが、PrimaryKeyが存在していますと出てしまいます。

[yaha4967]

そうですか。モデルを新しく作ったから「PrimaryKeyが存在しています」とでてるのだと思うのですが。。。
最後にsave(force_update=True)を試してみてください。saveにはインサートとアップデートがあるようでforce_update=Trueはアップデートを強制します。

これでできなかったらごめんない。私にはちょっとわからないです。

公式ドキュメントは役立つと思うので他に回答が付かず行き詰まるようでしたら是非みてみてください。
https://docs.djangoproject.com/ja/3.1/ref/models/instances/

[setuna0531]

yaha4967さん

お調べしていただきありがとうございます！
確かにモデルを新しく同じコードで作成すると「PrimaryKeyが存在しています」と出ることは確認できています。
yaha4967さんに調べていただいたsave(force_update=True)を試しましたが、Primaryが存在しますとやはり出てしまいました・・・。

公式ドキュメントを一度あさってみます。
お時間使わせてしまいすみませんでした。