[IMP] estate: Chapter 10: Constraints

- Add "Cancel" and "Sold" buttons to estate.property with state change logic and constraints.
- Add "Accept" and "Refuse" buttons to estate.property.offer.
- Accepting an offer sets the property's buyer and selling price.
- Added constraint that only one offer can be accepted per property.

[email protected]

Author: Ana Zurabashvili [anzu]

estate/models/estate_property.py

@@ -57,6 +57,19 @@ class EstateProperty(models.Model):
     best_price = fields.Float(string="Best price",
                               compute='_compute_best_price', store=True)
 
+    _sql_constraints = [
+        (
+            "check_expected_price_positive",
+            "CHECK(expected_price >= 0)",
+            "Expected price must be strictly positive.",
+        ),
+        (
+            "check_selling_price_positive",
+            "CHECK(selling_price >= 0)",
+            "Selling price must be strictly positive.",
+        ),
+    ]
+
     @api.depends('garden_area', 'living_area')
     def _compute_total_area(self):
         for val in self:
@@ -86,3 +99,20 @@ def _onchange_garden(self):
         else:
             self.garden_area = False
             self.garden_orientation = False
+
+    @api.constrains("expected_price", "selling_price")
+    def _check_selling_price_vs_expected(self):
+        for rec in self:
+            if float_is_zero(rec.selling_price or 0.0, precision_digits=2):
+                continue
+            if float_compare(
+                    rec.selling_price, rec.expected_price * 0.90, precision_digits=2
+            ) < 0:
+                raise ValidationError(
+                    "Here Selling price must be at least 90% of the expected price."
+                )
+            # if rec.selling_price and rec.selling_price <= rec.expected_price * 0.90:
+            #     raise ValidationError(
+            #         "Selling price should not be possible to accept an offer lower "
+            #         "than 90% of the expected price",
+            #     )

estate/models/estate_property_offer.py

@@ -22,6 +22,14 @@ class EstatePropertyOffer(models.Model):
                                 inverse="_inverse_date_deadline")
     validity = fields.Integer(string="Validity Days", default=7)
 
+    _sql_constraints = [
+        (
+            "check_price_positive",
+            "CHECK(price >= 0)",
+            "Offer price must be strictly positive.",
+        ),
+    ]
+
     @api.depends("validity")
     def _compute_date_deadline(self):
         for val in self:

estate/models/estate_property_tag.py

@@ -6,3 +6,8 @@ class EstatePropertyTag(models.Model):
     _description = "Estate Property Tag"
 
     name = fields.Char(string="Property Tag", required=True)
+
+    _sql_constraints = [
+        ('unique_name', 'unique(name)',
+         'Tags should have unique names.'),
+    ]

estate/models/estate_property_type.py

@@ -6,3 +6,8 @@ class EstatePropertyType(models.Model):
     _description = "Estate Property Type"
 
     name = fields.Char(string="Property Type", required=True)
+
+    _sql_constraints = [
+        ('unique_name', 'unique(name)',
+         'Types should have unique names.'),
+    ]