From f5b4edefcd4bf2cf4a78caed870189989bc84961 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 20 Jul 2026 14:01:45 +0200 Subject: [PATCH 01/41] [ADD] estate: 1. A New Application --- estate/__init__.py | 1 + estate/__manifest__.py | 6 ++++++ 2 files changed, 7 insertions(+) create mode 100644 estate/__init__.py create mode 100644 estate/__manifest__.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..73e1a1c2ad4 --- /dev/null +++ b/estate/__manifest__.py @@ -0,0 +1,6 @@ +{ + 'name': "Real Estate", + 'depends': ['base'], + 'author': 'bepro', + 'license': 'LGPL-3', +} From 5b8acd1650977a04277d26dd62364d6bbcc3a954 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 20 Jul 2026 14:06:48 +0200 Subject: [PATCH 02/41] [IMP] estate: 3. Models and fields --- estate/models/__init__.py | 1 + estate/models/property.py | 34 ++++++++++++++++++++++++++++++++++ 2 files changed, 35 insertions(+) create mode 100644 estate/models/__init__.py create mode 100644 estate/models/property.py diff --git a/estate/models/__init__.py b/estate/models/__init__.py new file mode 100644 index 00000000000..8120b005bb6 --- /dev/null +++ b/estate/models/__init__.py @@ -0,0 +1 @@ +from . import property diff --git a/estate/models/property.py b/estate/models/property.py new file mode 100644 index 00000000000..ba6f21dcb09 --- /dev/null +++ b/estate/models/property.py @@ -0,0 +1,34 @@ +from odoo import fields, models + + +class Property(models.Model): + _name = "estate.property" + _description = "Properties of our managed estates" + + name = fields.Char(string='Name', default="Unknown", required=True) + active = fields.Boolean(default=True) + state = fields.Selection(required=True, default='new', copy=False, selection=[ + ('new', 'New') + , ('offer_received', 'Offer Received') + , ('offer_accepted', 'Offer Accepted') + , ('sold', 'Sold') + , ('cancelled', 'Cancelled'), + ]) + + description = fields.Text(string='description') + postcode = fields.Char(string='postcode') + date_availability = fields.Date(string='Available from', default=lambda _: fields.Date.add(fields.Date.today(), months=3), copy=False) + expected_price = fields.Float(string='expected price', required=True) + selling_price = fields.Float(string='selling price', readonly=True, copy=False) + bedrooms = fields.Integer(string='# bedrooms', default=2) + living_area = fields.Integer(string='living area size') + facades = fields.Integer(string='# facades') + garage = fields.Boolean(string='Has garage') + garden = fields.Boolean(string='Has garden') + garden_area = fields.Integer(string='garden area size') + garden_orientation = fields.Selection(string='garden orientation', + selection=[ + ('north', 'North') + , ('south', 'South') + , ('east', 'East') + , ('west', 'West')]) From 999d42193b0f77c4dc1e73fc157a020f07066b1b Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 20 Jul 2026 14:07:50 +0200 Subject: [PATCH 03/41] [IMP] estate: 4. Security --- estate/__manifest__.py | 5 +++++ estate/security/ir.model.access.csv | 2 ++ 2 files changed, 7 insertions(+) create mode 100644 estate/security/ir.model.access.csv diff --git a/estate/__manifest__.py b/estate/__manifest__.py index 73e1a1c2ad4..4fc56a4886e 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -3,4 +3,9 @@ 'depends': ['base'], 'author': 'bepro', 'license': 'LGPL-3', + 'data': [ + 'security/ir.model.access.csv', + # 'views/estate_property_views.xml', + # 'views/estate_menus.xml' + ], } diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv new file mode 100644 index 00000000000..e5b8c9cc4dc --- /dev/null +++ b/estate/security/ir.model.access.csv @@ -0,0 +1,2 @@ +id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink +access_estate_property_model,access_estate_property_model,estate.model_estate_property,base.group_user,1,1,1,1 \ No newline at end of file From 69c5e669ce0e0b02ab3a7bbdc3fdeaac80bcb5a2 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 20 Jul 2026 14:11:32 +0200 Subject: [PATCH 04/41] [IMP] estate: 5. Some UI --- estate/__manifest__.py | 4 ++-- estate/views/estate_menus.xml | 9 +++++++++ estate/views/estate_property_views.xml | 8 ++++++++ 3 files changed, 19 insertions(+), 2 deletions(-) create mode 100644 estate/views/estate_menus.xml create mode 100644 estate/views/estate_property_views.xml diff --git a/estate/__manifest__.py b/estate/__manifest__.py index 4fc56a4886e..d34073eea7e 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -5,7 +5,7 @@ 'license': 'LGPL-3', 'data': [ 'security/ir.model.access.csv', - # 'views/estate_property_views.xml', - # 'views/estate_menus.xml' + 'views/estate_property_views.xml', + 'views/estate_menus.xml', ], } diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml new file mode 100644 index 00000000000..55c99310847 --- /dev/null +++ b/estate/views/estate_menus.xml @@ -0,0 +1,9 @@ + + + + + + + + + \ No newline at end of file diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml new file mode 100644 index 00000000000..02748fb409f --- /dev/null +++ b/estate/views/estate_property_views.xml @@ -0,0 +1,8 @@ + + + + Properties + estate.property + list,form + + \ No newline at end of file From 3cbf95acfa47a0997df09c6ccaa34a2a41c8aa8d Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 20 Jul 2026 14:11:49 +0200 Subject: [PATCH 05/41] [IMP] estate: 6. Basic views --- estate/views/estate_property_views.xml | 82 ++++++++++++++++++++++++++ 1 file changed, 82 insertions(+) diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 02748fb409f..eb07ccbd359 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -5,4 +5,86 @@ estate.property list,form + + + estate.properties.list + estate.property + + + + + + + + + + + + + + + estate.properties.form + estate.property + +
+ +
+
+
+

+ +

+ +
+ + + + + + + + + + + + + + + + + + + + + + + + + + +
+ + + + + estate.properties.search + estate.property + + + + + + + + + + + + + + + + + \ No newline at end of file From 00df62a4574415876f63a1d9a338579a4dc65649 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 20 Jul 2026 15:12:15 +0200 Subject: [PATCH 06/41] [IMP] estate: 7. Many2One --- estate/models/__init__.py | 2 +- estate/models/property.py | 3 +++ estate/models/property_type.py | 8 ++++++++ estate/security/ir.model.access.csv | 3 ++- estate/views/estate_menus.xml | 4 ++++ estate/views/estate_property_views.xml | 17 ++++++++++++++++- 6 files changed, 34 insertions(+), 3 deletions(-) create mode 100644 estate/models/property_type.py diff --git a/estate/models/__init__.py b/estate/models/__init__.py index 8120b005bb6..40f3a294b12 100644 --- a/estate/models/__init__.py +++ b/estate/models/__init__.py @@ -1 +1 @@ -from . import property +from . import property, property_type diff --git a/estate/models/property.py b/estate/models/property.py index ba6f21dcb09..4f0b7a73787 100644 --- a/estate/models/property.py +++ b/estate/models/property.py @@ -14,6 +14,9 @@ class Property(models.Model): , ('sold', 'Sold') , ('cancelled', 'Cancelled'), ]) + property_type_id = fields.Many2one("estate.property.type", string="Property Type") + salesman = fields.Many2one('res.users', string="Salesman", default=lambda self: self.env.user) + buyer = fields.Many2one('res.partner', string="Buyer", copy=False) description = fields.Text(string='description') postcode = fields.Char(string='postcode') diff --git a/estate/models/property_type.py b/estate/models/property_type.py new file mode 100644 index 00000000000..64359a59716 --- /dev/null +++ b/estate/models/property_type.py @@ -0,0 +1,8 @@ +from odoo import fields, models + + +class PropertyType(models.Model): + _name = "estate.property.type" + _description = "Type of property" + + name = fields.Char(string='Name', required=True) diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv index e5b8c9cc4dc..cc2adea6085 100644 --- a/estate/security/ir.model.access.csv +++ b/estate/security/ir.model.access.csv @@ -1,2 +1,3 @@ id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink -access_estate_property_model,access_estate_property_model,estate.model_estate_property,base.group_user,1,1,1,1 \ No newline at end of file +access_estate_property_model,access_estate_property_model,estate.model_estate_property,base.group_user,1,1,1,1 +access_estate_property_type_model,access_estate_property_type_model,estate.model_estate_property_type,base.group_user,1,1,1,1 \ No newline at end of file diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml index 55c99310847..3287a1ababd 100644 --- a/estate/views/estate_menus.xml +++ b/estate/views/estate_menus.xml @@ -4,6 +4,10 @@ + + + + \ No newline at end of file diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index eb07ccbd359..709daf716db 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -6,6 +6,12 @@ list,form + + Property Types + estate.property.type + list,form + + estate.properties.list estate.property @@ -39,6 +45,7 @@ + @@ -61,6 +68,14 @@ + + + + + + + + @@ -87,4 +102,4 @@ - \ No newline at end of file + \ No newline at end of file From 2beb69abc094fa912866cb1dc29b93ec229b613c Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 20 Jul 2026 15:19:22 +0200 Subject: [PATCH 07/41] [IMP] estate: 7. Many2Many --- estate/models/__init__.py | 2 +- estate/models/property.py | 1 + estate/models/property_tag.py | 8 ++++++++ estate/security/ir.model.access.csv | 3 ++- estate/views/estate_menus.xml | 3 ++- estate/views/estate_property_views.xml | 6 ++++++ 6 files changed, 20 insertions(+), 3 deletions(-) create mode 100644 estate/models/property_tag.py diff --git a/estate/models/__init__.py b/estate/models/__init__.py index 40f3a294b12..ff81f5a69ed 100644 --- a/estate/models/__init__.py +++ b/estate/models/__init__.py @@ -1 +1 @@ -from . import property, property_type +from . import property, property_tag, property_type diff --git a/estate/models/property.py b/estate/models/property.py index 4f0b7a73787..e9387854e6b 100644 --- a/estate/models/property.py +++ b/estate/models/property.py @@ -17,6 +17,7 @@ class Property(models.Model): property_type_id = fields.Many2one("estate.property.type", string="Property Type") salesman = fields.Many2one('res.users', string="Salesman", default=lambda self: self.env.user) buyer = fields.Many2one('res.partner', string="Buyer", copy=False) + tag_ids = fields.Many2many("estate.property.tag") description = fields.Text(string='description') postcode = fields.Char(string='postcode') diff --git a/estate/models/property_tag.py b/estate/models/property_tag.py new file mode 100644 index 00000000000..d6c1cf217a3 --- /dev/null +++ b/estate/models/property_tag.py @@ -0,0 +1,8 @@ +from odoo import fields, models + + +class PropertyTag(models.Model): + _name = "estate.property.tag" + _description = "Tag assigned to property" + + name = fields.Char(string='Name', required=True) diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv index cc2adea6085..0847cf48f26 100644 --- a/estate/security/ir.model.access.csv +++ b/estate/security/ir.model.access.csv @@ -1,3 +1,4 @@ id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink access_estate_property_model,access_estate_property_model,estate.model_estate_property,base.group_user,1,1,1,1 -access_estate_property_type_model,access_estate_property_type_model,estate.model_estate_property_type,base.group_user,1,1,1,1 \ No newline at end of file +access_estate_property_type_model,access_estate_property_type_model,estate.model_estate_property_type,base.group_user,1,1,1,1 +access_estate_property_tag_model,access_estate_property_tag_model,estate.model_estate_property_tag,base.group_user,1,1,1,1 \ No newline at end of file diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml index 3287a1ababd..48645ccd3ab 100644 --- a/estate/views/estate_menus.xml +++ b/estate/views/estate_menus.xml @@ -4,9 +4,10 @@ - + + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 709daf716db..c387ac77987 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -12,6 +12,12 @@ list,form + + Property Tags + estate.property.tag + list,form + + estate.properties.list estate.property From a7e7ef5bea49cf420c47972e8ea8beccf04e7230 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 20 Jul 2026 15:38:59 +0200 Subject: [PATCH 08/41] [IMP] estate: 7. One2Many --- estate/models/__init__.py | 2 +- estate/models/property.py | 1 + estate/models/property_offer.py | 13 ++++++++++ estate/security/ir.model.access.csv | 3 ++- estate/views/estate_property_views.xml | 33 ++++++++++++++++++++++++++ 5 files changed, 50 insertions(+), 2 deletions(-) create mode 100644 estate/models/property_offer.py diff --git a/estate/models/__init__.py b/estate/models/__init__.py index ff81f5a69ed..22c7e2d2c5b 100644 --- a/estate/models/__init__.py +++ b/estate/models/__init__.py @@ -1 +1 @@ -from . import property, property_tag, property_type +from . import property, property_offer, property_tag, property_type diff --git a/estate/models/property.py b/estate/models/property.py index e9387854e6b..e29f881e2be 100644 --- a/estate/models/property.py +++ b/estate/models/property.py @@ -18,6 +18,7 @@ class Property(models.Model): salesman = fields.Many2one('res.users', string="Salesman", default=lambda self: self.env.user) buyer = fields.Many2one('res.partner', string="Buyer", copy=False) tag_ids = fields.Many2many("estate.property.tag") + offer_ids = fields.One2many("estate.property.offer", "property_id") description = fields.Text(string='description') postcode = fields.Char(string='postcode') diff --git a/estate/models/property_offer.py b/estate/models/property_offer.py new file mode 100644 index 00000000000..c0fea318dd0 --- /dev/null +++ b/estate/models/property_offer.py @@ -0,0 +1,13 @@ +from odoo import fields, models + + +class PropertyOffer(models.Model): + _name = "estate.property.offer" + _description = "Bids for a property" + + price = fields.Float(string="Price") + status = fields.Selection(string="Status", copy=False + , selection=[('accepted', 'Accepted'), ('refused', 'Refused')]) + + partner_id = fields.Many2one("res.partner", required=True) + property_id = fields.Many2one("estate.property", required=True) diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv index 0847cf48f26..560cd2ef60f 100644 --- a/estate/security/ir.model.access.csv +++ b/estate/security/ir.model.access.csv @@ -1,4 +1,5 @@ id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink access_estate_property_model,access_estate_property_model,estate.model_estate_property,base.group_user,1,1,1,1 access_estate_property_type_model,access_estate_property_type_model,estate.model_estate_property_type,base.group_user,1,1,1,1 -access_estate_property_tag_model,access_estate_property_tag_model,estate.model_estate_property_tag,base.group_user,1,1,1,1 \ No newline at end of file +access_estate_property_tag_model,access_estate_property_tag_model,estate.model_estate_property_tag,base.group_user,1,1,1,1 +access_estate_property_offer_model,access_estate_property_offer_model,estate.model_estate_property_offer,base.group_user,1,1,1,1 \ No newline at end of file diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index c387ac77987..5f4bcac330f 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -74,6 +74,9 @@ + + + @@ -88,6 +91,36 @@ + + estate.property.offer + estate.property.offer + + + + + + + + + + + estate.property.offer + estate.property.offer + +
+ + + + + + + + + +
+ + + estate.properties.search estate.property From 47cb955b1949a5ad2a0cdebfcd85e1236f977128 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 20 Jul 2026 17:12:58 +0200 Subject: [PATCH 09/41] [IMP] estate: 8. Computed fields --- estate/models/property.py | 18 +++++++++++++++++- estate/models/property_offer.py | 22 +++++++++++++++++++++- estate/views/estate_property_views.xml | 4 ++++ 3 files changed, 42 insertions(+), 2 deletions(-) diff --git a/estate/models/property.py b/estate/models/property.py index e29f881e2be..2771694db3b 100644 --- a/estate/models/property.py +++ b/estate/models/property.py @@ -1,4 +1,4 @@ -from odoo import fields, models +from odoo import api, fields, models class Property(models.Model): @@ -37,3 +37,19 @@ class Property(models.Model): , ('south', 'South') , ('east', 'East') , ('west', 'West')]) + + total_area = fields.Float(string="Total area", compute="_compute_total_area") + best_price = fields.Float(string="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 + record.garden_area + + @api.depends("offer_ids.price") + def _compute_best_price(self): + for record in self: + if record.offer_ids: + record.best_price = max(record.offer_ids.mapped("price")) + else: + record.best_price = 0 diff --git a/estate/models/property_offer.py b/estate/models/property_offer.py index c0fea318dd0..fa353f3f042 100644 --- a/estate/models/property_offer.py +++ b/estate/models/property_offer.py @@ -1,4 +1,7 @@ -from odoo import fields, models +import datetime as dt +from math import floor + +from odoo import api, fields, models class PropertyOffer(models.Model): @@ -11,3 +14,20 @@ class PropertyOffer(models.Model): partner_id = fields.Many2one("res.partner", required=True) property_id = fields.Many2one("estate.property", required=True) + validity = fields.Integer(string="Validity of offer", default=7) + date_deadline = fields.Date(string="Offer expiry", compute="_compute_date_deadline", inverse="_inverse_date_deadline") + + @api.depends("create_date", "validity") + def _compute_date_deadline(self): + for record in self: + record.date_deadline = None + if record.create_date and record.validity: + record.date_deadline = fields.Date.add(record.create_date, days=record.validity) + + def _inverse_date_deadline(self): + for record in self: + if record.date_deadline: + raw_difference = record.date_deadline - (record.create_date.date() or fields.Date.today()) + difference = raw_difference.total_seconds() + if difference > 0: + record.validity = floor(difference / dt.timedelta(days=1).total_seconds()) diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 5f4bcac330f..5698ecc9a49 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -57,6 +57,7 @@ + @@ -71,6 +72,7 @@ + @@ -113,6 +115,8 @@ + + From 7f6bc8e71e107b7ae7d648bfda5fb8fc2ebbf884 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 20 Jul 2026 17:16:37 +0200 Subject: [PATCH 10/41] [IMP] estate: 8. Onchange fields --- estate/models/property.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/estate/models/property.py b/estate/models/property.py index 2771694db3b..b5c21011157 100644 --- a/estate/models/property.py +++ b/estate/models/property.py @@ -53,3 +53,9 @@ def _compute_best_price(self): record.best_price = max(record.offer_ids.mapped("price")) else: record.best_price = 0 + + @api.onchange("garden") + def _onchange_garden(self): + for record in self: + record.garden_area = 10 if record.garden else 0 + record.garden_orientation = 'north' if record.garden else None From 3351716a84434ada4ffbcc71929ba615bdb397e8 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Wed, 22 Jul 2026 09:10:56 +0200 Subject: [PATCH 11/41] [REF] estate: Add workspace --- .vscode/tutorials.code-workspace | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) create mode 100644 .vscode/tutorials.code-workspace diff --git a/.vscode/tutorials.code-workspace b/.vscode/tutorials.code-workspace new file mode 100644 index 00000000000..81d76912f0c --- /dev/null +++ b/.vscode/tutorials.code-workspace @@ -0,0 +1,30 @@ +{ + "folders": [ + { + "path": ".." + }, + { + "path": "../../odoo" + } + ], + "settings": { + "python.languageServer": "None", + }, + "launch": { + "version": "0.2.0", + "configurations": [ + { + "name": "Odoo onboarding", + "type": "debugpy", + "request": "launch", + "program": "${workspaceFolder:odoo}/odoo-bin", + "args": [ + "--addons-path=addons/,../enterprise,../tutorials/", + "-d", "19.0-onboarding", "-u", "estate" + ], + "cwd": "${workspaceFolder:odoo}", + "console": "integratedTerminal" + } + ] + } +} \ No newline at end of file From 5a2cd9f174a672e374c986ef7cc49c5cbde1dda8 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Wed, 22 Jul 2026 09:59:50 +0200 Subject: [PATCH 12/41] [IMP] estate: 9. Actions --- estate/models/property.py | 27 ++++++++++++++++++++++++- estate/models/property_offer.py | 28 +++++++++++++++++++++++++- estate/views/estate_property_views.xml | 7 +++++++ 3 files changed, 60 insertions(+), 2 deletions(-) diff --git a/estate/models/property.py b/estate/models/property.py index b5c21011157..00e99ae9c2f 100644 --- a/estate/models/property.py +++ b/estate/models/property.py @@ -1,4 +1,4 @@ -from odoo import api, fields, models +from odoo import _, api, exceptions, fields, models class Property(models.Model): @@ -59,3 +59,28 @@ def _onchange_garden(self): for record in self: record.garden_area = 10 if record.garden else 0 record.garden_orientation = 'north' if record.garden else None + + def action_sold(self): + for record in self: + if record.state == 'cancelled': + raise exceptions.UserError(_("Cancelled properties cannot be sold!")) + # + record.state = "sold" + return True + + def action_cancelled(self): + for record in self: + if record.state == "sold": + raise exceptions.UserError(_("Sold properties cannot be cancelled!")) + # + record.state = "cancelled" + return True + + def confirm_sale(self): + for property in self: + accepted_offer = property.offer_ids.filtered(lambda r: r.status == 'accepted') + accepted_offer.ensure_one() + property.selling_price = accepted_offer.price + property.buyer = accepted_offer.partner_id + # Refuse all other offers + (property.offer_ids - accepted_offer).action_cancel() diff --git a/estate/models/property_offer.py b/estate/models/property_offer.py index fa353f3f042..94aef2a9ec4 100644 --- a/estate/models/property_offer.py +++ b/estate/models/property_offer.py @@ -1,7 +1,8 @@ import datetime as dt from math import floor -from odoo import api, fields, models +from odoo import _, api, fields, models +from odoo.exceptions import UserError, ValidationError class PropertyOffer(models.Model): @@ -31,3 +32,28 @@ def _inverse_date_deadline(self): difference = raw_difference.total_seconds() if difference > 0: record.validity = floor(difference / dt.timedelta(days=1).total_seconds()) + + @api.constrains('status') + def _check_maximum_one_offer_accepted(self): + for record in self: + peer_offers = record.property_id.offer_ids + accepted_peers = peer_offers.filtered(lambda r: r.status == 'accepted') + if len(accepted_peers) > 1: + raise ValidationError(_("A single offer can be accepted at a time!")) + + def action_confirm(self): + for record in self: + if record.status == "refused": + raise UserError(_("Offer is already refused!")) + # + record.status = 'accepted' + record.property_id.confirm_sale() + return True + + def action_cancel(self): + for record in self: + if record.status == "accepted": + raise UserError(_("Offer is already accepted!")) + # + record.status = "refused" + return True diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 5698ecc9a49..aef14bf365b 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -39,6 +39,10 @@ estate.property
+
+
@@ -51,6 +55,7 @@
+ @@ -100,6 +105,8 @@ + +
+
+
+
+

+ +

+
+ + + + + + + + + + + + + + +
+ + + estate.properties.form estate.property
-
@@ -55,8 +116,7 @@
- - + @@ -75,14 +135,14 @@ - - + + - + @@ -102,16 +162,29 @@ estate.property.offer estate.property.offer - + - -
-
-
-

- -

-
- - - - - - - - - - - - - - -
- - - estate.properties.form estate.property diff --git a/estate/views/estate_property_type.xml b/estate/views/estate_property_type.xml index e69de29bb2d..51aafc63d77 100644 --- a/estate/views/estate_property_type.xml +++ b/estate/views/estate_property_type.xml @@ -0,0 +1,56 @@ + + + + Property Types + estate.property.type + list,form + + + + estate.property.types.list + estate.property.type + + + + + + + + + + estate.property.types.form + estate.property.type + +
+ +
+ +
+
+
+
+

+ +

+
+ + + + + + + + + + + + + + +
+ + + From 50b73e4ff2c6254e6fafe726d60d0f2296529391 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 27 Jul 2026 16:28:45 +0200 Subject: [PATCH 31/41] [MOV] estate: views estate.property tag --- estate/views/estate_property.xml | 17 ----------------- estate/views/estate_property_tag.xml | 18 ++++++++++++++++++ 2 files changed, 18 insertions(+), 17 deletions(-) diff --git a/estate/views/estate_property.xml b/estate/views/estate_property.xml index ab93b875f6f..bca83daed9c 100644 --- a/estate/views/estate_property.xml +++ b/estate/views/estate_property.xml @@ -7,12 +7,6 @@ {'search_default_available': True} - - Property Tags - estate.property.tag - list,form - - Property Offers estate.property.offer @@ -158,17 +152,6 @@ - - estate.property.tags.list - estate.property.tag - - - - - - - - estate.property.offer estate.property.offer diff --git a/estate/views/estate_property_tag.xml b/estate/views/estate_property_tag.xml index e69de29bb2d..a737d3f1420 100644 --- a/estate/views/estate_property_tag.xml +++ b/estate/views/estate_property_tag.xml @@ -0,0 +1,18 @@ + + + + Property Tags + estate.property.tag + list,form + + + + estate.property.tags.list + estate.property.tag + + + + + + + From c6f70fada49fe756e13819bf075bd2feeb7623e7 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 27 Jul 2026 16:29:52 +0200 Subject: [PATCH 32/41] [IMP] estate: views estate.property.offer --- estate/views/estate_property.xml | 44 ------------------------ estate/views/estate_property_offer.xml | 46 ++++++++++++++++++++++++++ 2 files changed, 46 insertions(+), 44 deletions(-) diff --git a/estate/views/estate_property.xml b/estate/views/estate_property.xml index bca83daed9c..5a63355c087 100644 --- a/estate/views/estate_property.xml +++ b/estate/views/estate_property.xml @@ -7,13 +7,6 @@ {'search_default_available': True} - - Property Offers - estate.property.offer - list,form - [('property_type_id', '=', active_id)] - - estate.properties.list estate.property @@ -135,43 +128,6 @@ - - estate.property.offer - estate.property.offer - - - - - - - From 6d89bae424f414a50e77994ec91f50f040c61ab5 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 27 Jul 2026 16:49:26 +0200 Subject: [PATCH 34/41] [REM] estate: remove vscode workspace file --- .vscode/tutorials.code-workspace | 30 ------------------------------ 1 file changed, 30 deletions(-) delete mode 100644 .vscode/tutorials.code-workspace diff --git a/.vscode/tutorials.code-workspace b/.vscode/tutorials.code-workspace deleted file mode 100644 index 81d76912f0c..00000000000 --- a/.vscode/tutorials.code-workspace +++ /dev/null @@ -1,30 +0,0 @@ -{ - "folders": [ - { - "path": ".." - }, - { - "path": "../../odoo" - } - ], - "settings": { - "python.languageServer": "None", - }, - "launch": { - "version": "0.2.0", - "configurations": [ - { - "name": "Odoo onboarding", - "type": "debugpy", - "request": "launch", - "program": "${workspaceFolder:odoo}/odoo-bin", - "args": [ - "--addons-path=addons/,../enterprise,../tutorials/", - "-d", "19.0-onboarding", "-u", "estate" - ], - "cwd": "${workspaceFolder:odoo}", - "console": "integratedTerminal" - } - ] - } -} \ No newline at end of file From f21a3e7587b0831c7040ea84cfc435bc6cd370fe Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 27 Jul 2026 16:52:40 +0200 Subject: [PATCH 35/41] [LINT] estate: remove empty comment line --- estate/models/property.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/estate/models/property.py b/estate/models/property.py index 5ffc8476512..1ff7c787f30 100644 --- a/estate/models/property.py +++ b/estate/models/property.py @@ -86,7 +86,7 @@ def action_cancelled(self): for record in self: if record.state == "sold": raise exceptions.UserError(_("Sold properties cannot be cancelled!")) - # + record.state = "cancelled" return True From 3768c172994c6365a9a8fe7aa246712f3f087126 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Mon, 27 Jul 2026 16:54:08 +0200 Subject: [PATCH 36/41] [FIX] estate: method today doesn't exist on datetime --- estate/models/property_offer.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/estate/models/property_offer.py b/estate/models/property_offer.py index 27cdc933f9d..12aa91d182b 100644 --- a/estate/models/property_offer.py +++ b/estate/models/property_offer.py @@ -24,7 +24,7 @@ class PropertyOffer(models.Model): @api.depends("create_date", "validity") def _compute_date_deadline(self): for record in self: - record.date_deadline = fields.Date.add(record.create_date or fields.Datetime.today(), days=record.validity) + record.date_deadline = fields.Date.add(record.create_date or fields.Datetime.now(), days=record.validity) @api.onchange('date_deadline') def _inverse_date_deadline(self): From 64d049a3d8e4db3fe91397a4776349c4ac4e6dee Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Tue, 28 Jul 2026 17:00:42 +0200 Subject: [PATCH 37/41] [LINT] estate: format estate_menus.xml --- estate/views/estate_menus.xml | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml index 1e68bb57dad..9e035007a3d 100644 --- a/estate/views/estate_menus.xml +++ b/estate/views/estate_menus.xml @@ -1,14 +1,14 @@ - - - - + + + + - - - + + + + - From bccf94ab4179bebb8040a78a070d4aa8f5cacefa Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Wed, 29 Jul 2026 14:17:08 +0200 Subject: [PATCH 38/41] [MOV] estate: rename view resource files --- estate/__manifest__.py | 8 ++++---- ...property_offer.xml => estate_property_offer_views.xml} | 0 ...ate_property_tag.xml => estate_property_tag_views.xml} | 0 ...e_property_type.xml => estate_property_type_views.xml} | 0 .../{estate_property.xml => estate_property_views.xml} | 0 5 files changed, 4 insertions(+), 4 deletions(-) rename estate/views/{estate_property_offer.xml => estate_property_offer_views.xml} (100%) rename estate/views/{estate_property_tag.xml => estate_property_tag_views.xml} (100%) rename estate/views/{estate_property_type.xml => estate_property_type_views.xml} (100%) rename estate/views/{estate_property.xml => estate_property_views.xml} (100%) diff --git a/estate/__manifest__.py b/estate/__manifest__.py index 7dc5aeb0d3c..a1df52a5f04 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -5,10 +5,10 @@ 'license': 'LGPL-3', 'data': [ 'security/ir.model.access.csv', - 'views/estate_property.xml', - 'views/estate_property_offer.xml', - 'views/estate_property_tag.xml', - 'views/estate_property_type.xml', + 'views/estate_property_views.xml', + 'views/estate_property_offer_views.xml', + 'views/estate_property_tag_views.xml', + 'views/estate_property_type_views.xml', 'views/estate_menus.xml', 'views/res_user_views.xml', ], diff --git a/estate/views/estate_property_offer.xml b/estate/views/estate_property_offer_views.xml similarity index 100% rename from estate/views/estate_property_offer.xml rename to estate/views/estate_property_offer_views.xml diff --git a/estate/views/estate_property_tag.xml b/estate/views/estate_property_tag_views.xml similarity index 100% rename from estate/views/estate_property_tag.xml rename to estate/views/estate_property_tag_views.xml diff --git a/estate/views/estate_property_type.xml b/estate/views/estate_property_type_views.xml similarity index 100% rename from estate/views/estate_property_type.xml rename to estate/views/estate_property_type_views.xml diff --git a/estate/views/estate_property.xml b/estate/views/estate_property_views.xml similarity index 100% rename from estate/views/estate_property.xml rename to estate/views/estate_property_views.xml From 0de649dfc2e597c2d5a93f0c2c9c2aa4f2b88bd5 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Wed, 29 Jul 2026 14:20:09 +0200 Subject: [PATCH 39/41] [LINT] estate: rename access rules according to guidelines --- estate/security/ir.model.access.csv | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv index 94af865def2..03ac00195c6 100644 --- a/estate/security/ir.model.access.csv +++ b/estate/security/ir.model.access.csv @@ -1,5 +1,5 @@ id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink -access_estate_property_model,access_estate_property_model,estate.model_estate_property,base.group_user,1,1,1,1 -access_estate_property_type_model,access_estate_property_type_model,estate.model_estate_property_type,base.group_user,1,1,1,1 -access_estate_property_tag_model,access_estate_property_tag_model,estate.model_estate_property_tag,base.group_user,1,1,1,1 -access_estate_property_offer_model,access_estate_property_offer_model,estate.model_estate_property_offer,base.group_user,1,1,1,1 +access_estate_property_model,access.estate.property.model,estate.model_estate_property,base.group_user,1,1,1,1 +access_estate_property_type_model,access.estate.property.type.model,estate.model_estate_property_type,base.group_user,1,1,1,1 +access_estate_property_tag_model,access.estate.property.tag.model,estate.model_estate_property_tag,base.group_user,1,1,1,1 +access_estate_property_offer_model,access.estate.property.offer.model,estate.model_estate_property_offer,base.group_user,1,1,1,1 From 118285da5c21b925611d9b5ac9b4705f8ed3f4c4 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Wed, 29 Jul 2026 14:25:07 +0200 Subject: [PATCH 40/41] [LINT] estate: order class properties according to guidelines --- estate/models/property.py | 51 ++++++++++++++++----------------- estate/models/property_offer.py | 25 ++++++++-------- estate/models/property_tag.py | 4 +-- estate/models/property_type.py | 5 ++-- 4 files changed, 41 insertions(+), 44 deletions(-) diff --git a/estate/models/property.py b/estate/models/property.py index 1ff7c787f30..909b011036e 100644 --- a/estate/models/property.py +++ b/estate/models/property.py @@ -7,17 +7,6 @@ class Property(models.Model): _description = "Properties of our managed estates" _order = "id desc" - _check_positive_expected_price = models.Constraint( - 'CHECK (expected_price >= 0)', 'Expected price must be positive!') - _check_positive_selling_price = models.Constraint( - 'CHECK (selling_price >= 0)', 'Selling price must be positive!') - _check_positive_living_area = models.Constraint( - 'CHECK (living_area >= 0)', 'Living area must be positive!') - _check_positive_amounts = models.Constraint( - 'CHECK (facades >= 0)', 'Number of facades must be positive!') - _check_positive_garden_area = models.Constraint( - 'CHECK (garden_area >= 0)', 'Garden area must be positive!') - name = fields.Char(string='Title', required=True) active = fields.Boolean(default=True) state = fields.Selection(required=True, default='new', copy=False, selection=[ @@ -32,7 +21,6 @@ class Property(models.Model): buyer_id = fields.Many2one('res.partner', string="Buyer", copy=False) tag_ids = fields.Many2many("estate.property.tag") offer_ids = fields.One2many("estate.property.offer", "property_id") - description = fields.Text(string='description') postcode = fields.Char(string='postcode') date_availability = fields.Date(string='Available from', default=lambda _: fields.Date.add(fields.Date.today(), months=3), copy=False) @@ -51,10 +39,20 @@ class Property(models.Model): ('east', 'East'), ('west', 'West'), ]) - total_area = fields.Float(string="Total area", compute="_compute_total_area") best_price = fields.Float(string="Best offer", compute="_compute_best_price") + _check_positive_expected_price = models.Constraint( + 'CHECK (expected_price >= 0)', 'Expected price must be positive!') + _check_positive_selling_price = models.Constraint( + 'CHECK (selling_price >= 0)', 'Selling price must be positive!') + _check_positive_living_area = models.Constraint( + 'CHECK (living_area >= 0)', 'Living area must be positive!') + _check_positive_amounts = models.Constraint( + 'CHECK (facades >= 0)', 'Number of facades must be positive!') + _check_positive_garden_area = models.Constraint( + 'CHECK (garden_area >= 0)', 'Garden area must be positive!') + @api.depends("living_area", "garden_area") def _compute_total_area(self): for record in self: @@ -68,12 +66,26 @@ def _compute_best_price(self): else: record.best_price = 0 + @api.constrains('selling_price', 'expected_price') + def _check_selling_price(self): + for property in self: + if float_is_zero(property.selling_price, 2): + return + if float_compare(property.selling_price, 0.9 * property.expected_price, 2) == -1: + raise exceptions.ValidationError(_("The accepted price is less than 90% of the expected price!")) + @api.onchange("garden") def _onchange_garden(self): for record in self: record.garden_area = 10 if record.garden else 0 record.garden_orientation = 'north' if record.garden else False + @api.ondelete(at_uninstall=False) + def _unlink_if_draft(self): + for record in self: + if record.state not in ['new', 'cancelled']: + raise exceptions.ValidationError(_("Only properties in state New or Cancelled can be deleted!")) + def action_sold(self): for record in self: if record.state == 'cancelled': @@ -90,13 +102,6 @@ def action_cancelled(self): record.state = "cancelled" return True - @api.constrains('selling_price', 'expected_price') - def _check_selling_price(self): - for property in self: - if float_is_zero(property.selling_price, 2): - return - if float_compare(property.selling_price, 0.9 * property.expected_price, 2) == -1: - raise exceptions.ValidationError(_("The accepted price is less than 90% of the expected price!")) def confirm_offer(self): for property in self: @@ -108,12 +113,6 @@ def confirm_offer(self): # Refuse all other offers (property.offer_ids - accepted_offer).action_cancel() - @api.ondelete(at_uninstall=False) - def _unlink_if_draft(self): - for record in self: - if record.state not in ['new', 'cancelled']: - raise exceptions.ValidationError(_("Only properties in state New or Cancelled can be deleted!")) - def _set_offer_received(self): self.ensure_one() if self.state == 'new': diff --git a/estate/models/property_offer.py b/estate/models/property_offer.py index 12aa91d182b..d1d86f31998 100644 --- a/estate/models/property_offer.py +++ b/estate/models/property_offer.py @@ -10,16 +10,15 @@ class PropertyOffer(models.Model): _description = "Bids for a property" _order = "price desc" - _check_positive_price = models.Constraint('CHECK(price >= 0)', 'Price has to be positive') - price = fields.Float(string="Price", required=True) status = fields.Selection(string="Status", copy=False, selection=[('accepted', 'Accepted'), ('refused', 'Refused')]) - 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) validity = fields.Integer(string="Validity of offer", default=7) date_deadline = fields.Date(string="Offer expiry", compute="_compute_date_deadline", inverse="_inverse_date_deadline") + + _check_positive_price = models.Constraint('CHECK(price >= 0)', 'Price has to be positive') @api.depends("create_date", "validity") def _compute_date_deadline(self): @@ -35,6 +34,16 @@ def _inverse_date_deadline(self): if difference > 0: record.validity = floor(difference / dt.timedelta(days=1).total_seconds()) + @api.constrains('status') + def _check_maximum_one_offer_accepted(self): + accepted_offers = self.filtered(lambda r: r.status == "accepted") + properties = accepted_offers.mapped('property_id') + for property in properties: + peer_offers = property.offer_ids + accepted_peers = peer_offers.filtered(lambda r: r.status == 'accepted') + if len(accepted_peers) > 1: + raise ValidationError(_("A single offer can be accepted at a time!")) + @api.model_create_multi def create(self, vals_list): property_ids = self.env["estate.property"].browse(vals["property_id"] for vals in vals_list) @@ -49,16 +58,6 @@ def create(self, vals_list): return super().create(vals_list) - @api.constrains('status') - def _check_maximum_one_offer_accepted(self): - accepted_offers = self.filtered(lambda r: r.status == "accepted") - properties = accepted_offers.mapped('property_id') - for property in properties: - peer_offers = property.offer_ids - accepted_peers = peer_offers.filtered(lambda r: r.status == 'accepted') - if len(accepted_peers) > 1: - raise ValidationError(_("A single offer can be accepted at a time!")) - def action_confirm(self): for record in self: if record.status == "refused": diff --git a/estate/models/property_tag.py b/estate/models/property_tag.py index bff492526a2..8a9265a814c 100644 --- a/estate/models/property_tag.py +++ b/estate/models/property_tag.py @@ -6,7 +6,7 @@ class PropertyTag(models.Model): _description = "Tag assigned to property" _order = "name" - _name_idx = models.UniqueIndex('(name)', 'Another record already exists with the same name!') - name = fields.Char(string='Name', required=True) color = fields.Integer(string='Color') + + _name_idx = models.UniqueIndex('(name)', 'Another record already exists with the same name!') \ No newline at end of file diff --git a/estate/models/property_type.py b/estate/models/property_type.py index 25dbaddd5ab..446293fc89e 100644 --- a/estate/models/property_type.py +++ b/estate/models/property_type.py @@ -6,14 +6,13 @@ class PropertyType(models.Model): _description = "Type of property" _order = "name" - _name_idx = models.UniqueIndex('(name)', 'Another record already exists with the same name!') - name = fields.Char(string='Name', required=True) sequence = fields.Integer('Sequence', default=1) - property_ids = fields.One2many("estate.property", "property_type_id") offer_ids = fields.One2many("estate.property.offer", "property_type_id") offer_count = fields.Integer(compute="_compute_offer_count") + + _name_idx = models.UniqueIndex('(name)', 'Another record already exists with the same name!') @api.depends("offer_ids") def _compute_offer_count(self): From f7f80beb8bd995c191b14d9c7fc3eb73df1805d0 Mon Sep 17 00:00:00 2001 From: "bepro@odoo" Date: Wed, 29 Jul 2026 15:23:09 +0200 Subject: [PATCH 41/41] [LINT] estate: run ruff --- estate/models/property.py | 1 - estate/models/property_offer.py | 2 +- estate/models/property_tag.py | 2 +- estate/models/property_type.py | 2 +- 4 files changed, 3 insertions(+), 4 deletions(-) diff --git a/estate/models/property.py b/estate/models/property.py index 909b011036e..411ffa592e7 100644 --- a/estate/models/property.py +++ b/estate/models/property.py @@ -102,7 +102,6 @@ def action_cancelled(self): record.state = "cancelled" return True - def confirm_offer(self): for property in self: accepted_offer = property.offer_ids.filtered(lambda r: r.status == 'accepted') diff --git a/estate/models/property_offer.py b/estate/models/property_offer.py index d1d86f31998..a7abfa4484a 100644 --- a/estate/models/property_offer.py +++ b/estate/models/property_offer.py @@ -17,7 +17,7 @@ class PropertyOffer(models.Model): property_type_id = fields.Many2one(related="property_id.property_type_id", store=True) validity = fields.Integer(string="Validity of offer", default=7) date_deadline = fields.Date(string="Offer expiry", compute="_compute_date_deadline", inverse="_inverse_date_deadline") - + _check_positive_price = models.Constraint('CHECK(price >= 0)', 'Price has to be positive') @api.depends("create_date", "validity") diff --git a/estate/models/property_tag.py b/estate/models/property_tag.py index 8a9265a814c..f83a3c78cd2 100644 --- a/estate/models/property_tag.py +++ b/estate/models/property_tag.py @@ -9,4 +9,4 @@ class PropertyTag(models.Model): name = fields.Char(string='Name', required=True) color = fields.Integer(string='Color') - _name_idx = models.UniqueIndex('(name)', 'Another record already exists with the same name!') \ No newline at end of file + _name_idx = models.UniqueIndex('(name)', 'Another record already exists with the same name!') diff --git a/estate/models/property_type.py b/estate/models/property_type.py index 446293fc89e..e1fb5cd94ef 100644 --- a/estate/models/property_type.py +++ b/estate/models/property_type.py @@ -11,7 +11,7 @@ class PropertyType(models.Model): property_ids = fields.One2many("estate.property", "property_type_id") offer_ids = fields.One2many("estate.property.offer", "property_type_id") offer_count = fields.Integer(compute="_compute_offer_count") - + _name_idx = models.UniqueIndex('(name)', 'Another record already exists with the same name!') @api.depends("offer_ids")