[MIG] maintenance_stock: Migration to 13.0

Author: mamcode

maintenance_stock/__manifest__.py

@@ -5,7 +5,7 @@
     "summary": "Links maintenance requests to stock",
     "author": "Odoo Community Association (OCA), Solvos",
     "license": "AGPL-3",
-    "version": "12.0.1.0.0",
+    "version": "13.0.1.0.0",
     "category": "Warehouse",
     "website": "https://github.com/OCA/maintenance",
     "depends": ["base_maintenance", "stock",],

maintenance_stock/data/demo_maintenance_stock.xml

@@ -5,7 +5,6 @@
         <record id="stock_warehouse_maintenance" model="stock.warehouse">
             <field name="name">Maintenance Warehouse</field>
             <field name="partner_id" ref="stock.res_partner_address_41" />
-            <field name="company_id" ref="stock.res_company_1" />
             <field name="code">Main</field>
         </record>
 
@@ -37,7 +36,6 @@
             <field name="standard_price">70.0</field>
             <field name="uom_id" ref="uom.product_uom_unit" />
             <field name="uom_po_id" ref="uom.product_uom_unit" />
-            <field name="property_stock_inventory" ref="stock.location_inventory" />
             <field
                 name="image"
                 type="base64"
@@ -47,8 +45,10 @@
 
         <record id="stock_inventory_toner" model="stock.inventory">
             <field name="name">Inventory for Toner</field>
-            <field name="filter">product</field>
-            <field name="product_id" ref="maintenance_stock.product_toner" />
+            <field
+                name="product_ids"
+                eval="[(4, ref('maintenance_stock.product_toner'))]"
+            />
         </record>
 
         <record id="stock_inventory_toner_line" model="stock.inventory.line">
@@ -59,14 +59,22 @@
             <field name="location_id" ref="maintenance_stock.main_location_stock" />
         </record>
 
-        <function model="stock.inventory" name="action_validate">
+        <function model="stock.inventory" name="_action_start">
             <function
                 eval="[[('state','=','draft'),('id', '=', ref('maintenance_stock.stock_inventory_toner'))]]"
                 model="stock.inventory"
                 name="search"
             />
         </function>
 
+        <function model="stock.inventory" name="action_validate">
+            <function
+                eval="[[('state','=','confirm'),('id', '=', ref('maintenance_stock.stock_inventory_toner'))]]"
+                model="stock.inventory"
+                name="search"
+            />
+        </function>
+
         <record id="maintenance.equipment_printer1" model="maintenance.equipment">
             <field name="allow_consumptions" eval="True" />
             <field

maintenance_stock/models/stock_picking.py

@@ -12,4 +12,5 @@ class StockPicking(models.Model):
     maintenance_equipment_id = fields.Many2one(
         comodel_name="maintenance.equipment",
         related="maintenance_request_id.equipment_id",
+        store=True,
     )

maintenance_stock/models/stock_warehouse.py

@@ -16,8 +16,8 @@ def _create_or_update_sequences_and_picking_types(self):
         PickingType = self.env["stock.picking.type"]
 
         # TODO when is called for an existing warehouse (e.g. during the
-        #      module installation in_type_id is not accesible). Temporary
-        #      solved with a hook
+        # module installation in_type_id is not accesible). Temporary
+        # solved with a hook
         if "cons_type_id" in warehouse_data:
             PickingType.browse(warehouse_data["cons_type_id"]).write(
                 {"return_picking_type_id": warehouse_data.get("in_type_id", False),}

maintenance_stock/tests/test_maintenance_stock.py

@@ -8,7 +8,11 @@ def setUp(self):
         super().setUp()
 
         self.maintenance_warehouse = self.env["stock.warehouse"].create(
+<<<<<<< HEAD
             {"name": "Test warehouse", "code": "TEST",}
+=======
+            {"name": "Test warehouse", "code": "TEST"}
+>>>>>>> 9a9e9cf ([MIG] maintenance_stock: Migration to 13.0)
         )
 
         self.product1 = self.env["product.product"].create(
@@ -107,6 +111,16 @@ def test_picking(self):
         self.assertEqual(len(self.equipment_1.stock_picking_ids), 0)
 
         qty_done = 5.0
+<<<<<<< HEAD
+=======
+        move_line_data = {
+            "product_id": self.product1.id,
+            "product_uom_id": self.env.ref("uom.product_uom_unit").id,
+            "qty_done": qty_done,
+            "location_id": self.maintenance_warehouse.lot_stock_id.id,
+            "location_dest_id": self.maintenance_warehouse.wh_cons_loc_id.id,
+        }
+>>>>>>> 9a9e9cf ([MIG] maintenance_stock: Migration to 13.0)
         picking = self.env["stock.picking"].create(
             {
                 "maintenance_request_id": self.request_1.id,
@@ -125,21 +139,7 @@ def test_picking(self):
                             "picking_type_id": self.maintenance_warehouse.cons_type_id.id,
                             "location_id": self.maintenance_warehouse.lot_stock_id.id,
                             "location_dest_id": self.maintenance_warehouse.wh_cons_loc_id.id,
-                            "move_line_ids": [
-                                (
-                                    0,
-                                    0,
-                                    {
-                                        "product_id": self.product1.id,
-                                        "product_uom_id": self.env.ref(
-                                            "uom.product_uom_unit"
-                                        ).id,
-                                        "qty_done": qty_done,
-                                        "location_id": self.maintenance_warehouse.lot_stock_id.id,
-                                        "location_dest_id": self.maintenance_warehouse.wh_cons_loc_id.id,
-                                    },
-                                )
-                            ],
+                            "move_line_ids": [(0, 0, move_line_data)],
                         },
                     )
                 ],

maintenance_stock/views/stock_move_views.xml

@@ -54,7 +54,6 @@
         <field name="name">Stock Moves</field>
         <field name="res_model">stock.move</field>
         <field name="type">ir.actions.act_window</field>
-        <field name="view_type">form</field>
         <field name="view_id" ref="stock.view_move_tree" />
         <field name="search_view_id" ref="stock.view_move_search" />
         <field name="domain">[('maintenance_request_id', '!=', False)]</field>