Merge lp:~akretion-team/openobject-addons/trunk-read-for-perf into lp:openobject-addons

[Benoit Guillot - http://www.akretion.com]

This proposal replaces the browse on the product in the method get_product_available by a read in order to greatly increase the perf. It also replaces an useless browse in the method _check_ean_key.

=== modified file 'product/product.py'
--- product/product.py	2012-06-05 17:12:31 +0000
+++ product/product.py	2012-07-09 16:21:28 +0000
@@ -559,8 +559,8 @@
         return False
 
     def _check_ean_key(self, cr, uid, ids, context=None):
-        for product in self.browse(cr, uid, ids, context=context):
-            res = check_ean(product.ean13)
+        for product in self.read(cr, uid, ids, ['ean13'], context=context):
+            res = check_ean(product['ean13'])
         return res
 
     _constraints = [(_check_ean_key, 'Error: Invalid ean code', ['ean13'])]

=== modified file 'stock/product.py'
--- stock/product.py	2012-05-22 14:54:33 +0000
+++ stock/product.py	2012-07-09 16:21:28 +0000
@@ -243,13 +243,16 @@
             child_location_ids = location_obj.search(cr, uid, [('location_id', 'child_of', location_ids)])
             location_ids = child_location_ids or location_ids
         
+        # this will be a dictionary of the product UoM by product id
+        product2uom = {}
+        uom_ids = []
+        for product in self.read(cr, uid, ids, ['uom_id'], context=context):
+            product2uom[product['id']] = product['uom_id'][0]
+            uom_ids.append(product['uom_id'][0])
         # this will be a dictionary of the UoM resources we need for conversion purposes, by UoM id
         uoms_o = {}
-        # this will be a dictionary of the product UoM by product id
-        product2uom = {}
-        for product in self.browse(cr, uid, ids, context=context):
-            product2uom[product.id] = product.uom_id.id
-            uoms_o[product.uom_id.id] = product.uom_id
+        for uom in self.pool.get('product.uom').browse(cr, uid, uom_ids, context=context):
+            uoms_o[uom.id] = uom
 
         results = []
         results2 = []