[IMP] estate: chapter 10: done

Author: abyo-odoo

estate/models/estate.py

@@ -1,6 +1,8 @@
 # Part of Odoo. See LICENSE file for full copyright and licensing details.
 
 from odoo import api, fields, models
+from odoo.exceptions import ValidationError
+from odoo.tools.float_utils import float_compare
 from dateutil.relativedelta import relativedelta
 
 
@@ -54,6 +56,19 @@ class Estate(models.Model):
     total_area = fields.Float(compute="_compute_total_area")
     best_price = fields.Float(compute="_compute_best_price")
 
+    _sql_constraints = [
+        (
+            "check_expected_price",
+            "CHECK(expected_price > 0)",
+            "Expected price must be positive.",
+        ),
+        (
+            "check_selling_price",
+            "CHECK(selling_price > -1)",
+            "Selling price must be positive.",
+        ),
+    ]
+
     @api.depends("garden_area", "living_area")
     def _compute_total_area(self):
         for record in self:
@@ -62,6 +77,9 @@ def _compute_total_area(self):
     @api.depends("property_offer_ids")
     def _compute_best_price(self):
         for record in self:
+            if not record.property_offer_ids:
+                record.best_price = 0
+                continue
             record.best_price = max(record.mapped("property_offer_ids.price"))
 
     @api.onchange("garden")
@@ -70,8 +88,8 @@ def _onchange_garden(self):
             self.garden_area = 10
             self.garden_orientation = "north"
         else:
-            self.garden_area = 0  # Or None if you prefer
-            self.garden_orientation = False  # Or None if you prefer
+            self.garden_area = 0
+            self.garden_orientation = False
 
     def action_sell_property(self):
         for record in self:
@@ -80,3 +98,14 @@ def action_sell_property(self):
     def action_cancel_property(self):
         for record in self:
             record.state = "cancelled"
+
+    @api.constrains("selling_price")
+    def _check_selling_price(self):
+        for record in self:
+            if not record.selling_price:
+                continue
+
+            if float_compare(record.selling_price, 0.9 * record.expected_price, precision_digits = 2) == -1:
+                raise ValidationError(
+                    "The selling price can not be less than 90 percent of the expected price"
+                )

estate/models/property_offer.py

@@ -2,11 +2,9 @@
 from odoo import api, fields, models
 from dateutil.relativedelta import relativedelta
 
-
 class PropertyOffer(models.Model):
     _name = "property_offer"
     _description = "Property Offer Model"
-    name = fields.Char(required=True)
     price = fields.Float()
     validity = fields.Integer(default=7)
     date_deadline = fields.Date(
@@ -22,6 +20,10 @@ class PropertyOffer(models.Model):
     partner_id = fields.Many2one("res.partner", required=True)
     property_id = fields.Many2one("estate_property", required=True)
 
+    _sql_constraints = [
+        ("check_price", "CHECK(price > 0)", "The offer price must be positive.")
+    ]
+
     @api.depends("validity", "create_date")
     def _compute_date_deadine(self):
         for record in self:
@@ -49,4 +51,4 @@ def action_refuse_offer(self):
             record.status = "refused"
             if record.property_id.selling_price == record.price:
                 record.property_id.selling_price = 0
-
+            

estate/models/property_tag.py

@@ -6,3 +6,10 @@ class PropertyTag(models.Model):
     _name = "property_tag"
     _description = "Property Tag Model"
     name = fields.Char(required=True)
+    _sql_constraints = [
+        (
+            "unique_tag_name",
+            "UNIQUE(name)",
+            "Property tag must be unique.",
+        ),
+    ]

estate/models/propery_type.py

@@ -7,3 +7,11 @@ class PropertyType(models.Model):
     _name = "property_type"
     _description = "Property Types"
     name = fields.Char(required=True)
+
+    _sql_constraints = [
+        (
+            "unique_type_name",
+            "UNIQUE(name)",
+            "Property type must be unique.",
+        ),
+    ]

estate/views/property_offer_views.xml

@@ -12,8 +12,8 @@
                 <field name="status" />
                 <field name="date_deadline"/>
                 <field name="validity"/>
-                <button name="action_accept_offer" type="object" icon="fa-check"/>
-                <button name="action_refuse_offer" type="object" icon="fa-times"/>
+                <button name="action_accept_offer" type="object" icon="fa-check" string ="accept"/>
+                <button name="action_refuse_offer" type="object" icon="fa-times" string = "refuse"/>
             </list>
         </field>
     </record>