Issue 11030
- Title
- Concurrent update in sao's wizard could result in incoherent behaviour
- Priority
- bug
- Status
- testing
- Nosy list
- ced, nicoe, reviewbot
- Assigned to
- nicoe
- Keywords
- backport, review
Created on 2021-12-09.09:36:55 by nicoe, last changed 6 days ago by ced.
Messages
* Cédric Krier [2021-12-09 09:40 +0100]: > >Cédric Krier <cedric.krier@b2ck.com> added the comment: > >On 2021-12-09 09:36, Nicolas Évrard wrote: >> Since setting the value of another might trigger an RPC call > >I do not understand what this means ? It should be "Since setting the value of another *field*". Setting values in the default might trigger an on_change or on_change_with RPC call.
On 2021-12-09 09:36, Nicolas Évrard wrote: > Since setting the value of another might trigger an RPC call I do not understand what this means ?
In sao when a StateView is sent to the wizard, we're setting the values of the form with a call to set_default.
When the value of a M2O is set it could happen that its rec_name is not sent thus it requires to make an RPC call in order to get it.
Since setting the value of another might trigger an RPC call we now have two concurrent RPC call. According to the order in which they will be treated the value of the M2O could be different. I think the behaviour should be always the same.
The review fixes that.
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-01-21 14:31:38 | ced | set | keyword:
+ backport status: chatting -> testing |
| 2022-01-21 11:15:54 | reviewbot | set | messages: + msg73372 |
| 2022-01-19 17:51:20 | reviewbot | set | messages: + msg73322 |
| 2021-12-09 09:59:16 | nicoe | set | messages: + msg72063 |
| 2021-12-09 09:54:36 | reviewbot | set | messages:
+ msg72062 nosy: + reviewbot |
| 2021-12-09 09:40:52 | ced | set | messages:
+ msg72058 nosy: + ced status: unread -> chatting |
| 2021-12-09 09:36:55 | nicoe | create | |
Showing 10 items. Show all history (warning: this could be VERY long)