[MIG] maintenance_stock: Migration to 14.0

OCA · Aug 16, 2023 · c3b5219 · c3b5219
1 parent b6a5612
commit c3b5219
Show file tree

Hide file tree

Showing 9 changed files with 77 additions and 94 deletions.
diff --git a/maintenance_stock/__manifest__.py b/maintenance_stock/__manifest__.py
@@ -5,18 +5,23 @@
     "summary": "Links maintenance requests to stock",
     "author": "Odoo Community Association (OCA), Solvos",
     "license": "AGPL-3",
-    "version": "13.0.1.0.0",
+    "version": "14.0.1.0.0",
     "category": "Warehouse",
     "website": "https://github.com/OCA/maintenance",
-    "depends": ["base_maintenance", "stock",],
+    "depends": [
+        "base_maintenance",
+        "stock",
+    ],
     "data": [
         "views/maintenance_equipment_views.xml",
         "views/maintenance_request_views.xml",
         "views/stock_move_views.xml",
         "views/stock_move_line_views.xml",
         "views/stock_picking_views.xml",
     ],
-    "demo": ["data/demo_maintenance_stock.xml",],
+    "demo": [
+        "data/demo_maintenance_stock.xml",
+    ],
     "post_init_hook": "post_init_hook",
     "installable": True,
 }
diff --git a/maintenance_stock/data/demo_maintenance_stock.xml b/maintenance_stock/data/demo_maintenance_stock.xml
@@ -1,11 +1,17 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <odoo>
     <data noupdate="1">
+        <record id="location_inventory" model="stock.location">
+            <field name="name">Inventory adjustment</field>
+            <field name="usage">inventory</field>
 
+        </record>
         <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="code">Main</field>
+            <field name="reception_steps">one_step</field>
+            <field name="delivery_steps">one_step</field>
         </record>
 
         <function model="ir.model.data" name="_update_xmlids">
@@ -37,18 +43,20 @@
             <field name="uom_id" ref="uom.product_uom_unit" />
             <field name="uom_po_id" ref="uom.product_uom_unit" />
             <field
-                name="image"
+                name="image_1920"
                 type="base64"
                 file="maintenance_stock/static/img/toner.png"
             />
+
         </record>
 
         <record id="stock_inventory_toner" model="stock.inventory">
             <field name="name">Inventory for Toner</field>
             <field
                 name="product_ids"
-                eval="[(4, ref('maintenance_stock.product_toner'))]"
+                eval="[(6,0,[ref('maintenance_stock.product_toner'),])]"
             />
+
         </record>
 
         <record id="stock_inventory_toner_line" model="stock.inventory.line">
@@ -58,8 +66,14 @@
             <field name="product_qty">50.0</field>
             <field name="location_id" ref="maintenance_stock.main_location_stock" />
         </record>
-
-        <function model="stock.inventory" name="_action_start">
+        <function name="action_start" model="stock.inventory">
+            <function
+                eval="[[('id', '=', ref('maintenance_stock.stock_inventory_toner'))]]"
+                model="stock.inventory"
+                name="search"
+            />
+        </function>
+        <function model="stock.inventory" name="action_validate">
             <function
                 eval="[[('state','=','draft'),('id', '=', ref('maintenance_stock.stock_inventory_toner'))]]"
                 model="stock.inventory"
@@ -69,7 +83,7 @@
 
         <function model="stock.inventory" name="action_validate">
             <function
-                eval="[[('state','=','confirm'),('id', '=', ref('maintenance_stock.stock_inventory_toner'))]]"
+                eval="[[('state','in', ['draft, 'confirm']),('id', '=', ref('maintenance_stock.stock_inventory_toner'))]]"
                 model="stock.inventory"
                 name="search"
             />
@@ -81,6 +95,7 @@
                 name="default_consumption_warehouse_id"
                 ref="maintenance_stock.stock_warehouse_maintenance"
             />
+
         </record>
 
         <record id="m_request_1" model="maintenance.request">
@@ -90,12 +105,13 @@
             <field name="equipment_id" ref="maintenance.equipment_printer1" />
             <field name="color">7</field>
             <field name="stage_id" ref="maintenance.stage_1" />
-            <field
-                name="maintenance_team_id"
-                ref="maintenance.equipment_team_maintenance"
-            />
-        </record>
 
+        </record>
+        <record model="stock.quant" id="stock_quant_toner">
+            <field name="product_id" ref="maintenance_stock.product_toner" />
+            <field name="location_id" ref="maintenance_stock.main_location_stock" />
+            <field name="quantity">1.0</field>
+        </record>
         <record id="consuption_picking_m_request_1" model="stock.picking">
             <field
                 name="picking_type_id"
@@ -105,6 +121,7 @@
             <field name="location_id" ref="maintenance_stock.main_location_stock" />
             <field name="location_dest_id" ref="maintenance_stock.main_location_cons" />
             <field name="maintenance_request_id" ref="maintenance_stock.m_request_1" />
