🚑 access error for poses created for different companies#1147
🚑 access error for poses created for different companies#1147KolushovAlexandr wants to merge 9 commits intoitpp-labs:11.0from
Conversation
There was a problem hiding this comment.
2 installable modules are updated:
├─ pos_longpolling/
| ├─ __manifest__.py
| ├─ doc/
| | └─ changelog.rst
| └─ static/
| └─ src/
| └─ js/
| └─ pos_longpolling.js
└─ pos_multi_session/
├─ __manifest__.py
├─ data/
| └─ pos_multi_session_data.xml
├─ demo/
| └─ demo.xml
├─ doc/
| └─ changelog.rst
├─ models/
| └─ pos_multi_session_models.py
├─ security/
| └─ ir_rule.xml
├─ static/
| └─ src/
| └─ js/
| └─ pos_multi_session.js
└─ views/
└─ pos_multi_session_views.xml
Not installable modules remain unchanged.
sent by ✌️ Odoo Review Bot
| -------- | ||
|
|
||
| - **Fix:** Access error for poses created for different companies | ||
|
|
1e40701 to
0d2c3fd
Compare
em230418
left a comment
There was a problem hiding this comment.
Конфликты надо поправить
| result = False | ||
| if 'multi_session_id' in vals: | ||
| multi_session_id = vals.get('multi_session_id') | ||
| if multi_session_id: |
There was a problem hiding this comment.
Я правильно понимаю, что multi_session_id может быть задан в словаре, но иметь значение вида 0, False или None? Меня тут смутило то, что мы сначала проверяем наличие multi_session_id, потом вызываем .get который не выкинет исключение, если его там нет.
There was a problem hiding this comment.
get по привычке наверное Саша написал.
yelizariev
left a comment
There was a problem hiding this comment.
Влить можно, но конфликты не очень хочется исправлять. Подождем выхода оду 14, сбросим поддержку оду 11 и закроем))
| </div> | ||
| </div> | ||
| </xpath> | ||
| <xpath expr="//field[@name='journal_id']/../../../../.." position="after"> |
There was a problem hiding this comment.
Попроще точно ничего нельзя сделать?)
No description provided.