Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
2189c2b
[ADD] estate: add new real estate module
robal-odoo Dec 15, 2025
d98554a
[IMP] ignore .vscode directory in git
robal-odoo Dec 15, 2025
783c0ed
[IMP] estate: add model estate_property
robal-odoo Dec 15, 2025
0a10db3
[IMP] estate: add access rules for estate.property model
robal-odoo Dec 15, 2025
66d6040
[IMP] estate: add basic menus to the module
robal-odoo Dec 15, 2025
84d7432
[IMP] estate: improve form, list and search views for estate property…
robal-odoo Dec 15, 2025
958549d
[CLN] estate: renamed estate menus
robal-odoo Dec 15, 2025
e206b8e
[IMP] estate: add relation fields for property types, tags and offers
robal-odoo Dec 16, 2025
8e0b93e
[LINT] estate: clean up models to match coding guidelines
robal-odoo Dec 16, 2025
bd767ca
[LINT] estate: clean up __init__ files
robal-odoo Dec 16, 2025
f5d3f43
[IMP] estate: add computed fields and "onchange" handler method
robal-odoo Dec 16, 2025
d8993d0
[IMP] estate: add actions for selling, cancelled properties and accep…
robal-odoo Dec 16, 2025
6940deb
[FIX] estate: remove property state from the property form view
robal-odoo Dec 17, 2025
f4fe545
[FIX] estate: reset a property's buyer and selling price when refusin…
robal-odoo Dec 17, 2025
dd59729
[LINT] estate: fix the CI warning for icon buttons without text
robal-odoo Dec 17, 2025
3ac9150
[IMP] estate: add contraints to models
robal-odoo Dec 17, 2025
b258392
[CLN] estate: align value of "nolabel" attributes in estate property …
robal-odoo Dec 17, 2025
55caa93
[IMP] estate: refine UI
robal-odoo Dec 17, 2025
c097387
[LINT] estate: remove unnecessary empty lines
robal-odoo Dec 18, 2025
9d14dfd
[LINT] estate: ruff linter pass
robal-odoo Dec 18, 2025
7c529c2
[IMP] estate: view improvements
robal-odoo Dec 18, 2025
9c60bb2
[FIX] fixes property_type_id related field not being stored due to a …
robal-odoo Dec 18, 2025
a776f13
[IMP] estate: refine CRUD access to models + extend user form
robal-odoo Dec 18, 2025
94d0b27
[ADD] estate_account: automatically create an invoice when a property…
robal-odoo Dec 18, 2025
d6444c7
[IMP] estate: add kanban view for estate properties
robal-odoo Dec 18, 2025
503f7f3
[MOV] estate_account: rename model file
robal-odoo Dec 18, 2025
cc3871e
[LINT] estate: fix quote consistency in manifest
robal-odoo Dec 18, 2025
70f7fc7
[LINT] estate_account: fix quote consistency in manifest
robal-odoo Dec 18, 2025
90be6c6
[IMP] estate: rename files and xmlids for match coding guidelines
robal-odoo Dec 18, 2025
0fd1b1d
[FIX] estate: use translation method for error messages
robal-odoo Dec 18, 2025
ea1aaa7
[FIX] estate_account: invoice line descriptions must be translated
robal-odoo Dec 18, 2025
2d8b90a
[FIX] estate: fix edge-case in validation of a new offer's price
robal-odoo Dec 18, 2025
dd5169f
[IMP] estate: renamed check constraint variables
robal-odoo Dec 18, 2025
0eedd7a
[FIX] estate: fix colors in estate property list form
robal-odoo Dec 18, 2025
a2c03a6
[CLN] estate: remove some unnecessary code
robal-odoo Dec 18, 2025
2fb7576
[LINT] estate_account: lint pass
robal-odoo Dec 18, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -127,3 +127,6 @@ dmypy.json

# Pyre type checker
.pyre/

# VScode
.vscode/
1 change: 1 addition & 0 deletions estate/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
from . import models
18 changes: 18 additions & 0 deletions estate/__manifest__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
{
"name": "Real Estate",
"description": "Real Estate Advertising",
"author": "robal",
"license": "LGPL-3",
"depends": ["base"],
"application": True,
"data": [
"security/ir.model.access.csv",

"views/estate_property_views.xml",
"views/estate_property_offer_views.xml",
"views/estate_property_type_views.xml",
"views/estate_property_tag_views.xml",
"views/estate_menus.xml",
"views/res_users_views.xml",
],
}
5 changes: 5 additions & 0 deletions estate/models/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
from . import estate_property_offer
from . import estate_property_tag
from . import estate_property_type
from . import estate_property
from . import res_users
113 changes: 113 additions & 0 deletions estate/models/estate_property.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,113 @@
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"

name = fields.Char(string="Title", required=True)
description = fields.Text()
postcode = fields.Char()
date_availability = fields.Date(string="Available From", default=fields.Date.today() + relativedelta(months=3), copy=False)
expected_price = fields.Float(required=True)
selling_price = fields.Float(readonly=True, copy=False)
bedrooms = fields.Integer(default=2)
living_area = fields.Integer(string="Living Area (sqm)")
facades = fields.Integer()
garage = fields.Boolean()
garden = fields.Boolean()
garden_area = fields.Integer(string="Garden Area (sqm)")
garden_orientation = fields.Selection(
selection=[("north", "North"), ("south", "South"), ("east", "East"), ("west", "West")],
)

# Reserved fields
active = fields.Boolean(default=True)
state = fields.Selection(
required=True,
selection=[("new", "New"), ("offer_received", "Offer Received"), ("offer_accepted", "Offer Accepted"), ("sold", "Sold"), ("cancelled", "Cancelled")],
default="new",
copy=False,
string="Status",
)

# Relations
property_type_id = fields.Many2one("estate.property.type", string="Property Type")
salesman_id = fields.Many2one("res.users", default=lambda self: self.env.user)
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

@api.ondelete(at_uninstall=False)
def _unlink_except_new_or_cancelled(self) -> None:
if any(record.state not in {"new", "cancelled"} for record in self):
raise UserError(_("Cannot delete properties unless they are new or cancelled."))

# Public methods
def action_set_sold(self) -> bool:
for record in self:
if record.state == "sold":
raise UserError(_("This property has already been sold."))

if record.state == "cancelled":
raise UserError(_("Cancelled properties cannot be sold."))

record.state = "sold"
return True

def action_set_cancelled(self) -> bool:
for record in self:
if record.state == "sold":
raise UserError(_("Sold properties cannot be cancelled."))

if record.state == "cancelled":
raise UserError(_("This property has already been cancelled."))

record.state = "cancelled"
return True

# Constraints
_check_expected_price_strict_positive = models.Constraint(
"CHECK(expected_price > 0)",
"A property's expected price must be strictly greater than 0.",
)

_check_selling_price_positive = models.Constraint(
"CHECK(selling_price >= 0)",
"A property's selling price must be equal to or greater than 0.",
)

@api.constrains("expected_price", "selling_price")
def _restrict_selling_price(self) -> None:
for record in self:
# Selling price is zero when no offer has been accepted
if not float_is_zero(record.selling_price, precision_digits=2) and float_compare(record.selling_price, 0.9 * record.expected_price, precision_digits=2) < 0:
raise ValidationError(_("A property's selling price cannot be lower that 90 percent of its expected price."))
75 changes: 75 additions & 0 deletions estate/models/estate_property_offer.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,75 @@
from datetime import date

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 = "Estate property offer"
_order = "price desc"

