From 4caf08d98a73efc8a5389e196e8290d435c099c3 Mon Sep 17 00:00:00 2001 From: Yousef Allam Date: Mon, 17 Aug 2026 14:02:44 +0200 Subject: [PATCH 01/15] [ADD] Chapter 2 done --- estate/__init__.py | 0 estate/__manifest__.py | 11 +++++++++++ 2 files changed, 11 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..e69de29bb2d diff --git a/estate/__manifest__.py b/estate/__manifest__.py new file mode 100644 index 00000000000..b32d09735e4 --- /dev/null +++ b/estate/__manifest__.py @@ -0,0 +1,11 @@ +{ + 'name': 'estate', + 'version': '0.0', + 'depends': [ + 'base', + ], + 'installable': True, + 'application': True, + 'author': 'me', + 'license': 'LGPL-3' +} \ No newline at end of file From 56c64b50a542ac36efd16f685f51fb4887ce99b6 Mon Sep 17 00:00:00 2001 From: Yousef Allam Date: Mon, 17 Aug 2026 15:08:01 +0200 Subject: [PATCH 02/15] [IMP] Chapter 3 done --- estate/__init__.py | 1 + estate/models/__init__.py | 1 + estate/models/estate_property.py | 36 ++++++++++++++++++++++++++++++++ 3 files changed, 38 insertions(+) create mode 100644 estate/models/__init__.py create mode 100644 estate/models/estate_property.py diff --git a/estate/__init__.py b/estate/__init__.py index e69de29bb2d..0650744f6bc 100644 --- a/estate/__init__.py +++ b/estate/__init__.py @@ -0,0 +1 @@ +from . import models diff --git a/estate/models/__init__.py b/estate/models/__init__.py new file mode 100644 index 00000000000..5e1963c9d2f --- /dev/null +++ b/estate/models/__init__.py @@ -0,0 +1 @@ +from . import estate_property diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py new file mode 100644 index 00000000000..42f9dacbf56 --- /dev/null +++ b/estate/models/estate_property.py @@ -0,0 +1,36 @@ +from odoo import models, fields + +class EstateProperty(models.Model): + _name = 'estate.property' + _description = 'Estate Property' + + name = fields.Char(required=True) + + description = fields.Text() + + postcode = fields.Char() + + date_availability = fields.Date() + + expected_price = fields.Float(required=True) + + selling_price = fields.Float() + + bedrooms = fields.Integer() + + living_area = fields.Integer() + + facades = fields.Integer() + + garage = fields.Boolean() + + garden = fields.Boolean() + + garden_area = fields.Integer() + + garden_orientation = fields.Selection([ + ('north', 'North'), + ('south', 'South'), + ('east', 'East'), + ('west', 'West') + ]) From 64eaa56f7bacf26f118b41eca8142850a280c201 Mon Sep 17 00:00:00 2001 From: Yousef Allam Date: Mon, 17 Aug 2026 16:09:56 +0200 Subject: [PATCH 03/15] [IMP] Chapter 4 done --- estate/__manifest__.py | 7 +++++-- estate/security/ir.model.access.csv | 2 ++ 2 files changed, 7 insertions(+), 2 deletions(-) create mode 100644 estate/security/ir.model.access.csv diff --git a/estate/__manifest__.py b/estate/__manifest__.py index b32d09735e4..0777e7d540a 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -7,5 +7,8 @@ 'installable': True, 'application': True, 'author': 'me', - 'license': 'LGPL-3' -} \ No newline at end of file + 'license': 'LGPL-3', + 'data': [ + 'security/ir.model.access.csv', + ] +} diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv new file mode 100644 index 00000000000..0e11f47e58d --- /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,access_estate_property,model_estate_property,base.group_user,1,1,1,1 \ No newline at end of file From af604d39e16b06fcfa7b60da9113d09eae411416 Mon Sep 17 00:00:00 2001 From: Yousef Allam Date: Mon, 17 Aug 2026 17:08:59 +0200 Subject: [PATCH 04/15] [IMP] Chapter 5: Finally, Some UI To Play With --- estate/__manifest__.py | 2 ++ estate/models/estate_property.py | 27 ++++++++++++-------------- estate/security/ir.model.access.csv | 2 +- estate/views/estate_menus.xml | 7 +++++++ estate/views/estate_property_views.xml | 7 +++++++ 5 files changed, 29 insertions(+), 16 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 0777e7d540a..db3a68f8b6c 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -10,5 +10,7 @@ 'license': 'LGPL-3', 'data': [ 'security/ir.model.access.csv', + 'views/estate_property_views.xml', + 'views/estate_menus.xml' ] } diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index 42f9dacbf56..f5516652b95 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -1,3 +1,4 @@ +from dateutil.relativedelta import relativedelta from odoo import models, fields class EstateProperty(models.Model): @@ -5,32 +6,28 @@ class EstateProperty(models.Model): _description = 'Estate Property' name = fields.Char(required=True) - description = fields.Text() - postcode = fields.Char() - - date_availability = fields.Date() - + date_availability = fields.Date(copy=False, default=lambda self: fields.Date.today() + relativedelta(months=3)) expected_price = fields.Float(required=True) - - selling_price = fields.Float() - - bedrooms = fields.Integer() - + 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() - + active = fields.Boolean(default=True) garden_orientation = fields.Selection([ ('north', 'North'), ('south', 'South'), ('east', 'East'), ('west', 'West') ]) + state = fields.Selection(required=True, copy=False, default='new', selection=[ + ('new', 'New'), + ('offer_received', 'Offer Received'), + ('offer_accepted', 'Offer Accepted'), + ('sold', 'Sold'), + ('canceled', 'Canceled') + ]) diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv index 0e11f47e58d..32389642d4f 100644 --- a/estate/security/ir.model.access.csv +++ b/estate/security/ir.model.access.csv @@ -1,2 +1,2 @@ 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 \ No newline at end of file +access_estate_property,access_estate_property,model_estate_property,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..d880b1717f4 --- /dev/null +++ b/estate/views/estate_menus.xml @@ -0,0 +1,7 @@ + + + + + + + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml new file mode 100644 index 00000000000..4d544c18597 --- /dev/null +++ b/estate/views/estate_property_views.xml @@ -0,0 +1,7 @@ + + + Properties + estate.property + list,form + + From 9d6aa19cb0132933242582576544f90aa9702d8d Mon Sep 17 00:00:00 2001 From: Yousef Allam Date: Tue, 18 Aug 2026 14:12:49 +0200 Subject: [PATCH 05/15] [IMP] Chapter 6: Basic Views --- estate/views/estate_property_views.xml | 76 ++++++++++++++++++++++++++ 1 file changed, 76 insertions(+) diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 4d544c18597..371b4f8633f 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -4,4 +4,80 @@ estate.property list,form + + + estate.property.tree + estate.property + + + + + + + + + + + + + + + estate.property.form + estate.property + +
+ +
+

