[IMP] estate: added computed fields,inverse fun & onchange

-Added computed fields with inverse functions and onchange methods,
-Improved real estate module with sold and cancel buttons to enhance the module
functionality.

Author: haahi-odoo

estate/__init__.py

@@ -1,2 +1 @@
 from . import models
-

estate/__manifest__.py

@@ -1,11 +1,10 @@
 {
     'name': 'Real Estate',
     'depends': ['base'],
-    'license': 'AGPL-3',
     'application': True,
     'installable': True,
     'author': 'Odoo S.A.',
-    'category':'Tutorials',
+    'license': 'LGPL-3',
     'data': [
         'security/ir.model.access.csv',
         'views/estate_property_views.xml',
@@ -14,4 +13,6 @@
         'views/estate_property_offer_views.xml',
         'views/estate_menus.xml'
     ],
+    
 }
+

estate/models/estate_property.py

@@ -1,5 +1,7 @@
-from odoo import models, fields
 from dateutil.relativedelta import relativedelta
+from odoo import models, fields, api
+from odoo.exceptions import UserError, ValidationError
+from odoo.tools.float_utils import float_compare, float_is_zero
 
 
 class EstateProperty(models.Model):
@@ -9,15 +11,18 @@ class EstateProperty(models.Model):
     name = fields.Char(required=True)
     description = fields.Text()
     postcode = fields.Char()
-    date_availability = fields.Date("Availability Date", default=fields.Date.today() + relativedelta(months=3))
+    date_availability = fields.Date(
+        "Availability Date",
+        default=lambda self: fields.Date.today() + relativedelta(months=3)
+    )
     expected_price = fields.Float("Expected Price", required=True)
     selling_price = fields.Float("Selling Price", readonly=True)
     bedrooms = fields.Integer(default=2)
-    living_area = fields.Integer("Living Area(sqm)")
+    living_area = fields.Integer("Living Area (sqm)")
     facades = fields.Integer()
     garage = fields.Boolean()
     garden = fields.Boolean()
-    garden_area = fields.Integer("Garden Area(sqm)")
+    garden_area = fields.Integer("Garden Area (sqm)")
     garden_orientation = fields.Selection(
         selection=[
             ("north", "North"),
@@ -41,9 +46,81 @@ class EstateProperty(models.Model):
         default="new",
     )
     active = fields.Boolean(default=True)
-    property_type_id = fields.Many2one("estate.property.type", string="Property Type")
-    buyer_id = fields.Many2one("res.partner", string="Buyer", copy=False)
-    salesperson_id = fields.Many2one(
-        "res.users", string="Salesperson")
+    property_type_id = fields.Many2one("estate.property.type", "Property Type")
+    buyer_id = fields.Many2one("res.partner", "Buyer", copy=False)
+    salesperson_id = fields.Many2one("res.users", string="Salesperson")
     tag_ids = fields.Many2many("estate.property.tag", string="Tags")
     offer_ids = fields.One2many("estate.property.offer", "property_id", string="Offers")
+    total_area = fields.Integer("Total Area (sqm)", compute="_compute_total_area")
+    best_price = fields.Float("Best Offer", compute="_compute_best_price")
+
+    @api.depends("living_area", "garden_area")
+    def _compute_total_area(self):
+        for record in self:
+            record.total_area = (record.living_area or 0) + (record.garden_area or 0)
+
+    @api.depends("offer_ids.price")
+    def _compute_best_price(self):
+        for record in self:
+            record.best_price = max(record.offer_ids.mapped("price") or [0])
+
+    @api.onchange("garden")
+    def _onchange_garden(self):
+        for record in self:
+            if record.garden:
+                if record.garden_area == 10:
+                    record.garden_orientation = "north"
+                elif record.garden_area == 20:
+                    record.garden_orientation = "south"
+                elif record.garden_area == 30:
+                    record.garden_orientation = "east"
+                elif record.garden_area == 40:
+                    record.garden_orientation = "west"
+                else:
+                    record.garden_area = 0
+                    record.garden_orientation = False
+            else:
+                record.garden_area = 0
+                record.garden_orientation = False
+
+    def action_cancel(self):
+        for record in self:
+            if record.state == "sold":
+                raise UserError("A sold property cannot be cancelled")
+            record.state = "cancelled"
+
+    def action_sold(self):
+        for record in self:
+            if record.state == "cancelled":
+                raise UserError("A cancelled property cannot be set as sold")
+            record.state = "sold"
+
+    _sql_constraints = [
+        (
+            "check_expected_price",
+            "CHECK(expected_price > 0)",
+            "The expected price must be strictly positive.",
+        ),
+        (
+            "check_selling_price_positive",
+            "CHECK(selling_price >= 0)",
+            "The selling price must be positive.",
+        ),
+    ]
+
+    @api.constrains("selling_price", "expected_price")
+    def _check_selling_price_ratio(self):
+        for record in self:
+            if not float_is_zero(record.selling_price, precision_digits=2):
+                if (
+                    float_compare(
+                        record.selling_price,
+                        record.expected_price * 0.9,
+                        precision_digits=2,
+                    )
+                    < 0
+                ):
+                    raise ValidationError(
+                        "The selling price cannot be lower than 90% of the expected price."
+                    )
+

estate/models/estate_property_offer.py

@@ -1,4 +1,7 @@
-from odoo import fields, models
+from dateutil.relativedelta import relativedelta
+
+from odoo import fields, models, api
+from odoo.exceptions import UserError
 
 
 class EstatePropertyOffer(models.Model):
@@ -12,4 +15,36 @@ class EstatePropertyOffer(models.Model):
     )
     partner_id = fields.Many2one("res.partner", string="Partner", required=True)
     property_id = fields.Many2one("estate.property", string="Property", required=True)
