From b4d66c5512c1fe1b74a733d25c1f4a5d07e62f45 Mon Sep 17 00:00:00 2001
From: sreedevk <srkod@odoo.com>
Date: Tue, 19 May 2026 14:33:52 +0000
Subject: [PATCH] [ADD] estate: estate module added for real estate businesses

[ADD] estate: added the module setup files, the estate_property model & basic default views
[LINT] estate: ruff linting & formatting
[FIX] estate: added all rights to estate_property model
[ADD] estate: estate_property model field level restrictions & properties added
[LINT] estate: ruff linting & formatting
[ADD] estate: custom list view for estate_property
[REF] estate: follow naming convention for view records
[REM] estate: remove test model
[ADD] estate: added search view with custom fields
[ADD] estate: add filters for available properties & group by postcode
[ADD] estate: add property type models & views
[ADD] estate: property type relationship to property added
[ADD] estate: buyer & seller fields added to estate_property
[ADD] estate: property tag added
[LINT] estate: linting & formatting
[ADD] estate: estate property offer added
[ADD] estate: best price & total area computed fields added
[ADD] estate: property offer date_deadline & validity fields added
[ADD] estate: property onchange method for garden based attributes
[ADD] estate: offer refuse & accept operations added
[ADD] estate: sold & cancelled buttons added to properties
[ADD] estate: accepting offer now sets buyer & selling price of property
[ADD] estate: property constraints
[LINT] satisfy the linting and formatting overlords
[FIX] estate: api.constrains was misspelt as api.constraint
[FIX] estate: missing precision_digits in float_is_zero call
[FIX] estate: inverse deadline computation type issue fixed
[ADD] estate: property_ids view for property_type form
[LINT] estate: sacrifice to the linter
[ADD] estate: property status now displayed as a statusbar widget
[ADD] estate: deterministic ordering of records
[ADD] estate: manual sequencing of types enabled using a handle widget
[ADD] estate: tag color fields added to property form & prevent type creation
[ADD] estate: sold and cancel buttons now only show up when appropriate
[ADD] estate: default available filter applied

Update estate/models/estate_property.py

Co-authored-by: Thomas THBE <65757639+Megaaaaaa@users.noreply.github.com>

tmp
---
 estate/__init__.py                     |   1 +
 estate/__manifest__.py                 |  14 +++
 estate/models/__init__.py              |   6 ++
 estate/models/estate_property.py       | 132 +++++++++++++++++++++++++
 estate/models/estate_property_offer.py |  76 ++++++++++++++
 estate/models/estate_property_tag.py   |  16 +++
 estate/models/estate_property_type.py  |  31 ++++++
 estate/security/ir.model.access.csv    |   5 +
 estate/views/estate_menus.xml          |  21 ++++
 estate/views/estate_property.xml       | 115 +++++++++++++++++++++
 estate/views/estate_property_offer.xml |  35 +++++++
 estate/views/estate_property_tag.xml   |   8 ++
 estate/views/estate_property_type.xml  |  40 ++++++++
 13 files changed, 500 insertions(+)
 create mode 100644 estate/__init__.py
 create mode 100644 estate/__manifest__.py
 create mode 100644 estate/models/__init__.py
 create mode 100644 estate/models/estate_property.py
 create mode 100644 estate/models/estate_property_offer.py
 create mode 100644 estate/models/estate_property_tag.py
 create mode 100644 estate/models/estate_property_type.py
 create mode 100644 estate/security/ir.model.access.csv
 create mode 100644 estate/views/estate_menus.xml
 create mode 100644 estate/views/estate_property.xml
 create mode 100644 estate/views/estate_property_offer.xml
 create mode 100644 estate/views/estate_property_tag.xml
 create mode 100644 estate/views/estate_property_type.xml