+ +

+ + + + + + + + + + +
+ + + + + + + + + + + + + + + +
+ + + + + estate.property.search + estate.property + + + + + + + + + + + + + + + + From 97e3e2d979c9f165b802842cefd505438f85b401 Mon Sep 17 00:00:00 2001 From: Yousef Allam Date: Tue, 18 Aug 2026 16:42:27 +0200 Subject: [PATCH 06/15] [IMP] Chapter 7: Relations Between Models --- estate/models/__init__.py | 3 + estate/models/estate_property.py | 5 ++ estate/models/estate_property_offer.py | 10 +++ estate/models/estate_property_tag.py | 7 ++ estate/models/estate_property_type.py | 7 ++ estate/security/ir.model.access.csv | 3 + estate/views/estate_menus.xml | 4 + estate/views/estate_property_views.xml | 106 +++++++++++++++++-------- 8 files changed, 113 insertions(+), 32 deletions(-) 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 diff --git a/estate/models/__init__.py b/estate/models/__init__.py index 5e1963c9d2f..2f1821a39c1 100644 --- a/estate/models/__init__.py +++ b/estate/models/__init__.py @@ -1 +1,4 @@ from . import estate_property +from . import estate_property_type +from . import estate_property_tag +from . import estate_property_offer diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index f5516652b95..4c78d49a40b 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -31,3 +31,8 @@ class EstateProperty(models.Model): ('sold', 'Sold'), ('canceled', 'Canceled') ]) + property_type_id = fields.Many2one('estate.property.type', string='Property Type') + user_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') + offer_ids = fields.One2many('estate.property.offer', 'property_id') diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py new file mode 100644 index 00000000000..e99c5e7ea6b --- /dev/null +++ b/estate/models/estate_property_offer.py @@ -0,0 +1,10 @@ +from odoo import models, fields + +class EstatePropertyOffer(models.Model): + _name = 'estate.property.offer' + _description = 'Estate Property Offer' + + price = fields.Float() + status = fields.Selection(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/models/estate_property_tag.py b/estate/models/estate_property_tag.py new file mode 100644 index 00000000000..24beb0208d7 --- /dev/null +++ b/estate/models/estate_property_tag.py @@ -0,0 +1,7 @@ +from odoo import models, fields + +class EstatePropertyTag(models.Model): + _name = 'estate.property.tag' + _description = 'Estate Property Tag' + + name = fields.Char(required=True) diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py new file mode 100644 index 00000000000..e0d9267df9f --- /dev/null +++ b/estate/models/estate_property_type.py @@ -0,0 +1,7 @@ +from odoo import models, fields + +class EstatePropertyType(models.Model): + _name = 'estate.property.type' + _description = 'Estate Property Type' + + name = fields.Char(required=True) diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv index 32389642d4f..89f97c50842 100644 --- a/estate/security/ir.model.access.csv +++ b/estate/security/ir.model.access.csv @@ -1,2 +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 index d880b1717f4..c974114807d 100644 --- a/estate/views/estate_menus.xml +++ b/estate/views/estate_menus.xml @@ -3,5 +3,9 @@ + + + + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 371b4f8633f..768cdcc7951 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -4,6 +4,22 @@ estate.property list,form + + Property Types + estate.property.type + list,form + + + Property Tags + estate.property.tag + list,form + + + + Property Offers + estate.property.offer + list,form + estate.property.tree @@ -21,43 +37,69 @@ + + estate.property.offer.tree + estate.property.offer + + + + + + + + + estate.property.form estate.property -
- -
-

