Eliminación de objetos en Django

Question

En una aplicación de mini blog, quiero crear una función de eliminación para que el propietario del blog pueda eliminar sus entradas (y solo sus entradas). Supongo que los únicos métodos para hacerlo, es usar un formulario. Aunque mi código de borrado parece claro y correcto, no funciona. Mi código:

def delete_new(request,id): 
    u = New.objects.get(pk=id).delete() 
    if request.method == 'POST': 
     form = DeleteNewForm(request.POST)  
     form.u.delete()    
     form.save() 
    return render_to_response('news/deleteNew.html', { 
      'form': form, 
      }, 
     context_instance=RequestContext(request)) 

y en la plantilla:

<a href='/news/delete_new/{{object.id}}/'> Delete</a> <br /> 

Es este un enfoque correcto? Quiero decir, ¿crear una forma para esto? también, la única forma de tomar la publicación de blog asociada con el enlace de eliminación es tener una identificación como parámetro. ¿Es correcto? Quiero decir, tal vez cualquier usuario puede escribir otra identificación, en la url, y eliminar otra entrada (eventualmente no una de las suyas)

Answer 1

En general, para eliminar objetos, debe usar POST (o ELIMINAR) HTTP methods.

Si realmente desea utilizar HTTP GET para el ejemplo, esto es lo que debe arreglar:

Si tiene URL que apunta a cierta URL como la suya: <a href='/news/delete_new/{{object.id}}/'> Delete</a> entonces simplemente puede escribir vista que va a comprobar si objeto pertenece al usuario conectado y eliminar esta entrada si es así, como en el código que ya ha escrito:

def delete_new(request,id): 
    #+some code to check if New belongs to logged in user 
    u = New.objects.get(pk=id).delete() 

para comprobar si los nuevos objetos belogs a algún usuario que necesita para crear realation entre User y New (como created_by = models.ForeignKey(User) en Modelo New).

Usted puede obtener usuario conectado de esta manera: request.user

espero que me dieron su punto de forma correcta y mi respuesta le ayuda de alguna manera.

PD: También puede considerar usar la etiqueta {% url %} en lugar de escribir urls directamente en sus plantillas.

Answer 2

Es necesario utilizar un formulario, o que son vulnerables a CSRF attacks. También está eliminando el modelo antes de verificar si la solicitud fue GET o POST.

Crear un simple ModelForm:

from django import forms 

from .models import New 

class DeleteNewForm(forms.ModelForm): 
    class Meta: 
     model = New 
     fields = [] 

En su views.py en la misma aplicación de Django:

from django.shortcuts import render, get_object_or_404 

from .forms import DeleteNewForm 
from .models import New 

def delete_new(request, new_id): 
    new_to_delete = get_object_or_404(New, id=new_id) 
    #+some code to check if this object belongs to the logged in user 

    if request.method == 'POST': 
     form = DeleteNewForm(request.POST, instance=new_to_delete) 

     if form.is_valid(): # checks CSRF 
      new_to_delete.delete() 
      return HttpResponseRedirect("/") # wherever to go after deleting 

    else: 
     form = DeleteNewForm(instance=new_to_delete) 

    template_vars = {'form': form} 
    return render(request, 'news/deleteNew.html', template_vars)