[IMP] search view generation: update fields set in place instead of creating a new set via union

bzr revid: xmo@openerp.com-20110915123155-a5ht8nhfp9kq0kst
2011-09-15 14:31:55 +02:00
parent eeb31efa88
commit 7bb856b834
1 changed files with 2 additions and 1 deletions
@@ -1794,11 +1794,12 @@ class orm_template(object):
        fields_to_search = set(
            field for field, descriptor in fields.iteritems()
            if descriptor.get('select'))
+
        for view in (form_view, tree_view):
            view_root = etree.fromstring(view['arch'])
            # Only care about select=1 in xpath below, because select=2 is covered
            # by the custom advanced search in clients
-            fields_to_search = fields_to_search.union(view_root.xpath("//field[@select=1]/@name"))
+            fields_to_search.update(view_root.xpath("//field[@select=1]/@name"))

        tree_view_root = view_root # as provided by loop above
        search_view = etree.Element("search", attrib={'string': tree_view_root.get("string", "")})