[IMP] stock_account_quantity_history_location: black, isort, prettier

stock_account_quantity_history_location/__init__.py

@@ -1 +1 @@
-from . import wizards
+from . import wizards

stock_account_quantity_history_location/__manifest__.py

@@ -3,22 +3,15 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 {
-    'name': 'Stock Account Quantity History Location',
-    'summary': """
+    "name": "Stock Account Quantity History Location",
+    "summary": """
         Glue module between Stock Account and Stock Quantity History Location
         modules""",
-    'version': '12.0.1.1.0',
-    'license': 'AGPL-3',
-    'author': 'Eficent,'
-              'Odoo Community Association (OCA)',
-    'website': 'https://github.com/OCA/stock-logistics-reporting',
-    'depends': [
-        'stock_account',
-        'stock_quantity_history_location',
-    ],
-    'data': [
-        'wizards/stock_quantity_history.xml',
-    ],
-    'auto_install': True,
-
+    "version": "12.0.1.1.0",
+    "license": "AGPL-3",
+    "author": "Eficent," "Odoo Community Association (OCA)",
+    "website": "https://github.com/OCA/stock-logistics-reporting",
+    "depends": ["stock_account", "stock_quantity_history_location",],
+    "data": ["wizards/stock_quantity_history.xml",],
+    "auto_install": True,
 }

stock_account_quantity_history_location/tests/test_valuation_location.py

@@ -1,40 +1,41 @@
 # Copyright 2020 Tecnativa - David Vidal
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 from odoo.addons.stock_quantity_history_location.tests import (
-    test_stock_quantity_history_location
+    test_stock_quantity_history_location,
 )
 
 
-class TestValuationLocation(test_stock_quantity_history_location
-                            .TestStockQuantityHistoryLocation):
+class TestValuationLocation(
+    test_stock_quantity_history_location.TestStockQuantityHistoryLocation
+):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
-        cls.env['product.price.history'].create({
-            'datetime': '2019-08-12',
-            'product_id': cls.product.id,
-            'cost': 35.01,
-        })
-        cls.env['product.price.history'].create({
-            'datetime': '2020-01-01',
-            'product_id': cls.product.id,
-            'cost': 33.33,
-        })
-        cls.product.categ_id.property_cost_method = 'standard'
-        cls.test_stock_loc2 = cls.env['stock.location'].create({
-            'usage': 'internal',
-            'name': 'Test Stock Location 2',
-            'company_id': cls.main_company.id
-        })
+        cls.env["product.price.history"].create(
+            {"datetime": "2019-08-12", "product_id": cls.product.id, "cost": 35.01,}
+        )
+        cls.env["product.price.history"].create(
+            {"datetime": "2020-01-01", "product_id": cls.product.id, "cost": 33.33,}
+        )
+        cls.product.categ_id.property_cost_method = "standard"
+        cls.test_stock_loc2 = cls.env["stock.location"].create(
+            {
+                "usage": "internal",
+                "name": "Test Stock Location 2",
+                "company_id": cls.main_company.id,
+            }
+        )
         # Create a move for the past
-        move = cls.env['stock.move'].create({
-            'name': 'Stock move in',
-            'location_id': cls.supplier_location.id,
-            'location_dest_id': cls.test_stock_loc2.id,
-            'product_id': cls.product.id,
-            'product_uom': cls.product.uom_id.id,
-            'product_uom_qty': 50.0,
-        })
+        move = cls.env["stock.move"].create(
+            {
+                "name": "Stock move in",
+                "location_id": cls.supplier_location.id,
+                "location_dest_id": cls.test_stock_loc2.id,
+                "product_id": cls.product.id,
+                "product_uom": cls.product.uom_id.id,
+                "product_uom_qty": 50.0,
+            }
+        )
         move._action_confirm()
         move._action_assign()
         move_line = move.move_line_ids[0]
@@ -43,13 +44,15 @@ class TestValuationLocation(test_stock_quantity_history_location
         move.date = "2020-01-01"
 
     def test_stock_location_valuation(self):
-        wizard = self.env['stock.quantity.history'].create({
-            "location_id": self.test_stock_loc2.id,
-            "include_child_locations": True,
-            "compute_at_date": 1,
-            "date": "2020-01-01",
-        })
-        action = wizard.with_context(
-            company_owned=True, valuation=True).open_table()
+        wizard = self.env["stock.quantity.history"].create(
+            {
+                "location_id": self.test_stock_loc2.id,
+                "include_child_locations": True,
+                "compute_at_date": 1,
+                "date": "2020-01-01",
+            }
+        )
+        action = wizard.with_context(company_owned=True, valuation=True).open_table()
         self.assertAlmostEqual(
-            self.product.with_context(action['context']).stock_value, 1666.5)
+            self.product.with_context(action["context"]).stock_value, 1666.5
+        )

stock_account_quantity_history_location/wizards/__init__.py

@@ -1,2 +1 @@
 from . import stock_quantity_history
-

stock_account_quantity_history_location/wizards/stock_quantity_history.py

@@ -2,35 +2,38 @@
 # Copyright 2019 Aleph Objects, Inc.
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 import ast
+
 from odoo import models
 
 
 class StockQuantityHistory(models.TransientModel):
-    _inherit = 'stock.quantity.history'
+    _inherit = "stock.quantity.history"
 
     def open_table(self):
         """Stock valuation goes by a different path than stock report, so
            we ensure the correct context as well"""
         action = super().open_table()
-        if not self.env.context.get('valuation'):
+        if not self.env.context.get("valuation"):
             return action
         # Show 0 quantities on Inventory Valuation to display Account Valuation
         # anomalies, such as, non 0 stock_value on cost_method FIFO
         if not self.location_id:
-            domain = ast.literal_eval(action['domain'])
-            domain.pop(domain.index(('qty_available', '!=', 0)))
-            action['domain'] = domain
-        ctx = action['context']
+            domain = ast.literal_eval(action["domain"])
+            domain.pop(domain.index(("qty_available", "!=", 0)))
+            action["domain"] = domain
+        ctx = action["context"]
         if isinstance(ctx, str):
             ctx = ast.literal_eval(ctx)
         if self.location_id:
-            ctx['location'] = self.location_id.id
-            ctx['compute_child'] = self.include_child_locations
-            if ctx.get('company_owned', False):
-                ctx.pop('company_owned')
+            ctx["location"] = self.location_id.id
+            ctx["compute_child"] = self.include_child_locations
+            if ctx.get("company_owned", False):
+                ctx.pop("company_owned")
             # Ensure the context isn't added later and catch it
-            ctx['drop_company_owned'] = True
-            action['name'] = '%s (%s)' % (action['name'],
-                                          self.location_id.complete_name)
-            action['context'] = ctx
+            ctx["drop_company_owned"] = True
+            action["name"] = "%s (%s)" % (
+                action["name"],
+                self.location_id.complete_name,
+            )
+            action["context"] = ctx
         return action

stock_account_quantity_history_location/wizards/stock_quantity_history.xml

@@ -1,14 +1,16 @@
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8" ?>
 <odoo>
     <record id="view_stock_account_quantity_history_location" model="ir.ui.view">
         <field name="name">Valuation Report</field>
         <field name="model">stock.quantity.history</field>
-        <field name="inherit_id" ref="stock_account.view_stock_quantity_history"/>
+        <field name="inherit_id" ref="stock_account.view_stock_quantity_history" />
         <field name="arch" type="xml">
             <field name="compute_at_date" position="before">
-                <field name="location_id"/>
-                <field name="include_child_locations"
-                       attrs="{'invisible': [('location_id', '=', False)]}"/>
+                <field name="location_id" />
+                <field
+                    name="include_child_locations"
+                    attrs="{'invisible': [('location_id', '=', False)]}"
+                />
             </field>
         </field>
     </record>