[FIX] function_pattern_chunk_run allow_commit

Author: Kev-Roche

pattern_import_export/data/queue_job_function_data.xml

@@ -15,6 +15,7 @@
         <field name="model_id" ref="model_pattern_chunk" />
         <field name="method">run</field>
         <field name="channel_id" ref="channel_pattern_import" />
+        <field name="allow_commit" eval="True" />
     </record>
     <record id="job_function_pattern_chunk_check_last" model="queue.job.function">
         <field name="model_id" ref="model_pattern_chunk" />

pattern_import_export/demo/demo.xml

@@ -21,6 +21,12 @@
         <field name="model_id">res.company</field>
         <field name="user_id" />
     </record>
+    <record id="demo_filter_categories" model="ir.filters">
+        <field name="name">Main category</field>
+        <field name="domain">[("active", "=", True)]</field>
+        <field name="model_id">res.partner.category</field>
+        <field name="user_id" />
+    </record>
 
     <!--Exports-->
     <record id="demo_export" model="ir.exports">
@@ -68,6 +74,7 @@
         <field name="name">category_id/name</field>
         <field name="export_id" ref="pattern_import_export.demo_export" />
         <field name="add_select_tab" eval="True" />
+        <field name="tab_filter_id" ref="demo_filter_categories" />
     </record>
     <record id="demo_export_m2m_line_1" model="ir.exports.line">
         <field name="name">id</field>

pattern_import_export/models/ir_exports_line.py

@@ -59,10 +59,14 @@ def _get_last_relation_field(self, model, path, level=1):
             path = path + "/"
         field, path = path.split("/", 1)
         if path:
-            next_model = self.env[model]._fields[field]._related_comodel_name
-            next_field = path.split("/", 1)[0]
-            if self.env[next_model]._fields[next_field]._related_comodel_name:
-                return self._get_last_relation_field(next_model, path, level=level + 1)
+            field_obj = self.env[model]._fields.get(field)
+            if field_obj:
+                next_model = getattr(field_obj, "_related_comodel_name", False)
+                if next_model and next_model in self.env:
+                    next_field = path.split("/", 1)[0]
+                    next_field_obj = self.env[next_model]._fields.get(next_field)
+                    if next_field_obj and getattr(next_field_obj, "_related_comodel_name", False):
+                        return self._get_last_relation_field(next_model, path, level=level + 1)
         return field, model, level
 
     @api.depends("name", "add_select_tab")
@@ -78,30 +82,54 @@ def _compute_required_fields(self):
                 "tab_filter_id",
                 "add_select_tab",
             ]
-            if record.name:
-                required = []
-                field, model, level = self._get_last_relation_field(
-                    record.model1_id.model, record.name
-                )
-                ftype = self.env[model]._fields[field].type
-                if ftype in ["many2one", "many2many"]:
-                    level += 1
-                    hidden_fields.remove("add_select_tab")
-                for idx in range(2, level + 1):
-                    required.append(f"field{idx}_id")
-                if ftype in ["one2many", "many2many"]:
-                    required.append("number_occurence")
-                if ftype in "one2many":
-                    required.append("sub_pattern_config_id")
-                record.required_fields = ",".join(required)
-                if record.add_select_tab:
-                    # this field is optionnal
-                    required.append("tab_filter_id")
-                hidden_fields = list(set(hidden_fields) - set(required))
-                record.hidden_fields = ",".join(hidden_fields)
-            else:
+
+            if not record.name:
                 record.required_fields = ""
                 record.hidden_fields = ""
+                continue
+
+            required = []
+
+            field, model, level = self._get_last_relation_field(
+                record.model1_id.model, record.name
+            )
+
+            if model not in self.env:
+                record.required_fields = ""
+                record.hidden_fields = ",".join(hidden_fields)
+                continue
+
+            model_obj = self.env[model]
+
+            field_obj = model_obj._fields.get(field)
+            if not field_obj:
+                record.required_fields = ""
+                record.hidden_fields = ",".join(hidden_fields)
+                continue
+
+            ftype = field_obj.type
+
+            if ftype in ["many2one", "many2many"]:
+                level += 1
+                if "add_select_tab" in hidden_fields:
+                    hidden_fields.remove("add_select_tab")
+
+            for idx in range(2, level + 1):
+                required.append(f"field{idx}_id")
+
+            if ftype in ["one2many", "many2many"]:
+                required.append("number_occurence")
+
+            if ftype == "one2many":
+                required.append("sub_pattern_config_id")
+
+            if record.add_select_tab:
+                required.append("tab_filter_id")
+
+            record.required_fields = ",".join(required)
+
+            hidden_fields = list(set(hidden_fields) - set(required))
+            record.hidden_fields = ",".join(hidden_fields)
 
     def _inverse_name(self):
         super()._inverse_name()

pattern_import_export/tests/test_pattern_export.py

@@ -393,7 +393,9 @@ def test_get_metadata(self):
         tab_country_name = (
             f"({self.filter_countries_1.id}) {self.filter_countries_1.name}"
         )
-        self.assertEqual(list(tabs.keys()), [tab_country_name, "Tags"])
+        filter_categories = self.env.ref("pattern_import_export.demo_filter_categories")
+        tab_category_name = f"({filter_categories.id}) {filter_categories.name}"
+        self.assertEqual(list(tabs.keys()), [tab_country_name, tab_category_name])
         self.assertEqual(
             tabs[tab_country_name],
             {
@@ -403,7 +405,7 @@ def test_get_metadata(self):
             },
         )
         self.assertEqual(
-            tabs["Tags"],
+            tabs[tab_category_name],
             {
                 "headers": ["name"],
                 "data": [