Merge PR #736 into 10.0

Signed-off-by jbeficent
OCA · Sep 16, 2019 · ae9bc05 · ae9bc05
2 parents 3e577f0 + 597a2dd
commit ae9bc05
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/purchase_location_by_line/models/purchase.py b/purchase_location_by_line/models/purchase.py
@@ -52,6 +52,6 @@ def _create_stock_moves(self, picking):
 
             location = line.location_dest_id or default_picking_location
             if location:
-                line.move_ids.write(
+                line.move_ids.filtered(lambda m: m.state != 'done').write(
                     {'location_dest_id': location.id})
         return res
diff --git a/purchase_location_by_line/tests/test_purchase_delivery.py b/purchase_location_by_line/tests/test_purchase_delivery.py
@@ -181,3 +181,13 @@ def test_check_multiple_locations_multiple_dates_02(self):
         self.assertEquals(
             sorted_pickings[2].min_date[:10], self.date_later,
             "The second picking must be planned at the latest date")
+
+    def test_modify_confirmed(self):
+        # if all moves contain the location no need to update
+
+        self.po.button_confirm()
+        self.po.order_line[0].write({'product_qty': 43})
+        self.assertEqual(
+            self.po.order_line[0].move_ids.mapped('location_dest_id'),
+            self.l1
+        )