Как удалить несколько объектов в отношениях ManyToMany на основе фильтра?

Учитывая эти две модели:

class Item(models.Model):
    timestamp = models.DateTimeField()

class Source(models.Model):
    items = models.ManyToManyField(Item, related_name="sources")

Я могу найти все исходные элементы до заданного времени, используя это:

source.items.filter(timestamp__lte=some_datetime)

Как эффективно удалить все элементы, соответствующие этому запросу? Полагаю, я мог бы попробовать что-то вроде этого:

items_to_remove = list(source.items.filter(timestamp__lte=some_datetime))
source.items.remove(*items_to_remove)

но это кажется плохим.

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

Ответы

Ответ 1

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

source.items.remove(*source.items.filter(*args))

Метод remove/add выглядит следующим образом

remove(self, *objs)
add(self, *objs)

и docs http://www.djangoproject.com/documentation/models/many_to_many/ используйте несколько примеров в форме [p1, p2, p3], поэтому я бы сделал то же самое для remove, видя, что аргументы одинаковы.

>>> a2.publications.add(p1, p2, p3)

Копаясь немного больше, функция remove выполняет итерацию по *objs один за другим, проверяя, действительно ли она действительной модели, в противном случае используя значения как PK, а затем удаляет элементы с помощью pk__in, поэтому я скажем, да, лучший способ - сначала запросить таблицу m2m для удаления объектов, а затем передать эти объекты в менеджер m2m.

    # django.db.models.related.py
    def _remove_items(self, source_field_name, target_field_name, *objs):
        # source_col_name: the PK colname in join_table for the source object
        # target_col_name: the PK colname in join_table for the target object
        # *objs - objects to remove

        # If there aren't any objects, there is nothing to do.
        if objs:
            # Check that all the objects are of the right type
            old_ids = set()
            for obj in objs:
                if isinstance(obj, self.model):
                    old_ids.add(obj.pk)
                else:
                    old_ids.add(obj)
            if self.reverse or source_field_name == self.source_field_name:
                # Don't send the signal when we are deleting the
                # duplicate data row for symmetrical reverse entries.
                signals.m2m_changed.send(sender=rel.through, action="pre_remove",
                    instance=self.instance, reverse=self.reverse,
                    model=self.model, pk_set=old_ids)
            # Remove the specified objects from the join table
            db = router.db_for_write(self.through.__class__, instance=self.instance)
            self.through._default_manager.using(db).filter(**{
                source_field_name: self._pk_val,
                '%s__in' % target_field_name: old_ids
            }).delete()
            if self.reverse or source_field_name == self.source_field_name:
                # Don't send the signal when we are deleting the
                # duplicate data row for symmetrical reverse entries.
                signals.m2m_changed.send(sender=rel.through, action="post_remove",
                    instance=self.instance, reverse=self.reverse,
                    model=self.model, pk_set=old_ids)