Как для m2m_changed задать условие для выполнения?

Question

[zkweb]

Подскажите, пожалуйста, как мне сделать следующее:

Модели: Service, Request, Rzi, RziRequest

В модели Service - лежат договора
В модели Request - лежат вызовы по договорам

В Rzi, RziRequest - по сути всё тоже самое, что и выше.

В модели Rzi есть поле номер договора,

• - если оно заполнено (т.е есть номер договора), то всё значения выбранные в ManyToManyField необходимо записать в другую промежуточную таблицу Service_Request
  
• - иначе записать всё в промежуточную таблицу Rzi_RziRequest
  

Сейчас все выбранные данные из ManyToManyField записываются в промежуточную таблицу Rzi_RziRequest

Как это всё сделать?

Comments

[sim3x]

> иначе записать всё в промежуточную таблицу
зачем все усложнять?

Answer 1

[sim3x]

Достаточно сделать метод у модели, из которого возвращать bool если

есть поле номер договора,

Comments

[zkweb]

подскажите код для примера, как это сделать. В джанго новичок еще

[sim3x]

zkweb: сначала добавь в вопрос, последовательность событий

[zkweb]

sim3x: Если в модели Rzi заполнено поле "номер договора", то все элементы добавляем в промежуточную таблицу Service_Request
Иначе добавляем в Rzi_RziRequest

[zkweb]

sim3x: Промежуточная таблица Service_Request: id, request_id, servicelist_id

[zkweb]

sim3x: Промежуточная таблица Rzi_RziRequest: id, rzi_id, servicelist_id

[zkweb]

sim3x: т.е если договора нет, то это выезд платный и т.д и в договору это не привязывается

[zkweb]

sim3x: понятно всё. я просто просил помочь... спасибо и на этом

[sim3x]

zkweb:
> Промежуточная таблица Rzi_RziRequest: id, rzi_id, servicelist_id
забудь про промежуточные таблицы -- у тебя есть m2m

Начни с упорядочения моделей, хотя б в таком виде https://toster.ru/answer?answer_id=671144#answers_...

[zkweb]

sim3x:

# ДОГОВОРА
class Service(models.Model):

SMS_CHOICES = (
(0, 'запрещенно'),
(1, 'разрешено'),
)

branch = models.ForeignKey('Branch',verbose_name = u"Филиал",)#
number = models.CharField(max_length=255,verbose_name = u"Номер договора",)#
date_begin = models.DateField(verbose_name = u"Дата заключения",)#
date_end = models.DateField(verbose_name = u"Дата окончания",blank=True,null=True)#
date_equip_sold = models.DateField(verbose_name = u"Дата продажи котла",blank=True,null=True)#
date_launch = models.DateField(verbose_name = u"Дата пуска-наладки",blank=True,null=True)#
client = models.ForeignKey('Client',verbose_name = u"Клиент",null=True,blank=True)#
equipment = models.ForeignKey('Equipment',verbose_name = u"Оборудование")#
dealer = models.ForeignKey('Dealer',verbose_name = u"Дилер",blank=True,null=True)#
serial_number = models.CharField(max_length=255,verbose_name = u"Серийный номер",blank=True)#
sum_contract = models.FloatField(u"Сумма контракта",null=True)#
sum_launch = models.FloatField(u"Сумма пуско-наладки", blank=True, default=0)#
engineer = models.ForeignKey('Engineer',verbose_name = u"Инженер",blank=True,null=True)#
address = models.CharField(max_length=255,verbose_name = u"Адрес", blank=True)#
type = models.ForeignKey('ServiceType',verbose_name = u"Вид договора",)#
notes = models.TextField(u"Комментарии",blank=True)#
equipment2 = models.ForeignKey('Equipment',verbose_name = u"Доп.оборудование", related_name='second_equipment',null=True,blank=True)#
sms_delivery = models.PositiveSmallIntegerField(u'подписка на смс', choices=SMS_CHOICES,)#

def count_request(self):
return Request.objects.filter(contract = self.id).count();
count_request.short_description = 'Вызовов'


def count_call(self):
return Call.objects.filter(contract = self.id).count();
count_call.short_description = u'Звонков'

