[IMP] estate: Added constraint on estate property and tag and type

Author: yagho-odoo

estate/models/estate_property.py

@@ -41,6 +41,11 @@ class EstateProperty(models.Model):
     tag_ids = fields.Many2many("estate.property.tag", string="tag")
     offer_ids = fields.One2many("estate.property.offer", "property_id", string="offer")
 
+    _check_expected_price = models.Constraint(
+        'CHECK(expected_price > 0)',
+        'The expected_price of any property must be strictly positive.',
+    )
+
     @api.depends('living_area', 'garden_area')
     def _compute_area(self):
         for property in self:

estate/models/estate_property_offer.py

@@ -1,7 +1,8 @@
 from odoo import api, fields, models
 from datetime import date
 from dateutil.relativedelta import relativedelta
-from odoo.exceptions import UserError
+from odoo.exceptions import UserError, ValidationError
+from odoo.tools.float_utils import float_compare, float_is_zero, float_round
 
 
 class EstatePropertyOffer(models.Model):
@@ -45,3 +46,11 @@ def action_accept(self):
     def action_refuse(self):
         self.status = 'Refused'
         return True
+
+    @api.constrains('price')
+    def _check_price_validity(self):
+        if (float_is_zero(self.price,0)):
+            raise ValidationError("Offered price must be at least a euro")
+        min_amount = self.property_id.expected_price * 0.9
+        if (float_compare(self.price,min_amount,2) < 0):
+            raise ValidationError("Offered price must be at least 90% of the expected price")

estate/models/estate_property_tag.py

@@ -6,3 +6,8 @@ class EstatePropertyTag(models.Model):
     _description = "property tags"
 
     name = fields.Char(required=True)
+
+    _unique_name = models.Constraint(
+        'unique(name)',
+        'A tag must have a unique name.',
+    )

estate/models/estate_property_type.py

@@ -6,3 +6,8 @@ class EstatePropertyType(models.Model):
     _description = "property types"
 
     name = fields.Char(required=True)
+
+    _unique_name = models.Constraint(
+        'unique(name)',
+        'A property type must have a unique name.',
+    )