[MIG] dms_field_auto_classification: Migration to 18.0

TT55508

Author: victoralmau

dms_field_auto_classification/__manifest__.py

@@ -2,7 +2,7 @@
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 {
     "name": "Auto classify files into embedded DMS",
-    "version": "16.0.1.0.3",
+    "version": "18.0.1.0.0",
     "category": "Document Management",
     "website": "https://github.com/OCA/dms",
     "author": "Tecnativa, Odoo Community Association (OCA)",

dms_field_auto_classification/i18n/es.po

@@ -112,6 +112,3 @@ msgstr "Plantilla"
 #: model:ir.model,name:dms_field_auto_classification.model_wizard_dms_classification_detail
 msgid "Wizard Dms Classification Detail"
 msgstr "Asistente de Clasificación Dms Detalle"
-
-#~ msgid "DMS Directories"
-#~ msgstr "Directorios DMS"

dms_field_auto_classification/tests/test_dms_field_auto_classification.py

@@ -50,21 +50,23 @@ def setUpClass(cls):
                 "vat": "58426469Y",
             }
         )
-        cls.wizard = cls._create_wizard_dms_classification(cls, cls.template)
-        cls.extra_wizard = cls._create_wizard_dms_classification(cls, cls.template)
+        cls.wizard = cls._create_wizard_dms_classification(cls.template)
+        cls.extra_wizard = cls._create_wizard_dms_classification(cls.template)
 
-    def _data_file(self, filename, encoding=None):
+    @classmethod
+    def _data_file(cls, filename, encoding=None):
         mode = "rt" if encoding else "rb"
         with open(path.join(path.dirname(__file__), filename), mode) as file:
             data = file.read()
             if encoding:
                 data = data.encode(encoding)
             return b64encode(data)
 
-    def _create_wizard_dms_classification(self, template):
-        wizard_form = Form(self.env["wizard.dms.classification"])
+    @classmethod
+    def _create_wizard_dms_classification(cls, template):
+        wizard_form = Form(cls.env["wizard.dms.classification"])
         wizard_form.template_id = template
-        wizard_form.data_file = self._data_file(self, "data/test.zip")
+        wizard_form.data_file = cls._data_file("data/test.zip")
         return wizard_form.save()
 
     def test_partner_misc(self):

dms_field_auto_classification/views/dms_classification_template_views.xml

@@ -23,16 +23,13 @@
         <field name="arch" type="xml">
             <field name="filename_pattern" position="after">
                 <field name="model_id" />
-                <field
-                    name="detail_ids"
-                    attrs="{'invisible' : [('model_id', '=', False)]}"
-                >
-                    <tree editable="bottom">
+                <field name="detail_ids" invisible="not model_id">
+                    <list editable="bottom">
                         <field name="sequence" widget="handle" />
-                        <field name="model_id" invisible="1" />
-                        <field name="field_id" required="1" />
+                        <field name="model_id" column_invisible="1" />
+                        <field name="field_id" column_invisible="1" />
                         <field name="index" />
-                    </tree>
+                    </list>
                 </field>
             </field>
         </field>

dms_field_auto_classification/wizards/wizard_dms_classification.py

@@ -63,7 +63,7 @@ def _compute_directory_id(self):
                     for detail in item.template_id.detail_ids:
                         matches_value = matches.groups()[detail.index]
                         # Change directory pattern if index in pattern
-                        expected = "{%s}" % detail.index
+                        expected = "{%s}" % detail.index  # noqa: UP031
                         if expected in directory_pattern:
                             directory_pattern = directory_pattern.replace(
                                 expected, matches_value

dms_field_auto_classification/wizards/wizard_dms_classification_views.xml

@@ -9,7 +9,7 @@
         />
         <field name="arch" type="xml">
             <xpath
-                expr="//field[@name='detail_ids']/tree/field[@name='file_name']"
+                expr="//field[@name='detail_ids']/list/field[@name='file_name']"
                 position="after"
             >
                 <field name="record_ref" />