[MIG] account_analytic_tag: Migration to 17.0

OCA · Mar 26, 2024 · 21d8bf5 · 21d8bf5
1 parent 65292ed
commit 21d8bf5
Show file tree

Hide file tree

Showing 7 changed files with 20 additions and 9 deletions.
diff --git a/account_analytic_tag/__manifest__.py b/account_analytic_tag/__manifest__.py
@@ -3,7 +3,7 @@
 
 {
     "name": "Account Analytic Tag",
-    "version": "16.0.1.0.0",
+    "version": "17.0.1.0.0",
     "author": "Tecnativa, Odoo Community Association (OCA)",
     "category": "Account",
     "website": "https://github.com/OCA/account-analytic",

diff --git a/account_analytic_tag/models/account_move_line.py b/account_analytic_tag/models/account_move_line.py
@@ -15,9 +15,16 @@ def _prepare_analytic_lines(self):
         if self.analytic_tag_ids:
             for val in vals:
                 account_id = val.get("account_id")
+                if not account_id:
+                    account_field_name = next(
+                        (key for key in val.keys() if key.startswith("x_plan")), None
+                    )
+                    account_id = val.get(account_field_name)
                 tags = self.analytic_tag_ids.filtered(
-                    lambda x: not x.account_analytic_id
-                    or x.account_analytic_id.id == account_id
+                    lambda x, account_id=account_id: (
+                        not x.account_analytic_id
+                        or x.account_analytic_id.id == account_id
+                    )
                 )
                 val.update({"tag_ids": [(6, 0, tags.ids)]})
         return vals
diff --git a/account_analytic_tag/security/analytic_security.xml b/account_analytic_tag/security/analytic_security.xml
@@ -10,7 +10,7 @@
             >['|',('company_id','=',False),('company_id', 'in', company_ids)]</field>
         </record>
     </data>
-    <data noupdate="0">
+    <data>
         <record id="group_analytic_tags" model="res.groups">
             <field name="name">Analytic Accounting Tags</field>
             <field name="category_id" ref="base.module_category_hidden" />

diff --git a/account_analytic_tag/tests/common.py b/account_analytic_tag/tests/common.py
@@ -21,8 +21,7 @@ def setUpClass(cls):
         cls.user = new_test_user(
             cls.env,
             login="test-analytic-tag-user",
-            groups="%s,%s,%s"
-            % (
+            groups="{},{},{}".format(
                 "account.group_account_invoice",
                 "analytic.group_analytic_accounting",
                 "account_analytic_tag.group_analytic_tags",

diff --git a/account_analytic_tag/tests/test_account_analytic_tag.py b/account_analytic_tag/tests/test_account_analytic_tag.py
@@ -20,10 +20,15 @@ def setUpClass(cls):
         with invoice_form.invoice_line_ids.new() as line_form:
             line_form.product_id = cls.product_a
             line_form.analytic_tag_ids.add(cls.account_analytic_tag_a)
+            line_form.analytic_distribution = {cls.analytic_account_a.id: 100}
         with invoice_form.invoice_line_ids.new() as line_form:
             line_form.product_id = cls.product_b
             line_form.analytic_tag_ids.add(cls.account_analytic_tag_a)
             line_form.analytic_tag_ids.add(cls.account_analytic_tag_b)
+            line_form.analytic_distribution = {
+                cls.analytic_account_a.id: 50,
+                cls.analytic_account_b.id: 50,
+            }
         cls.invoice = invoice_form.save()
         cls.line_a = cls.invoice.invoice_line_ids.filtered(
             lambda x: x.product_id == cls.product_a

diff --git a/account_analytic_tag/views/account_analytic_tag_views.xml b/account_analytic_tag/views/account_analytic_tag_views.xml
@@ -5,7 +5,7 @@
         <field name="arch" type="xml">
             <tree>
                 <field name="name" />
-                <field name="display_name" invisible="1" />
+                <field name="display_name" column_invisible="True" />
                 <field name="account_analytic_id" />
                 <field name="color" widget="color_picker" />
                 <field name="company_id" groups="base.group_multi_company" />
@@ -22,7 +22,7 @@
                         name="web_ribbon"
                         title="Archived"
                         bg_color="bg-danger"
-                        attrs="{'invisible': [('active', '=', True)]}"
+                        invisible="active"
                     />
                     <group>
                         <group>

diff --git a/account_analytic_tag/views/res_config_settings_views.xml b/account_analytic_tag/views/res_config_settings_views.xml
@@ -7,7 +7,7 @@
         <field name="model">res.config.settings</field>
         <field name="inherit_id" ref="account.res_config_settings_view_form" />
         <field name="arch" type="xml">
-            <xpath expr="//div[@id='track_costs_revenues']" position="after">
+            <xpath expr="//setting[@id='track_costs_revenues']" position="after">
                 <div
                     class="col-12 col-lg-6 o_setting_box"
                     id="tag_analytic_entries"