使用ModelForm编辑模型实例会创建新实例而不是更新

时间:2011-05-28 04:42:47

标签: django django-models django-forms

我正在尝试创建一个允许用户编辑模型实例的视图(在这种情况下是一个事件)。不幸的是,提交此表单会创建一个新实例(带有新ID),甚至不会删除旧实例。我的印象是,在这种情况下,save方法应该更新实例...

注意:EventForm是ModelForm

我已尝试使用每个https://docs.djangoproject.com/en/dev/ref/models/instances/#forcing-an-insert-or-update的force_update arg,但没有骰子。我还尝试删除form.is_valid()块中的原始事件(通过调用event.delete()),但是......没有骰子。

我觉得commit=False是个问题?我不确定!

感谢。

(请忽略代码段中的间距问题)

def edit_event(request, event_id):
 event = Event.objects.get(pk=event_id)  

 if request.method == 'POST':
     post_data = request.POST.copy()

     # here is some validation that can't be done in the ModelForm...

     #form = EventForm(post_data, request.FILES, instance=event)
     form = EventForm(post_data, request.FILES)

     if form.is_valid():

         edited_event = form.save(commit=False)

         edited_event.save(force_update=True) # doesn't work with or without force_update arg

         #form.save_m2m() # needed for ManyToMany relationship

         return HttpResponseRedirect('/events/view/%s' % edited_event.id)
  else:
      form = EventForm(instance=event)

return render_to_response('create_event.html', {'form': form,}, context_instance=RequestContext(request) )

更新

我摆脱了模型上的M2M关系,所以我能够摆脱form.save_m2m()线。那仍然行不通。

我还尝试在提交表单时不传递实例,假设在用户提交时将预先填充正确的字段(现在就是这种情况)。这仍然无效。

在更新模型时,我是否遗漏了一个重要的细节?

4 个答案:

答案 0 :(得分:5)

我认为你应该像这样初始化表单(因为你已经注释掉了)。

form = EventForm(post_data, request.FILES, instance=event)

然后事件对象与表单相关联,form.save()可以执行更新而不是插入。

答案 1 :(得分:2)

根据我的理解,您的代码的问题在于您始终在保存之前创建新实例。既然你已经在“event = Event.objects.get(pk = event_id)”中获得了事件的实例,你可以简单地改变这个实例并保存它,我很确定这会导致django制作一个更新

我建议你查看这个链接: https://docs.djangoproject.com/en/dev/ref/models/instances/?from=olddocs#how-django-knows-to-update-vs-insert

它对Django用于区分插入和更新的算法有一个非常简单的解释。如果你的对象没有它的主键集(可能是你在syncdb时django自动创建的“id”字段),那么它将被视为一个新对象。如果它有该字段集,那么Django将尝试在数据库中找到该特定记录,如果它存在,它将被更新。

希望答案很清楚。

答案 2 :(得分:0)

个人资料显示和更新工作代码:

@login_required(login_url=LOGIN_URL)
@csrf_exempt
def profile(request):
    """
    :param request:
    :param:
    :return:
    """
    val = get_object_or_404(Profil, user=request.user)

if request.method == 'POST':
    form = ProfilForm(request.POST, request.FILES, instance=val)
    if form.is_valid():
        form.save()
        msg_ok = u'Profil updated'
    else:
        msg_err = u"Invalid form."
else:
    form = ProfilForm(instance=val)

return render(request, "back/profile.html", locals())

答案 3 :(得分:-3)

你的例子太复杂了。尝试从简单的模型/视图/表单开始。使用一个简单字段创建新模型,然后创建模型表单并查看并确保它们有效。然后开始修改视图,使其更接近实际状态。