[IMP] estate: add compute fields and onchange to add some automation

Axel927 · Axel927 · commit b8c4800f2d4c · 2024-03-20T09:51:59.000+01:00
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
@@ -1,4 +1,4 @@
-from odoo import fields, models
+from odoo import api, fields, models
 
 
 class EstateProperty(models.Model):
@@ -8,8 +8,8 @@ class EstateProperty(models.Model):
     name = fields.Char(required=True)
     description = fields.Text()
     postcode = fields.Char()
-    date_availability = fields.Date("Available from", default=lambda _: fields.Date().add(fields.Date().today(), months=3),
-                                    copy=False)
+    date_availability = fields.Date("Available from",
+                                    default=lambda _: fields.Date().add(fields.Date().today(), months=3), copy=False)
     expected_price = fields.Float(required=True)
     selling_price = fields.Float(readonly=True, copy=False)
     bedrooms = fields.Integer(default=2)
@@ -19,8 +19,8 @@ class EstateProperty(models.Model):
     garden = fields.Boolean()
     garden_area = fields.Integer("Garden Area (sqm)")
     garden_orientation = fields.Selection(string="Garden Orientation",
-                                          selection=[("north", "North"), ("south", "South"), ("east", "East"),
-                                                     ("west", "West")])
+                                          selection=[("north", "North"), ("south", "South"),
+                                                     ("east", "East"), ("west", "West")])
     active = fields.Boolean("Active", default=True)
     state = fields.Selection(string="State", default="new", copy=False, required=True,
                              selection=[("new", "New"), ("offer received", "Offer Received"),
@@ -31,3 +31,24 @@ class EstateProperty(models.Model):
     salesperson_id = fields.Many2one("res.users", string="Salesperson", index=True, default=lambda self: self.env.user)
     property_tag_ids = fields.Many2many("estate.property.tag", string="Tag")
     offer_ids = fields.One2many("estate.property.offer", "property_id", string="Offer")
+    total_area = fields.Integer("Total Area (sqm)", compute="_compute_total_area")
+    best_price = fields.Float(compute="_compute_best_price")
+
+    @api.depends("living_area", "garden_area")
+    def _compute_total_area(self):
+        for estate in self:
+            estate.total_area = estate.living_area + estate.garden_area
+
+    @api.depends("offer_ids")
+    def _compute_best_price(self):
+        for estate in self:
+            estate.best_price = max(estate.offer_ids.mapped("price")) if estate.offer_ids else 0
+
+    @api.onchange("garden")
+    def _onchange_garden(self):
+        if self.garden:
+            self.garden_area = 10
+            self.garden_orientation = "north"
+        else:
+            self.garden_area = 0
+            self.garden_orientation = ""
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
@@ -1,4 +1,4 @@
-from odoo import fields, models
+from odoo import api, fields, models
 
 
 class EstatePropertyOffer(models.Model):
@@ -9,3 +9,15 @@ class EstatePropertyOffer(models.Model):
     status = fields.Selection(string="Status", copy=False, selection=[("accepted", "Accepted"),("refused", "Refused")])
     partner_id = fields.Many2one("res.partner", string="Partner", required=True)
     property_id = fields.Many2one("estate.property", string="Property", required=True)
+    validity = fields.Integer(default=7)
+
+    date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline")
+    create_date = fields.Date(default=lambda _: fields.Date().today())
+    @api.depends("validity", "create_date")
+    def _compute_date_deadline(self):
+        for offer in self:
+            offer.date_deadline = fields.Date().add(offer.create_date, days=offer.validity)
+
+    def _inverse_date_deadline(self):
+        for offer in self:
+            offer.validity = (offer.date_deadline - offer.create_date).days
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
@@ -12,6 +12,8 @@
                                 <field name="price"/>
                                 <field name="partner_id"/>
                                 <field name="status"/>
+                                <field name="validity"/>
+                                <field name="date_deadline"/>
                             </group>
                         </page>
                     </notebook>
@@ -28,6 +30,8 @@
                 <field name="price"/>
                 <field name="partner_id"/>
                 <field name="status"/>
+                <field name="validity"/>
+                <field name="date_deadline"/>
             </tree>
         </field>
     </record>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
@@ -40,6 +40,7 @@
 
                         <group>
                             <field name="expected_price"/>
+                            <field name="best_price"/>
                             <field name="selling_price"/>
                         </group>
 
@@ -57,6 +58,7 @@
                                     <field name="garden_area"/>
                                     <field name="garden_orientation"/>
                                     <field name="state"/>
+                                    <field name="total_area"/>
                                 </group>
                             </page>
 
