[14.0][MIG] agreement_legal (Version 12.0 to 14.0)

[MIG] Black & Cleanup

[IMP] Reviewer Requested Changes

[IMP] Black

[IMP] Reviewer Suggestion

[FIX] Tests

[FIX] Multiple Issues

- Added 'is_old_version' field to better display old versions. Before these were marked as inactive therefore not showing in the agreement.
- 'is_template' field now copies when making a template revision or copying a template.
- New Versions now retain original create date.
- Moved version/revision and created on/by fields out of footer to top in order to have the save buttons when creating children.
- Moved demo data out of data folder into the demo.xml

[FIX] demo data

[FIX] Revert back to only migration

[FIX] Report dynamic field rendering

[FIX] Test

agreement_legal/tests/test_agreement_appendix.py

@@ -0,0 +1,81 @@
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html)
+
+from datetime import timedelta
+
+from odoo import fields
+from odoo.tests.common import TransactionCase
+
+
+class TestAgreementAppendices(TransactionCase):
+    def setUp(self):
+        super().setUp()
+        self.test_customer = self.env["res.partner"].create({"name": "TestCustomer"})
+        self.agreement_type = self.env["agreement.type"].create(
+            {
+                "name": "Test Agreement Type",
+                "domain": "sale",
+            }
+        )
+        self.test_agreement = self.env["agreement"].create(
+            {
+                "name": "TestAgreement",
+                "description": "Test",
+                "special_terms": "Test",
+                "partner_id": self.test_customer.id,
+                "start_date": fields.Date.today(),
+                "end_date": fields.Date.today() + timedelta(days=365),
+            }
+        )
+        self.test_appendices = self.env["agreement.appendix"].create(
+            {
+                "name": "TestAppendices",
+                "title": "Test",
+                "content": "Test",
+                "agreement_id": self.test_agreement.id,
+            }
+        )
+
+    # TEST 01: Set 'Field' for dynamic placeholder, test onchange method
+    def test_onchange_copyvalue(self):
+        appendix_01 = self.test_appendices
+        field_01 = self.env["ir.model.fields"].search(
+            [
+                ("model", "=", "agreement.appendix"),
+                ("name", "=", "active"),
+            ]
+        )
+        appendix_01.field_id = field_01.id
+        appendix_01.onchange_copyvalue()
+        self.assertEqual(appendix_01.copyvalue, "${object.active or ''}")
+
+    # TEST 02: Set related 'Field' for dynamic placeholder to
+    # test onchange method
+    def test_onchange_copyvalue2(self):
+        appendix_01 = self.test_appendices
+        field_01 = self.env["ir.model.fields"].search(
+            [
+                ("model", "=", "agreement.appendix"),
+                ("name", "=", "agreement_id"),
+            ]
+        )
+        sub_field_01 = self.env["ir.model.fields"].search(
+            [
+                ("model", "=", "agreement"),
+                ("name", "=", "active"),
+            ]
+        )
+        appendix_01.field_id = field_01.id
+        appendix_01.onchange_copyvalue()
+        self.assertEqual(appendix_01.sub_object_id.model, "agreement")
+        appendix_01.sub_model_object_field_id = sub_field_01.id
+        appendix_01.onchange_copyvalue()
+        self.assertEqual(appendix_01.copyvalue, "${object.agreement_id.active or ''}")
+
+    # TEST 03: Test Dynamic Field
+    def test_compute_dynamic_content(self):
+        appendix_01 = self.test_appendices
+        appendix_01.content = "${object.name}"
+        self.assertEqual(
+            appendix_01.dynamic_content,
+            "<p>{" + str(appendix_01.id) + ": '</p><p>TestAppendices</p>'}",
+        )