从Django中的视图中删除对象

时间:2018-05-13 17:45:52

标签: django django-forms django-urls

我有一个显示对象列表的页面,我想在每个对象旁边添加一个按钮,以便用户删除它。我在网上找到了一些关于类似场景的问题但由于某些原因我在尝试复制解决方案时遇到了错误。

这是views.py中的删除功能:

def delete_dish(request, pk):
    query = Dish.objects.get_object_or_404(pk)
    supermenu = query['supermenu'].pk
    query.delete()
    return redirect('foodmenu:menu', supermenu)

以下是HTML模板中的表单:

            {% if not supermenu.dish_set.count == 0 %}
            <ul>
            {% for dish in supermenu.dish_set.all %}


                <li>
                {{ dish.name }} - {{ dish.description }}
                <form action="{%  url 'foodmenu:delete_dish' dish.id %}" method="POST">
                    {% csrf_token %}
                    <button type="submit">X</button>
                </form>

                </li>


                    {% if not dish.price_set.count == 0 %}
                    <ul>
                    {% for price in dish.price_set.all %}
                        {% if price.label %}
                            <li>{{ price.label }}: {{ price.cost }}</li>
                        {% else %}
                            <li>{{ price.cost }}</li>
                        {% endif %}
                    {% endfor %}
                    </ul>
                    {% endif %}


                {% endfor %}
            </ul>

            {% else %}
            <p>No dishes on this menu!</p>

        {% endif %}

这是urls.py:

app_name = 'foodmenu'
urlpatterns = [
    ...
    path('dish/delete/<int:dish.id>', views.delete_dish, name="delete_dish")
]

当我单击按钮时,浏览器转到./dish/delete/1(1是对象的pk),但Django返回404错误。

1 个答案:

答案 0 :(得分:2)

而不是query = Dish.objects.get_object_or_404(pk)您应该使用get_object_or_404快捷方式:

from django.shortcuts import get_object_or_404

from django.views.decorators.http import require_POST

@require_POST
def delete_dish(request, pk):
    if request.method
    query = get_object_or_404(Dish, pk=pk)
    supermenu = query.supermenu.pk
    query.delete()
    return redirect('foodmenu:menu', supermenu)

同时将您的网址格式更改为:

path('dish/delete/<int:pk>/', views.delete_dish, name="delete_dish")

<强> UPD

正如@daniherrera在评论中提到的,您可能想要检查请求的方法,以防止意外删除。为此,您可以使用require_POST装饰器。