我是Django的新手,但我似乎在另一个网站上有相同的代码。我可以更新Django shell中的记录,但在view.py中,相同的代码在我运行此表单时坚持要插入新记录。
所以view.py的相关部分是这样的:
if request.method == 'POST':
incidentId = request.POST['id']
editedEvent = DisciplineEvent.objects.get(pk=int(incidentId))
form = DisciplineEventEntryForm(request.POST, instance=editedEvent)
form.save()
variables = Context({
'account': account,
'date': request.POST['event_date'],
'description': request.POST['incident_description'],
'incident_id':incidentId,
})
template = get_template('disciplineform_confirm_entry.html')
output = template.render(variables)
response = HttpResponse(output)
return response
我认为这会拉出有问题的记录,将新的表单数据保存到其中,并更新记录。相反,它会创建一个包含所有数据和增量主键的新记录。
答案 0 :(得分:9)
您要做的是非传统的和可能的安全漏洞。
您不应该从您在表单中填充的隐藏ID键中获取对象的实例。用户可以轻松更改此代码,并让您的代码覆盖他们甚至可能无权访问的其他模型实例。
执行此操作的标准方法是根据URL获取对象。
def view_function(request,id):
object_to_edit = get_object_or_404(Model,id=id) #Or slug=slug
form = ModelForm(data = request.POST or None, instance=object_to_edit)
if form.is_valid():
form.save()
redirect()
return render_to_response('template_name',{},RequestContext(request))
希望它有所帮助!