class Meta():
db_table = "service_main"
verbose_name = u"Договор"
verbose_name_plural = u"Сервис"

def __unicode__(self):
return self.number;

# ВЫЗОВЫ
class Request(models.Model):
date = models.DateField(verbose_name = u"дата",)
contract = models.ForeignKey(Service,verbose_name = u"договор",)
engineer = models.ForeignKey(Engineer,verbose_name = u"инженер",blank=True,null=True)
service = models.ManyToManyField(ServiceList, verbose_name=u"услуга",blank=True,null=True)
description = models.TextField(u"описание", blank=True)

class Meta:
db_table = "service_request"
verbose_name = u"вызов"
verbose_name_plural = u"вызовы"

def __unicode__(self):
return u'Вызов от %s для договора №%s' % (self.date,self.contract)

# СПИСОК ВЫПОНЯЕМЫХ УСЛУГ, ЭТО БУДЕТ ManyToManyField в моделях Service, Rzi
class ServiceList(models.Model):
name = models.CharField(u'наименование',max_length=255,)

class Meta:
db_table = "service_service"
verbose_name = u"услуга"
verbose_name_plural = u"услуги"

def __unicode__(self):
return '%s' % (self.name,)

# Сигнал, автоматически добавляем запись (выезд) в Rzi если форма была заполнена в модели Service
@receiver(post_save, sender=Request)
def go_request_rzi(sender, instance, created, **kwargs):
contract = Service.objects.filter(pk=instance.contract_id)
for item in contract:
Rzi.objects.create(
contract_number_id=instance.contract_id,
client_id=item.client_id,
serial_number=item.serial_number,
date_equip_sold=item.date_equip_sold,
date_launch=item.date_launch,
date_create=datetime.date.today(),
date_visit=instance.date,
address=item.address,
description=instance.description,
engineer_id=instance.engineer_id,
equipment_id=item.equipment_id,
equipment2_id=item.equipment2_id,
status="1",
time_visit="2",
request_id=instance.id)
post_save.connect(go_request_rzi, sender=Request)

# ВЫЕЗДЫ ИНЖЕНЕРОВ. Как по договору так и без него
class Rzi(models.Model):

STATUS_CHOICES = (
(1, 'платная'),
(2, 'по договору'),
)

TIME_VIZIT_CHOICES = (
(1, 'рабочее'),
(2, 'не рабочее'),
)

contract_number = models.ForeignKey(Service, blank = True, null = True, verbose_name = u"Номер договора")
client = models.ForeignKey(Client, verbose_name = u"Клиент")#
equipment = models.ForeignKey(Equipment, verbose_name = u"Оборудование")#
equipment2 = models.ForeignKey(Equipment, verbose_name = u"Доп.оборудование",blank=True,null=True)#
serial_number = models.CharField(max_length=255,verbose_name = u"Серийный номер",blank=True)#
address = models.CharField(max_length=255,verbose_name = u"Адрес", blank=True)#
date_equip_sold = models.DateField(verbose_name = u"Дата продажи котла",blank=True,null=True)#
date_launch = models.DateField(verbose_name = u"Дата пуска-наладки",blank=True,null=True)#
date_create = models.DateField(verbose_name = u"Дата публикации",blank=True,null=True)#
who_launch = models.CharField(max_length=255,verbose_name = u"Кем производился запуск", blank=True)#
description = models.TextField(u"Описание",blank=True)#
note = models.TextField(u"Комментарии",blank=True)#
status = models.PositiveSmallIntegerField(u'Статус заявки', choices=STATUS_CHOICES,blank=True,null=True)#
date_visit = models.DateField(verbose_name = u"Дата выезда",blank=True,null=True)#
engineer = models.ForeignKey(Engineer,verbose_name = u"Инженер",blank=True,null=True)#
time_visit = models.PositiveSmallIntegerField(u'Время выезда', choices=TIME_VIZIT_CHOICES,blank=True,null=True)#
request_id = models.PositiveSmallIntegerField(verbose_name = u"Вызов",blank=True,null=True)#
service = models.ManyToManyField(ServiceList, verbose_name=u"Выполненные работы",blank=True,null=True)