+
             <field
                 name="move_lines"
                 model="stock.move"
@@ -116,13 +133,7 @@
                 'picking_type_id': ref('maintenance_stock.main_picking_type_cons'),
                 'location_id': ref('maintenance_stock.main_location_stock'),
                 'location_dest_id': ref('maintenance_stock.main_location_cons'),
-                'move_line_ids': [(0, 0, {
-                    'product_id': ref('maintenance_stock.product_toner'),
-                    'qty_done': 1,
-                    'product_uom_id': ref('uom.product_uom_unit'),
-                    'location_id': ref('maintenance_stock.main_location_stock'),
-                    'location_dest_id': ref('maintenance_stock.main_location_cons'),
-                })],
+
             })]"
             />
         </record>
@@ -134,8 +145,14 @@
                 obj().env.ref('maintenance_stock.consuption_picking_m_request_1').id]"
             />
         </function>
-
-        <function model="stock.picking" name="action_done">
+        <function model="stock.picking" name="action_assign">
+            <value
+                model="stock.picking"
+                eval="[
+                obj().env.ref('maintenance_stock.consuption_picking_m_request_1').id]"
+            />
+        </function>
+        <function model="stock.picking" name="button_validate">
             <value
                 model="stock.picking"
                 eval="[

diff --git a/maintenance_stock/models/maintenance_equipment.py b/maintenance_stock/models/maintenance_equipment.py
@@ -6,11 +6,6 @@
 class MaintenanceEquipment(models.Model):
     _inherit = "maintenance.equipment"
 
-    stock_picking_ids = fields.One2many(
-        comodel_name="stock.picking",
-        inverse_name="maintenance_equipment_id",
-        groups="stock.group_stock_user",
-    )
     allow_consumptions = fields.Boolean(
         groups="stock.group_stock_user",
     )

diff --git a/maintenance_stock/models/stock_warehouse.py b/maintenance_stock/models/stock_warehouse.py
@@ -20,7 +20,9 @@ def _create_or_update_sequences_and_picking_types(self):
         # 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),}
+                {
+                    "return_picking_type_id": warehouse_data.get("in_type_id", False),
+                }
             )
         return warehouse_data
 
@@ -41,7 +43,7 @@ def _get_picking_type_create_values(self, max_sequence):
                     "use_existing_lots": True,
                     "default_location_src_id": self.lot_stock_id.id,
                     "default_location_dest_id": self.wh_cons_loc_id.id,
-                    "sequence": max_sequence_new,
+                    "sequence_code": max_sequence_new,
                     "barcode": self.code.replace(" ", "").upper() + "-CONS",
                 },
             },
@@ -64,8 +66,8 @@ def _get_sequence_values(self):
             },
         }
 
-    def _get_locations_values(self, vals):
-        sub_locations = super()._get_locations_values(vals)
+    def _get_locations_values(self, vals, code=False):
+        sub_locations = super()._get_locations_values(vals, code)
         code = vals.get("code") or self.code
         code = code.replace(" ", "").upper()
         company_id = vals.get("company_id", self.company_id.id)

diff --git a/maintenance_stock/tests/test_maintenance_stock.py b/maintenance_stock/tests/test_maintenance_stock.py
@@ -1,15 +1,13 @@
 # © 2020 Solvos Consultoría Informática (<http://www.solvos.es>)
 # License AGPL-3 - See http://www.gnu.org/licenses/agpl-3.0.html
 import odoo.tests.common as test_common
+from odoo.tests.common import Form
 
 
 class TestMaintenanceStock(test_common.TransactionCase):
     def setUp(self):
         super().setUp()
-
         self.maintenance_warehouse = self.env["stock.warehouse"].create(
-            {"name": "Test warehouse", "code": "TEST",}
-            {"name": "Test warehouse", "code": "TEST"}
             {
                 "name": "Test warehouse",
                 "code": "TEST",
@@ -30,6 +28,7 @@ def setUp(self):
             {
                 "name": "Test equipment",
                 "allow_consumptions": True,
+                "equipment_assign_to": "employee",
                 "default_consumption_warehouse_id": self.maintenance_warehouse.id,
             }
         )
@@ -109,61 +108,26 @@ def test_request(self):
 
     def test_picking(self):
         self.assertEqual(len(self.request_1.stock_picking_ids), 0)
-        self.assertEqual(len(self.equipment_1.stock_picking_ids), 0)
-
-        qty_done = 5.0
-        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,
-        }
-        location_id = self.maintenance_warehouse.lot_stock_id.id
-        location_dest_id = self.maintenance_warehouse.wh_cons_loc_id.id
-        picking_type_id = self.maintenance_warehouse.cons_type_id.id
-        picking = self.env["stock.picking"].create(
+        location_id = self.maintenance_warehouse.lot_stock_id
+        location_dest_id = self.maintenance_warehouse.wh_cons_loc_id
+        picking_type_id = self.maintenance_warehouse.cons_type_id
+        self.env["stock.quant"].create(
             {
-                "maintenance_request_id": self.request_1.id,
-                "picking_type_id": picking_type_id,
-                "location_id": location_id,
-                "location_dest_id": location_dest_id,
-                "move_lines": [
-                    (
-                        0,
-                        0,
-                        {
-                            "name": "Test move",
-                            "product_id": self.product1.id,
-                            "product_uom": self.env.ref("uom.product_uom_unit").id,
-                            "product_uom_qty": 5.0,
-                            "picking_type_id": picking_type_id,
-                            "location_id": location_id,
-                            "location_dest_id": location_dest_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": location_id,
-                                        "location_dest_id": location_dest_id,
-                                    },
-                                )
-                            ],
-                        },
-                    )
-                ],
+                "product_id": self.product1.id,
+                "location_id": location_id.id,
+                "quantity": 5,
             }
         )
