[IMP] estate: add computed fields and "onchange" handler method

Framework 101 tutorial: chapter 8
- Estate properties now have computed fields to display the best offer (i.e. best price) and the total area (i.e. sum of living area + garden)
- Default values are set or cleared when clicking the garden checkbox
- Introduced validity period fields for offers

Author: robal-odoo

estate/models/estate_property.py

@@ -1,5 +1,5 @@
 from dateutil.relativedelta import relativedelta
-from odoo import fields, models
+from odoo import api, fields, models
 
 
 class EstateProperty(models.Model):
@@ -37,3 +37,27 @@ class EstateProperty(models.Model):
     buyer_id = fields.Many2one("res.partner", copy=False)
     tag_ids = fields.Many2many("estate.property.tag")
     offer_ids = fields.One2many("estate.property.offer", "property_id")
+
+    # Computed
+    total_area = fields.Integer(compute="_compute_total_area", string="Total Area (sqm)")
+    best_price = fields.Float(compute="_compute_best_price", string="Best Offer")
+
+    @api.depends("living_area", "garden_area")
+    def _compute_total_area(self) -> None:
+        for record in self:
+            record.total_area = record.living_area + record.garden_area
+
+    @api.depends("offer_ids.price")
+    def _compute_best_price(self) -> None:
+        for record in self:
+            record.best_price = max(record.offer_ids.mapped("price"), default=0)
+
+    # Methods that trigger on changes
+    @api.onchange("garden")
+    def _compute_garden_defaults(self) -> None:
+        if self.garden:
+            self.garden_area = 10
+            self.garden_orientation = "north"
+        else:
+            self.garden_area = None
+            self.garden_orientation = None

estate/models/estate_property_offer.py

@@ -1,4 +1,7 @@
-from odoo import fields, models
+from datetime import date
+
+from dateutil.relativedelta import relativedelta
+from odoo import api, fields, models
 
 
 class EstatePropertyOffer(models.Model):
@@ -10,7 +13,24 @@ class EstatePropertyOffer(models.Model):
         selection=[("accepted", "Accepted"), ("refused", "Refused")],
         copy=False,
     )
+    validity = fields.Integer(default=7, string="Validity (days)")
 
     # Relations
     partner_id = fields.Many2one("res.partner", required=True)
     property_id = fields.Many2one("estate.property", required=True)
+
+    # Computed
+    date_deadline = fields.Date(compute="_compute_deadline", inverse="_inverse_deadline", string="Deadline")
+
+    @api.depends("create_date", "validity")
+    def _compute_deadline(self) -> None:
+        for record in self:
+            record.date_deadline = _get_reference_date(record) + relativedelta(days=record.validity)
+
+    def _inverse_deadline(self) -> None:
+        for record in self:
+            record.validity = (record.date_deadline - _get_reference_date(record)).days
+
+
+def _get_reference_date(offer: EstatePropertyOffer) -> date:
+    return fields.Date.today() if not offer.create_date else offer.create_date.date()

estate/views/estate_property_offer_views.xml

@@ -8,6 +8,8 @@
                 <field name="price"/>
                 <field name="partner_id"/>
                 <field name="status"/>
+                <field name="validity"/>
+                <field name="date_deadline"/>
             </list>
         </field>
     </record>
@@ -22,9 +24,11 @@
                         <field name="price"/>
                         <field name="partner_id"/>
                         <field name="status"/>
+                        <field name="validity"/>
+                        <field name="date_deadline"/>
                     </group>
                 </sheet>
             </form>
         </field>
     </record>
-</odoo>
+</odoo>

estate/views/estate_property_views.xml

@@ -35,6 +35,7 @@
                         </group>
                         <group>
                             <field name="expected_price"/>
+                            <field name="best_price"/>
                             <field name="selling_price"/>
                         </group>
                     </group>
@@ -49,6 +50,7 @@
                                 <field name="garden"/>
                                 <field name="garden_area"/>
                                 <field name="garden_orientation"/>
+                                <field name="total_area"/>
                             </group>
                         </page>
                         <page string="Offers">
@@ -89,4 +91,4 @@
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
     </record>
-</odoo>
+</odoo>