diff --git a/estate/__init__.py b/estate/__init__.py
new file mode 100644
index 00000000000..0650744f6bc
--- /dev/null
+++ b/estate/__init__.py
@@ -0,0 +1 @@
+from . import models
diff --git a/estate/__manifest__.py b/estate/__manifest__.py
new file mode 100644
index 00000000000..6da37ad0852
--- /dev/null
+++ b/estate/__manifest__.py
@@ -0,0 +1,14 @@
+{
+    "name": "Estate",
+    "depends": ["base"],
+    "author": "Odoo S.A.",
+    "license": "LGPL-3",
+    "data": [
+        "security/ir.model.access.csv",
+        "views/estate_property.xml",
+        "views/estate_property_type.xml",
+        "views/estate_property_tag.xml",
+        "views/estate_property_offer.xml",
+        "views/estate_menus.xml",
+    ],
+}
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
new file mode 100644
index 00000000000..3683ff97b61
--- /dev/null
+++ b/estate/models/__init__.py
@@ -0,0 +1,6 @@
+from . import (
+    estate_property,
+    estate_property_offer,
+    estate_property_tag,
+    estate_property_type,
+)
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
new file mode 100644
index 00000000000..c97b429c470
--- /dev/null
+++ b/estate/models/estate_property.py
@@ -0,0 +1,132 @@
+from dateutil.relativedelta import relativedelta
+
+from odoo import _, api, fields, models
+from odoo.exceptions import UserError, ValidationError
+from odoo.tools.float_utils import float_compare, float_is_zero
+
+
+class EstateProperty(models.Model):
+    _name = "estate.property"
+    _description = "Estate Property"
+    _order = "id desc"
+
+    active = fields.Boolean(default=True)
+    state = fields.Selection(
+        string="State",
+        copy=False,
+        default="new",
+        required=True,
+        selection=[
+            ("new", "New"),
+            ("offer_received", "Offer Received"),
+            ("offer_accepted", "Offer Accepted"),
+            ("sold", "Sold"),
+            ("cancelled", "Cancelled"),
+        ],
+    )
+    name = fields.Char(required=True)
+    description = fields.Text()
+    postcode = fields.Char()
+    date_availability = fields.Date(
+        default=(fields.Date.today() + relativedelta(months=3)),
+        copy=False,
+    )
+    seller_id = fields.Many2one(
+        "res.users",
+        string="Sales Person",
+        default=lambda self: self.env.user,
+    )
+    buyer_id = fields.Many2one("res.partner", string="Buyer")
+
+    expected_price = fields.Float(required=True)
+    selling_price = fields.Float(readonly=True, copy=False)
+    best_price = fields.Float(compute="_compute_best_price")
+    property_tag_ids = fields.Many2many("estate.property.tag", string="Property Tags")
+    bedrooms = fields.Integer(default=2)
+    living_area = fields.Integer()
+    facades = fields.Integer()
+    garage = fields.Boolean()
+    garden = fields.Boolean()
+    garden_area = fields.Integer("Property Garden Area")
+    total_area = fields.Integer(compute="_compute_total_area")
+    property_type_id = fields.Many2one("estate.property.type", string="Property Type")
+    offer_ids = fields.One2many("estate.property.offer", "property_id", string="Offers")
+    garden_orientation = fields.Selection(
+        string="Garden Orientation",
+        selection=[
+            ("north", "North"),
+            ("south", "South"),
+            ("east", "East"),
+            ("west", "West"),
+        ],
+        help="Select garden orientation",
+    )
+
+    _check_expected_price = models.Constraint(
+        "CHECK(expected_price > 0)",
+        "Expected price should be positive.",
+    )
+    _check_selling_price = models.Constraint(
+        "CHECK(selling_price > 0)",
+        "Selling price must be positive.",
+    )
+
+    @api.depends("living_area", "garden_area")
+    def _compute_total_area(self):
+        for property in self:
+            property.total_area = property.living_area + property.garden_area
+
+    @api.depends("offer_ids.price")
+    def _compute_best_price(self):
+        for property in self:
+            property.best_price = max(property.offer_ids.mapped("price"), default=0)
+
+    @api.onchange("garden")
+    def _onchange_garden(self):
+        for property in self:
+            if property.garden:
+                property.garden_area = 10
+                property.garden_orientation = "north"
+            else:
+                property.garden_area = None
+                property.garden_orientation = None
+
+    def action_cancel_property(self):
+        for property in self:
+            if property.state == "sold":
+                raise UserError(_("Sold properties cannot be cancelled."))
+            else:
+                property.state = "cancelled"
+                return True
+
+    def action_sold_property(self):
+        for property in self:
+            if property.state == "cancelled":
+                raise UserError(_("Cancelled properties cannot be sold."))
+            else:
+                property.state = "sold"
+                return True
+
+    @api.constrains("selling_price")
+    def _check_selling_price(self):
+        for property in self:
+            selling_price = property.selling_price
+            expected_price = property.expected_price
+            if (
+                not float_is_zero(property.selling_price, precision_digits=2)
+                and float_compare(
+                    selling_price,
+                    0.9 * expected_price,
+                    precision_digits=2,
+                )
+                == -1
+            ):
+                raise ValidationError(
+                    _("Selling price is not atleast 90% of expected price."),
+                )
+
+    @api.ondelete(at_uninstall=False)
+    def _ensure_state_before_deletion(self):
+        for property in self:
+            if property.state in ("new", "cancelled"):
+                raise UserError(_("can't delete a properties in intermediate states."))
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
new file mode 100644
index 00000000000..feb9dd60dae
--- /dev/null
+++ b/estate/models/estate_property_offer.py
@@ -0,0 +1,76 @@
+from dateutil.relativedelta import relativedelta
+
+from odoo import _, api, fields, models
+from odoo.exceptions import UserError
+
+
+class EstatePropertyOffer(models.Model):
+    _name = "estate.property.offer"
+    _description = "Offer made on properties"
+    _order = "price desc"
+
+    property_type_id = fields.Many2one(related="property_id.property_type_id")
+    price = fields.Float()
+    status = fields.Selection(
+        string="status",
+        selection=[
+            ("accepted", "Accepted"),
+            ("refused", "Refused"),
+        ],
+        help="Offer Status",
+    )
+    partner_id = fields.Many2one("res.partner", required=True)
+    property_id = fields.Many2one("estate.property", required=True)
+    validity = fields.Integer(default=7)
+    date_deadline = fields.Date(
+        compute="_compute_date_deadline",
+        inverse="_compute_validity",
+    )
+
+    _check_price = models.Constraint(
+        "CHECK(price > 0)",
+        "Offer price must be positive.",
+    )
+
+    @api.depends("validity", "create_date")
+    def _compute_date_deadline(self):
+        for offer in self:
+            offer.date_deadline = (
+                offer.create_date or fields.Date.today()
+            ) + relativedelta(days=offer.validity)
+
+    def action_accept_offer(self):
+        for offer in self:
+            is_not_actionable = any(
+                status == "accepted"
+                for status in offer.property_id.offer_ids.mapped("status")
+            )
+            if is_not_actionable:
+                raise UserError(
+                    _("An offer has already been accepted for this property."),
+                )
+
+            offer.status = "accepted"
+            offer.property_id.buyer_id = offer.partner_id
+            offer.property_id.selling_price = offer.price
+            offer.property_id.state = "sold"
+            return True
+
+    def action_reject_offer(self):
+        self.status = "refused"
+        return True
+
+    @api.depends("create_date", "date_deadline")
+    def _compute_validity(self):
+        for offer in self:
+            self.validity = (
+                offer.date_deadline - (offer.create_date or fields.Date.today()).date()
+            ).days
+
+    @api.model
+    def create(self, vals):
+        property = self.env["estate.property"].browse(vals["property_id"])
+        if property.state == "new":
+            property.state = "offer_recevied"
+
+        return super(EstatePropertyOffer, self).create(vals)
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
new file mode 100644
index 00000000000..f8554c4c554
--- /dev/null
+++ b/estate/models/estate_property_tag.py
@@ -0,0 +1,16 @@
+from odoo import fields, models
+
+
+class EstatePropertyTag(models.Model):
+    _name = "estate.property.tag"
+    _description = "Tag representing an attribute's presence in a property"
+    _order = "name"
+
+    active = fields.Boolean(default=True)
+    color = fields.Integer(string="Color")
+    name = fields.Char(required=True)
+
+    _name_uniq = models.Constraint(
+        "unique (name)",
+        "Each tag name must be unique.",
+    )
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
new file mode 100644
index 00000000000..8f0ec7a0fa1
--- /dev/null
+++ b/estate/models/estate_property_type.py
@@ -0,0 +1,31 @@
+from odoo import api, fields, models
+
+
+class EstatePropertyType(models.Model):
+    _name = "estate.property.type"
+    _description = "type of the estate property"
+    _order = "name"
+
+    offer_ids = fields.One2many(
+        "estate.property.offer",
+        "property_type_id",
+        string="Offers",
+    )
+    offer_count = fields.Integer(compute="_compute_offer_count")
+    active = fields.Boolean(default=True)
+    name = fields.Char(required=True)
+    sequence = fields.Integer(string="Sequence", default=1, help="Used for ordering")
+    property_ids = fields.One2many(
+        "estate.property",
+        "property_type_id",
+        string="Properties",
+    )
+
+    _name_uniq = models.Constraint(
+        "unique (name)",
+        "Each property type name must be unique.",
+    )
+
+    @api.depends("offer_ids")
+    def _compute_offer_count(self):
+        self.offer_count = len(property.offer_ids)
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
new file mode 100644
index 00000000000..a90b03a38c6
--- /dev/null
+++ b/estate/security/ir.model.access.csv
@@ -0,0 +1,5 @@
+id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
+estate.access_estate_property,access_estate_property,model_estate_property,base.group_user,1,1,1,1
+estate.access_estate_property_type,access_estate_property_type,model_estate_property_type,base.group_user,1,1,1,1
+estate.access_estate_property_tag,access_estate_property_tag,model_estate_property_tag,base.group_user,1,1,1,1
+estate.access_estate_property_offer,access_estate_property_offer,model_estate_property_offer,base.group_user,1,1,1,1
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
new file mode 100644
index 00000000000..1cee15f2511
--- /dev/null
+++ b/estate/views/estate_menus.xml
@@ -0,0 +1,21 @@
+<?xml version="1.0"?>
+<odoo>
+    <menuitem id="estate.app_menu" name="Estate">
+        <menuitem id="advertisements_list_menu" name="Advertisements">
+            <menuitem 
+                id="estate.advertisements_properties_list_menu_item"
+                name="Properties"
+                action="estate.estate_property_list_form"/>
+        </menuitem>
+        <menuitem id="estate.settings_list_menu" name="Settings">
+            <menuitem 
+                id="estate.advertisements_property_types_list_menu_item"
+                name="Property Types"
+                action="estate.estate_property_type_list_form"/>
+            <menuitem 
+                id="estate.advertisements_property_tag_list_menu_item"
+                name="Property Tags"
+                action="estate.estate_property_tag_list_form"/>
+        </menuitem>
+    </menuitem>
+</odoo>
diff --git a/estate/views/estate_property.xml b/estate/views/estate_property.xml
new file mode 100644
index 00000000000..e563dbb0572
--- /dev/null
+++ b/estate/views/estate_property.xml
@@ -0,0 +1,115 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate.estate_property_view_list" model="ir.ui.view">
+        <field name="name">estate.property.list</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <list string="Properties" 
+                decoration-success="state in ('offer_received', 'offer_accepted')" 
+                decoration-bf="state == 'offer_accepted'"
+                decoration-muted="state == 'sold'">
+                <field name="name" string="Name"/>
+                <field name="postcode" string="Postcode"/>
+                <field name="property_tag_ids" string="Tags" widget="many2many_tags"/>
+                <field name="bedrooms" string="Bedrooms"/>
+                <field name="living_area" string="Living Area (sqm)"/>
+                <field name="expected_price" string="Expected Price"/>
+                <field name="selling_price" string="Selling Price"/>
+                <field name="date_availability" string="Available From" optional="hidden"/>
+            </list>
+        </field>
+    </record>
+    <record id="estate.estate_property_view_form" model="ir.ui.view">
+        <field name="name">estate.property.form</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <form>
+                <header>
+                    <group invisible="state in ('sold', 'cancelled')">
+                        <button name="action_cancel_property" string="Cancel" type="object" icon="fa-trash"/>
+                        <button name="action_sold_property" string="Sold" type="object" icon="fa-check"/>
+                    </group> 
+                    <field name="state" widget="statusbar" statusbar_visible="new,offer_received,offer_accepted,sold"/>
+                </header>
+                <group>
+                    <group>
+                        <field name="name" />
+                        <field name="property_tag_ids" string="Tags" widget="many2many_tags" options="{'color_field': 'color'}"/>
+                    </group>  
+                </group>
+                <group>
+                    <group>
+                        <field name="postcode" string="Postcode"/>
+                        <field name="date_availability" string="Available From"/>
+                    </group>
+                    <group>
+                        <field name="expected_price" string="Expected Price"/>
+                        <field name="selling_price" string="Selling Price"/>
+                        <field name="best_price" string="Best Price"/>
+                    </group>
+                </group>
+                <notebook>
+                    <page string="Description">
+                        <group>
+                            <group>
+                                <field name="description" string="Description"/>
+                                <field name="garage" string="Garage"/>
+                                <field name="garden" string="Garden"/>
+                                <field name="garden_area" string="Garden Area (sqm)" invisible="not garden"/>
+                                <field name="garden_orientation" string="Garden Orientation" invisible="not garden"/>
+                            </group>
+                            <group>
+                                <group>
+                                    <field name="bedrooms" string="Bedrooms"/>
+                                    <field name="living_area" string="Living Area (sqm)"/>
+                                    <field name="facades" string="Facades"/>
+                                    <field name="total_area" string="Total Area"/>
+                                </group>
+                            </group>
+                        </group>
+                        <group>
+                            <field name="property_type_id" string="Property Type" options="{'no_create': true}"/>
+                            <field name="seller_id" string="Sales Person"/>
+                            <field name="buyer_id" string="Buyer"/>
+                        </group>
+                    </page>
+                    <page string="Offers">
+                        <group>
+                            <field 
+                                name="offer_ids"
+                                context="{'list_view_ref': 'estate.estate_property_offer_list_form'}"
+                                readonly="state == 'offer_accepted' or state == 'sold' or state == 'cancelled'"/>
+                        </group>
+                    </page>
+                </notebook>
+            </form>
+        </field>
+    </record>
+
+    <record id="estate.estate_property_view_search" model="ir.ui.view">
+        <field name="name">estate.property.search</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <search string="Search">
+                <field name="name" string="Title"/>
+                <field name="postcode" string="Postcode"/>
+                <field name="expected_price" string="Expected Price"/>
+                <field name="bedrooms" string="Bedrooms"/>
+                <field name="living_area" string="Living Area (sqm)" filter_domain="[('living_area', '&gt;=', self)]"/>
+                <field name="facades" string="Facades"/>
+                <separator/>
+                <filter name="available" string="Available" domain="['|', ('state', '=', 'new'), ('state', '=', 'offer_received')]"/>
+                <group>
+                    <filter name="group_by_postcode" string="Group by Postcode" context="{'group_by':'postcode'}"/>
+                </group>
+            </search>
+        </field>
+    </record>
+
+    <record id="estate.estate_property_list_form" model="ir.actions.act_window">
+        <field name="name">Estate Properties</field>
+        <field name="res_model">estate.property</field>
+        <field name="view_mode">list,form</field>
+        <field name="context">{'search_default_available': True}</field>
+    </record>
+</odoo>
diff --git a/estate/views/estate_property_offer.xml b/estate/views/estate_property_offer.xml
new file mode 100644
index 00000000000..2418eb6a34e
--- /dev/null
+++ b/estate/views/estate_property_offer.xml
@@ -0,0 +1,35 @@
+<?xml version="1.0"?>
+<odoo>
+
+    <record id="estate.estate_property_offer_view_list" model="ir.ui.view">
+        <field name="name">estate.property.offer.list</field>
+        <field name="model">estate.property.offer</field>
+        <field name="arch" type="xml">
+            <list editable="top" decoration-success="status == 'accepted'" decoration-danger="status == 'refused'">
+                <field name="price" string="Price"/>
+                <field name="partner_id" string="Partner"/>
+                <field name="status" string="Status" optional="hidden"/>
+                <field name="validity" string="Validity (days)"/>
+                <field name="date_deadline" string="Deadline"/>
+                <button 
+                    name="action_accept_offer"
+                    string="Accept"
+                    type="object"
+                    icon="fa-check"
+                    invisible="status == 'accepted' or status == 'refused'"/>
+                <button
+                    name="action_reject_offer"
+                    string="Refuse"
+                    type="object"
+                    icon="fa-trash"
+                    invisible="status == 'accepted' or status == 'refused'"/>
+            </list>
+        </field>
+    </record>
+
+    <record id="estate.estate_property_offer_list_form" model="ir.actions.act_window">
+        <field name="name">Offers</field>
+        <field name="res_model">estate.property.offer</field>
+        <field name="view_mode">list,form</field>
+    </record>
+</odoo>
diff --git a/estate/views/estate_property_tag.xml b/estate/views/estate_property_tag.xml
new file mode 100644
index 00000000000..30d99f73a25
--- /dev/null
+++ b/estate/views/estate_property_tag.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate.estate_property_tag_list_form" model="ir.actions.act_window">
+        <field name="name">Property Tags</field>
+        <field name="res_model">estate.property.tag</field>
+        <field name="view_mode">list,form</field>
+    </record>
+</odoo>
diff --git a/estate/views/estate_property_type.xml b/estate/views/estate_property_type.xml
new file mode 100644
index 00000000000..c3397d9807d
--- /dev/null
+++ b/estate/views/estate_property_type.xml
@@ -0,0 +1,40 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="estate.estate_property_type_view_list" model="ir.ui.view">
+        <field name="name">estate.property.type.list</field>
+        <field name="model">estate.property.type</field>
+        <field name="arch" type="xml">
+            <list>
+                <field name="sequence" widget="handle"/>
+                <field name="name" string="Name"/>
+            </list>
+        </field>
+    </record>
+
+    <record id="estate.estate_property_type_view_form" model="ir.ui.view">
+        <field name="name">estate.property.type.form</field>
+        <field name="model">estate.property.type</field>
+        <field name="arch" type="xml">
+            <form>
+                <notebook>
+                    <page string="Details">
+                        <group>
+                            <field name="name"/>
+                        </group>
+                    </page>
+                    <page string="Properties">
+                        <group>
+                            <field name="property_ids" context="{'list_view_ref': 'estate.estate_property_list_form'}"/>
+                        </group>
+                    </page>
+                </notebook>
+            </form>
+        </field>
+    </record>
+
+    <record id="estate.estate_property_type_list_form" model="ir.actions.act_window">
+        <field name="name">Property Types</field>
+        <field name="res_model">estate.property.type</field>
+        <field name="view_mode">list,form</field>
+    </record>
+</odoo>