Tryton - Issues

 

Issue7433

Title Wrong on_change* calls order
Priority bug Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type behavior Components sao
Assigned To ced Keywords review
Reviews 50361002
View: 50361002

Created on 2018-05-16.10:28:03 by ced, last changed by roundup-bot.

Messages
New changeset eedc83c547e6 by Cédric Krier in branch '4.8':
Ensure sequential order of on_change calls
https://hg.tryton.org/sao/rev/eedc83c547e6

New changeset d195c5096ed1 by Cédric Krier in branch '4.6':
Ensure sequential order of on_change calls
https://hg.tryton.org/sao/rev/d195c5096ed1

New changeset c0b96d267fae by Cédric Krier in branch '4.4':
Ensure sequential order of on_change calls
https://hg.tryton.org/sao/rev/c0b96d267fae

New changeset 0572d37d8e22 by Cédric Krier in branch '4.2':
Ensure sequential order of on_change calls
https://hg.tryton.org/sao/rev/0572d37d8e22

New changeset e7a2920a4e85 by Cédric Krier in branch '4.0':
Ensure sequential order of on_change calls
https://hg.tryton.org/sao/rev/e7a2920a4e85
New changeset e38b8b836a7f by Cédric Krier in branch 'default':
Ensure sequential order of on_change calls
https://hg.tryton.org/sao/rev/e38b8b836a7f
review50361002 updated at https://codereview.tryton.org/50361002/#ps1
msg40696 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-05-16.10:28:02
The on_change_with are called with the value prior to the on_change calls, so the result is wrong because it is based on the wrong data.

Ex: on a sale line, if you change the quantity which result in changing the unit price, the amount is computed with the previous unit price.
History
Date User Action Args
2018-05-23 16:40:05roundup-botsetmessages: + msg40966
2018-05-17 18:26:13roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg40745
2018-05-16 10:39:39reviewbotsetnosy: + reviewbot
messages: + msg40697
2018-05-16 10:31:21cedsetstatus: in-progress -> testing
reviews: 50361002
keyword: + review
2018-05-16 10:28:03cedcreate

Showing 10 items. Show all history (warning: this could be VERY long)