Дополнительная функция, написанная для класса DeleteView, не работает

#django #django-models #django-views #django-urls

Вопрос:

Я добавил функцию, которая, если сообщение удалено пользователем, оно будет удалено, если есть загруженная фотография, но функция не работает.

мой взгляд:

 class NewsDeleteView(LoginRequiredMixin, UserPassesTestMixin, DeleteView):
    model = News
    template_name = 'news/news_delete.html'
    success_url = reverse_lazy('news_list')

    def delete_header_image(self, pk):
        news = get_object_or_404(News, id = pk)
        header_image = news.header_image
        if header_image is not None:
            os.remove(str(header_image))
            return HttpResponseRedirect(reverse('news_list'))
        else:
            return HttpResponseRedirect(reverse('news_list'))

    def test_func(self):
        obj = self.get_object()
        if self.request.user.has_perm('news.all') or self.request.user.has_perm('news.delete_news') or obj.author == self.request.user:
            return True

url-адреса:

 urlpatterns = [
    path('<int:pk>/delete', NewsDeleteView.as_view(), name='news_delete'),
]

Модели:

 def get_header_image_filepath(self, filepath):
    return f'images/news/header/{self.author.id}/{self.header_image}'


class News(models.Model):
    title = models.CharField(max_length=255)
    header_image = models.ImageField(null=True, blank=True, upload_to=get_header_image_filepath)
    body = RichTextUploadingField()
    datetime = models.DateTimeField(auto_now_add=True)
    author = models.ForeignKey(
        AUTH_USER_MODEL,
        on_delete=models.CASCADE,
    )
    def __str__(self):
        return self.title

    def get_absolute_url(self):
        return reverse("news_detail", args=[str(self.id)])

1. Ты об этом говоришь delete_header_image ? Как вы думаете, почему это будет называться?

2. Также неясно, хотите ли вы просто удалить изображение или хотите удалить News экземпляр, пожалуйста, уточните.

3. Я хочу удалить экземпляр, но когда он будет удален, загруженное изображение не будет удалено, поэтому я написал эту функцию(delete_header_image), но она не работает.

Ответ №1:

Представления, основанные на классах, не работают волшебным образом. У них уже написаны некоторые методы, которые вызываются для выполнения действий. Следовательно, то, что вы пишете метод delete_header_image , не означает, что он будет вызван автоматически. Вместо этого вы должны переопределить какой-либо подходящий метод класса, который будет вызываться внутренне. Для DeleteView метода, который выполняет удаление delete , и, следовательно, вы должны переопределить его. Также условие if header_image is not None не будет работать, так как даже если файла нет, он не будет представлен, None вместо этого вы должны просто написать if header_image для проверки. Кроме того, вместо удаления вручную просто вызовите ieldFile.delete [Django Docs]:

 class NewsDeleteView(LoginRequiredMixin, UserPassesTestMixin, DeleteView):
    model = News
    template_name = 'news/news_delete.html'
    success_url = reverse_lazy('news_list')

    def delete(self, request, *args, **kwargs):
        object = self.get_object()
        if object.header_image:
            object.header_image.delete(save=False)
        return super().delete(request, *args, **kwargs)

    def test_func(self):
        obj = self.get_object()
        if self.request.user.has_perm('news.all') or self.request.user.has_perm('news.delete_news') or obj.author == self.request.user:
            return True