- -

- - - - - - - - - - -
- - - - - - - - - - - + + +
+
+

+ +

+ +
+ + + + + + + + + + + - - - - +
+ + + + + + + + + + + + + + + + + + + + + + + + +
From eeb03573550a63d6949c877b0b36f8898c4cb633 Mon Sep 17 00:00:00 2001 From: Yousef Allam Date: Wed, 19 Aug 2026 12:56:50 +0200 Subject: [PATCH 07/15] [IMP] Chapter 8: Computed Fields And Onchanges --- estate/models/estate_property.py | 26 +++++++++++++++++++++++++- estate/models/estate_property_offer.py | 18 +++++++++++++++++- estate/views/estate_property_views.xml | 4 ++++ 3 files changed, 46 insertions(+), 2 deletions(-) diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index 4c78d49a40b..ecd15fc1646 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -1,5 +1,5 @@ from dateutil.relativedelta import relativedelta -from odoo import models, fields +from odoo import models, fields, api class EstateProperty(models.Model): _name = 'estate.property' @@ -36,3 +36,27 @@ class EstateProperty(models.Model): buyer_id = fields.Many2one('res.partner', string='Buyer', copy=False) tag_ids = fields.Many2many('estate.property.tag', string='Tags') offer_ids = fields.One2many('estate.property.offer', 'property_id') + total_area = fields.Integer(compute='_compute_total_area') + best_price = fields.Float(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.0 + + @api.onchange("garden") + def _onchange_garden(self): + if not self.garden: + self.garden_area = 0 + self.garden_orientation = False + else: + self.garden_area = 10 + self.garden_orientation = 'north' diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py index e99c5e7ea6b..e81f9dc7a78 100644 --- a/estate/models/estate_property_offer.py +++ b/estate/models/estate_property_offer.py @@ -1,4 +1,4 @@ -from odoo import models, fields +from odoo import models, fields, api class EstatePropertyOffer(models.Model): _name = 'estate.property.offer' @@ -8,3 +8,19 @@ class EstatePropertyOffer(models.Model): status = fields.Selection(copy=False, selection=[('accepted', 'Accepted'), ('refused', 'Refused')]) partner_id = fields.Many2one('res.partner', required=True) property_id = fields.Many2one('estate.property', required=True) + date_deadline = fields.Date(compute='_compute_date_deadline', inverse='_inverse_date_deadline', store=True) + validity = fields.Integer(default=7) + + @api.depends('validity') + def _compute_date_deadline(self): + for record in self: + base_date = record.create_date.date() if record.create_date else fields.Date.today() + record.date_deadline = fields.Date.add(base_date, days=record.validity) + + @api.depends('date_deadline') + def _inverse_date_deadline(self): + for record in self: + if not record.date_deadline: + continue + base_date = record.create_date.date() if record.create_date else fields.Date.today() + record.validity = (record.date_deadline - base_date).days diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 768cdcc7951..0d8b32da21d 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -45,6 +45,8 @@ + + @@ -71,6 +73,7 @@ + @@ -86,6 +89,7 @@ + From 9ac0ca94df9bc2ecdc8da3839ae545db12c0079c Mon Sep 17 00:00:00 2001 From: Yousef Allam Date: Wed, 19 Aug 2026 14:57:18 +0200 Subject: [PATCH 08/15] [IMP] Chapter 9: Ready For Some Action --- estate/models/estate_property.py | 70 ++++++++++++++++---------- estate/models/estate_property_offer.py | 49 ++++++++++++++---- estate/views/estate_property_views.xml | 11 +++- 3 files changed, 92 insertions(+), 38 deletions(-) diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index ecd15fc1646..2048c5402b8 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -1,14 +1,18 @@ from dateutil.relativedelta import relativedelta from odoo import models, fields, api +from odoo.exceptions import UserError + class EstateProperty(models.Model): - _name = 'estate.property' - _description = 'Estate Property' + _name = "estate.property" + _description = "Estate Property" name = fields.Char(required=True) description = fields.Text() postcode = fields.Char() - date_availability = fields.Date(copy=False, default=lambda self: fields.Date.today() + relativedelta(months=3)) + date_availability = fields.Date( + copy=False, default=lambda self: fields.Date.today() + relativedelta(months=3) + ) expected_price = fields.Float(required=True) selling_price = fields.Float(readonly=True, copy=False) bedrooms = fields.Integer(default=2) @@ -18,28 +22,32 @@ class EstateProperty(models.Model): garden = fields.Boolean() garden_area = fields.Integer() active = fields.Boolean(default=True) - garden_orientation = fields.Selection([ - ('north', 'North'), - ('south', 'South'), - ('east', 'East'), - ('west', 'West') - ]) - state = fields.Selection(required=True, copy=False, default='new', selection=[ - ('new', 'New'), - ('offer_received', 'Offer Received'), - ('offer_accepted', 'Offer Accepted'), - ('sold', 'Sold'), - ('canceled', 'Canceled') - ]) - property_type_id = fields.Many2one('estate.property.type', string='Property Type') - user_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') - offer_ids = fields.One2many('estate.property.offer', 'property_id') - total_area = fields.Integer(compute='_compute_total_area') - best_price = fields.Float(compute='_compute_best_price') - - @api.depends('living_area', 'garden_area') + garden_orientation = fields.Selection( + [("north", "North"), ("south", "South"), ("east", "East"), ("west", "West")] + ) + state = fields.Selection( + required=True, + copy=False, + default="new", + selection=[ + ("new", "New"), + ("offer_received", "Offer Received"), + ("offer_accepted", "Offer Accepted"), + ("sold", "Sold"), + ("canceled", "Canceled"), + ], + ) + property_type_id = fields.Many2one("estate.property.type", string="Property Type") + user_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") + offer_ids = fields.One2many("estate.property.offer", "property_id") + total_area = fields.Integer(compute="_compute_total_area") + best_price = fields.Float(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 @@ -59,4 +67,14 @@ def _onchange_garden(self): self.garden_orientation = False else: self.garden_area = 10 - self.garden_orientation = 'north' + self.garden_orientation = "north" + + def action_sold(self): + if self.state == "canceled": + raise UserError("You cannot mark a canceled property as sold.") + self.state = "sold" + + def action_canceled(self): + if self.state == "sold": + raise UserError("You cannot cancel a sold property.") + self.state = "canceled" diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py index e81f9dc7a78..4cebb3a7b17 100644 --- a/estate/models/estate_property_offer.py +++ b/estate/models/estate_property_offer.py @@ -1,26 +1,55 @@ from odoo import models, fields, api +from odoo.exceptions import UserError + class EstatePropertyOffer(models.Model): - _name = 'estate.property.offer' - _description = 'Estate Property Offer' + _name = "estate.property.offer" + _description = "Estate Property Offer" price = fields.Float() - status = fields.Selection(copy=False, selection=[('accepted', 'Accepted'), ('refused', 'Refused')]) - partner_id = fields.Many2one('res.partner', required=True) - property_id = fields.Many2one('estate.property', required=True) - date_deadline = fields.Date(compute='_compute_date_deadline', inverse='_inverse_date_deadline', store=True) + status = fields.Selection( + copy=False, selection=[("accepted", "Accepted"), ("refused", "Refused")] + ) + partner_id = fields.Many2one("res.partner", required=True) + property_id = fields.Many2one("estate.property", required=True) + date_deadline = fields.Date( + compute="_compute_date_deadline", inverse="_inverse_date_deadline", store=True + ) validity = fields.Integer(default=7) - @api.depends('validity') + @api.depends("validity") def _compute_date_deadline(self): for record in self: - base_date = record.create_date.date() if record.create_date else fields.Date.today() + base_date = ( + record.create_date.date() if record.create_date else fields.Date.today() + ) record.date_deadline = fields.Date.add(base_date, days=record.validity) - @api.depends('date_deadline') + @api.depends("date_deadline") def _inverse_date_deadline(self): for record in self: if not record.date_deadline: continue - base_date = record.create_date.date() if record.create_date else fields.Date.today() + base_date = ( + record.create_date.date() if record.create_date else fields.Date.today() + ) record.validity = (record.date_deadline - base_date).days + + def action_accept_offer(self): + for record in self: + if record.property_id.state not in ["new", "offer_received"]: + raise UserError( + "You can only accept offers for properties that are new or have received offers." + ) + record.status = "accepted" + record.property_id.selling_price = record.price + record.property_id.buyer_id = record.partner_id + record.property_id.state = "offer_accepted" + + def action_refuse_offer(self): + for record in self: + if record.status == "accepted": + raise UserError( + "You cannot refuse an offer that has already been accepted." + ) + record.status = "refused" diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 0d8b32da21d..020b23c0757 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -43,10 +43,12 @@ - - + + + +

+ +

+ + + + + + + + + + + + + + + + estate.property.form estate.property
-
-
-

- +

+

- +

- - - - + + + - - - + + +
- - - - - - - - - + + + + + + + + + - + - - + + @@ -119,16 +175,18 @@ estate.property - - - - - - - - + + + + + + + + + - + From 8e6d89b30652f60f29641733a81919c3762fd734 Mon Sep 17 00:00:00 2001 From: Yousef Allam Date: Thu, 20 Aug 2026 15:40:00 +0200 Subject: [PATCH 11/15] [IMP] Chapter 12: Inheritance --- estate/models/__init__.py | 1 + estate/models/estate_property.py | 8 ++++++++ estate/models/estate_property_offer.py | 20 ++++++++++++++++++++ estate/models/res_users.py | 12 ++++++++++++ estate/views/estate_property_views.xml | 13 +++++++++++++ 5 files changed, 54 insertions(+) create mode 100644 estate/models/res_users.py diff --git a/estate/models/__init__.py b/estate/models/__init__.py index 2f1821a39c1..9a2189b6382 100644 --- a/estate/models/__init__.py +++ b/estate/models/__init__.py @@ -2,3 +2,4 @@ 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 index 2c9bcccf13b..5a36450ccef 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -106,3 +106,11 @@ def action_canceled(self): if self.state == "sold": raise UserError("You cannot cancel a sold property.") self.state = "canceled" + + @api.ondelete(at_uninstall=False) + def _delete_if_new_or_canceled(self): + for record in self: + if record.state in ["new", "canceled"]: + raise UserError( + "You cannot delete a new or cancelled property." + ) diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py index 57d52a237b1..e015fedb6bf 100644 --- a/estate/models/estate_property_offer.py +++ b/estate/models/estate_property_offer.py @@ -1,5 +1,6 @@ from odoo import api, fields, models from odoo.exceptions import UserError +from odoo.tools.float_utils import float_compare class EstatePropertyOffer(models.Model): @@ -60,3 +61,22 @@ def action_refuse_offer(self): "You cannot refuse an offer that has already been accepted." ) record.status = "refused" + + @api.model + def create(self, vals): + for val in vals: + estate_property = self.env["estate.property"].browse(val["property_id"]) + estate_property.status = "offer_received" + + previous_offers_prices = estate_property.property_offer_ids.mapped("price") + if ( + float_compare( + val["price"], min(previous_offers_prices), precision_digits=2 + ) + < 0 + ): + raise UserError( + f"The offer {val['price']} cannot be lower than the other offers." + ) + + return super().create(vals) diff --git a/estate/models/res_users.py b/estate/models/res_users.py new file mode 100644 index 00000000000..2af37881fd8 --- /dev/null +++ b/estate/models/res_users.py @@ -0,0 +1,12 @@ +from odoo import fields, models + + +class ResUsers(models.Model): + _inherit = "res.users" + + property_ids = fields.One2many( + "estate.property", + "user_id", + string="Properties", + domain="[('state', 'in', ['new', 'offer_received'])]", + ) diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index bdb0b78453c..f1ef21c1304 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -191,4 +191,17 @@ + + + res.users.view.form.inherit.property_ids + res.users + + + + + + + + + From 0e68bf1dd514b0b5eee5fb0fd0102282e3d50e2f Mon Sep 17 00:00:00 2001 From: Yousef Allam Date: Thu, 20 Aug 2026 16:54:42 +0200 Subject: [PATCH 12/15] [IMP] Chapter 13: Interact With Other Modules --- estate_account/__init__.py | 1 + estate_account/__manifest__.py | 12 ++++++++ estate_account/models/__init__.py | 1 + estate_account/models/estate_property.py | 31 +++++++++++++++++++++ estate_account/security/ir.model.access.csv | 2 ++ 5 files changed, 47 insertions(+) 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 create mode 100644 estate_account/security/ir.model.access.csv 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..492652cd47c --- /dev/null +++ b/estate_account/__manifest__.py @@ -0,0 +1,12 @@ +{ + "name": "estate_account", + "version": "0.0", + "depends": ["estate", "account"], + "installable": True, + "application": True, + "author": "me", + "license": "LGPL-3", + "data": [ + "security/ir.model.access.csv", + ], +} 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..a1671cad821 --- /dev/null +++ b/estate_account/models/estate_property.py @@ -0,0 +1,31 @@ +from odoo import Command, models + + +class EstateProperty(models.Model): + _inherit = ["estate.property"] + + def action_sold(self): + 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": property.name, + "quantity": 1.0, + "price_unit": property.selling_price * 0.06, + } + ), + Command.create( + { + "name": "Administrative fees", + "quantity": 1.0, + "price_unit": 100.0, + } + ), + ], + } + ) + return super().action_sold() diff --git a/estate_account/security/ir.model.access.csv b/estate_account/security/ir.model.access.csv new file mode 100644 index 00000000000..32389642d4f --- /dev/null +++ b/estate_account/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,access_estate_property,model_estate_property,base.group_user,1,1,1,1 From 9dad5121e222512f6519253736de571c85366a1b Mon Sep 17 00:00:00 2001 From: Yousef Allam Date: Fri, 21 Aug 2026 09:27:21 +0200 Subject: [PATCH 13/15] [IMP] Chapter 14: A Brief History Of QWeb --- estate/models/estate_property_offer.py | 21 +++++++-------- estate/views/estate_property_views.xml | 37 +++++++++++++++++++++++--- 2 files changed, 43 insertions(+), 15 deletions(-) diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py index e015fedb6bf..c07d53f3e85 100644 --- a/estate/models/estate_property_offer.py +++ b/estate/models/estate_property_offer.py @@ -66,17 +66,14 @@ def action_refuse_offer(self): def create(self, vals): for val in vals: estate_property = self.env["estate.property"].browse(val["property_id"]) - estate_property.status = "offer_received" - - previous_offers_prices = estate_property.property_offer_ids.mapped("price") - if ( - float_compare( - val["price"], min(previous_offers_prices), precision_digits=2 - ) - < 0 - ): - raise UserError( - f"The offer {val['price']} cannot be lower than the other offers." - ) + estate_property.state = "offer_received" + existing_prices = estate_property.offer_ids.mapped("price") + + if existing_prices: + min_price = min(existing_prices) + if float_compare(val["price"], min_price, precision_digits=2) < 0: + raise UserError( + f"The offer {val['price']} cannot be lower than the other offers." + ) return super().create(vals) diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index f1ef21c1304..488ff8a9d62 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -2,7 +2,7 @@ Properties estate.property - list,form + list,form,kanban {'search_default_state_filter': 1} @@ -115,7 +115,7 @@ + +

