在Django中刪除對象

Question

在一個迷你博客應用程序中，我想創建一個刪除函數，以便博客的所有者可以刪除他的條目（僅限他的條目）。 我想做的唯一方法就是使用表單。 雖然我的刪除代碼似乎清晰和正確，但它不起作用。 我的代碼：

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)) 

，並在模板：

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

這是一個正確的做法？我的意思是，爲此創建一個表單？ 也是，只有將刪除鏈接關聯的博客文章的ID作爲參數。這樣對嗎？我的意思是，也許任何用戶可以在url中輸入另一個ID，並刪除另一個條目（最終不是他的）

Answer 1

通常，對於刪除對象，您應該使用POST（或DELETE）HTTP methods。

如果你真的想使用HTTP GET爲你的例子，這裏是你需要解決什麼：

如果URL指向像你這樣的一些網址：<a href='/news/delete_new/{{object.id}}/'> Delete</a>那麼你可以簡單地寫視圖，將檢查對象屬於在用戶登錄，如果是刪除該條目，就像在代碼中你已經寫好：

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

要檢查是否新對象belogs你需要User和New（之間創建realation像created_by = models.ForeignKey(User)在一些用戶New模型）。

可以得到登錄的用戶是這樣的：request.user

我希望我正確地得到了你點我的回答可以幫助你以某種方式。

PS：您也可以考慮使用{% url %}標籤，而不是直接在您的模板中編寫url。

Answer 2

您需要使用表格，否則您很容易受到CSRF attacks的影響。在檢查請求是GET還是POST之前，您還要刪除模型。

創建一個簡單的ModelForm：

from django import forms 

from .models import New 

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

在你views.py在同一個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)