[IMP] barcodes: use builtin python sets when it makes sense

[odoo/odoo.git] / addons / stock / stock.py

diff --git a/addons/stock/stock.py b/addons/stock/stock.py
index 443ad6a..e6db4b5 100644 (file)
--- a/addons/stock/stock.py
+++ b/addons/stock/stock.py
@@ -23,6 +23,7 @@ from datetime import date, datetime
 from dateutil import relativedelta
 import json
 import time
+import sets
 
 import openerp
 from openerp.osv import fields, osv
@@ -4307,22 +4308,13 @@ class barcode_rule(models.Model):
     _inherit = 'barcode.rule'
 
     def _get_type_selection(self):
-        types = super(barcode_rule,self)._get_type_selection() 
-
-        new_types = [('weight','Weighted Product'),
-                     ('location','Location'),
-                     ('lot','Lot'),
-                     ('package','Package')]
-
-        for (key, value) in new_types:
-            add = True
-            for (key2, value2) in types:
-                if key == key2:
-                    add = False
-                    break
-            if add:
-                types += [(key, value)]
-
-        return types
+        types = sets.Set(super(barcode_rule,self)._get_type_selection()) 
+        types.update([
+            ('weight','Weighted Product'),
+            ('location','Location'),
+            ('lot','Lot'),
+            ('package','Package')
+        ])
+        return list(types)
 
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4: