Created on 2020-09-23.18:47:43 by nicoe, last changed 1 month ago by roundup-bot.
New changeset 856f5c6f7d4f by Nicolas Évrard in branch 'default': Ensure dependencies of fields are also loaded on Form display https://hg.tryton.org/tryton-env/rev/856f5c6f7d4f
New changeset ccaef9daa811 by Nicolas Évrard in branch 'default': Ensure dependencies of fields are also loaded on Form display https://hg.tryton.org/trytond/rev/ccaef9daa811
New changeset dc0520e83643 by Nicolas Évrard in branch 'default': Ensure dependencies of fields are also loaded on Form display https://hg.tryton.org/tryton/rev/dc0520e83643
New changeset 9f7cab2d5c3a by Nicolas Évrard in branch 'default': Ensure dependencies of fields are also loaded on Form display https://hg.tryton.org/sao/rev/9f7cab2d5c3a
I don't think it's required to load the depends of the depends because those fields are used to evaluate the states of a field so only their value is important not the state in which they are.
I guess you mean on Form.display to add to the fields loaded the depends of those fields? But then should it not be required to also load the depends of the depends etc.
This a bit convoluted to reproduce so I'll give a small example. Let's consider a simple model: class A: a = fields.Char('a', depends=['b']) b = fields.Char('b', depends=['c']) c = fields.Char('c') Let's say the first form view of A consists of only displaying "a" and the second consists of displaying both "a" and "b". If a button in the first form view will trigger a switch to the second view then the value of "c" will stay unset. Which could be an issue if the visibility of "b" depends on it. What happens is that the first view is loaded and the fields_view_get returns only the fields "a" and "b", they are both loaded. When the second view is loaded "c" is added to the group but since only "a" and "b" have a widget the display on the form will load them and since they are already loaded no RPC call will happen. I think the way to solve this is to add the depends of the fields with a widget when displaying a form.
|2021-03-11 23:35:04||roundup-bot||set||messages: + msg65429|
|2021-03-11 23:35:01||roundup-bot||set||messages: + msg65428|
|2021-03-11 23:34:57||roundup-bot||set||messages: + msg65427|
nosy: + roundup-bot
status: testing -> resolved
|2021-01-07 11:26:53||reviewbot||set||messages: + msg63798|
|2020-10-26 10:25:59||reviewbot||set||messages: + msg61410|
|2020-10-22 00:04:47||ced||set||status: chatting -> testing|
|2020-10-09 18:57:06||reviewbot||set||messages: + msg60714|
|2020-10-06 01:27:26||reviewbot||set||messages: + msg60597|
|2020-10-05 14:28:46||reviewbot||set||messages: + msg60583|
Showing 10 items. Show all history (warning: this could be VERY long)