Thibault Delavallée (OpenERP) has proposed merging 
lp:~openerp-dev/openerp-web/trunk-kanban-tags-chm into lp:openerp-web.

Requested reviews:
  OpenERP R&D Web Team (openerp-dev-web)

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-kanban-tags-chm/+merge/136113

[FIX]: when changing the column of a kanban vignette, the display of tags could 
be duplicated.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-kanban-tags-chm/+merge/136113
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/trunk-kanban-tags-chm.
=== modified file 'addons/web_kanban/static/src/js/kanban.js'
--- addons/web_kanban/static/src/js/kanban.js	2012-11-15 18:17:30 +0000
+++ addons/web_kanban/static/src/js/kanban.js	2012-11-26 09:14:25 +0000
@@ -494,18 +494,21 @@
                     field.raw_value.forEach(function(id) {
                         rel.ids.push(id);
                         if (!rel.elements[id]) {
-                            rel.elements[id] = [];
+                            rel.elements[id] = $el;
                         }
-                        rel.elements[id].push($el[0]);
                     });
                 });
             });
         });
-       _.each(relations, function(rel, rel_name) {
+       _.each(relations, function (rel, rel_name) {
             var dataset = new instance.web.DataSetSearch(self, rel_name, self.dataset.get_context());
-            dataset.name_get(_.uniq(rel.ids)).done(function(result) {
-                result.forEach(function(nameget) {
-                    $(rel.elements[nameget[0]]).append('<span class="oe_tag">' + _.str.escapeHTML(nameget[1]) + '</span>');
+            dataset.name_get(_.uniq(rel.ids)).done(function (result) {
+                var htmllist = [];
+                result.forEach(function (nameget) {
+                    htmllist[nameget[0]] = (htmllist[nameget[0]] || '') + '<span class="oe_tag">' + _.str.escapeHTML(nameget[1]) + '</span>';
+                });
+                _.each(htmllist, function (value, key) {
+                    rel.elements[key].html(value);
                 });
             });
         });

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to