web-dev-qa-db-ja.com

Django --ModelFormの作成または更新？

計画に関するセッションの作成または追加を可能にするフォームが必要です

モデル

class Session(models.Model):
    tutor = models.ForeignKey(User)
    start_time = models.DateTimeField()
    end_time = models.DateTimeField()

形

class SessionForm(forms.ModelForm):
    class Meta:
        model = Session
        exclude = ['tutor']

フォームをレンダリングするためのビュー

def editor(request):
    if request.method == 'GET':
        if request.GET['id'] != '0':
            # The user has selected a session
            session = Session.objects.get(id=request.GET['id'])
            form = SessionForm(instance=session)
        else:
            # The user wants to add a new session
            form = SessionForm()
        return render_to_response('planner/editor.html',
            {'form': form,}, context_instance=RequestContext(request),)

テンプレートeditor.html

<form action="/planner/post" method="post">{% csrf_token %}
{{ form.as_p }}
</form>

値を投稿するために表示

def post(request):
    if request.method == 'POST':
        form = SessionForm(request.POST)
        if form.is_valid():
            form.instance.tutor = request.user
            form.save()
            obj = {'posted': True}
            return HttpResponse(json.dumps(obj), mimetype='application/json')
        else:
            return render_to_response('planner/editor.html',
                form, context_instance=RequestContext(request),)

問題

セッションは常に作成されます（更新されません）

質問

  • 私の見解では、postセッションを更新する必要があり、作成する必要がないことをどのように知ることができますか？
  • このコードを単純化する方法はありますか？
djangodjango-modelsdjango-formsdjango-views
12
Pierre de LESPINAY

セッションを更新する場合は、フォームをバインドするときにインスタンスを提供する必要があります。

フォームが有効な場合は、 save with commit=False、そして家庭教師を更新します。

form = SessionForm(instance=instance, data=request.POST)
if form.is_valid():
    instance = form.save(commit=False)
    instance.tutor = request.user
    instance.save()
17
Alasdair
from Django.shortcuts import render_to_response, get_object_or_404
from Django.http import HttpResponseRedirect, Http404
from Django.template import RequestContext
from application.models import Session
from application.forms import SessionForm

def allInOneView(request):
    session_id = request.POST.get('session_id')

    if session_id:
        session = get_object_or_404(Session, pk=session_id)
    else:
        session = None

    """
    A subclass of ModelForm can accept an existing model instance 
    as the keyword argument instance; 
    if this is supplied, save() will update that instance. 
    If it's not supplied, save() will create a new instance of the specified model.
    """
    form = SessionForm(instance=session)

    if request.method == 'POST':
        form = SessionForm(request.POST, instance=session)
        if form.is_valid():
            form.save()
            return HttpResponseRedirect(request.path)

    return render_to_response('planner/editor.html', {
        'form': form
    }, context_instance=RequestContext(request))
1
niko.makela

私が今（ここで述べたアドバイスに従って）通常行うことは、オプションのsession_id（作成用のsession_idなし）をURLディスパッチャーに渡す1つのビューのみを使用することです。

<form action="{% url session_edit session_id=session_id|default_if_none:"" %}"
    method="post">{% csrf_token %}
{{ form.as_p }}
</form>

url('^planner/edit$', session_edit, name='session_edit'),
url('^planner/edit/(?P<session_id>\d+)$', session_edit, name='session_edit'),

4つのケースすべてを再グループ化すると

  • 作成フォームを取得する
  • 更新フォームを入手する
  • 投稿作成フォーム
  • 更新後のフォーム

1つのビューにまとめると、はるかに保守しやすくなります。

1
Pierre de LESPINAY

すべてを1つのビューで実行します。何かのようなもの：

def session_manager(request):

    session = None
    try:
        session = Session.objects.get(id=request.POST['id'])
    except Session.DoesNotExist:
        pass

    if request.method == "POST":
       kwargs = {
            data = request.POST
       }
       if session:
            # Update
            kwargs['instance'] session 
       form = SessionForm(**kwargs)
       if form.is_valid():
            ...
    else:
        form = SessionForm(instance=session)
0
Timmy O'Mahony