[FIX] agreement_legal: Change create_new_version() function in agreements

TT32359

agreement_legal/models/agreement.py

@@ -225,7 +225,6 @@ class Agreement(models.Model):
         "parent_agreement_id",
         string="Previous Versions",
         copy=False,
-        domain=[("active", "=", False)],
         context={"active_test": False},
     )
     child_agreements_ids = fields.One2many(
@@ -408,8 +407,7 @@ class Agreement(models.Model):
                     "date_created": fields.Datetime.now(),
                 }
             )
-            # Reset revision to 0 since it's a new version
-            rec.revision = 0
+        return super().write({"revision": 0})
 
     def _get_new_agreement_default_vals(self):
         self.ensure_one()

agreement_legal/tests/test_agreement.py

@@ -143,3 +143,8 @@ class TestAgreement(TransactionCase):
         self.assertEqual(
             field[0].get("modifiers", ""), '{"readonly": [["readonly", "=", true]]}'
         )
+
+    def test_action_create_new_version(self):
+        self.test_agreement.create_new_version()
+        self.assertEqual(self.test_agreement.state, "draft")
+        self.assertEqual(len(self.test_agreement.previous_version_agreements_ids), 1)