Merge lp:~akretion-team/openobject-addons/trunk-addons-mrp-perf-improve into lp:openobject-addons

[Alexis de Lattre]

2 reasons for this one-line patch :

- performance : most of the time, production.move_prod_id.location_id = "Stock" and production.location_dest_id = "Stock", so, in those frequent cases, a write is not required... the stock_move already has the right value

- usability : in stock/stock.py line 1731, you can see that writing to a stock_move in 'done' state is very restricted. When you have already validated the outgoing product but haven't validated the production order (I know, you should first validate the production order and THEN the outgoing product), and you try to validate the production order, then you get the error message "Quantities, UoMs, Products and Locations cannot be modified on stock moves that have already been processed (except by the Administrator)". This patch allows this scenario without the error message when the stock_move already has the right value for 'location_id'.

=== modified file 'mrp/mrp.py'
--- mrp/mrp.py	2012-04-03 17:21:08 +0000
+++ mrp/mrp.py	2012-04-16 09:08:31 +0000
@@ -660,7 +660,7 @@
 
         for (production_id,name) in self.name_get(cr, uid, ids):
             production = self.browse(cr, uid, production_id)
-            if production.move_prod_id:
+            if production.move_prod_id and production.move_prod_id.location_id.id != production.location_dest_id.id:
                 move_obj.write(cr, uid, [production.move_prod_id.id],
                         {'location_id': production.location_dest_id.id})
             self.action_ready_send_note(cr, uid, [production_id], context)