Merge lp:~openerp-dev/openerp-web/7.0-opw-597678-msh into lp:openerp-web/7.0

Proposed by Mohammed Shekha(Open ERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openerp-web/7.0-opw-597678-msh
Merge into: lp:openerp-web/7.0
Diff against target: 13 lines (+3/-0)
1 file modified
addons/web/static/src/js/view_list.js (+3/-0)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/7.0-opw-597678-msh
Reviewer Review Type Date Requested Status
Martin Trigaux (OpenERP) Pending
Xavier (Open ERP) Pending
Review via email: mp+185699@code.launchpad.net

Description of the change

Hello,

Fixed the issue of many2many tag update/refresh issue with editable list view.

Demo: Go to editable list view where there is many2many_tag field, for e.g. Customer Invoice and Supplier Invoice, Create Supplier Invoice, create Invoice Line(if it is not editable tree then make it editable tree), add tax, then click outside, again edit that line and add second tax, second tax value not updated.

Expected: Tags added should be immediately get reflected after adding new tag, the issue is many2many_tag is not refreshed with new tags.

Thanks.

To post a comment you must log in.

Unmerged revisions

4028. By Mohammed Shekha(OpenERP)<email address hidden>

[FIX]Fixed the issue of many2many tag update/refresh issue with editable list view.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/web/static/src/js/view_list.js'
2--- addons/web/static/src/js/view_list.js 2013-06-10 07:10:36 +0000
3+++ addons/web/static/src/js/view_list.js 2013-09-16 04:55:24 +0000
4@@ -533,6 +533,9 @@
5 return;
6 }
7 _(_.keys(values)).each(function(key){
8+ if(record.get(key + '__display')) {
9+ delete record.attributes[key + '__display'];
10+ }
11 record.set(key, values[key], {silent: true});
12 });
13 record.trigger('change', record);