class Meta():
db_table = "service_rzi_main"
verbose_name = u"Реестр заявок инженеров"
verbose_name_plural = u"РЗИ"

def __unicode__(self):
return u'Запись под №%s' % (self.id)

[zkweb]

sim3x: это вам нужно было?

[sim3x]

zkweb: конечно нет

Общие замечания
используй db_table если нужно прикрутить орм к существующей таблице
Договор=Контракт=Contract
request_id ??? для чего, если он уже есть

# Сигнал, автоматически добавляем запись (выезд) в Rzi если форма была заполнена в модели Service

тут вообще не нужен сигнал - тут в форме на save все делается

У тебя бардак в моделях. Пропиши на бумажке как происходит весь процесс и агрегируй общие данные в одной модели, потом посмотри на модели с точки зрения 3 нормальной формы

[zkweb]

sim3x: посмотрите, пожалуйста, это фаил. Может я что-то не так объяснял
https://yadi.sk/i/IUfh4SAvjhWDP

[sim3x]

zkweb:
Странное ТЗ

>В моделе Rzi же попадают определенные поля с договора + с блока «Вызовы» поля Дата,

>Если выезд был без договора, то данные об «Выполненных услугах» также сохраняются только уже в другой промежуточной таблице.

Его нужно переписывать

[zkweb]

sim3x: ))) так пользователи видят свою систему. не мне же выбирать. Ладно спасибо, кроме непонятной критики от вас я ничего больше не услышал. Я просто хотел понял как распарсить m2m и всё.

[sim3x]

zkweb: я просто старался показать, что пхп стайл в джанго применять не стоит - его поддерживать тяжело

[zkweb]

sim3x: Это понятно!!! Джанго мне понравилась, и сейчас часть проекта с PHP переношу на неё. Смотря на всё это дело, я не думаю что в моделях бардак, как вы говорите. Но, и сейчас на этой задаче я просто "встал" ))) не знаю куда дальше... Мне на данном этапе хотя бы решить задачу как m2m если поле договора пустое, значения сохранить в другую промежут. таблицу. Если я глето-то туплю вы мне прямо скажите, где ошибка и как бы вы сделали... может где-то я и запутался

[sim3x]

zkweb:
> я не думаю
проверь модели на соответствие НФ, тогда вопрос
>начения сохранить в другую промежут. таблицу
пропадет сам

[zkweb]

sim3x: Модель Rzi лишняя что-ли?

[zkweb]

sim3x: расширить модель Service и удалить Rzi? или я уже туплю совсем.... (

[zkweb]

sim3x: или расширить модель Request (где сами вызовы хранятся)

[zkweb]

sim3x: я решил удалить полность Rzi модель, и переделать модель самих вызовов Request. Добавить туда необходимы поля. И если поле договора там будет заполнено, то оно и автоматом появится в теле самого договора. Вроде так.

[sim3x]

zkweb:
Стоит идти от того как у заказчика организовано все
Если у заказчика большая часть запросов идет без контракта, то стоит сместить акцент на Запрос. Если больше контрактников, то - Контракт

Нормализация данних предполагает отсутствие дублирования в моделях - ей все равно в какой модели все будет лежать

Оборудование
   номер
   специфика

Контракт
   номер контракта
   оборудование = м2м(Оборудование)  

Заказ на выезд
   контракт = fk(Контракт, blank=True, null=True)
   оборудование = м2м(Оборудование, blank=True, null=True)
   
   is_заказ_выполнен = BooleanField

>Добавить туда необходимы поля.
тебе пора отказаться от джанго админки, тк дальше ты дальше упрешься в ее ограничения.
Также ознакомься с Forms - ето позволит тебе соединять несколько моделей в одной форме

[zkweb]

sim3x: для заказчика главное был удобный функционал. То что сейчас сделал, т.е удалением модели Rzi и переделал модель Request мне вроде и помогло. Сейчас всё работает как надо. Всё-ли верно с точки зрения организации кода и т.д сейчас не могу сказать. Раз работает как надо - то всё хорошо. По поводу админки - она хорошо конечно и сейчас просто отказаться не могу от неё.