From 99e5de2a76954a6280c52e9900d4110c988eea46 Mon Sep 17 00:00:00 2001 From: Mark Orban Date: Tue, 19 May 2026 13:37:20 +0200 Subject: [PATCH] [ADD] estate, estate_account: Implement a Real Estate Application w/ Invoicing Problem: There is currently no module to manage selling properties. Solution: Two new modules create a Real Estate application which manages multiple properties, handles offers, and creates invoices once properties are sold. Task: 6229439 --- estate/__init__.py | 1 + estate/__manifest__.py | 18 +++ estate/models/__init__.py | 5 + estate/models/estate_property.py | 127 +++++++++++++++++ estate/models/estate_property_offer.py | 76 ++++++++++ estate/models/estate_property_tag.py | 14 ++ estate/models/estate_property_type.py | 32 +++++ estate/models/res_users.py | 11 ++ estate/security/ir.model.access.csv | 5 + estate/views/estate_menus.xml | 12 ++ estate/views/estate_property_offer_views.xml | 42 ++++++ estate/views/estate_property_tag_views.xml | 32 +++++ estate/views/estate_property_type_views.xml | 53 +++++++ estate/views/estate_property_views.xml | 139 +++++++++++++++++++ estate/views/res_users_views.xml | 15 ++ estate_account/__init__.py | 1 + estate_account/__manifest__.py | 11 ++ estate_account/models/__init__.py | 1 + estate_account/models/estate_account.py | 27 ++++ 19 files changed, 622 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/models/res_users.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_offer_views.xml create mode 100644 estate/views/estate_property_tag_views.xml create mode 100644 estate/views/estate_property_type_views.xml create mode 100644 estate/views/estate_property_views.xml create mode 100644 estate/views/res_users_views.xml create mode 100644 estate_account/__init__.py create mode 100644 estate_account/__manifest__.py create mode 100644 estate_account/models/__init__.py create mode 100644 estate_account/models/estate_account.py 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..13a5e43234d --- /dev/null +++ b/estate/__manifest__.py @@ -0,0 +1,18 @@ +{ + 'name': 'Real Estate', + 'author': 'Odoo S.A.', + 'depends': [ + 'base', + ], + 'application': True, + 'license': 'LGPL-3', + 'data': [ + 'views/estate_property_offer_views.xml', + 'security/ir.model.access.csv', + 'views/estate_property_views.xml', + 'views/estate_property_type_views.xml', + 'views/estate_property_tag_views.xml', + 'views/estate_menus.xml', + 'views/res_users_views.xml', + ] +} diff --git a/estate/models/__init__.py b/estate/models/__init__.py new file mode 100644 index 00000000000..fea9f441d6d --- /dev/null +++ b/estate/models/__init__.py @@ -0,0 +1,5 @@ +from . import estate_property +from . import estate_property_offer +from . import estate_property_tag +from . import estate_property_type +from . import res_users diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py new file mode 100644 index 00000000000..d89ee34b0a0 --- /dev/null +++ b/estate/models/estate_property.py @@ -0,0 +1,127 @@ +from odoo import _, api, fields, models +from odoo.exceptions import UserError, ValidationError +from odoo.tools.float_utils import float_compare + + +class EstateProperty(models.Model): + _name = "estate.property" + _description = "Estate Property Model" + _order = "id desc" + + name = fields.Char( + string="Property Name", + required=True, + ) + description = fields.Text() + postcode = fields.Char() + date_availability = fields.Date( + copy=False, + default=fields.Date.today().replace(month=(fields.Date.today().month + 3) % 12), + ) + expected_price = fields.Float(required=True) + selling_price = fields.Float( + readonly=True, + copy=False, + ) + best_price = fields.Float( + compute="_compute_best_price", + readonly=True, + copy=False, + ) + bedrooms = fields.Integer(default=2) + living_area = fields.Integer() + facades = fields.Integer() + garage = fields.Boolean() + garden = fields.Boolean() + garden_area = fields.Integer() + garden_orientation = fields.Selection( + selection=[("north", "North"), ("east", "East"), ("south", "South"), ("west", "West")], + string="Orientation", + ) + active = fields.Boolean(default=True) + state = fields.Selection( + selection=[("new", "New"), ("offer_received", "Offer Received"), ("offer_accepted", "Offer Accepted"), ("sold", "Sold"), ("cancelled", "Cancelled")], + string="Status", + default="new", + required=True, + copy=False, + ) + property_type_id = fields.Many2one( + "estate.property.type", + string="Property Type", + ) + salesperson_id = fields.Many2one( + "res.users", + string="Salesperson", + default=lambda self: self.env.user, + ) + buyer_id = fields.Many2one( + "res.partner", + string="Buyer", + copy=False, + ) + tag_ids = fields.Many2many( + "estate.property.tag", + string="Tags", + copy=False, + ) + offer_ids = fields.One2many( + "estate.property.offer", + "property_id", + copy=False, + ) + total_area = fields.Integer( + compute="_compute_total_area", + copy=False, + ) + + _check_expected_price = models.Constraint("CHECK (expected_price >= 0)", "The expected price must be strictly positive.") + _check_selling_price = models.Constraint("CHECK (selling_price >=0)", "The selling price must be strictly positive.") + + @api.depends("living_area", "garden_area") + def _compute_total_area(self): + for record in self: + record.total_area = record.living_area + record.garden_area + + @api.depends("offer_ids") + def _compute_best_price(self): + for record in self: + if record.offer_ids: + record.best_price = min(record.offer_ids.mapped("price")) + else: + record.best_price = 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 = None + + @api.constrains("selling_price") + def _check_minimum_selling_price(self): + for record in self: + if record.selling_price and float_compare(record.expected_price * 0.9, record.selling_price, precision_digits=2) > 0: + raise ValidationError(_("The selling price must be at least 90%% of the expected price. You must reduce the expected price in order to accept this offer.")) + + @api.ondelete(at_uninstall=False) + def _unlink_except_active_property(self): + for record in self: + if record.state in ("offer_received", "offer_accepted", "sold"): + raise UserError(_("Only new or cancelled properties may be deleted.")) + + def action_sold_button(self): + if self.state != "cancelled": + self.state = "sold" + else: + raise UserError(_("A cancelled property cannot be sold!")) + return True + + def action_cancel_button(self): + if self.state != "sold": + self.state = "cancelled" + else: + raise UserError(_("A sold property cannot be cancelled!")) + return True diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py new file mode 100644 index 00000000000..df5f16b6eb3 --- /dev/null +++ b/estate/models/estate_property_offer.py @@ -0,0 +1,76 @@ +from odoo import _, api, fields, models +from odoo.exceptions import UserError +from odoo.tools import float_compare + + +class EstatePropertyOffer(models.Model): + _name = "estate.property.offer" + _description = "Estate Property Offer Model" + _order = "price desc" + _check_price_positive = models.Constraint("CHECK (price >= 0)", "Offer price must be strictly positive.") + + price = fields.Float() + status = fields.Selection( + string="Status", + selection=[("accepted", "Accepted"), ("refused", "Refused")], + copy=False, + ) + partner_id = fields.Many2one( + "res.partner", + required=True, + ) + property_id = fields.Many2one( + "estate.property", + required=True, + ) + validity = fields.Integer( + string="Validity (days)", + default=7, + ) + date_deadline = fields.Date( + compute="_compute_date_deadline", + inverse="_inverse_date_deadline", + readonly=False, + ) + property_type_id = fields.Many2one( + related="property_id.property_type_id", + store=True, + ) + + @api.depends("validity") + def _compute_date_deadline(self): + for record in self: + if record.create_date: + record.date_deadline = fields.Date.add(record.create_date, days=record.validity) + else: + record.date_deadline = fields.Date.add(fields.Date.today(), days=record.validity) + + @api.model_create_multi + def create(self, vals_list): + for vals in vals_list: + if vals.get("property_id") and vals.get("price"): + prop = self.env["estate.property"].browse(vals["property_id"]) + if prop.offer_ids: + highest_offer = max(prop.mapped("offer_ids.price")) + if float_compare(vals["price"], highest_offer, precision_rounding=0.01) <= 0: + raise UserError(_("The offer must be higher than %(offer)s.", offer=highest_offer)) + prop.state = "offer_received" + return super().create(vals_list) + + def _inverse_date_deadline(self): + for record in self: + record.validity = ((record.date_deadline or fields.Date.today()) - record.create_date.date()).days + + def action_status_accepted(self): + if self.property_id.state in ("new", "offer_received"): + self.status = "accepted" + self.property_id.state = "offer_accepted" + self.property_id.selling_price = self.price + self.property_id.buyer_id = self.partner_id + else: + raise UserError(_("This property has already accepted an offer, been sold, or is cancelled!")) + return True + + def action_status_refused(self): + self.status = "refused" + return True diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py new file mode 100644 index 00000000000..5ccb29756dd --- /dev/null +++ b/estate/models/estate_property_tag.py @@ -0,0 +1,14 @@ +from odoo import fields, models + + +class EstatePropertyTag(models.Model): + _name = "estate.property.tag" + _description = "Estate Property Tag Model" + _order = "name" + + name = fields.Char( + string="Property Tag", + required=True) + color = fields.Integer() + + _check_name_unique = models.Constraint("UNIQUE(name)", "Tag 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..f6387385cf3 --- /dev/null +++ b/estate/models/estate_property_type.py @@ -0,0 +1,32 @@ +from odoo import fields, models + + +class EstatePropertyType(models.Model): + _name = "estate.property.type" + _description = "Estate Property Type Model" + _order = "sequence" + + name = fields.Char( + string="Property Type", + required=True, + ) + property_ids = fields.One2many( + "estate.property", + "property_type_id", + ) + sequence = fields.Integer( + 'Sequence', + default=1, + help="Used to order stages. Lower is better.", + ) + offer_ids = fields.One2many( + "estate.property.offer", + "property_type_id", + ) + offer_count = fields.Integer(compute="_compute_total_offers") + + _check_name_unique = models.Constraint("UNIQUE(name)", "Property type must be unique.") + + def _compute_total_offers(self): + for record in self: + record.offer_count = len(record.offer_ids) diff --git a/estate/models/res_users.py b/estate/models/res_users.py new file mode 100644 index 00000000000..0386275971d --- /dev/null +++ b/estate/models/res_users.py @@ -0,0 +1,11 @@ +from odoo import fields, models + + +class ResUsers(models.Model): + _inherit = "res.users" + + property_ids = fields.One2many( + "estate.property", + "salesperson_id", + domain=[("state", "in", ["new", "offer_received"])], + ) diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv new file mode 100644 index 00000000000..49bca99cac8 --- /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 +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 diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml new file mode 100644 index 00000000000..020003da75c --- /dev/null +++ b/estate/views/estate_menus.xml @@ -0,0 +1,12 @@ + + + + + + + + + + + + diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml new file mode 100644 index 00000000000..2aa1269a871 --- /dev/null +++ b/estate/views/estate_property_offer_views.xml @@ -0,0 +1,42 @@ + + + + estate.property.offer.form + estate.property.offer + +
+ + + + + + + + + +
+ + + + + estate.property.offer.list + estate.property.offer + + + + + + + + + + + + + + + + + + + + + + + + + + + + + estate.property.type.list + estate.property.type + + + + + + + + + + Property Types + estate.property.type + list,form + + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml new file mode 100644 index 00000000000..925c5830d5d --- /dev/null +++ b/estate/views/estate_property_views.xml @@ -0,0 +1,139 @@ + + + + estate.property.search + estate.property + + + + + + + + + + + + + + + + + + + estate.property.form + estate.property + +
+
+
+ + +

+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+ + + + + estate.property.kanban + estate.property + + + + + +
+
+

+
+
+ Expected Price: +
+
+ Best Offer: +
+
+ Selling Price: +
+
+ +
+
+ + + + + + + + estate.property.list + estate.property + + + + + + + + + + + + + + + + + + Properties + estate.property + list,form,kanban + {"search_default_available_filter": True} + + diff --git a/estate/views/res_users_views.xml b/estate/views/res_users_views.xml new file mode 100644 index 00000000000..6e55552630e --- /dev/null +++ b/estate/views/res_users_views.xml @@ -0,0 +1,15 @@ + + + + res.users.form.inherit.estate + res.users + + + + + + + + + + diff --git a/estate_account/__init__.py b/estate_account/__init__.py new file mode 100644 index 00000000000..0650744f6bc --- /dev/null +++ b/estate_account/__init__.py @@ -0,0 +1 @@ +from . import models diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py new file mode 100644 index 00000000000..38febc31b6a --- /dev/null +++ b/estate_account/__manifest__.py @@ -0,0 +1,11 @@ +{ + 'name': 'Estate Account', + 'author': 'Odoo S.A.', + 'depends': [ + 'estate', + 'account', + ], + 'application': True, + 'license': 'LGPL-3', + 'data': [] +} diff --git a/estate_account/models/__init__.py b/estate_account/models/__init__.py new file mode 100644 index 00000000000..02b688798a3 --- /dev/null +++ b/estate_account/models/__init__.py @@ -0,0 +1 @@ +from . import estate_account diff --git a/estate_account/models/estate_account.py b/estate_account/models/estate_account.py new file mode 100644 index 00000000000..3069453dd34 --- /dev/null +++ b/estate_account/models/estate_account.py @@ -0,0 +1,27 @@ +from odoo import Command, models + + +class EstateAccount(models.Model): + _inherit = "estate.property" + + def action_sold_button(self): + for prop in self: + self.env["account.move"].create( + { + "partner_id": prop.buyer_id.id, + "move_type": "out_invoice", + "invoice_line_ids": [ + Command.create({ + "name": prop.name, + "quantity": 1, + "price_unit": prop.selling_price * 0.06, + }), + Command.create({ + "name": "Administrative fees", + "quantity": 1, + "price_unit": 100, + }), + ], + } + ) + return super().action_sold_button()