From deb80464ab7b0c47aba59f3104eb6799d664b271 Mon Sep 17 00:00:00 2001 From: "Abdelrahman Gad (abgad)" Date: Tue, 19 May 2026 16:43:37 +0200 Subject: [PATCH] [ADD] estate, estate_account: create Real Estate and Real Estate Account modules for managing real estate business. PROBLEM Odoo does not have a built-in module to handle real estate properties, track buyer offers, or manage the property sales pipeline. GOAL Create Real Estate module and Real Estate Account modules to manage properties from the moment they hit the market until the final sale and invoice. SOLUTION Added Real Estate and Real Estate Account modules task-6229482 --- estate/__init__.py | 1 + estate/__manifest__.py | 16 ++ estate/models/__init__.py | 5 + estate/models/estate_property.py | 147 ++++++++++++++++++ estate/models/estate_property_offer.py | 98 ++++++++++++ estate/models/estate_property_tag.py | 19 +++ estate/models/estate_property_type.py | 48 ++++++ 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 | 48 ++++++ estate/views/estate_property_tag_views.xml | 19 +++ estate/views/estate_property_type_views.xml | 58 +++++++ estate/views/estate_property_views.xml | 151 +++++++++++++++++++ estate/views/res_users_views.xml | 17 +++ estate_account/__init__.py | 1 + estate_account/__manifest__.py | 7 + estate_account/models/__init__.py | 1 + estate_account/models/estate_property.py | 30 ++++ 19 files changed, 694 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_property.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..e3e2c96d7f6 --- /dev/null +++ b/estate/__manifest__.py @@ -0,0 +1,16 @@ +{ + 'name': 'Real Estate', + 'application': True, + 'depends': [ + 'base', + ], + 'data': [ + 'security/ir.model.access.csv', + 'views/estate_property_offer_views.xml', + 'views/estate_property_tag_views.xml', + 'views/estate_property_type_views.xml', + 'views/estate_property_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..9a2189b6382 --- /dev/null +++ b/estate/models/__init__.py @@ -0,0 +1,5 @@ +from . import estate_property +from . import estate_property_type +from . import estate_property_tag +from . import estate_property_offer +from . import res_users diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py new file mode 100644 index 00000000000..553c4efd3d4 --- /dev/null +++ b/estate/models/estate_property.py @@ -0,0 +1,147 @@ +from odoo import api, fields, models +from odoo.exceptions import UserError +from odoo.tools.float_utils import float_compare, float_is_zero + +class EstateProperty(models.Model): + _name = "estate.property" + _description = "It's free real estate" + + # Order attributes + _order = "id desc" + + name = fields.Char(required=True,) + description = fields.Text() + postcode = fields.Char() + date_availability = fields.Date( + default=lambda self: 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,) + living_area = fields.Integer() + facades = fields.Integer() + garage = fields.Boolean() + garden = fields.Boolean() + garden_area = fields.Integer() + garden_orientation = fields.Selection( + string='Garden Orientation', + selection=[('north', 'North'), ('south', 'South'), ('east', 'East'), ('west', 'West'),], + ) + state = fields.Selection( + string='State', + selection=[ + ('new', 'New'), + ('offer_received', 'Offer Received'), + ('offer_accepted', 'Offer Accepted'), + ('sold', 'Sold'), + ('canceled', 'Cancelled'), + ], + default='new', + required=True, + copy=False, + ) + + # Reserved fields + active = fields.Boolean(default=True, string='Active',) + + # Relations + property_type_id = fields.Many2one("estate.property.type",) + property_tag_ids = fields.Many2many("estate.property.tag",) + salesperson_id = fields.Many2one('res.users', string='Salesman', default=lambda self: self.env.user,) + buyer_id = fields.Many2one('res.partner', copy=False,) + offer_ids = fields.One2many("estate.property.offer", "property_id", string="Offers",) + + # ------------------------------------------------------------------------- + # COMPUTED FIELDS + # ------------------------------------------------------------------------- + total_area = fields.Integer( + string="Total Area (sqm)", + compute="_compute_total_area", + ) + + best_price = fields.Float( + string="Best Offer", + compute="_compute_best_price", + ) + + # ------------------------------------------------------------------------- + # SQL Constraints + # ------------------------------------------------------------------------- + _check_expected_price = models.Constraint( + 'CHECK(expected_price > 0)', + 'A property expected price must be strictly positive.', + ) + + _check_selling_price = models.Constraint( + 'CHECK(selling_price >= 0)', + 'A property selling price must be positive.', + ) + + # ------------------------------------------------------------------------- + # COMPUTE METHODS + # ------------------------------------------------------------------------- + @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: + if property.offer_ids: + property.best_price = max(property.offer_ids.mapped("price"), default=0.0) + else: + property.best_price = 0.0 + + # ------------------------------------------------------------------------- + # CONSTRAINTs METHODS + # ------------------------------------------------------------------------- + @api.constrains('expected_price', 'selling_price',) + def _check_selling_price(self,): + for property in self: + # Selling price = 0 => Do nothing + if float_is_zero(property.selling_price, precision_digits=2): + continue + + minimum_price = property.expected_price * 0.90 + + if float_compare(property.selling_price, minimum_price, precision_digits=2) < 0: + raise UserError(self.env._("The selling price cannot be lower than 90% of the expected price.")) + + # ------------------------------------------------------------------------- + # ONCHANGE METHODS + # ------------------------------------------------------------------------- + @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 = False + + # ------------------------------------------------------------------------- + # CRUD METHODS + # ------------------------------------------------------------------------- + @api.ondelete(at_uninstall=False,) + def _unlink_except_new_or_canceled(self,): + for property in self: + if property.state not in ('new', 'canceled',): + raise UserError(self.env._("You can only delete properties that are New or Canceled.")) + + # ------------------------------------------------------------------------- + # ACTIONS + # ------------------------------------------------------------------------- + def action_sold(self,): + for property in self: + if property.state == 'canceled': + raise UserError(self.env._("You cannot sell a canceled property.")) + self.state = 'sold' + return True + + def action_cancel(self,): + for property in self: + if property.state == 'sold': + raise UserError(self.env._("You cannot cancel a sold property.")) + self.state = 'canceled' + return True diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py new file mode 100644 index 00000000000..756b26d7eb9 --- /dev/null +++ b/estate/models/estate_property_offer.py @@ -0,0 +1,98 @@ +from datetime import timedelta +from odoo import api, fields, models +from odoo.exceptions import UserError + +class EstatePropertyOffer(models.Model): + _name = "estate.property.offer" + _description = "Real Estate Property Offer" + + # Order attributes + _order = "price desc" + + price = fields.Float() + status = fields.Selection( + selection=[ + ('accepted', 'Accepted'), + ('refused', 'Refused'), + ], + copy=False, + ) + validity = fields.Integer(default=7,) + date_deadline = fields.Date( + compute="_compute_date_deadline", + inverse="_inverse_date_deadline", + ) + partner_id = fields.Many2one("res.partner", required=True,) + property_id = fields.Many2one("estate.property", required=True,) + property_type_id = fields.Many2one( + "estate.property.type", + related="property_id.property_type_id", + string="Property Type", + store=True, + ) + + # ------------------------------------------------------------------------- + # CONSTRAINTS + # ------------------------------------------------------------------------- + _check_price = models.Constraint( + 'CHECK(price > 0)', + 'An offer price must be strictly positive.', + ) + + @api.depends("create_date", "validity",) + def _compute_date_deadline(self,): + for offer in self: + base_date = (offer.create_date or fields.Datetime.now()).date() + offer.date_deadline = base_date + timedelta(days=offer.validity,) + + def _inverse_date_deadline(self,): + for offer in self: + base_date = (offer.create_date or fields.Datetime.now()).date() + if offer.date_deadline: + offer.validity = (offer.date_deadline - base_date).days + + # ------------------------------------------------------------------------- + # CRUD methods + # ------------------------------------------------------------------------- + @api.model_create_multi + def create(self, vals_list,): + for vals in vals_list: + property_record = self.env['estate.property'].browse(vals['property_id']) + + if property_record.offer_ids: + best_price = max(property_record.offer_ids.mapped('price')) + if vals.get('price', 0) < best_price: + raise UserError(self.env._("You cannot create an offer with a price lower than the current best offer.")) + + property_record.state = 'offer_received' + + return super().create(vals_list) + + # ------------------------------------------------------------------------- + # ACTIONS + # ------------------------------------------------------------------------- + def action_accept(self,): + for offer in self: + if offer.property_id.state in ('sold', 'canceled',): + raise UserError(self.env._("You cannot accept an offer on a sold or canceled property.")) + + if offer.property_id.offer_ids.filtered(lambda o: o.status == "accepted"): + raise UserError(self.env._("An offer has already been accepted for this property.")) + + offer.status = 'accepted' + offer.property_id.selling_price = offer.price + offer.property_id.buyer_id = offer.partner_id + + other_offers = offer.property_id.offer_ids - offer + other_offers.status = 'refused' + + return True + + def action_refuse(self,): + for offer in self: + if offer.property_id.state in ('sold', 'canceled',): + raise UserError(self.env._("You cannot refuse an offer on a sold or canceled property.")) + + 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..75364a0ccd7 --- /dev/null +++ b/estate/models/estate_property_tag.py @@ -0,0 +1,19 @@ +from odoo import fields, models + +class EstatePropertyTag(models.Model): + _name = "estate.property.tag" + _description = "Property's tag" + + # Order attributes + _order = "name" + + name = fields.Char(required=True,) + color = fields.Integer(string="Color",) + + # ------------------------------------------------------------------------- + # CONSTRAINTS + # ------------------------------------------------------------------------- + _unique_tag_name = models.Constraint( + 'UNIQUE(name)', + 'A property 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..4a68c827a3c --- /dev/null +++ b/estate/models/estate_property_type.py @@ -0,0 +1,48 @@ +from odoo import api, fields, models + +class EstatePropertyType(models.Model): + _name = "estate.property.type" + _description = "Property's type" + + # Order attributes + _order = "sequence, name" + + name = fields.Char(required=True,) + + sequence = fields.Integer(string="Sequence", default=1, help="Used to order stages. Lower is better.",) + + # Relations + property_ids = fields.One2many( + "estate.property", + "property_type_id", + string="Properties", + ) + + offer_ids = fields.One2many( + "estate.property.offer", + "property_type_id", + string="Offers", + ) + + # ------------------------------------------------------------------------- + # COMPUTED FIELDS + # ------------------------------------------------------------------------- + offer_count = fields.Integer( + string="Offers Count", + compute="_compute_offer_count", + ) + + + # ------------------------------------------------------------------------- + # CONSTRAINTS + # ------------------------------------------------------------------------- + _unique_type_name = models.Constraint( + 'UNIQUE(name)', + 'A property type name must be unique.', + ) + + + @api.depends('offer_ids') + def _compute_offer_count(self): + for property_type in self: + property_type.offer_count = len(property_type.offer_ids) diff --git a/estate/models/res_users.py b/estate/models/res_users.py new file mode 100644 index 00000000000..7f5d032c14e --- /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", + string="Properties", + 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..0c0b62b7fee --- /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,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 +estate.access_estate_property_tag,access_estate_property_tag,estate.model_estate_property_tag,base.group_user,1,1,1,1 +estate.access_estate_property_offer,access_estate_property_offer,estate.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..3cdc83c8d3e --- /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..8e11eaab42f --- /dev/null +++ b/estate/views/estate_property_offer_views.xml @@ -0,0 +1,48 @@ + + + + + Offers + estate.property.offer + list,form + [('property_type_id', '=', active_id)] + + + + estate.property.offer.list + estate.property.offer + + + + + + + + + +
+

+ +

+
+ + + + + + + + + + + + + + + + + + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml new file mode 100644 index 00000000000..ddc2e8d40fc --- /dev/null +++ b/estate/views/estate_property_views.xml @@ -0,0 +1,151 @@ + + + + Properties + estate.property + kanban,list,form + + {'search_default_available': True} + + + + estate.property.list + estate.property + + + + + + + + + + + + + + + + + estate.property.form + estate.property + +
+
+
+ +
+

+ +

+
+
+ +
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+ + + + + estate.property.view.kanban + estate.property + + + + + + +
+ +
+ +
+ Expected Price: +
+ +
+ Best Price: +
+ +
+ Selling Price: +
+ +
+ +
+ + + + + + + + estate.property.search + estate.property + + + + + + + + + + + + + + + + + + + + diff --git a/estate/views/res_users_views.xml b/estate/views/res_users_views.xml new file mode 100644 index 00000000000..b03046e6776 --- /dev/null +++ b/estate/views/res_users_views.xml @@ -0,0 +1,17 @@ + + + + res.users.view.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..b4ef4f85db8 --- /dev/null +++ b/estate_account/__manifest__.py @@ -0,0 +1,7 @@ +{ + "name": "Real Estate Accounting", + "depends": [ + "estate", + "account", + ], +} diff --git a/estate_account/models/__init__.py b/estate_account/models/__init__.py new file mode 100644 index 00000000000..5e1963c9d2f --- /dev/null +++ b/estate_account/models/__init__.py @@ -0,0 +1 @@ +from . import estate_property diff --git a/estate_account/models/estate_property.py b/estate_account/models/estate_property.py new file mode 100644 index 00000000000..14b31b1cfaf --- /dev/null +++ b/estate_account/models/estate_property.py @@ -0,0 +1,30 @@ +from odoo import models + + +class EstateProperty(models.Model): + _inherit = "estate.property" + + + def action_sold(self): + res = super().action_sold() + + for property in self: + self.env["account.move"].create( + { + "partner_id": property.buyer_id.id, + "move_type": "out_invoice", + "invoice_line_ids": [ + Command.create({ + 'name': '6% Commission on Sales', + 'quantity': 1, + 'price_unit': property.selling_price * 0.06, + }), + Command.create({ + 'name': 'Administrative Fees', + 'quantity': 1, + 'price_unit': 100.00, + }), + ], + } + ) + return res