[17.0][MIG] stock_quant_manual_assign: Migration to 17.0

stock_quant_manual_assign/__manifest__.py

@@ -7,7 +7,7 @@
 
 {
     "name": "Stock - Manual Quant Assignment",
-    "version": "16.0.1.0.1",
+    "version": "17.0.1.0.0",
     "category": "Warehouse",
     "license": "AGPL-3",
     "author": "AvanzOSC, "

stock_quant_manual_assign/tests/test_stock_quant_manual_assign.py

@@ -48,7 +48,7 @@ def setUpClass(cls):
                 "location_id": cls.location_src.id,
             }
         )
-        cls.picking_type = cls.env.ref("stock.picking_type_out")
+        cls.picking_type = cls.env.ref("stock.warehouse0").out_type_id
         cls.quant1 = cls.quant_model.sudo().create(
             {
                 "product_id": cls.product.id,
@@ -100,7 +100,7 @@ def test_quant_assign_wizard(self):
         self.assertEqual(wizard.lines_qty, 0.0, "None selected must give 0")
         self.assertEqual(
             sum(line.qty for line in wizard.quants_lines),
-            self.move.reserved_availability,
+            self.move.quantity,
         )
         self.assertEqual(wizard.move_qty, self.move.product_uom_qty)
 
@@ -149,7 +149,7 @@ def test_quant_manual_assign(self):
             "There are 2 quants selected",
         )
         self.assertFalse(self.move.picking_type_id.auto_fill_qty_done)
-        self.assertEqual(sum(self.move.move_line_ids.mapped("qty_done")), 0.0)
+        self.assertEqual(sum(self.move.move_line_ids.mapped("quantity")), 0.0)
 
     def test_quant_manual_assign_auto_fill_qty_done(self):
         wizard = self.quant_assign_wizard.with_context(active_id=self.move.id).create(
@@ -162,7 +162,7 @@ def test_quant_manual_assign_auto_fill_qty_done(self):
         self.picking_type.auto_fill_qty_done = True
         wizard.assign_quants()
         self.assertTrue(self.move.picking_type_id.auto_fill_qty_done)
-        self.assertEqual(sum(self.move.move_line_ids.mapped("qty_done")), 150.0)
+        self.assertEqual(sum(self.move.move_line_ids.mapped("quantity")), 150.0)
 
     def test_quant_assign_wizard_after_availability_check(self):
         self.move._action_assign()
@@ -186,5 +186,5 @@ def test_quant_assign_wizard_after_availability_check(self):
         )
         self.assertEqual(
             sum(line.qty for line in wizard.quants_lines),
-            self.move.reserved_availability,
+            self.move.quantity,
         )

stock_quant_manual_assign/views/stock_move_view.xml

@@ -5,14 +5,14 @@
         <field name="model">stock.picking</field>
         <field name="inherit_id" ref="stock.view_picking_form" />
         <field name="arch" type="xml">
-            <button name="action_show_details" position="before">
+            <button name="action_assign_serial" position="before">
                 <button
                     name="%(assign_manual_quants_action)d"
                     type="action"
                     icon="fa-tags"
                     title="Manual Quants"
                     options='{"warn": true}'
-                    attrs="{'invisible':['|',('picking_code','=','incoming'),('state','not in',('confirmed','assigned','partially_available'))]}"
+                    invisible="picking_code == 'incoming' or state not in ['confirmed','assigned','partially_available']"
                 />
             </button>
         </field>

stock_quant_manual_assign/views/stock_picking_type_views.xml

@@ -5,11 +5,9 @@
         <field name="model">stock.picking.type</field>
         <field name="inherit_id" ref="stock.view_picking_type_form" />
         <field name="arch" type="xml">
-            <xpath expr="//field[@name='show_reserved']" position="after">
-                <field
-                    name="auto_fill_qty_done"
-                    attrs="{'invisible':[('code','=','incoming')]}"
-                />
+            <xpath expr="//field[@name='create_backorder']" position="after">
+                <field name="code" invisible="1" />
+                <field name="auto_fill_qty_done" invisible="code == 'incoming'" />
             </xpath>
         </field>
     </record>

stock_quant_manual_assign/wizard/assign_manual_quants.py

@@ -63,10 +63,6 @@ def assign_quants(self):
         move._do_unreserve()
         for line in self.quants_lines:
             line._assign_quant_line()
-        if move.picking_type_id.auto_fill_qty_done:
-            # Auto-fill all lines as done
-            for ml in move.move_line_ids:
-                ml.qty_done = ml.reserved_uom_qty
         move._recompute_state()
         move.mapped("picking_id")._compute_state()
         return {}
@@ -115,7 +111,7 @@ def _prepare_wizard_line(self, move, quant):
                 and ml.package_id == quant.package_id
             )
         )
-        line["qty"] = sum(move_lines.mapped("reserved_uom_qty"))
+        line["qty"] = sum(move_lines.mapped("quantity_product_uom"))
         line["selected"] = bool(line["qty"])
         line["reserved"] = quant.reserved_quantity - line["qty"]
         return line
@@ -200,11 +196,11 @@ def _check_qty(self):
             quant = record.quant_id
             move_lines = record.assign_wizard.move_id.move_line_ids.filtered(
                 lambda ml: (
-                    ml.location_id == quant.location_id and ml.lot_id == quant.lot_id
+                    ml.location_id == quant.location_id and ml.lot_id == quant.lot_id  # noqa: B023
                 )
             )
             reserved = quant.reserved_quantity - sum(
-                move_lines.mapped("reserved_uom_qty")
+                move_lines.mapped("quantity_product_uom")
             )
             if (
                 float_compare(
@@ -247,8 +243,8 @@ def _assign_quant_line(self):
                 return
             move._update_reserved_quantity(
                 self.qty,
-                available_quantity,
                 self.quant_id.location_id,
+                self.quant_id,
                 lot_id=self.quant_id.lot_id,
                 package_id=self.quant_id.package_id,
                 owner_id=self.quant_id.owner_id,

stock_quant_manual_assign/wizard/assign_manual_quants_view.xml

@@ -12,14 +12,10 @@
                         <field name="package_id" groups="stock.group_tracking_lot" />
                         <field name="owner_id" groups="stock.group_tracking_owner" />
                         <field name="location_id" />
-                        <field name="on_hand" force_save="1" />
-                        <field name="reserved" force_save="1" />
-                        <field name="selected" widget="boolean_toggle" />
-                        <field
-                            name="qty"
-                            attrs="{'readonly':[('selected', '=', False)]}"
-                            sum="qty"
-                        />
+                        <field name="on_hand" />
+                        <field name="reserved" />
+                        <field name="selected" />
+                        <field name="qty" readonly="selected == False" sum="qty" />
                     </tree>
                 </field>
                 <group col='4' colspan="4">