[FIX] stock_release_channel: Don't use related for open pickings field

foodles-tech · Nov 19, 2024 · ccca46c · ccca46c
1 parent 9d4611a
commit ccca46c
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/stock_release_channel/models/stock_release_channel.py b/stock_release_channel/models/stock_release_channel.py
@@ -104,7 +104,8 @@ class StockReleaseChannel(models.Model):
     )
     open_picking_ids = fields.One2many(
         string="Open Transfers",
-        related="picking_ids",
+        comodel_name="stock.picking",
+        inverse_name="release_channel_id",
         readonly=True,
         domain=[("state", "not in", ("done", "cancel"))],
     )

diff --git a/stock_release_channel/tests/test_release_channel.py b/stock_release_channel/tests/test_release_channel.py
@@ -126,3 +126,13 @@ def test_recompute_channel(self):
         self.env.company.recompute_channel_on_pickings_at_release = True
         move.release_available_to_promise()
         self.assertEqual(move.picking_id.release_channel_id, channel)
+
+    def test_open_picking(self):
+        self.assertFalse(self.default_channel.open_picking_ids)
+        move = self._create_single_move(self.product1, 10)
+        move.picking_id.assign_release_channel()
+        self.assertEqual(move.picking_id.release_channel_id.id, self.default_channel.id)
+        self.assertEqual(self.default_channel.open_picking_ids, move.picking_id)
+        move.quantity_done = move.product_uom_qty
+        move.picking_id._action_done()
+        self.assertFalse(self.default_channel.open_picking_ids)