Tryton - Issues

 

Issue6473

Title Client - One2Many wrong item opening after refreshing view
Priority bug Status resolved
Superseder Nosy List Pijuli, afibanez, ced, pokoli, reviewbot, roundup-bot
Type behavior Components sao, tryton
Assigned To ced Keywords review
Reviews 35251002,40591002
View: 35251002, 40591002

Created on 2017-04-26.16:48:03 by Pijuli, last changed by roundup-bot.

Messages
New changeset 793555a6e7e3 by Cédric Krier in branch '4.4':
Mark tree states as done for view without tree_state attribute
http://hg.tryton.org/sao/rev/793555a6e7e3

New changeset ee142d24f6b2 by Cédric Krier in branch '4.2':
Mark tree states as done for view without tree_state attribute
http://hg.tryton.org/sao/rev/ee142d24f6b2
New changeset bebb94a6d75e by Cédric Krier in branch '4.4':
Mark tree states as done for view without tree_state attribute
http://hg.tryton.org/tryton/rev/bebb94a6d75e

New changeset f990cdd42f90 by Cédric Krier in branch '4.2':
Mark tree states as done for view without tree_state attribute
http://hg.tryton.org/tryton/rev/f990cdd42f90
New changeset 537095f33687 by Cédric Krier in branch 'default':
Mark tree states as done for view without tree_state attribute
http://hg.tryton.org/sao/rev/537095f33687
New changeset 252241d9bbd9 by Cédric Krier in branch 'default':
Mark tree states as done for view without tree_state attribute
http://hg.tryton.org/tryton/rev/252241d9bbd9
review40591002 updated at https://codereview.tryton.org/40591002/#ps1
review35251002 updated at https://codereview.tryton.org/35251002/#ps1
msg35077 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2017-08-07.22:58:11
Even if I can not reproduce the issue on sao, here is review40591002 for the sake of consistency.
msg35076 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2017-08-07.22:53:59
Indeed it is well a side effect of r 33caa31d27bc. Here is review35251002 that should fix it.
msg33634 (view) Author: [hidden] (afibanez) Date: 2017-05-11.17:09:49
I have done the bisect and the bug was introduced between 4.0 and 4.2, in this commit: https://hg.tryton.org/tryton/rev/33c1a31d27bc

But I don't have any idea of tryton client development and how to solve it except by removing the two new returns.
msg33633 (view) Author: [hidden] (pokoli) (Tryton committer) (Tryton translator) Date: 2017-05-11.16:27:52
I confirm I can reproduce it with tryton, but I can not reproduce it with sao. So it seems sao is not afected. 

@Pijuli, can you do an hg bisect to determine which changeset introduced it? From your comments, it must be something introduced after 4.0 release.
msg33632 (view) Author: [hidden] (Pijuli) Date: 2017-05-11.16:14:34
If you disable "options -> form -> save tree state" it stops behaving unexpectedly.
But this is just a workaround.
It's not happening in 4.0 and before. It is for 4.2 and 4.4
msg33631 (view) Author: [hidden] (Pijuli) Date: 2017-05-11.15:40:37
This bug is also reproducible in tryton client v4.4
msg33427 (view) Author: [hidden] (Pijuli) Date: 2017-04-26.16:48:03
When you reload a view that has a One2Many list and you have already opened one item, the next item you open will be the last you opened before reloading

How to reproduce:
Login into demo4.2 with client 4.2.5
Financial -> Invoices -> Customer Invoices
Create a customer invoice
Create 2 lines with different products so it's easily distinguishable
open line2
reload view
open line1

As you can see line2 has opened altough you chose line1
History
Date User Action Args
2017-09-05 23:37:52roundup-botsetmessages: + msg35554
2017-09-05 23:37:21roundup-botsetmessages: + msg35553
2017-08-28 18:27:40roundup-botsetmessages: + msg35382
2017-08-28 18:27:04roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg35381
2017-08-07 23:05:04reviewbotsetmessages: + msg35080
2017-08-07 23:04:47reviewbotsetnosy: + reviewbot
messages: + msg35079
2017-08-07 22:58:11cedsetreviews: 35251002 -> 35251002,40591002
component: + sao
messages: + msg35077
2017-08-07 22:54:00cedsetstatus: chatting -> testing
keyword: + review
nosy: + ced
messages: + msg35076
reviews: 35251002
assignedto: ced
2017-05-11 17:09:49afibanezsetnosy: + afibanez
messages: + msg33634
2017-05-11 16:27:52pokolisetstatus: unread -> chatting
nosy: + pokoli
messages: + msg33633

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