[IMP] estate: Ch 10

Author: hazei-odoo

estate/models/estate_property.py

@@ -2,6 +2,7 @@
 from datetime import date
 from dateutil.relativedelta import relativedelta
 from odoo.exceptions import UserError
+from odoo.tools import float_compare
 
 
 class EstateProperty(models.Model):
@@ -14,8 +15,8 @@ class EstateProperty(models.Model):
     postcode = fields.Char()
     state = fields.Text()
     date_availability = fields.Date(copy=False, readonly=True, default=lambda self: date.today() + relativedelta(months=3))
-    expected_price = fields.Float(required=True)
-    selling_price = fields.Float(readonly=True, copy=False)
+    expected_price = fields.Float("Expected Price", required=True)
+    selling_price = fields.Float("Selling Price", readonly=True, copy=False)
     bedrooms = fields.Integer(default=2)
     living_area = fields.Integer(string='Living Area (m2)')
     facades = fields.Integer()
@@ -83,3 +84,17 @@ def _onchange_garden(self):
         else:
             self.garden_area = 0
             self.garden_orientation = False
+
+    _check_expected_price = models.Constraint(
+         'CHECK(expected_price >= 0)', 'The expected price must be strictly positive.')
+
+    _check_selling_price = models.Constraint(
+         'CHECK(selling_price > 0)', 'The selling price must be positive.')
+    
+    @api.constrains('selling_price', 'expected_price')
+    def _check_selling_price_expected_price(self):
+        for record in self:
+            if record.selling_price == 0:
+                continue    
+            if float_compare(record.selling_price, 0.9 * record.expected_price, precision_digits=2) == -1:
+                raise UserError("The selling price must be at least 90% of the expected price.")

estate/models/estate_property_offer.py

@@ -2,12 +2,13 @@
 from dateutil.relativedelta import relativedelta
 from odoo.exceptions import UserError
 
+
 class EstatePropertyOffer(models.Model):
     _name = 'estate.property.offer'
     _description = 'Estate Property Offer'
 
     price = fields.Float(string='price', required=True)
-    status = fields.Selection(selection=[('accepted', 'Accepted'),
+    state = fields.Selection(selection=[('accepted', 'Accepted'),
         ('refused', 'Refused')],
         string="Status",
         copy=False,
@@ -30,16 +31,16 @@ def _compute_date_deadline(self):
 
     def action_accept(self):
         for offer in self:
-            if offer.state != 'Refused':
-                offer.state = 'Accepted'
+            if offer.state != 'refused':
+                offer.state = 'accepted'
             else:
                 raise UserError("A refused offer cannot be accepted.")
         return True
 
     def action_refuse(self):
         for offer in self:
-            if offer.state != 'Accepted':
-                offer.state = 'Refused'
+            if offer.state != 'accepted':
+                offer.state = 'refused'
             else:
                 raise UserError("An accepted offer cannot be refused.")
-        return True
+        return True

estate/views/estate_property_views.xml

@@ -51,6 +51,7 @@
             </group>
             <group>
                 <field name="expected_price"/>
+                <field name="best_price"/>
                 <field name="selling_price"/>
             </group>
         </group>
@@ -82,7 +83,7 @@
                     <list>
                         <field name="partner_id"/>
                         <field name="price"/>
-                        <field name="status"/>
+                        <field name="state"/>
                         <field name="validity"/>
                         <field name="date_deadline"/>
                         <button name="action_accept" type="object" icon="fa-check" title="Accept"/>
@@ -91,7 +92,7 @@
                     <form>
                         <field name="partner_id"/>
                         <field name="price"/>
-                        <field name="status"/>
+                        <field name="state"/>
                         <field name="validity"/>
                         <field name="date_deadline"/>
                     </form>