Merge lp:~akretion-team/openerp-product-attributes/dont-blow-if-same-field-twice into lp:~product-core-editors/openerp-product-attributes/7.0

[Raphaël Valyi - http://www.akretion.com]

[FIX][product_custom_attributes] 2 attr groups can include the same field. In such case, present the field in the first group where it's found only instead of twice which leads to bugs

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

Seems fine to me

review: Approve

[Niels Huylebroeck (red15)]

Looks sensible, just not sure why it's showing 3 revisions unmerged but the preview diff is only showing change from the last rev?

review: Approve

[Numérigraphe (numerigraphe)]

Dear Niels Huylebroeck,
The prerequisite branch specified in this proposal has the first two unmerged revisions.
Lionel Sausin.

=== modified file 'product_custom_attributes/product.py'
--- product_custom_attributes/product.py	2013-02-15 04:39:23 +0000
+++ product_custom_attributes/product.py	2013-02-15 04:39:23 +0000
@@ -110,8 +110,9 @@
         for group in grp_obj.browse(cr, uid, attribute_group_ids, context=context):
             page = etree.SubElement(notebook, 'page', string=group.name.capitalize())
             for attribute in group.attribute_ids:
-                toupdate_fields.append(attribute.name)
-                self._build_attribute_field(cr, uid, page, attribute, context=context)
+                if attribute.name not in toupdate_fields:
+                    toupdate_fields.append(attribute.name)
+                    self._build_attribute_field(cr, uid, page, attribute, context=context)
         return notebook, toupdate_fields
 
     def fields_view_get(self, cr, uid, view_id=None, view_type='form', context=None, toolbar=False, submenu=False):