-
+        picking_form = Form(self.env["stock.picking"])
+        picking_form.picking_type_id = picking_type_id
+        picking_form.location_id = location_id
+        picking_form.location_dest_id = location_dest_id
+        with picking_form.move_ids_without_package.new() as move:
+            move.product_id = self.product1
+            move.product_uom_qty = 5.0
+        picking = picking_form.save()
+        picking.write({"maintenance_request_id": self.request_1.id})
         self.assertEqual(len(self.request_1.stock_picking_ids), 1)
-        self.assertEqual(len(self.equipment_1.stock_picking_ids), 1)
-
         stock_quant_obj = self.env["stock.quant"]
         domain_from = [
             ("product_id", "=", self.product1.id),
@@ -173,11 +137,12 @@ def test_picking(self):
             ("product_id", "=", self.product1.id),
             ("location_id", "=", self.maintenance_warehouse.wh_cons_loc_id.id),
         ]
-        self.assertEqual(stock_quant_obj.search(domain_from).quantity, 0)
+        self.assertEqual(stock_quant_obj.search(domain_from).quantity, 5)
         self.assertEqual(stock_quant_obj.search(domain_to).quantity, 0)
 
         picking.action_confirm()
-        picking.action_done()
-
-        self.assertEqual(stock_quant_obj.search(domain_from).quantity, -qty_done)
-        self.assertEqual(stock_quant_obj.search(domain_to).quantity, qty_done)
+        picking.action_assign()
+        picking.move_line_ids.write({"qty_done": 5.0})
+        picking.button_validate()
+        self.assertEqual(stock_quant_obj.search(domain_from).quantity, 0)
+        self.assertEqual(stock_quant_obj.search(domain_to).quantity, 5)
diff --git a/maintenance_stock/views/maintenance_request_views.xml b/maintenance_stock/views/maintenance_request_views.xml
@@ -6,7 +6,7 @@
         <field name="inherit_id" ref="maintenance.hr_equipment_request_view_search" />
         <field name="groups_id" eval="[(4, ref('stock.group_stock_user'))]" />
         <field name="arch" type="xml">
-            <xpath expr="//filter[@name='top_priority']" position="after">
+            <xpath expr="//filter[@name='high_priority']" position="after">
                 <separator />
                 <filter
                     string="Allow Consumptions"

diff --git a/maintenance_stock/views/stock_move_line_views.xml b/maintenance_stock/views/stock_move_line_views.xml
@@ -62,7 +62,6 @@
         <field name="name">Product Moves</field>
         <field name="res_model">stock.move.line</field>
         <field name="type">ir.actions.act_window</field>
-        <field name="view_type">form</field>
         <field name="view_mode">tree,kanban,pivot,form</field>
         <field name="view_id" ref="stock.view_move_line_tree" />
         <field name="domain">[('maintenance_request_id', '!=', False)]</field>

diff --git a/maintenance_stock/views/stock_move_views.xml b/maintenance_stock/views/stock_move_views.xml
@@ -49,11 +49,11 @@
             </xpath>
         </field>
     </record>
-
     <record id="stock_move_action" model="ir.actions.act_window">
         <field name="name">Stock Moves</field>
         <field name="res_model">stock.move</field>
         <field name="type">ir.actions.act_window</field>
+        <field name="view_mode">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>

diff --git a/maintenance_stock/views/stock_picking_views.xml b/maintenance_stock/views/stock_picking_views.xml
@@ -5,7 +5,7 @@
         <field name="model">stock.picking</field>
         <field name="inherit_id" ref="stock.view_picking_internal_search" />
         <field name="arch" type="xml">
-            <xpath expr="//field[@name='group_id']" position="after">
+            <xpath expr="//field[@name='picking_type_id']" position="after">
                 <field name="maintenance_equipment_id" />
                 <field name="maintenance_request_id" />
             </xpath>