[IMP] auditlog: make the line views using a non auto model

auditlog/models/__init__.py

@@ -4,4 +4,5 @@ from . import rule
 from . import http_session
 from . import http_request
 from . import log
+from . import auditlog_log_line_view
 from . import autovacuum

auditlog/models/auditlog_log_line_view.py

@@ -0,0 +1,64 @@
+from odoo import fields, models
+
+
+class AuditlogLogLineView(models.Model):
+    _name = "auditlog.log.line.view"
+    _inherit = "auditlog.log.line"
+    _description = "Auditlog - Log details (fields updated)"
+    _auto = False
+    _log_access = True
+
+    name = fields.Char()
+    model_id = fields.Many2one("ir.model")
+    model_name = fields.Char()
+    model_model = fields.Char()
+    res_id = fields.Integer()
+    user_id = fields.Many2one("res.users")
+    method = fields.Char()
+    http_session_id = fields.Many2one(
+        "auditlog.http.session", string="Session", index=True
+    )
+    http_request_id = fields.Many2one(
+        "auditlog.http.request", string="HTTP Request", index=True
+    )
+    log_type = fields.Selection(
+        selection=lambda r: r.env["auditlog.rule"]._fields["log_type"].selection,
+        string="Type",
+    )
+
+    def _select_query(self):
+        return """
+            alogl.id,
+            alogl.create_date,
+            alogl.create_uid,
+            alogl.write_uid,
+            alogl.write_date,
+            alogl.field_id,
+            alogl.log_id,
+            alogl.old_value,
+            alogl.new_value,
+            alogl.old_value_text,
+            alogl.new_value_text,
+            alogl.field_name,
+            alogl.field_description,
+            alog.name,
+            alog.model_id,
+            alog.model_name,
+            alog.model_model,
+            alog.res_id,
+            alog.user_id,
+            alog.method,
+            alog.http_session_id,
+            alog.http_request_id,
+            alog.log_type
+        """
+
+    def _from_query(self):
+        return """
+            auditlog_log_line alogl
+            JOIN auditlog_log alog ON alog.id = alogl.log_id
+        """
+
+    @property
+    def _table_query(self):
+        return "SELECT %s FROM %s" % (self._select_query(), self._from_query())

auditlog/models/log.py

@@ -66,21 +66,6 @@ class AuditlogLogLine(models.Model):
     new_value_text = fields.Text("New value Text")
     field_name = fields.Char("Technical name", readonly=True)
     field_description = fields.Char("Description", readonly=True)
-    # From log auditlog.log
-    name = fields.Char(related="log_id.name", store=True)
-    model_id = fields.Many2one(related="log_id.model_id", store=True)
-    model_name = fields.Char(related="log_id.model_name", store=True)
-    model_model = fields.Char(related="log_id.model_model", store=True)
-    res_id = fields.Integer(related="log_id.res_id", store=True)
-    user_id = fields.Many2one(related="log_id.user_id", store=True)
-    method = fields.Char(related="log_id.method", store=True)
-    http_session_id = fields.Many2one(
-        related="log_id.http_session_id", store=True, index=True
-    )
-    http_request_id = fields.Many2one(
-        related="log_id.http_request_id", store=True, index=True
-    )
-    log_type = fields.Selection(related="log_id.log_type", store=True)
 
     @api.model_create_multi
     def create(self, vals_list):

auditlog/security/ir.model.access.csv

@@ -11,3 +11,4 @@ access_auditlog_log_line_manager,auditlog_log_line_manager,model_auditlog_log_li
 access_auditlog_http_session_manager,auditlog_http_session_manager,model_auditlog_http_session,auditlog.group_auditlog_manager,1,1,1,1
 access_auditlog_http_request_manager,auditlog_http_request_manager,model_auditlog_http_request,auditlog.group_auditlog_manager,1,1,1,1
 access_auditlog_autovacuum,access_auditlog_autovacuum,model_auditlog_autovacuum,auditlog.group_auditlog_user,1,1,1,1
+access_auditlog_log_line_view_manager,auditlog_log_line_view,model_auditlog_log_line_view,base.group_erp_manager,1,0,0,0

auditlog/views/auditlog_view.xml

@@ -268,7 +268,7 @@
     <!-- auditlog.log.line -->
     <record model="ir.ui.view" id="view_auditlog_line_tree">
         <field name="name">view.auditlog.line.tree</field>
-        <field name="model">auditlog.log.line</field>
+        <field name="model">auditlog.log.line.view</field>
         <field name="arch" type="xml">
             <tree create="0">
                 <field name="create_date" optional="show" />
@@ -292,7 +292,7 @@
     </record>
     <record id="view_auditlog_line_search" model="ir.ui.view">
         <field name="name">auditlog.line.search</field>
-        <field name="model">auditlog.log.line</field>
+        <field name="model">auditlog.log.line.view</field>
         <field name="arch" type="xml">
             <search string="Log Lines">
                 <field name="name" />
@@ -348,7 +348,7 @@
     </record>
     <record id="action_auditlog_line" model="ir.actions.act_window">
         <field name="name">Log Lines</field>
-        <field name="res_model">auditlog.log.line</field>
+        <field name="res_model">auditlog.log.line.view</field>
         <field name="view_mode">tree</field>
         <field name="search_view_id" ref="view_auditlog_line_search" />
         <field name="context">{'search_default_group_by_model_id': 1}</field>