[MIG] stock_quantity_history_location: Migration to 17.0

- refactored .js code by inheriting `get_views` method to display a different button text for those users having the Multi Locations group enabled. Added a test case to cover the change
OCA · Oct 21, 2024 · f28ee0e · f28ee0e
1 parent 910696f
commit f28ee0e
Show file tree

Hide file tree

Showing 12 changed files with 86 additions and 50 deletions.
diff --git a/stock_quantity_history_location/README.rst b/stock_quantity_history_location/README.rst
@@ -39,7 +39,14 @@ report by location, for a past date or for current date.
 Usage
 =====
 
--  Go to: *Inventory / Reporting / Inventory or Inventory Valuation*
+First, you need to activate Storage Locations. To do so, go to Inventory
+/ Configuration / Settings / Warehouse and activate it.
+
+To use this module, go to:
+
+-  *Inventory / Reporting / Locations*
+
+-  Select Inventory at Date & Location
 
 -  Filter by location
 

diff --git a/stock_quantity_history_location/__init__.py b/stock_quantity_history_location/__init__.py
@@ -1 +1,2 @@
 from . import wizards
+from . import models
diff --git a/stock_quantity_history_location/__manifest__.py b/stock_quantity_history_location/__manifest__.py
@@ -5,7 +5,7 @@
 {
     "name": "Stock Quantity History Location",
     "summary": "Provides stock quantity by location on past date",
-    "version": "16.0.1.0.1",
+    "version": "17.0.1.0.0",
     "license": "AGPL-3",
     "author": "ForgeFlow, Odoo Community Association (OCA)",
     "maintainers": [
@@ -15,12 +15,4 @@
     "website": "https://github.com/OCA/stock-logistics-reporting",
     "depends": ["stock"],
     "data": ["wizards/stock_quantity_history.xml"],
-    "assets": {
-        "web.assets_backend": [
-            "stock_quantity_history_location/static/src/components"
-            "/inventory_report/inventory_report.esm.js",
-            "stock_quantity_history_location/static/src/components/"
-            "inventory_report/inventory_report.xml",
-        ],
-    },
 }
diff --git a/stock_quantity_history_location/models/__init__.py b/stock_quantity_history_location/models/__init__.py
@@ -0,0 +1 @@
+from . import stock_quant
diff --git a/stock_quantity_history_location/models/stock_quant.py b/stock_quantity_history_location/models/stock_quant.py
@@ -0,0 +1,25 @@
+# Copyright 2024 ForgeFlow S.L.
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from lxml import etree
+
+from odoo import api, models
+
+
+class StockQuant(models.Model):
+    _inherit = "stock.quant"
+
+    @api.model
+    def get_views(self, views, options=None):
+        res = super().get_views(views, options=options)
+        user_has_group = self.user_has_groups("stock.group_stock_multi_locations")
+        list_view = res.get("views", {}).get("list", {})
+        if user_has_group and list_view:
+            arch = list_view.get("arch", "")
+            arch_tree = etree.XML(arch)
+            buttons = arch_tree.xpath('//button[@name="action_inventory_at_date"]')
+            for button in buttons:
+                button.set("string", "Inventory at Date & Location")
+            new_arch = etree.tostring(arch_tree, encoding="unicode")
+            list_view["arch"] = new_arch
+        return res
diff --git a/stock_quantity_history_location/readme/USAGE.md b/stock_quantity_history_location/readme/USAGE.md
@@ -1,9 +1,15 @@
-- Go to: *Inventory / Reporting / Inventory or Inventory Valuation*
+First, you need to activate Storage Locations. To do so, go to
+Inventory / Configuration / Settings / Warehouse and activate it.
+
+To use this module, go to:
+- *Inventory / Reporting / Locations*
+
+- Select Inventory at Date & Location
 
 - Filter by location
 
 - **Optionally: Mark if you want to include child location**
 
-- Choose a moment in time:  
+- Choose a moment in time:
   - Current Inventory
   - At a Specific Date
diff --git a/stock_quantity_history_location/static/description/index.html b/stock_quantity_history_location/static/description/index.html
@@ -1,4 +1,3 @@
-<?xml version="1.0" encoding="utf-8"?>
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
 <head>
@@ -387,8 +386,12 @@ <h1 class="title">Stock Quantity History Location</h1>
 </div>
 <div class="section" id="usage">
 <h1><a class="toc-backref" href="#toc-entry-1">Usage</a></h1>
+<p>First, you need to activate Storage Locations. To do so, go to Inventory
+/ Configuration / Settings / Warehouse and activate it.</p>
+<p>To use this module, go to:</p>
 <ul class="simple">
-<li>Go to: <em>Inventory / Reporting / Inventory or Inventory Valuation</em></li>
+<li><em>Inventory / Reporting / Locations</em></li>
+<li>Select Inventory at Date &amp; Location</li>
 <li>Filter by location</li>
 <li><strong>Optionally: Mark if you want to include child location</strong></li>
 <li>Choose a moment in time:<ul>

diff --git a/..._quantity_history_location/static/src/components/inventory_report/inventory_report.esm.js b/..._quantity_history_location/static/src/components/inventory_report/inventory_report.esm.js
diff --git a/stock_quantity_history_location/static/src/components/inventory_report/inventory_report.xml b/stock_quantity_history_location/static/src/components/inventory_report/inventory_report.xml
diff --git a/stock_quantity_history_location/tests/common.py b/stock_quantity_history_location/tests/common.py
@@ -22,7 +22,8 @@ def _create_stock_move(self, location_dest_id, qty):
         )
         move._action_confirm()
         move._action_assign()
