[FIX] stock_move_location: fill with stock

Always compute put-aways

Author: jbaudoux

stock_move_location/models/stock_picking.py

@@ -1,4 +1,4 @@
-# Copyright Jacques-Etienne Baudoux 2016 Camptocamp
+# Copyright 2016 Jacques-Etienne Baudoux (BCIM) <je@bcim.be>
 # Copyright Iryna Vyshnevska 2020 Camptocamp
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl)
 
@@ -19,6 +19,7 @@ def button_fillwithstock(self):
             "active_model": "stock.quant",
             "planned": True,
         }
+        # FIXME: this action should not bypass the call to action_assign !!!
         move_wizard = (
             self.env["wiz.stock.move.location"]
             .with_context(**context)
@@ -28,6 +29,7 @@ def button_fillwithstock(self):
                     "origin_location_id": self.location_id.id,
                     "picking_type_id": self.picking_type_id.id,
                     "picking_id": self.id,
+                    "apply_putaway_strategy": True,
                 }
             )
         )

stock_move_location/readme/ROADMAP.md

@@ -9,3 +9,8 @@ ACSONE):
 - Nice to have: add a magic button on locations that with context
   creates a new picking of that type with the origin location already
   filled in.
+
+Note from Jacques-Etienne Baudoux (BCIM)
+- The fill with stock action on stock.picking should not create manually the
+  move lines but instead call `action_assign()`. There are many modules hooking
+  `action_assign` that are not called due to this.