Created on 2019-11-03.15:53:30 by risto3, last changed 10 months ago by roundup-bot.
New changeset a825c6456c1e by Sergi Almacellas Abellana in branch 'default': Add lines on Move's check_modify_exclude https://hg.tryton.org/tryton-env/rev/a825c6456c1e
New changeset f8d219211cc1 by Sergi Almacellas Abellana in branch 'default': Add lines on Move's check_modify_exclude https://hg.tryton.org/modules/account/rev/f8d219211cc1
I've implmented issue9194 and updated the review to require it. Everything is not ready to be tested.
Indeed I think we should implement issue9194 so it will not be needed to exclude move.
On 2019-12-10 19:42, Sergi Almacellas Abellana wrote: > Indeed it was also required to exclude the move from the lines modification as the field is rewritten by the clients. Then it is a no go. We can not allow to modify the move of a posted line.
Here is review286381004 that fixes it. Indeed it was also required to exclude the move from the lines modification as the field is rewritten by the clients.
On from move form but not on line. lines should probably be added to Move._check_modify_exclude as check_modify is called by the lines at creation and deletion.
as this apparently was possible with https://bugs.tryton.org/issue7276 I get an exception if I try to save a move with the maturity date updated on a line. The message is to the tune 'you cannot modify a posted move " 16429 "' I'm using the 5.2 tryton gtk client.
|2020-06-17 14:00:17||roundup-bot||set||messages: + msg58761|
|2020-06-17 14:00:13||roundup-bot||set||status: testing -> resolved|
nosy: + roundup-bot
messages: + msg58760
|2020-05-10 14:00:54||reviewbot||set||messages: + msg57933|
|2020-05-10 13:41:28||pokoli||set||status: chatting -> testing|
messages: + msg57931
superseder: + Do not write on target already added to One2Many
|2020-03-13 00:35:27||ced||set||status: in-progress -> chatting|
|2019-12-11 09:48:27||pokoli||set||status: testing -> in-progress|
assignedto: pokoli -> (no value)
|2019-12-10 23:09:06||ced||set||messages: + msg54425|
messages: + msg54423
|2019-12-10 19:56:56||reviewbot||set||reviews: 286381004|
keyword: + review
Showing 10 items. Show all history (warning: this could be VERY long)