[IMP] vijan-Serer101-10

Author: VincentJanssen-Code

estate/models/estate_property.py

@@ -1,13 +1,14 @@
 from dateutil.relativedelta import relativedelta
 from odoo import fields, models,api,exceptions
+from odoo.tools.float_utils import float_compare, float_is_zero;
+from odoo.exceptions import ValidationError;
 
 
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Property"
     _positif_expected_price = models.Constraint("CHECK (expected_price > 0)","A price can't be negatif");
     _positif_selling_price = models.Constraint("CHECK (selling_price > 0)","A price can't be negatif");
-    _postif_best_price = models.Constraint("CHECK (best_price > 0)", "A price can't be negatif");
 
     state = fields.Selection(selection = [("New","New"), ("Offer_Received","Offer Received") ,("Offer_Accepted","Offer Accepted"), ("Sold","Sold"), ("Cancelled","Cancelled")])
     active = fields.Boolean('Active',default=True)
@@ -70,4 +71,12 @@ def _get_best_price(self):
     @api.onchange("garden")
     def _garden_pre_fill(self):
         self.garden_area = 10 if self.garden else 0
-        self.garden_orientation = 'North' if self.garden else ''
+        self.garden_orientation = 'North' if self.garden else ''
+
+    @api.constrains('selling_price','expected_price')
+    def _check_prices(self):
+            for record in self:
+                if float_is_zero(record.selling_price,2):
+                    return;
+                if(float_compare(record.selling_price,record.expected_price*.8,2) == -1):
+                    raise ValidationError(f"Selling price is too low {record.selling_price}");

estate/models/estate_property_offer.py

@@ -6,7 +6,7 @@
 class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
     _description = "Property Offer"
-    _postif_price = models.Constraint("CHECK (best_price > 0)", "A price can't be negatif");
+    _postif_price = models.Constraint("CHECK (price > 0)", "A price can't be negatif");
     price = fields.Float(string="Price")
     status = fields.Selection(copy=False,selection = [("Accepted","Accepted"),("Refused","Refused")])
     partner_id = fields.Many2one('res.partner',required=True)

estate/models/estate_property_tag.py

@@ -4,4 +4,5 @@
 class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "Property Tag"
-    name = fields.Char(required=True)
+    _unique_tag = models.UniqueIndex("(name)","Tag name must be unique in database")
+    name = fields.Char(required=True, unique=True)

estate/models/estate_property_type.py

@@ -4,4 +4,5 @@
 class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "Property Type"
-    name = fields.Char(required=True)
+    name = fields.Char(required=True)
+    _unique_type = models.UniqueIndex("(name)","Property type name must be unique in database")