0

Я пытаюсь запретить пользователям удалять посты других пользователей. То есть Вася Пупкин не может удалять посты Ирины Ивановой по URL. Для этого я пишу данный код. Но он выдает ошибку AttributeError 'tuple' object has no attribute 'owner'

views.py

@login_required
def deleteAll(request):
    todos = Todo.objects.all().delete()

    if todos.owner != request.user:
        raise Http404

    return redirect('list:index')

models.py

from django.contrib.auth.models import User
from django.db import models

class Todo(models.Model):
    body = models.TextField()
    created_at = models.DateTimeField(auto_now_add=True)
    complete = models.BooleanField(default=False)
    owner = models.ForeignKey(User, on_delete=models.CASCADE, default='1')

    def __str__(self):
        return self.body

Интересно то что raise Http404 работает у меня в коде ниже

@login_required
def update(request, pk):
   todo = Todo.objects.get(id=pk)

   if todo.owner != request.user:
      raise Http404

И при переходе update/20 пользователем которому не принадлежит данный пост, он выдает ошибку 404 (чего я и добивался), но с методом deleteAll такая фишка не проходит и любой желающий может стереть все посты, всех пользователей, при переходе по URL, а я хочу чтобы каждый мог удалять только свои посты и не удаляя чужие. В чем моя ошибка подскажите ? Полная ошибка выглядит так введите сюда описание изображения

2
  • Полный текст ошибки добавьте в вопрос. 4 янв 2019 в 12:01
  • @Sergey Gornostaev, добавил
    – user290664
    4 янв 2019 в 12:34

1 ответ 1

1

Во-первых, зачем все эти манипуляции, если можно просто выбирать записи только определённого владельца?

@login_required
def delete_all(request):
    Todo.objects.filter(owner=request.user).delete()
    return redirect('list:index')

Во-вторых, нужно сначала проверить владельца, а уж потом удалять. Наконец, метод delete() возвращает кортеж хранящий информацию о том, сколько записей удалено. Естественно, в нём нет поля owner.

1
  • спасибо большое, помогло!
    – user290664
    4 янв 2019 в 14:32

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки