[FIX] stock_average_daily_sale: Use a new cursor to avoid closed one

stock_average_daily_sale/models/stock_average_daily_sale.py

@@ -6,7 +6,7 @@ import logging
 from psycopg2.errors import ObjectNotInPrerequisiteState
 from psycopg2.extensions import AsIs
 
-from odoo import _, api, fields, models
+from odoo import _, api, fields, models, registry
 
 from odoo.addons.stock_storage_type_putaway_abc.models.stock_location import (
     ABC_SELECTION,
@@ -101,8 +101,9 @@ class StockAverageDailySale(models.Model):
     @api.model
     def _check_view(self):
         try:
-            with self.env.cr.savepoint():
+            cr = registry(self._cr.dbname).cursor()
-                self.env.cr.execute("SELECT COUNT(1) FROM %s", (AsIs(self._table),))
+            new_self = self.with_env(self.env(cr=cr))  # TDE FIXME
+            new_self.env.cr.execute("SELECT COUNT(1) FROM %s", (AsIs(self._table),))
             return True
         except ObjectNotInPrerequisiteState:
             _logger.warning(
@@ -111,6 +112,8 @@ class StockAverageDailySale(models.Model):
             return False
         except Exception as e:
             raise e
+        finally:
+            new_self.env.cr.close()
 
     # pylint: disable=redefined-outer-name
     @api.model

stock_average_daily_sale/tests/test_average_daily_sale.py

@@ -218,3 +218,9 @@ class TestAverageSale(CommonAverageSaleTest, TransactionCase):
             str("The materialized view has not been populated. Launch the cron."),
             str(logger.output),
         )
+        # Check if we can still query database
+        product = self.env["product.product"].search([("id", "=", self.product_1.id)])
+        self.assertEqual(
+            product,
+            self.product_1,
+        )