price = fields.Float()
status = fields.Selection(
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)
property_type_id = fields.Many2one(related="property_id.property_type_id", store=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

# CRUD overrides
@api.model
def create(self, vals):
for val in vals:
estate_property = self.env["estate.property"].browse(val["property_id"])
best_price = estate_property.best_price or 0.0
if val["price"] < best_price:
raise UserError(_("A new offer must match or exceed the price of the current best offer."))
estate_property.state = "offer_received"
return super().create(vals)

# Public methods
def action_accept(self) -> bool:
# TODO double check validation
for record in self:
if record.property_id.offer_ids.filtered(lambda r: r.status == "accepted"):
raise UserError(_("Cannot accept this offer because another offer has already been accepted for the property."))

record.status = "accepted"
record.property_id.selling_price = record.price
record.property_id.buyer_id = record.partner_id
record.property_id.state = "offer_accepted"
return True

def action_refuse(self) -> bool:
# TODO double check validation
for record in self:
record.status = "refused"
return True

# Constraints
_check_price_strict_positive = models.Constraint(
"CHECK(price > 0)",
"An offer's price must be strictly greater than 0.",
)


def _get_reference_date(offer: EstatePropertyOffer) -> date:
return fields.Date.today() if not offer.create_date else offer.create_date.date()
16 changes: 16 additions & 0 deletions estate/models/estate_property_tag.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
from odoo import fields, models


class EstatePropertyTag(models.Model):
_name = "estate.property.tag"
_description = "Estate property tag"
_order = "name"

name = fields.Char(required=True)
color = fields.Integer()

# Constraints
_uniq_name = models.Constraint(
"UNIQUE(name)",
"A property tag's name must be unique.",
)
28 changes: 28 additions & 0 deletions estate/models/estate_property_type.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
from odoo import api, fields, models


class EstatePropertyType(models.Model):
_name = "estate.property.type"
_description = "Estate property type"
_order = "name"

name = fields.Char(required=True)
sequence = fields.Integer(default=1)

# Relations
property_ids = fields.One2many("estate.property", "property_type_id")
offer_ids = fields.One2many("estate.property.offer", "property_type_id")

# Computed
offer_count = fields.Integer(compute="_compute_offer_count")

@api.depends("offer_ids")
def _compute_offer_count(self) -> None:
for record in self:
record.offer_count = len(record.offer_ids)

# Constraints
_uniq_name = models.Constraint(
"UNIQUE(name)",
"A property type's name must be unique.",
)
7 changes: 7 additions & 0 deletions estate/models/res_users.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
from odoo import fields, models


class User(models.Model):
_inherit = "res.users"

property_ids = fields.One2many("estate.property", "salesman_id", domain=[("state", "in", ["new", "offer_received"])])
5 changes: 5 additions & 0 deletions estate/security/ir.model.access.csv
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink
access_estate_property,access_estate_property,model_estate_property,base.group_user,1,1,1,1
access_estate_property_type,access_estate_property_type,model_estate_property_type,base.group_user,1,1,1,1
access_estate_property_tag,access_estate_property_tag,model_estate_property_tag,base.group_user,1,1,1,1
access_estate_property_offer,access_estate_property_offer,model_estate_property_offer,base.group_user,1,1,1,1
12 changes: 12 additions & 0 deletions estate/views/estate_menus.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
<?xml version="1.0"?>
<odoo>
<menuitem id="estate_menu_root" name="Real Estate">
<menuitem id="estate_property_menu" name="Properties">
<menuitem id="estate_property_menu_action" action="estate_property_action"/>
</menuitem>
<menuitem id="estate_settings_menu" name="Settings">
<menuitem id="estate_property_type_menu" action="estate_property_type_action"/>
<menuitem id="estate_property_tag_menu" action="estate_property_tag_action"/>
</menuitem>
</menuitem>
</odoo>
42 changes: 42 additions & 0 deletions estate/views/estate_property_offer_views.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
<?xml version="1.0"?>
<odoo>
<record id="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="bottom" decoration-danger="status == 'refused'" decoration-success="status == 'accepted'">
<field name="price"/>
<field name="partner_id"/>
<field name="validity"/>
<field name="date_deadline"/>
<button name="action_accept" type="object" icon="fa-check" string="Accept" invisible="status"/>
<button name="action_refuse" type="object" icon="fa-times" string="Refuse" invisible="status"/>
</list>
</field>
</record>

<record id="estate_property_offer_view_form" model="ir.ui.view">
<field name="name">estate.property.offer.form</field>
<field name="model">estate.property.offer</field>
<field name="arch" type="xml">
<form>
<sheet>
<group>
<field name="price"/>
<field name="partner_id"/>
<field name="status"/>
<field name="validity"/>
<field name="date_deadline"/>
</group>
</sheet>
</form>
</field>
</record>

<record id="estate_property_offer_action" 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>
<field name="domain">[('property_type_id', '=', active_id)]</field>
</record>
</odoo>
32 changes: 32 additions & 0 deletions estate/views/estate_property_tag_views.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
<?xml version="1.0"?>
<odoo>
<record id="estate_property_tag_lview_list" model="ir.ui.view">
<field name="name">estate.property.tag.list</field>
<field name="model">estate.property.tag</field>
<field name="arch" type="xml">
<list editable="bottom">
<field name="name"/>
</list>
</field>
</record>

<record id="estate_property_tag_view_form" model="ir.ui.view">
<field name="name">estate.property.tag.form</field>
<field name="model">estate.property.tag</field>
<field name="arch" type="xml">
<form>
<sheet>
<group>
<field name="name"/>
</group>
</sheet>
</form>
</field>
</record>

<record id="estate_property_tag_action" 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>
Loading