From 6430cf4ccfb2914a1a866fb5d01b503af4021ecc Mon Sep 17 00:00:00 2001
From: Guillem Barba <guillembarba@gmail.com>
Date: Mon, 21 Mar 2016 11:06:22 +0100
Subject: [PATCH] Improve Move.assign() making a single call to Move.write()

issue5155
review20581002
---
 move.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/move.py b/move.py
index e149683944..6dec253599 100644
--- a/move.py
+++ b/move.py
@@ -627,7 +627,7 @@ class Move(Workflow, ModelSQL, ModelView):
         cls.check_origin(moves)
         for move in moves:
             move.set_effective_date()
-            move.save()
+        cls.save(moves)
 
     @classmethod
     @ModelView.button
@@ -638,6 +638,8 @@ class Move(Workflow, ModelSQL, ModelView):
             move.set_effective_date()
             move._do()
             move.state = 'done'
+            # This save() call can't be grouped because the average computation
+            # of product cost price requires each move to be done separately
             move.save()
 
     def _do(self):