Телеграмм чат группы pydjango страница 12255

А если тебе надо удялять именно файлы, используй django-cleanup

15:33пожаловаться #1

NIkita Gromov in Django [ru] #STAY HOME

Можно ли в drf не отправлять body?

Ситуация такая:
есть путь album/{album_pk}/track/{pk}/like/ он принимает либо POST, для установки лайка, либо DELETE для убирания лайка, создание пустого сериализатора выглядит как костыль, так что как это лучше реализовать?

15:33пожаловаться #2

А если тебе надо удялять именно файлы, используй django-cleanup

Нет. Мне нужно удалять объекты модели

15:34пожаловаться #3

А если тебе надо удялять именно файлы, используй django-cleanup

Энивей, я уже сделал на сигналах. Спасибо, кто учасвствовал в дискуссии

15:34пожаловаться #4

🔐

Grigory Surkov

Нет. Мне нужно удалять объекты модели

А как они связаны, не проще использовать on_delete=CASCADE?

15:34пожаловаться #5

А как они связаны, не проще использовать on_delete=CASCADE?

Через ManyToMany невозможно поставить on_delete. Таким образом было бы совсем просто

15:35пожаловаться #6

🔐

Grigory Surkov

Через ManyToMany невозможно поставить on_delete. Таким образом было бы совсем просто

Ну тогда лучше delete переопределять

15:35пожаловаться #7

🔐

Сигналы часто используют там где не надо, усложняя логику

15:36пожаловаться #8

Ну тогда лучше delete переопределять

😕 Оу, мне не стоило тратить на это время. Тогда хоть дай совет, когда лучше следует употреблять сигналы

15:37пожаловаться #9

🔐

Grigory Surkov

😕 Оу, мне не стоило тратить на это время. Тогда хоть дай совет, когда лучше следует употреблять сигналы

Ну, например как django-cleanup делает, чекает и удаляет файлы из хранилища перед удалением модели с файловыми полями

15:38пожаловаться #10

Ну тогда лучше delete переопределять

15:39пожаловаться #11

🔐

ilya

А ему надо удалять кверисеты?)

15:41пожаловаться #12

А ему надо удалять кверисеты?)

думаешь у него одиночный объект?)

15:42пожаловаться #13

🔐

ilya

думаешь у него одиночный объект?)

Он же сказал что m2m, я редко видел чтоб была нужда чистить кверисет с m2m, так что скорее всего да

15:44пожаловаться #14

Он же сказал что m2m, я редко видел чтоб была нужда чистить кверисет с m2m, так что скорее всего да

очень сомневаюсь, ну да ладно. Суть в том что сигналы подойдут для обоих случаев, переопределение delete() для модели - только для удаления одиночных объектов

15:45пожаловаться #15

🔐

ilya

Но если ему не надо, он все усложнит

15:45пожаловаться #16

🔐

Там по контексту надо смотреть

15:45пожаловаться #17

ilya

def delete(self, *args, **kwargs):
photos = Photo.objects.filter(ad_ident=self.ident)
for photo in photos:
photo.delete()
super(Ad, self).delete(*args, **kwargs)

15:46пожаловаться #18

NIkita Gromov in Django [ru] #STAY HOME

Я думаю что сигналы в любом случае более верный подход, хотя бы из-за таких тонкостей.

15:46пожаловаться #19

Но если ему не надо, он все усложнит

что там можно усложнить одиночным описанием вызова сигнала в несколько строчек...