+ +

+ + + + + + + + + + + + +
+ + + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 488ff8a9d62..20daf13384e 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -1,28 +1,4 @@ - - Properties - estate.property - list,form,kanban - {'search_default_state_filter': 1} - - - Property Types - estate.property.type - list,form - - - Property Tags - estate.property.tag - list,form - - - - Property Offers - estate.property.offer - list,form - [('property_type_id', '=', active_id)] - - estate.property.tree estate.property @@ -42,69 +18,6 @@ - - estate.property.offer.tree - estate.property.offer - - - - - - - - - -

- -

- - - - - - - - - - - - - - - - estate.property.form estate.property @@ -192,19 +105,6 @@ - - res.users.view.form.inherit.property_ids - res.users - - - - - - - - - - estate.property.kanban estate.property diff --git a/estate/views/res_users_views.xml b/estate/views/res_users_views.xml new file mode 100644 index 00000000000..7428663dc8e --- /dev/null +++ b/estate/views/res_users_views.xml @@ -0,0 +1,14 @@ + + + res.users.view.form.inherit.property_ids + res.users + + + + + + + + + + diff --git a/estate_account/models/estate_property.py b/estate_account/models/estate_property.py index a1671cad821..1c7ea4e4844 100644 --- a/estate_account/models/estate_property.py +++ b/estate_account/models/estate_property.py @@ -2,20 +2,20 @@ class EstateProperty(models.Model): - _inherit = ["estate.property"] + _inherit = "estate.property" def action_sold(self): - for property in self: + for estate_property in self: self.env["account.move"].create( { - "partner_id": property.buyer_id.id, + "partner_id": estate_property.buyer_id.id, "move_type": "out_invoice", "invoice_line_ids": [ Command.create( { - "name": property.name, + "name": estate_property.name, "quantity": 1.0, - "price_unit": property.selling_price * 0.06, + "price_unit": estate_property.selling_price * 0.06, } ), Command.create( From 27e3fc25c9a773f0a5883a845aa8e7e8b4c5b6e8 Mon Sep 17 00:00:00 2001 From: Yousef Allam Date: Mon, 24 Aug 2026 10:23:52 +0200 Subject: [PATCH 15/15] [REF] Estate: clean up and added views to module manifest --- estate/__manifest__.py | 3 +++ estate/models/estate_property.py | 4 +-- estate/models/estate_property_offer.py | 26 ++++++++++++-------- estate/models/estate_property_tag.py | 2 +- estate/models/estate_property_type.py | 10 +++++--- estate/models/res_users.py | 2 +- estate/views/estate_property_offer_views.xml | 4 +-- estate/views/estate_property_type_views.xml | 6 ++--- estate/views/estate_property_views.xml | 6 ++--- 9 files changed, 38 insertions(+), 25 deletions(-) diff --git a/estate/__manifest__.py b/estate/__manifest__.py index 7bc8ae3d35d..6b1e3d02d86 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -13,5 +13,8 @@ "views/estate_property_views.xml", "views/estate_property_actions.xml", "views/estate_menus.xml", + "views/estate_property_type_views.xml", + "views/estate_property_offer_views.xml", + "views/res_users_views.xml", ], } diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index 58695062063..b4d158d86e6 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -8,8 +8,9 @@ class EstateProperty(models.Model): _name = "estate.property" _description = "Estate Property" + _order = "id desc" - name = fields.Char(required=True) + name = fields.Char(string="Title", required=True) description = fields.Text() postcode = fields.Char() date_availability = fields.Date( @@ -60,7 +61,6 @@ class EstateProperty(models.Model): "CHECK(selling_price >= 0)", "The selling price of a property cannot be negative.", ) - _order = "id desc" @api.constrains("selling_price") def _check_selling_price(self): diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py index 17d4bcb038d..977e3ee8ab6 100644 --- a/estate/models/estate_property_offer.py +++ b/estate/models/estate_property_offer.py @@ -6,25 +6,30 @@ class EstatePropertyOffer(models.Model): _name = "estate.property.offer" _description = "Estate Property Offer" + _order = "price desc" price = fields.Float() status = fields.Selection( - copy=False, selection=[("accepted", "Accepted"), ("refused", "Refused")] + 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 + related="property_id.property_type_id", + store=True, ) date_deadline = fields.Date( - compute="_compute_date_deadline", inverse="_inverse_date_deadline", store=True + compute="_compute_date_deadline", + inverse="_inverse_date_deadline", + store=True, ) validity = fields.Integer(default=7) _positive_price = models.Constraint( - "CHECK(price > 0)", "The price of an offer cannot be negative." + "CHECK(price > 0)", + "The price of an offer cannot be negative.", ) - _order = "price desc" @api.depends("create_date", "validity") def _compute_date_deadline(self): @@ -36,7 +41,6 @@ def _compute_date_deadline(self): ) record.date_deadline = fields.Date.add(base_date, days=record.validity) - @api.depends("date_deadline") def _inverse_date_deadline(self): for record in self: if not record.date_deadline: @@ -51,8 +55,9 @@ def _inverse_date_deadline(self): def action_accept_offer(self): for record in self: if record.property_id.state not in ["new", "offer_received"]: + msg = "You can only accept offers for properties that are new or have received offers." raise UserError( - "You can only accept offers for properties that are new or have received offers." + msg, ) record.status = "accepted" record.property_id.state = "offer_accepted" @@ -62,8 +67,9 @@ def action_accept_offer(self): def action_refuse_offer(self): for record in self: if record.status == "accepted": + msg = "You cannot refuse an offer that has already been accepted." raise UserError( - "You cannot refuse an offer that has already been accepted." + msg, ) record.status = "refused" @@ -71,7 +77,7 @@ def action_refuse_offer(self): def create(self, vals_list): for vals in vals_list: estate_property = self.env["estate.property"].browse( - vals.get("property_id") + vals.get("property_id"), ) existing_prices = estate_property.offer_ids.mapped("price") @@ -82,7 +88,7 @@ def create(self, vals_list): < 0 ): raise UserError( - f"The offer {vals.get('price', 0.0)} cannot be lower than the other offers." + f"The offer {vals.get('price', 0.0)} cannot be lower than the other offers.", ) estate_property.state = "offer_received" diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py index b6573bb202a..c6778c26e8d 100644 --- a/estate/models/estate_property_tag.py +++ b/estate/models/estate_property_tag.py @@ -4,6 +4,7 @@ class EstatePropertyTag(models.Model): _name = "estate.property.tag" _description = "Estate Property Tag" + _order = "name" name = fields.Char(required=True) color = fields.Integer() @@ -12,4 +13,3 @@ class EstatePropertyTag(models.Model): "UNIQUE (name)", "The name of the tag must be unique!", ) - _order = "name" diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py index e358483c212..923e6333869 100644 --- a/estate/models/estate_property_type.py +++ b/estate/models/estate_property_type.py @@ -4,6 +4,7 @@ class EstatePropertyType(models.Model): _name = "estate.property.type" _description = "Estate Property Type" + _order = "sequence, name" name = fields.Char(required=True) _name_uniq = models.Constraint( @@ -12,14 +13,17 @@ class EstatePropertyType(models.Model): ) property_ids = fields.One2many( - "estate.property", "property_type_id", string="Properties" + "estate.property", + "property_type_id", + string="Properties", ) offer_ids = fields.One2many( - "estate.property.offer", "property_type_id", string="Offers" + "estate.property.offer", + "property_type_id", + string="Offers", ) offer_count = fields.Integer(compute="_compute_offer_count") sequence = fields.Integer("Sequence", default=1) - _order = "sequence, name" @api.depends("offer_ids") def _compute_offer_count(self): diff --git a/estate/models/res_users.py b/estate/models/res_users.py index 2af37881fd8..9cf76344815 100644 --- a/estate/models/res_users.py +++ b/estate/models/res_users.py @@ -7,6 +7,6 @@ class ResUsers(models.Model): property_ids = fields.One2many( "estate.property", "user_id", - string="Properties", + string="Estate Properties", domain="[('state', 'in', ['new', 'offer_received'])]", ) diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml index e0b11b82b99..ef4cd5b8366 100644 --- a/estate/views/estate_property_offer_views.xml +++ b/estate/views/estate_property_offer_views.xml @@ -1,6 +1,6 @@ - - estate.property.offer.tree + + estate.property.offer.list estate.property.offer - - estate.property.type.tree + + estate.property.type.list estate.property.type @@ -29,7 +29,7 @@ - + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 20daf13384e..fce36d0ca9b 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -1,6 +1,6 @@ - - estate.property.tree + + estate.property.list estate.property estate.property - +