-        move_line = move.move_line_ids[0]
-        move_line.qty_done = qty
+        for move_line in move.move_line_ids:
+            move_line.quantity = qty
+            move_line.picked = True
         move._action_done()
         return move
diff --git a/stock_quantity_history_location/tests/test_stock_quantity_history_location.py b/stock_quantity_history_location/tests/test_stock_quantity_history_location.py
@@ -2,7 +2,9 @@
 # Copyright 2021 Tecnativa - Víctor Martínez
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
-from odoo import fields
+from lxml import etree
+
+from odoo import Command, fields
 
 from .common import TestCommon
 
@@ -30,6 +32,7 @@ def setUpClass(cls):
             }
         )
         cls._create_stock_move(cls, location_dest_id=cls.child_test_stock_loc, qty=100)
+        cls.group_multi_locations = cls.env.ref("stock.group_stock_multi_locations")
 
     def test_01_wizard_past_date(self):
         wizard = self.env["stock.quantity.history"].create(
@@ -63,3 +66,29 @@ def test_02_wizard_current(self):
         action = wizard.with_context().open_at_date()
         self.assertEqual(action["context"]["compute_child"], True)
         self.assertEqual(action["context"]["location"], self.test_stock_loc.id)
+
+    def test_03_get_stock_quant_list_view(self):
+        # 1. Get Stock Quant list view without the `group_stock_multi_locations` group
+        self.env.user.write(
+            {"groups_id": [Command.unlink(self.group_multi_locations.id)]}
+        )
+        views = [[False, "list"]]
+        sq_views = self.env["stock.quant"].get_views(views=views)
+        list_view = sq_views.get("views", {}).get("list", {})
+        arch = list_view.get("arch", "")
+        arch_tree = etree.XML(arch)
+        buttons = arch_tree.xpath('//button[@name="action_inventory_at_date"]')
+        for button in buttons:
+            self.assertEqual(button.get("string"), "Inventory at Date")
+        # 2. Now, with the group
+        self.env.user.write(
+            {"groups_id": [Command.link(self.group_multi_locations.id)]}
+        )
+        views = [[False, "list"]]
+        sq_views = self.env["stock.quant"].get_views(views=views)
+        list_view = sq_views.get("views", {}).get("list", {})
+        arch = list_view.get("arch", "")
+        arch_tree = etree.XML(arch)
+        buttons = arch_tree.xpath('//button[@name="action_inventory_at_date"]')
+        for button in buttons:
+            self.assertEqual(button.get("string"), "Inventory at Date & Location")
diff --git a/stock_quantity_history_location/wizards/stock_quantity_history.xml b/stock_quantity_history_location/wizards/stock_quantity_history.xml
@@ -2,15 +2,17 @@
 <odoo>
 
     <record id="view_stock_quantity_history_location" model="ir.ui.view">
-        <field name="name">Inventory Report</field>
+        <field
+            name="name"
+        >stock.quantity.history.form - stock_quantity_history_location</field>
         <field name="model">stock.quantity.history</field>
         <field name="inherit_id" ref="stock.view_stock_quantity_history" />
         <field name="arch" type="xml">
             <field name="inventory_datetime" position="before">
                 <field name="location_id" groups="stock.group_stock_multi_locations" />
                 <field
                     name="include_child_locations"
-                    attrs="{'invisible': [('location_id', '=', False)]}"
+                    invisible="not location_id"
                     groups="stock.group_stock_multi_locations"
                 />
             </field>