[FIX] disgusting hack to skip around race condition
authorXavier Morel <xmo@openerp.com>
Tue, 25 Oct 2011 08:51:12 +0000 (10:51 +0200)
committerXavier Morel <xmo@openerp.com>
Tue, 25 Oct 2011 08:51:12 +0000 (10:51 +0200)
commitb6a0874f4cc406788f0bf7045e279f43f8bd82ed
tree4cf3eb5db81405c65dbec59ed9d93ca4503cfe35
parentc68d85140742ff1090175e64815ffa09f2f77104
[FIX] disgusting hack to skip around race condition

When clicking on an action button, form view first saves then executes
the action. In doing so, it causes the o2m field to call
ListView#reload_content twice without waiting for the reloading to
end, generating a race condition in ListView.List#render_dataset.

As a result, the read_slice callback fills the list twice (once for
each #reload_content call), resulting in everything being duplicated.

Clear the content of the records collection before adding stuff.

note: collections should dedup on ids.

lp bug: https://launchpad.net/bugs/877965 fixed

bzr revid: xmo@openerp.com-20111025085112-q38q8m0k2mopwadg
addons/web/static/src/js/view_list.js