如何更新模型modelforms和Django

Question

i have a problem when i try update the data for model usuario

this is my model

class Usuario(models.Model):
 user = models.ForeignKey(User)
 nombres = models.CharField(max_length = 50)
 correo = models.EmailField()


 class Meta:
    db_table = u'utp_users'


 def __str__(self):
    return str(self.nombres.encode('utf-8')  )

this is mi form. py

class UsuarioForm(forms.ModelForm):

 class Meta:
     model = Usuario
     fields = ['nombres','correo']

my url

url(r'^menu/update/(?P<usuario_id>\d+)$', 'utpapp.views.update'),

this my view update

@login_required(login_url='/')
def update(request,usuario_id):



 form = UsuarioForm(request.POST)

 if form.is_valid():

     user = Usuario.objects.get(pk=usuario_id)
     form = UsuarioForm(request.POST, instance = user)
     form.save()
     return redirect('/menu/')
 else:
     user = Usuario.objects.get(pk = usuario_id)       
     form = UsuarioForm(instance=user)

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

and the template i am do this

<a title="Editar informacion" href="/menu/update/{{usuario.id}}"><img src="/media/imagenes/usuario.png" class=" col-xs-3 col-md-7 quitar-float" alt="Editar informacion" ></a>

the problem is when i select the option update i get the this msj "Page not found (404)"

but i change in template href {{usuario.id}} for {{user.id}} is work but with user different what is the problem ??

Answer 1

您忘记将usuario变量传递给模板。

Answer 2

Try this, you need to pass usuario as context to the template, and you can much simplify the code (especially the form for GET/POST, they are basically the same form, so don't repeat it) as like:

from django.shortcuts import get_object_or_404

@login_required(login_url='/')
def update(request,usuario_id):
    user = get_object_or_404(Usuario, pk=usuario_id)
    form = UsuarioForm(request.POST or None, instance=user)

    if request.method == 'POST' and form.is_valid():
       form.save()
       return redirect('/menu/')
    # you need to pass `usuario` as part of the context to template
    # so you can access usuario.id in template
    return render_to_response('form.html',{ 'form':form, 'usuario': user }, context_instance=RequestContext(request))

Also make sure you have a url route for /menu/ , or else redirect will get you 404 .