[IMP] portal_event_tickets: fixed error in communications #11
Conversation
| if rself.event_id.state not in ["confirm", "done"]: | ||
| if not rself.event_id.is_done and not rself.event_id.is_finished: |
There was a problem hiding this comment.
Посмотрел по коду, как определяются is_done и is_finished. Определяются они примерно одинаково и соотвествуют старому состоянию "done". Поэтому оставь is_done, потому-что он определен в website_event
Замены для старого состояния "confirm" тут нету. Надо добавить
There was a problem hiding this comment.
Замены для старого состояния "confirm" тут нету. Надо добавить
Мне самому добавить поле is_confirmed в "event.event"? Если да, то каким должен быть его compute?
There was a problem hiding this comment.
Мне самому добавить поле is_confirmed в "event.event"? Если да, то каким должен быть его compute?
Не знаю. Тебе надо выяснить, чему соответствует состояние state = "confirm" из более ранних odoo в odoo 17.0
There was a problem hiding this comment.
Не знаю. Тебе надо выяснить, чему соответствует состояние state = "confirm" из более ранних odoo в odoo 17.0
Я просто не нашел никакого поля, связанного с confirm. Можешь подсказать, какое поле является его заменой?
There was a problem hiding this comment.
Посмотел коммит, который удаляет этот confirm
odoo/odoo@412ff99
Походу полноценной замены состояния state = 'confirm' тут нет.
В таком случае просто оставь упомянутый is_done
|
Также исправление надо обозначать как [FIX]. Как тут |
kwhatk0k3
force-pushed
the
17.0-mig-portal_event_tickets
branch
from
a69ab63 to
299b6a5
Compare
kwhatk0k3
force-pushed
the
17.0-mig-portal_event_tickets
branch
from
299b6a5 to
288068b
Compare
[IMP] portal_event_tickets: fixed error in communications