-   
+    validity = fields.Integer(string="Validity(days)", default=7)
+    date_deadline = fields.Date(string="Deadline Date", compute="_compute_date_deadline", inverse="_inverse_date_deadline")
+
+    @api.depends('validity')
+    def _compute_date_deadline(self):
+        for record in self:
+            creation_date = record.create_date or fields.Date.today()
+            record.date_deadline =  relativedelta(days=record.validity) +  creation_date
+
+    def _inverse_date_deadline(self):
+        for record in self:
+            creation_date = record.create_date or fields.Date.today()
+            record.validity = (record.date_deadline - fields.Date.to_date(creation_date)).days
+
+    def action_accept(self):
+        for record in self:
+            if record.property_id.buyer_id:
+                raise UserError("Property already accepted")
+            else:
+                record.status = 'accepted'
+                record.property_id.selling_price = record.price
+                record.property_id.state = 'offer_accepted'
+                record.property_id.buyer_id= record.partner_id
+
+    def action_refuse(self):
+        for record in self:
+            record.status = 'refused'
+        return True
+
+    _check_offer_price = models.Constraint(
+        'CHECK(price > 0)',
+        'The price of an offer must be strictly positive.'
+    )

estate/models/estate_property_tag.py

@@ -6,3 +6,8 @@ class EstatePropertyTag(models.Model):
     _description = "Real Estate Property Tag"
 
     name = fields.Char(required=True)
+
+    _check_tag_name_unique = models.Constraint(
+    'UNIQUE(name)', 
+    'The name of the property tag must be unique.'
+    )

estate/models/estate_property_type.py

@@ -6,3 +6,8 @@ class EstatePropertyType(models.Model):
     _description = "Real Estate Property Type"
 
     name = fields.Char(required=True)
+
+    _check_type_name_unique = models.Constraint(
+    'UNIQUE(name)', 
+    'The name of the property type must be unique.'
+    )

estate/security/ir.model.access.csv

@@ -2,4 +2,4 @@ id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
 estate.access_estate_property,access_estate_property,estate.model_estate_property,base.group_user,1,1,1,1
 estate.access_estate_property_type,access_estate_property_type,estate.model_estate_property_type,base.group_user,1,1,1,1
 access_estate_property_tag,estate.property.tag.access,model_estate_property_tag,base.group_user,1,1,1,1
-access_estate_property_offer,estate.property.offer.access,model_estate_property_offer,base.group_user,1,1,1,1
+access_estate_property_offer,estate.property.offer.access,model_estate_property_offer,base.group_user,1,1,1,1

estate/views/estate_property_offer_views.xml

@@ -7,6 +7,10 @@
                 <field name="price" />
                 <field name="partner_id" />
                 <field name="status" />
+                <field name="validity" />
+                <field name="date_deadline" />
+                <button name="action_accept" type="object" string="Accept" icon="fa-check"/>
+                <button name="action_refuse" type="object" string="Refuse" icon="fa-times"/>
             </list>
         </field>
     </record>
@@ -18,12 +22,15 @@
             <form>
                 <sheet>
                     <group>
-                        <field name="price"/>
-                        <field name="partner_id"/>
-                        <field name="status"/>
+                        <field name="price" />
+                        <field name="partner_id" />
+                        <field name="status" />
+                        <field name="validity" />
+                        <field name="date_deadline" />
                     </group>
                 </sheet>
             </form>
         </field>
     </record>
 </odoo>
+

estate/views/estate_property_tag_views.xml

@@ -4,5 +4,4 @@
         <field name="res_model">estate.property.tag</field>
         <field name="view_mode">list,form</field>
     </record>
-</odoo>
-
+</odoo>

estate/views/estate_property_views.xml

@@ -33,14 +33,20 @@
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
             <form string="form">
+                <header>
+                    <button name="action_sold" type="object" string="Sold" />
+                    <button name="action_cancel" type="object" string="Cancel" />
+                </header>
                 <sheet>
                     <group>
                         <group>
                             <field name="name" />
                             <field name="postcode" />
                             <field name="date_availability" />
                             <field name="property_type_id" />
+                            <field name="state" />
                             <field name="tag_ids" widget="many2many_tags" />
+                            <field name="best_price" />
                         </group>
                         <group>
                             <field name="expected_price" />
@@ -52,6 +58,7 @@
                             <field name="garden" />
                             <field name="garden_area" />
                             <field name="garden_orientation" />
+                            <field name="total_area" />
                         </group>
                     </group>
                     <notebook>
@@ -61,18 +68,6 @@
 
                         <page string="Offers">
                             <field name="offer_ids">
-                                <list>
-                                    <field name="price" />
-                                    <field name="partner_id" />
-                                    <field name="status" />
-                                </list>
-                                <form>
-                                    <group>
-                                        <field name="price" />
-                                        <field name="partner_id" />
-                                        <field name="status" />
-                                    </group>
-                                </form>
                             </field>
                         </page>