Django: изменение ManyToManyField в методе save()?

Question

[meteozond]

При созданении модели нужно проанализировать содержимое полей и внести изменения в ManyToManyField.



Если save вызывается напрямую (во view или из другой модели) — никаких проблем. Но при вызове из admin — результат работы save заменятся результатом обработки form.



Подозреваю, что редактирование ManyToManyField в .save() не соотвествует концепции django. Посоветуйте как это корректно реализовать.

Answer 1

[savados]

Нет никакого «результата обработки form», любая ModelForm всегда вызывает метод .save модели. И неважно, в админке она, или еще где.

Проблема в другом, и тут не помешал бы код метода .save.

Похожая проблема может быть при оверрайде save в ModelForm, там нужно явно вызывать метод .save_m2m, но это пальцем в небо, нужно увидеть код.

Не вижу, как редактирование ManyToManyField в .save противоречит концепции Django, по-моему, совершенно нормальное дело (хоть и малость странное).

Comments

[meteozond]

в коде ничего сверхественного:
1. Парсим строчку введенну пользователем
2. Находим записи соотвествующие найденному
3. Добавляем их в связь
4. Сохраняем
* Это пример для изменения, при добавлении, естественно, выдаст ошибку.
def save(self, force_insert=False, force_update=False, using=None:
matches = SYMBOL_PARSER.finditer(self.body)
symbols = [match.groups()[0] for match in matches]
self.indicators.clear()
for symbol in Indicator.objects.filter(symbol__in=symbols):
self.indicators.add(symbol)
super(Formula, self).save(force_insert=force_insert, force_update=force_update, using=using)


[meteozond]

def save(self, force_insert=False, force_update=False, using=None:
    matches = SYMBOL_PARSER.finditer(self.body)
    symbols = [match.groups()[0] for match in matches]
    self.indicators.clear()
    for symbol in Indicator.objects.filter(symbol__in=symbols):
        self.indicators.add(symbol)
    super(Formula, self).save(force_insert=force_insert, force_update=force_update, using=using)

[meteozond]

при изменении из админки метод отрабатывает, но потом его результат затирается

[savados]

1. Надо, видимо, сначала вызвать родительский метод, а потом менять m2m. А у вас наоборот.
2. Можно (если у вас не используется through) записать это

self.indicators.clear()
for symbol in Indicator.objects.filter(symbol__in=symbols):
    self.indicators.add(symbol)

как

self.indicators = Indicator.objects.filter(symbol__in=symbols)

3. Тут точно m2m лучшее решение? Не лучше ли сделать метод indicators, который будет возвращать нужный queryset? Минус JOIN в запросе. Хотя тут надо смотреть, что менее затратно.

Answer 2

[Влад Фролов]

Честно говоря долго вникал в вопрос, при сохранении формы всё-равно вызывается .save() у модели. Как способ решения проблемы посмотрите в сторону pre_save, post_save

Answer 3

[adsinger]

Как вариант, задать manytomany через through и перегузить сейв в объекте связки, для данных запросить саму модель через related manager.