Created on 2020-05-05.14:46:17 by oqueralto, last changed yesterday by roundup-bot.
New changeset 37c3df0a3007 by Cédric Krier in branch 'default': Keep record added/modified after cancel from popup window https://hg.tryton.org/tryton-env/rev/37c3df0a3007
New changeset f813fb56a83d by Cédric Krier in branch 'default': Keep record added/modified after cancel from popup window https://hg.tryton.org/tryton/rev/f813fb56a83d
New changeset e474e7cddb63 by Cédric Krier in branch 'default': Keep record added/modified after cancel from popup window https://hg.tryton.org/sao/rev/e474e7cddb63
So it's ok for me.
This is the expected behavior because you edit a saved record so canceling makes it be reloaded from the database.
This is needed to correctly detect the user modification only (otherwise we will resent to the server all the fields because they were restored). The button names could be improved see issue9710.
There's still something strange. I did my tests using stock_lot. So i add an incoming move (without lot number).
Then i double-click on this line to set a lot number, then OK. there is now a lot number displayed in the treeview.
Then i double-click again on this line and Cancel -> lot number which was displayed on the form view disappear in the tree view.
Here is review296601002 which keep the record being added on save.
I think it is linked to issue3325.
It's happening on both.
Are you using tryton, sao or both?
It happens on any o2m field with the many2one widget, but I will explain how to reproduce it on a concrete one: 1. Create a new Supplier shipment. 2. Add an existing incoming move to the shipment. 3. Open the move form. 4. Press cancel button or 'Esc' key. 5. Save the shipment or receive it. If you follow the steps, the move disappears, but for me the expected behavior is to keep it. I'm using 5.2 version but it's also happening in 5.6.
|2020-10-24 13:15:19||roundup-bot||set||messages: + msg61201|
|2020-10-24 13:15:16||roundup-bot||set||messages: + msg61200|
nosy: + roundup-bot
status: testing -> resolved
|2020-10-21 14:31:41||mrichez||set||messages: + msg61098|
|2020-10-21 14:28:55||ced||set||messages: + msg61097|
nosy: + mrichez
|2020-10-17 02:05:41||ced||link||issue9600 superseder|
messages: + msg61019
|2020-10-17 00:38:23||ced||set||status: chatting -> testing|
messages: + msg61018
keyword: + backport, review
messages: + msg57841
Showing 10 items. Show all history (warning: this could be VERY long)