[MIG] contract_price_revision: Migration to 17.0

contract_price_revision/README.rst

@@ -83,6 +83,7 @@ Contributors
    -  Vicent Cubells
    -  Carlos Dauden
    -  Carolina Fernandez
+   -  Juan José Seguí
 
 Maintainers
 -----------

contract_price_revision/__manifest__.py

@@ -6,7 +6,7 @@
 {
     "name": "Contract Price Revision",
     "summary": "Easy revision of contract prices",
-    "version": "15.0.1.0.0",
+    "version": "17.0.1.0.0",
     "category": "Contract",
     "author": "ACSONE SA/NV, Tecnativa, Odoo Community Association (OCA)",
     "license": "AGPL-3",

contract_price_revision/models/contract_line.py

@@ -30,7 +30,7 @@ class ContractLine(models.Model):
 
     @api.depends_context("date_start")
     def _compute_price_can_be_revised(self):
-        date_start = self.env.context.get("date_start", fields.Date.today())
+        date_start = self.env.context.get("date_start", fields.Date.context_today(self))
         lines_can_be_revised = self.filtered(
             lambda line: not line.never_revise_price
             and not line.automatic_price

contract_price_revision/readme/CONTRIBUTORS.md

@@ -2,3 +2,4 @@
   - Vicent Cubells
   - Carlos Dauden
   - Carolina Fernandez
+  - Juan José Seguí

contract_price_revision/static/description/index.html

@@ -429,6 +429,7 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
 <li>Vicent Cubells</li>
 <li>Carlos Dauden</li>
 <li>Carolina Fernandez</li>
+<li>Juan José Seguí</li>
 </ul>
 </li>
 </ul>

contract_price_revision/tests/test_contract_price_revision.py

@@ -82,3 +82,9 @@ class TestContractPriceRevision(TestContractBase):
         self.assertEqual(len(lines), 2)
         lines = lines.filtered(lambda x: x.price_unit == 200.0)
         self.assertEqual(len(lines), 1)
+
+    def test_compute_variation_percent(self):
+        self.acct_line.write({"price_unit": 200.0, "previous_price": 100.0})
+        self.assertEqual(self.acct_line.variation_percent, 100.0)
+        self.acct_line.write({"price_unit": 200.0, "previous_price": 0.0})
+        self.assertEqual(self.acct_line.variation_percent, 0.0)

contract_price_revision/views/contract_line.xml

@@ -19,7 +19,7 @@
                 <field
                     name="variation_percent"
                     groups="base.group_no_one"
-                    attrs="{'invisible': [('predecessor_contract_line_id', '=', False)]}"
+                    invisible="not predecessor_contract_line_id"
                 />
             </field>
         </field>

contract_price_revision/wizards/contract_price_revision_views.xml

@@ -18,16 +18,10 @@
                         <field name="date_end" />
                         <field name="variation_type" />
                     </group>
-                    <group
+                    <group name="percentage" invisible="variation_type != 'percentage'">
-                        name="percentage"
-                        attrs="{'invisible': [('variation_type', '!=', 'percentage')]}"
-                    >
                         <field name="variation_percent" />
                     </group>
-                    <group
+                    <group name="fixed" invisible="variation_type != 'fixed'">
-                        name="fixed"
-                        attrs="{'invisible': [('variation_type', '!=', 'fixed')]}"
-                    >
                         <field name="fixed_price" />
                     </group>
                 </group>