[IMP] estate: Chapter 10 - Constraints

Author: magai2002

estate/models/estate_property.py

@@ -1,7 +1,8 @@
 from odoo import api, fields, models
 from dateutil.relativedelta import relativedelta
 from datetime import date
-from odoo.exceptions import UserError
+from odoo.exceptions import UserError, ValidationError
+from odoo.tools import float_compare, float_is_zero
 
 
 class EstatePropertyModel(models.Model):
@@ -56,6 +57,9 @@ class EstatePropertyModel(models.Model):
     total_area = fields.Integer(compute='_compute_total_area', string="Total Area (sqm)")
     best_price = fields.Float(compute='_compute_best_price', string="Best Offer")
 
+    _expected_price_check = models.Constraint('CHECK(expected_price > 0)', "The expected price must be strictly positive.")
+    _selling_price_check = models.Constraint('CHECK(selling_price >= 0)', "The selling price must be positive.")
+
     @api.depends('living_area', 'garden_area')
     def _compute_total_area(self):
         for record in self:
@@ -92,3 +96,10 @@ def action_cancel(self):
                 raise UserError("Sold properties cannot be canceled.")
             record.state = "cancel"
         return True
+
+    @api.constrains('selling_price', 'expected_price')
+    def _check_selling_price(self):
+        for record in self:
+            if not float_is_zero(record.selling_price, precision_digits=2):
+                if float_compare(record.selling_price, record.expected_price * 0.9, precision_digits=2) == -1:
+                    raise ValidationError("The selling price cannot be lower than 90% of the expected price!")

estate/models/estate_property_offer.py

@@ -17,6 +17,8 @@ class EstatePropertyOffer(models.Model):
     validity = fields.Integer(string="Validity (days)", default=7)
     date_deadline = fields.Date(string="Deadline", compute='_compute_date_deadline', inverse='_inverse_date_deadline')
 
+    _price_check = models.Constraint('CHECK(price > 0)', "The offer price must be strictly positive.")
+
     @api.depends('create_date', 'validity')
     def _compute_date_deadline(self):
         for record in self:

estate/models/estate_property_tag.py

@@ -2,7 +2,9 @@
 
 
 class EstatePropertyTag(models.Model):
-    _name = "estate.property.tag"
+    _name = 'estate.property.tag'
     _description = "Property Tag"
 
     name = fields.Char(required=True)
+
+    _name_check = models.Constraint('UNIQUE(name)', "The name must be unique.")

estate/models/estate_property_type.py

@@ -2,6 +2,9 @@
 
 
 class EstatePropertyTypeModel(models.Model):
-    _name = "estate.property.type"
+    _name = 'estate.property.type'
     _description = "Real Estate property types database"
+    
     name = fields.Char(required=True)
+
+    _name_check = models.Constraint('UNIQUE(name)', "The name must be unique.")