From 748a2b7c39760ae44f2bf5bd4c62170de1d0cad7 Mon Sep 17 00:00:00 2001 From: tskv Date: Tue, 18 Nov 2025 14:05:20 +0100 Subject: [PATCH 01/16] [ADD] estate:taskv --- estate/__init__.py | 0 estate/__manifest__.py | 14 ++++++++++++++ 2 files changed, 14 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..6fb5b82d9fe --- /dev/null +++ b/estate/__manifest__.py @@ -0,0 +1,14 @@ +{ + 'name': "Real Estate", + 'version': '1.0', + 'depends': ['base'], + 'author': "taskv", + 'category': 'Category', + 'description': """ + Description text + """, + 'data': [ + ], + 'demo': [ + ], +} \ No newline at end of file From 07ee005b359ab41012116f8c78f72920e438fcb3 Mon Sep 17 00:00:00 2001 From: tskv Date: Tue, 18 Nov 2025 14:06:46 +0100 Subject: [PATCH 02/16] [ADD] estate: init --- estate/__manifest__.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/estate/__manifest__.py b/estate/__manifest__.py index 6fb5b82d9fe..5660bb5c774 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -5,7 +5,7 @@ 'author': "taskv", 'category': 'Category', 'description': """ - Description text + Description """, 'data': [ ], From 0db978f363b912e8a444c9ffb1afa6c57aba9769 Mon Sep 17 00:00:00 2001 From: tskv Date: Tue, 18 Nov 2025 14:25:46 +0100 Subject: [PATCH 03/16] [IMP] new model: estate_property --- estate/__init__.py | 4 ++++ estate/__manifest__.py | 3 +++ estate/models/__init__.py | 4 ++++ estate/models/estate_property.py | 11 +++++++++++ 4 files changed, 22 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..e9917144f69 100644 --- a/estate/__init__.py +++ b/estate/__init__.py @@ -0,0 +1,4 @@ +# -*- coding: utf-8 -*- +# Part of Odoo. See LICENSE file for full copyright and licensing details. + +from . import models \ No newline at end of file diff --git a/estate/__manifest__.py b/estate/__manifest__.py index 5660bb5c774..090a184a7cd 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -11,4 +11,7 @@ ], 'demo': [ ], + 'installable': True, + 'application': True, + 'license': 'LGPL-3', } \ No newline at end of file diff --git a/estate/models/__init__.py b/estate/models/__init__.py new file mode 100644 index 00000000000..0c4a347c60d --- /dev/null +++ b/estate/models/__init__.py @@ -0,0 +1,4 @@ +# -*- coding: utf-8 -*- +# Part of Odoo. See LICENSE file for full copyright and licensing details. + +from . import estate_property \ No newline at end of file diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py new file mode 100644 index 00000000000..7a179bb239c --- /dev/null +++ b/estate/models/estate_property.py @@ -0,0 +1,11 @@ +# -*- coding: utf-8 -*- +# Part of Odoo. See LICENSE file for full copyright and licensing details. + +from odoo import fields, models + +class EstateProperty(models.Model): + _name = 'estate.property' + + name = fields.Char('Estate Property', quired=True, translate='True') + active = fields.Boolean('Active', default=True) + price = fields.Float('Price', default=0) \ No newline at end of file From b78f9c1689bd7f760c43339a3d2893f1a1ad3b38 Mon Sep 17 00:00:00 2001 From: tskv Date: Tue, 18 Nov 2025 14:41:07 +0100 Subject: [PATCH 04/16] [IMP] estate_property:new fields --- estate/__manifest__.py | 2 +- estate/models/estate_property.py | 23 +++++++++++++++++++++-- 2 files changed, 22 insertions(+), 3 deletions(-) diff --git a/estate/__manifest__.py b/estate/__manifest__.py index 090a184a7cd..f1741a8de6a 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -5,7 +5,7 @@ 'author': "taskv", 'category': 'Category', 'description': """ - Description + Tutorial Project """, 'data': [ ], diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index 7a179bb239c..8061c6cf264 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -5,7 +5,26 @@ class EstateProperty(models.Model): _name = 'estate.property' + _description = 'Estate Property' - name = fields.Char('Estate Property', quired=True, translate='True') + name = fields.Char('Estate Property', required=True, translate='True') active = fields.Boolean('Active', default=True) - price = fields.Float('Price', default=0) \ No newline at end of file + description = fields.Text('Description') + postcode = fields.Char('Postcode') + date_availability = fields.Date('Date Availability') + expected_price = fields.Float('Expected Price', required=True) + selling_price = fields.Float('Selling Price') + bedrooms = fields.Integer('Bedrooms') + living_area = fields.Integer('Living Area') + facades = fields.Integer('Facades') + garage = fields.Boolean('Garage') + garden = fields.Boolean('Garden') + garden_area = fields.Integer('Garden Area') + garden_orientation = fields.Selection(string='Garden orientation', + selection=[('north', 'North'), + ('south', 'South'), + ('east', 'East'), + ('west', 'West')] + ) + + From c66d2ad57a20ef29157612cdbda77111f2fb04ad Mon Sep 17 00:00:00 2001 From: tskv Date: Tue, 18 Nov 2025 14:51:54 +0100 Subject: [PATCH 05/16] [IMP] estate: access rights --- estate/__init__.py | 3 +-- estate/__manifest__.py | 7 +++++- estate/models/__init__.py | 3 +-- estate/models/estate_property.py | 8 +++---- estate/security/ir.model.access.csv | 2 ++ estate/views/estate_menus.xml | 19 +++++++++++++++ estate/views/estate_property_views.xml | 32 ++++++++++++++++++++++++++ 7 files changed, 64 insertions(+), 10 deletions(-) create mode 100644 estate/security/ir.model.access.csv create mode 100644 estate/views/estate_menus.xml create mode 100644 estate/views/estate_property_views.xml diff --git a/estate/__init__.py b/estate/__init__.py index e9917144f69..d6210b1285d 100644 --- a/estate/__init__.py +++ b/estate/__init__.py @@ -1,4 +1,3 @@ -# -*- coding: utf-8 -*- # Part of Odoo. See LICENSE file for full copyright and licensing details. -from . import models \ No newline at end of file +from . import models diff --git a/estate/__manifest__.py b/estate/__manifest__.py index f1741a8de6a..e132f8bd045 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -8,10 +8,15 @@ Tutorial Project """, 'data': [ + 'security/ir.model.access.csv', + + + 'views/estate_property_views.xml', + 'views/estate_menus.xml', ], 'demo': [ ], 'installable': True, 'application': True, 'license': 'LGPL-3', -} \ No newline at end of file +} diff --git a/estate/models/__init__.py b/estate/models/__init__.py index 0c4a347c60d..09b94f90f8d 100644 --- a/estate/models/__init__.py +++ b/estate/models/__init__.py @@ -1,4 +1,3 @@ -# -*- coding: utf-8 -*- # Part of Odoo. See LICENSE file for full copyright and licensing details. -from . import estate_property \ No newline at end of file +from . import estate_property diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index 8061c6cf264..a4d74e976c3 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -1,18 +1,18 @@ -# -*- coding: utf-8 -*- # Part of Odoo. See LICENSE file for full copyright and licensing details. from odoo import fields, models + class EstateProperty(models.Model): _name = 'estate.property' _description = 'Estate Property' - name = fields.Char('Estate Property', required=True, translate='True') + name = fields.Char('Estate Property', translate='True') active = fields.Boolean('Active', default=True) description = fields.Text('Description') postcode = fields.Char('Postcode') date_availability = fields.Date('Date Availability') - expected_price = fields.Float('Expected Price', required=True) + expected_price = fields.Float('Expected Price') selling_price = fields.Float('Selling Price') bedrooms = fields.Integer('Bedrooms') living_area = fields.Integer('Living Area') @@ -26,5 +26,3 @@ class EstateProperty(models.Model): ('east', 'East'), ('west', 'West')] ) - - diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv new file mode 100644 index 00000000000..d9d6ba57cc5 --- /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 diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml new file mode 100644 index 00000000000..9b6f1ae4075 --- /dev/null +++ b/estate/views/estate_menus.xml @@ -0,0 +1,19 @@ + + + + + + + + \ 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..6ddba1aa607 --- /dev/null +++ b/estate/views/estate_property_views.xml @@ -0,0 +1,32 @@ + + + + estate.property.view.form + estate.property + +
+ + + +
+ + + + + estate.property.view.list + estate.property + + + + + + + + + + Estate Property + estate.property + list,form + + + \ No newline at end of file From 821328f1c78c4002c9c7fd720267bc9d75327438 Mon Sep 17 00:00:00 2001 From: tskv Date: Tue, 18 Nov 2025 17:09:23 +0100 Subject: [PATCH 06/16] [IMP] estate: menu + view form --- estate/__manifest__.py | 1 - estate/models/estate_property.py | 18 ++++++++++++----- estate/views/estate_menus.xml | 2 +- estate/views/estate_property_views.xml | 27 +++++++++++++++++++++++--- 4 files changed, 38 insertions(+), 10 deletions(-) diff --git a/estate/__manifest__.py b/estate/__manifest__.py index e132f8bd045..548c729ac60 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -10,7 +10,6 @@ '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 a4d74e976c3..291daf83a41 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -1,20 +1,21 @@ # Part of Odoo. See LICENSE file for full copyright and licensing details. from odoo import fields, models +from datetime import timedelta class EstateProperty(models.Model): _name = 'estate.property' _description = 'Estate Property' - name = fields.Char('Estate Property', translate='True') + name = fields.Char('Estate Property', required=True, default='Unknown', translate='True') active = fields.Boolean('Active', default=True) description = fields.Text('Description') postcode = fields.Char('Postcode') - date_availability = fields.Date('Date Availability') - expected_price = fields.Float('Expected Price') - selling_price = fields.Float('Selling Price') - bedrooms = fields.Integer('Bedrooms') + date_availability = fields.Date('Date Availability', copy=False, default=fields.Datetime.now() + timedelta(days=90)) + expected_price = fields.Float('Expected Price', required=True) + selling_price = fields.Float('Selling Price', readonly=True, copy=False) + bedrooms = fields.Integer('Bedrooms', default=2) living_area = fields.Integer('Living Area') facades = fields.Integer('Facades') garage = fields.Boolean('Garage') @@ -26,3 +27,10 @@ class EstateProperty(models.Model): ('east', 'East'), ('west', 'West')] ) + state = fields.Selection(string='State', + selection=[('new', 'New'), + ('offer_received', 'Offer Received'), + ('offer_accepted', 'Offer Accepted'), + ('sold', 'Sold'), + ('cancelled', 'Cancelled')], + default='new', required=True, copy=False) diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml index 9b6f1ae4075..df87fa407b3 100644 --- a/estate/views/estate_menus.xml +++ b/estate/views/estate_menus.xml @@ -16,4 +16,4 @@ action="estate.estate_property_action" sequence="1"/> - \ No newline at end of file + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 6ddba1aa607..1de456c498d 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -6,7 +6,28 @@
- + + + + + + + + + + + + + + + + + + + + + +
@@ -16,7 +37,7 @@ estate.property.view.list estate.property - + @@ -29,4 +50,4 @@ list,form - \ No newline at end of file + From afec0e3fda7ee7f452634458df6bb2141f3ea82e Mon Sep 17 00:00:00 2001 From: tskv Date: Wed, 19 Nov 2025 10:20:40 +0100 Subject: [PATCH 07/16] [IMP] estate: chapter 6 (views) --- estate/models/estate_property.py | 8 ++-- estate/views/estate_property_views.xml | 66 ++++++++++++++++++++------ 2 files changed, 55 insertions(+), 19 deletions(-) diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index 291daf83a41..81cf54c0528 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -8,19 +8,19 @@ class EstateProperty(models.Model): _name = 'estate.property' _description = 'Estate Property' - name = fields.Char('Estate Property', required=True, default='Unknown', translate='True') + name = fields.Char('Title', required=True, default='Unknown', translate='True') active = fields.Boolean('Active', default=True) description = fields.Text('Description') postcode = fields.Char('Postcode') - date_availability = fields.Date('Date Availability', copy=False, default=fields.Datetime.now() + timedelta(days=90)) + date_availability = fields.Date('Available From', copy=False, default=fields.Datetime.now() + timedelta(days=90)) expected_price = fields.Float('Expected Price', required=True) selling_price = fields.Float('Selling Price', readonly=True, copy=False) bedrooms = fields.Integer('Bedrooms', default=2) - living_area = fields.Integer('Living Area') + living_area = fields.Integer('Living Area (sqm)') facades = fields.Integer('Facades') garage = fields.Boolean('Garage') garden = fields.Boolean('Garden') - garden_area = fields.Integer('Garden Area') + garden_area = fields.Integer('Garden Area (sqm)') garden_orientation = fields.Selection(string='Garden orientation', selection=[('north', 'North'), ('south', 'South'), diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 1de456c498d..5d60cad84e6 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -1,33 +1,63 @@ + + estate.property.search + estate.property + + + + + + + + + + + + + + + + + + + + estate.property.view.form estate.property
- - - - +
+

+ +

+
- - - - - + - + - - - - + + + + + + + + + + + + + +
@@ -38,8 +68,13 @@ estate.property - + + + + + + @@ -48,6 +83,7 @@ Estate Property estate.property list,form + From b87b0a4da91160cc56c23f9ee76dae93733f02d1 Mon Sep 17 00:00:00 2001 From: tskv Date: Wed, 19 Nov 2025 12:53:21 +0100 Subject: [PATCH 08/16] [IMP] estate: chapter 7 (relational fields) --- estate/__manifest__.py | 2 ++ estate/models/__init__.py | 3 ++ estate/models/estate_property.py | 7 +++++ estate/models/estate_property_offer.py | 15 +++++++++ estate/models/estate_property_tag.py | 10 ++++++ estate/models/estate_property_type.py | 10 ++++++ estate/security/ir.model.access.csv | 3 ++ estate/views/estate_menus.xml | 22 +++++++++---- estate/views/estate_property_tag_views.xml | 33 +++++++++++++++++++ estate/views/estate_property_type_views.xml | 35 +++++++++++++++++++++ estate/views/estate_property_views.xml | 27 ++++++++++++++-- 11 files changed, 158 insertions(+), 9 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 create mode 100644 estate/views/estate_property_tag_views.xml create mode 100644 estate/views/estate_property_type_views.xml diff --git a/estate/__manifest__.py b/estate/__manifest__.py index 548c729ac60..49887de0548 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -11,6 +11,8 @@ 'security/ir.model.access.csv', 'views/estate_property_views.xml', + 'views/estate_property_type_views.xml', + 'views/estate_property_tag_views.xml', 'views/estate_menus.xml', ], 'demo': [ diff --git a/estate/models/__init__.py b/estate/models/__init__.py index 09b94f90f8d..7b28f0f82ad 100644 --- a/estate/models/__init__.py +++ b/estate/models/__init__.py @@ -1,3 +1,6 @@ # Part of Odoo. See LICENSE file for full copyright and licensing details. 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 81cf54c0528..5f25c472db7 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -34,3 +34,10 @@ class EstateProperty(models.Model): ('sold', 'Sold'), ('cancelled', 'Cancelled')], default='new', required=True, copy=False) + property_type_id = fields.Many2one('estate.property.type', string='Property Type') + property_tag_ids = fields.Many2many('estate.property.tag', string='Property Tag') + buyer_id = fields.Many2one('res.partner', string='Buyer', copy=False, + domain=[('is_company', '=', False)]) + salesperson_id = fields.Many2one('res.users', string='Salesperson', + default=lambda self: self.env.user) + offer_ids = fields.Many2many('estate.property.offer', 'property_id', string='Offers') diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py new file mode 100644 index 00000000000..0d27b000eeb --- /dev/null +++ b/estate/models/estate_property_offer.py @@ -0,0 +1,15 @@ +# Part of Odoo. See LICENSE file for full copyright and licensing details. + +from odoo import fields, models + + +class EstatePropertyOffer(models.Model): + _name = 'estate.property.offer' + _description = 'Estate Property Offer' + + + price = fields.Float(string='Price') + status = fields.Selection(string='Status', copy=False, + selection=[('accepted', 'Accepted'), ('refused', 'Refused')]) + partner_id = fields.Many2one('res.partner', string='Customer', required=True) + property_id = fields.Many2one('estate.property', string='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..12586dcd3c3 --- /dev/null +++ b/estate/models/estate_property_tag.py @@ -0,0 +1,10 @@ +# Part of Odoo. See LICENSE file for full copyright and licensing details. + +from odoo import fields, models + + +class EstatePropertyTag(models.Model): + _name = 'estate.property.tag' + _description = 'Estate Property Tag' + + name = fields.Char(string='Property Tag', required=True) diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py new file mode 100644 index 00000000000..c157e02370a --- /dev/null +++ b/estate/models/estate_property_type.py @@ -0,0 +1,10 @@ +# Part of Odoo. See LICENSE file for full copyright and licensing details. + +from odoo import fields, models + + +class EstatePropertyType(models.Model): + _name = 'estate.property.type' + _description = 'Estate Property Type' + + name = fields.Char(string='Property Type', required=True) diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv index d9d6ba57cc5..49bca99cac8 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 df87fa407b3..f619f1ce708 100644 --- a/estate/views/estate_menus.xml +++ b/estate/views/estate_menus.xml @@ -1,6 +1,5 @@ - @@ -8,12 +7,23 @@ id="properties_menu" name="Properties" parent="estate_menu_root" + action="estate.estate_property_action" sequence="1"/> + - + diff --git a/estate/views/estate_property_tag_views.xml b/estate/views/estate_property_tag_views.xml new file mode 100644 index 00000000000..05750dca150 --- /dev/null +++ b/estate/views/estate_property_tag_views.xml @@ -0,0 +1,33 @@ + + + + estate.property.tag.view.form + estate.property.tag + +
+ + + + + +
+ + + + + estate.property.tag.view.list + estate.property.tag + + + + + + + + + Estate Property Tag + estate.property.tag + list,form + + + diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml new file mode 100644 index 00000000000..d38ee04d82a --- /dev/null +++ b/estate/views/estate_property_type_views.xml @@ -0,0 +1,35 @@ + + + + estate.property.type.view.form + estate.property.type + +
+ +
+

+ +

+
+ +
+ + + + + estate.property.type.view.list + estate.property.type + + + + + + + + + Estate Property Type + estate.property.type + list,form + + + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 5d60cad84e6..9deea421839 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -10,10 +10,13 @@ + + - + @@ -30,12 +33,14 @@
-

- +

+

+

+ @@ -57,6 +62,21 @@ + + + + + + + + + + + + + + +
@@ -70,6 +90,7 @@ + From ae446452ac1361f760794af8ff7820c8a92902bf Mon Sep 17 00:00:00 2001 From: tskv Date: Wed, 19 Nov 2025 14:14:48 +0100 Subject: [PATCH 09/16] [IMP] estate: chapter 8 (compute and onchange) --- estate/models/estate_property.py | 26 +++++++++++++++++++++++++- estate/models/estate_property_offer.py | 15 +++++++++++++-- estate/views/estate_property_views.xml | 10 +++++++--- 3 files changed, 45 insertions(+), 6 deletions(-) diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index 5f25c472db7..62637107e78 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -1,6 +1,6 @@ # Part of Odoo. See LICENSE file for full copyright and licensing details. -from odoo import fields, models +from odoo import fields, models, api from datetime import timedelta @@ -41,3 +41,27 @@ class EstateProperty(models.Model): salesperson_id = fields.Many2one('res.users', string='Salesperson', default=lambda self: self.env.user) offer_ids = fields.Many2many('estate.property.offer', 'property_id', string='Offers') + total_area = fields.Integer('Total Area (sqm)', compute='_compute_total_area') + best_offer = fields.Float('Best Offer', compute='_compute_best_offer') + + @api.depends('garden_area', 'living_area') + def _compute_total_area(self): + for record in self: + record.total_area = record.garden_area + record.living_area + + @api.depends('offer_ids.price') + def _compute_best_offer(self): + for record in self: + if record.offer_ids: + record.best_offer = max(record.offer_ids.mapped('price')) + else: + record.best_offer = 0 + + @api.onchange('garden') + def _onchange_garden(self): + if self.garden: + self.garden_area = 10 + self.garden_orientation = 'north' + else: + self.garden_area = 0 + self.garden_orientation = '' diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py index 0d27b000eeb..49f9aa2e07e 100644 --- a/estate/models/estate_property_offer.py +++ b/estate/models/estate_property_offer.py @@ -1,15 +1,26 @@ # Part of Odoo. See LICENSE file for full copyright and licensing details. -from odoo import fields, models +from odoo import fields, models, api +from datetime import date, timedelta class EstatePropertyOffer(models.Model): _name = 'estate.property.offer' _description = 'Estate Property Offer' - price = fields.Float(string='Price') status = fields.Selection(string='Status', copy=False, selection=[('accepted', 'Accepted'), ('refused', 'Refused')]) partner_id = fields.Many2one('res.partner', string='Customer', required=True) property_id = fields.Many2one('estate.property', string='Property', required=True) + validity = fields.Integer(string='Validity (days)', default=7) + date_deadline = fields.Date(string='Deadline', compute='_compute_deadline', inverse='_inverse_deadline') + + @api.depends('validity') + def _compute_deadline(self): + for record in self: + record.date_deadline = (record.create_date if record.create_date else date.today()) + timedelta(days=record.validity) + + def _inverse_deadline(self): + for record in self: + record.validity = (record.date_deadline - (record.create_date.date() if record.create_date else date.today())).days diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 9deea421839..c78594e1070 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -46,6 +46,7 @@ + @@ -60,6 +61,7 @@ + @@ -67,6 +69,8 @@ + + @@ -89,9 +93,9 @@ - - - + + + From 898c085478703eb98f3effb72b92437ac540e578 Mon Sep 17 00:00:00 2001 From: tskv Date: Wed, 19 Nov 2025 15:21:58 +0100 Subject: [PATCH 10/16] [IMP] estate: chapter 9 (actions) --- estate/__manifest__.py | 1 - estate/models/estate_property.py | 31 +++++++++++++++++++++++--- estate/models/estate_property_offer.py | 20 +++++++++++++++++ estate/models/estate_property_tag.py | 5 +++++ estate/models/estate_property_type.py | 5 +++++ estate/views/estate_menus.xml | 6 ++--- estate/views/estate_property_views.xml | 7 ++++++ 7 files changed, 68 insertions(+), 7 deletions(-) diff --git a/estate/__manifest__.py b/estate/__manifest__.py index 49887de0548..a9c9f5a15e1 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -9,7 +9,6 @@ """, 'data': [ 'security/ir.model.access.csv', - 'views/estate_property_views.xml', 'views/estate_property_type_views.xml', 'views/estate_property_tag_views.xml', diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index 62637107e78..1028f8ee7fa 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -1,7 +1,6 @@ # Part of Odoo. See LICENSE file for full copyright and licensing details. -from odoo import fields, models, api -from datetime import timedelta +from odoo import fields, models, api, exceptions class EstateProperty(models.Model): @@ -12,7 +11,7 @@ class EstateProperty(models.Model): active = fields.Boolean('Active', default=True) description = fields.Text('Description') postcode = fields.Char('Postcode') - date_availability = fields.Date('Available From', copy=False, default=fields.Datetime.now() + timedelta(days=90)) + date_availability = fields.Date('Available From', copy=False, default=fields.Date.add(fields.Date.today(), months=3)) expected_price = fields.Float('Expected Price', required=True) selling_price = fields.Float('Selling Price', readonly=True, copy=False) bedrooms = fields.Integer('Bedrooms', default=2) @@ -44,6 +43,16 @@ class EstateProperty(models.Model): total_area = fields.Integer('Total Area (sqm)', compute='_compute_total_area') best_offer = fields.Float('Best Offer', compute='_compute_best_offer') + _check_expected_price = models.Constraint( + 'CHECK(expected_price > 0)', + 'The expected price must be strictly positive', + ) + + _check_selling_price = models.Constraint( + 'CHECK(selling_price >= 0)', + 'The selling price must be positive', + ) + @api.depends('garden_area', 'living_area') def _compute_total_area(self): for record in self: @@ -65,3 +74,19 @@ def _onchange_garden(self): else: self.garden_area = 0 self.garden_orientation = '' + + def property_sold(self): + for record in self: + if record.state != 'cancelled': + record.state = 'sold' + else: + raise exceptions.UserError("Sold properties cannot be cancelled.") + return True + + def property_cancelled(self): + for record in self: + if record.state != 'sold': + record.state = 'cancelled' + else: + raise exceptions.UserError("Cancelled properties cannot be sold.") + return True diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py index 49f9aa2e07e..51583350544 100644 --- a/estate/models/estate_property_offer.py +++ b/estate/models/estate_property_offer.py @@ -16,6 +16,11 @@ class EstatePropertyOffer(models.Model): validity = fields.Integer(string='Validity (days)', default=7) date_deadline = fields.Date(string='Deadline', compute='_compute_deadline', inverse='_inverse_deadline') + _check_price = models.Constraint( + 'CHECK(price > 0)', + 'The offer price must be strictly positive', + ) + @api.depends('validity') def _compute_deadline(self): for record in self: @@ -24,3 +29,18 @@ def _compute_deadline(self): def _inverse_deadline(self): for record in self: record.validity = (record.date_deadline - (record.create_date.date() if record.create_date else date.today())).days + + def action_accept(self): + for record in self: + if record.property_id.state in ('new', 'offer_received'): + record.property_id.state = 'offer_accepted' + record.status = 'accepted' + record.property_id.selling_price = record.price + record.property_id.buyer_id = record.partner_id + return True + + def action_refuse(self): + for record in self: + if not record.status: + record.status = 'refused' + return True diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py index 12586dcd3c3..125c59ac8b7 100644 --- a/estate/models/estate_property_tag.py +++ b/estate/models/estate_property_tag.py @@ -8,3 +8,8 @@ class EstatePropertyTag(models.Model): _description = 'Estate Property Tag' name = fields.Char(string='Property Tag', required=True) + + _tag_name_uniq = models.Constraint( + 'unique(name)', + "The property tag name must be unique", + ) diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py index c157e02370a..7789de50e06 100644 --- a/estate/models/estate_property_type.py +++ b/estate/models/estate_property_type.py @@ -8,3 +8,8 @@ class EstatePropertyType(models.Model): _description = 'Estate Property Type' name = fields.Char(string='Property Type', required=True) + + _type_name_uniq = models.Constraint( + 'unique(name)', + "The property type name must be unique", + ) diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml index f619f1ce708..18b79afccf6 100644 --- a/estate/views/estate_menus.xml +++ b/estate/views/estate_menus.xml @@ -7,7 +7,7 @@ id="properties_menu" name="Properties" parent="estate_menu_root" - action="estate.estate_property_action" + action="estate_property_action" sequence="1"/> diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index c78594e1070..2ede2aa522e 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -31,6 +31,10 @@ estate.property
+
+

@@ -40,6 +44,7 @@

+ @@ -72,6 +77,8 @@ + +

+ + + + + + + + + + +
@@ -21,6 +40,7 @@ estate.property.type + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 9c32cae552c..aa0c4255fd1 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -8,11 +8,11 @@ - + - +
-

- +
- - + @@ -64,13 +66,13 @@ - - + + - + @@ -88,7 +90,11 @@ estate.property.view.list estate.property - + @@ -96,7 +102,7 @@ - + @@ -106,6 +112,7 @@ estate.property list,form + {'search_default_filter_available':1} From 220923dc8a8e518ce41dbe4bacafc14807c9154b Mon Sep 17 00:00:00 2001 From: tskv Date: Thu, 20 Nov 2025 15:14:47 +0100 Subject: [PATCH 13/16] [IMP] estate: chapter 12 (inheritance) --- estate/__manifest__.py | 1 + estate/models/__init__.py | 1 + estate/models/estate_property.py | 6 ++++++ estate/models/estate_property_offer.py | 13 ++++++++++++- estate/models/res_users.py | 10 ++++++++++ estate/views/res_users_views.xml | 19 +++++++++++++++++++ 6 files changed, 49 insertions(+), 1 deletion(-) create mode 100644 estate/models/res_users.py create mode 100644 estate/views/res_users_views.xml diff --git a/estate/__manifest__.py b/estate/__manifest__.py index bf69561325a..a8843cea817 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -14,6 +14,7 @@ 'views/estate_property_type_views.xml', 'views/estate_property_tag_views.xml', 'views/estate_menus.xml', + 'views/res_users_views.xml', ], 'demo': [ ], diff --git a/estate/models/__init__.py b/estate/models/__init__.py index 7b28f0f82ad..f7e4cc6f3dd 100644 --- a/estate/models/__init__.py +++ b/estate/models/__init__.py @@ -4,3 +4,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 3048919d048..439b69b8c72 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -83,6 +83,12 @@ def _onchange_garden(self): self.garden_area = 0 self.garden_orientation = '' + @api.ondelete(at_uninstall=False) + def _unlink_check_state(self): + for record in self: + if record.state not in ('new', 'cancelled'): + raise exceptions.UserError("Only new and cancelled properties can be deleted.") + def property_sold(self): for record in self: if record.state != 'cancelled': diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py index 38bfab06491..9507cff2133 100644 --- a/estate/models/estate_property_offer.py +++ b/estate/models/estate_property_offer.py @@ -1,6 +1,6 @@ # Part of Odoo. See LICENSE file for full copyright and licensing details. -from odoo import fields, models, api +from odoo import fields, models, api, exceptions from datetime import date, timedelta @@ -23,6 +23,17 @@ class EstatePropertyOffer(models.Model): 'The offer price must be strictly positive', ) + @api.model + def create(self, vals_list): + for vals in vals_list: + if 'property_id' in vals and vals.get('property_id'): + current_property = self.env['estate.property'].browse(vals['property_id']) + if 'price' in vals and current_property.best_offer > vals.get('price', 0): + raise exceptions.ValidationError(f'The offer must be higher than {current_property.best_offer}') + if current_property.state == 'new': + current_property.state = 'offer_received' + return super().create(vals_list) + @api.depends('validity') def _compute_deadline(self): for record in self: diff --git a/estate/models/res_users.py b/estate/models/res_users.py new file mode 100644 index 00000000000..bd1f23c6ac7 --- /dev/null +++ b/estate/models/res_users.py @@ -0,0 +1,10 @@ +# Part of Odoo. See LICENSE file for full copyright and licensing details. + +from odoo import fields, models + + +class Users(models.Model): + _inherit = 'res.users' + + property_ids = fields.One2many('estate.property', 'salesperson_id', string='Estate Property', + domain=[('date_availability', '<=', fields.Date.today())]) diff --git a/estate/views/res_users_views.xml b/estate/views/res_users_views.xml new file mode 100644 index 00000000000..6dc864c5665 --- /dev/null +++ b/estate/views/res_users_views.xml @@ -0,0 +1,19 @@ + + + + + + res.users.view.form.inherit.real.estate + res.users + + + + + + + + + + + + From b722b3e849fbd81f304f2b9a40a0e6aad8192936 Mon Sep 17 00:00:00 2001 From: tskv Date: Thu, 20 Nov 2025 16:59:38 +0100 Subject: [PATCH 14/16] [ADD] estate: invoicing (chapter 13) --- estate/__manifest__.py | 2 +- estate/models/estate_property.py | 4 +-- estate_account/__init__.py | 3 +++ estate_account/__manifest__.py | 18 ++++++++++++++ estate_account/models/__init__.py | 3 +++ estate_account/models/estate_property.py | 31 ++++++++++++++++++++++++ 6 files changed, 58 insertions(+), 3 deletions(-) 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/__manifest__.py b/estate/__manifest__.py index a8843cea817..e45dd9af1fc 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -3,7 +3,7 @@ 'version': '1.0', 'depends': ['base'], 'author': "taskv", - 'category': 'Category', + 'category': 'Tutorials', 'description': """ Tutorial Project """, diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index 439b69b8c72..3bebf34eff7 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -94,7 +94,7 @@ def property_sold(self): if record.state != 'cancelled': record.state = 'sold' else: - raise exceptions.UserError("Sold properties cannot be cancelled.") + raise exceptions.UserError("Cancelled properties cannot be sold.") return True def property_cancelled(self): @@ -102,5 +102,5 @@ def property_cancelled(self): if record.state != 'sold': record.state = 'cancelled' else: - raise exceptions.UserError("Cancelled properties cannot be sold.") + raise exceptions.UserError("Sold properties cannot be cancelled.") return True diff --git a/estate_account/__init__.py b/estate_account/__init__.py new file mode 100644 index 00000000000..d6210b1285d --- /dev/null +++ b/estate_account/__init__.py @@ -0,0 +1,3 @@ +# Part of Odoo. See LICENSE file for full copyright and licensing details. + +from . import models diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py new file mode 100644 index 00000000000..5d132fb958c --- /dev/null +++ b/estate_account/__manifest__.py @@ -0,0 +1,18 @@ + +{ + 'name': "Real Estate Invoicing", + 'version': '1.0', + 'depends': ['estate', 'account'], + 'author': "taskv", + 'category': 'Tutorials', + 'description': """ + Tutorial Project + """, + 'data': [ + ], + 'demo': [ + ], + 'installable': True, + 'application': True, + 'license': 'LGPL-3', +} diff --git a/estate_account/models/__init__.py b/estate_account/models/__init__.py new file mode 100644 index 00000000000..09b94f90f8d --- /dev/null +++ b/estate_account/models/__init__.py @@ -0,0 +1,3 @@ +# Part of Odoo. See LICENSE file for full copyright and licensing details. + +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..1ed1b57fe27 --- /dev/null +++ b/estate_account/models/estate_property.py @@ -0,0 +1,31 @@ +# Part of Odoo. See LICENSE file for full copyright and licensing details. + +from odoo import models, Command + + +class EstateProperty(models.Model): + _inherit = 'estate.property' + + def property_sold(self): + invoice_vals_list = [] + for record in self: + if record.state == 'offer_accepted' and record.buyer_id: + invoice_vals = { + 'move_type': 'out_invoice', + 'partner_id': record.buyer_id.id, + 'invoice_line_ids': [ + Command.create({ + "name": "Commission 6%", + "quantity": 1, + "price_unit": record.selling_price * 0.06, + }), + Command.create({ + "name": "Administrative fees", + "quantity": 1, + "price_unit": 100., + }), + ], + } + invoice_vals_list.append(invoice_vals) + self.env['account.move'].sudo().create(invoice_vals_list) + return super().property_sold() From 64e100c442a30c822f447fca001b75e2ce24ca4d Mon Sep 17 00:00:00 2001 From: tskv Date: Fri, 21 Nov 2025 10:01:11 +0100 Subject: [PATCH 15/16] [ADD] estate: kanban view (chapter 14) --- estate/views/estate_property_views.xml | 32 +++++++++++++++++++++++++- 1 file changed, 31 insertions(+), 1 deletion(-) diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index aa0c4255fd1..e4728228487 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -107,10 +107,40 @@ + + estate.property.view.kanban + estate.property + + + + + +
+ +
+ Expected price: + +
+
+ Best offer: + +
+
+ Selling price: + +
+ +
+ + + + + + Estate Property estate.property - list,form + list,form,kanban {'search_default_filter_available':1} From 03cee0d8afacf873b173ff3e2bc3dda553668f6a Mon Sep 17 00:00:00 2001 From: tskv Date: Fri, 21 Nov 2025 10:34:08 +0100 Subject: [PATCH 16/16] [IMP] estate: chapter 15 --- estate/models/estate_property.py | 37 ++++++++++++------------ estate/models/estate_property_offer.py | 36 +++++++++++------------ estate/views/estate_property_views.xml | 4 +-- estate_account/models/estate_property.py | 4 +-- 4 files changed, 41 insertions(+), 40 deletions(-) diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index 3bebf34eff7..775621559d7 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -55,12 +55,6 @@ class EstateProperty(models.Model): 'The selling price must be positive', ) - @api.constrains('selling_price', 'expected_price') - def _check_selling_price(self): - for record in self: - if not float_is_zero(record.selling_price, 4) and float_compare(record.selling_price, record.expected_price * 0.9, 4) < 0: - raise exceptions.ValidationError("The selling price must be at least 90% of the expected price.") - @api.depends('garden_area', 'living_area') def _compute_total_area(self): for record in self: @@ -74,6 +68,13 @@ def _compute_best_offer(self): else: record.best_offer = 0 + @api.constrains('selling_price', 'expected_price') + def _check_selling_price(self): + for record in self: + if not float_is_zero(record.selling_price, 4) and float_compare(record.selling_price, + record.expected_price * 0.9, 4) < 0: + raise exceptions.ValidationError("The selling price must be at least 90% of the expected price.") + @api.onchange('garden') def _onchange_garden(self): if self.garden: @@ -89,18 +90,18 @@ def _unlink_check_state(self): if record.state not in ('new', 'cancelled'): raise exceptions.UserError("Only new and cancelled properties can be deleted.") - def property_sold(self): - for record in self: - if record.state != 'cancelled': - record.state = 'sold' - else: - raise exceptions.UserError("Cancelled properties cannot be sold.") + def action_property_sold(self): + self.ensure_one() + if self.state != 'cancelled': + self.state = 'sold' + else: + raise exceptions.UserError("Cancelled properties cannot be sold.") return True - def property_cancelled(self): - for record in self: - if record.state != 'sold': - record.state = 'cancelled' - else: - raise exceptions.UserError("Sold properties cannot be cancelled.") + def action_property_cancelled(self): + self.ensure_one() + if self.state != 'sold': + self.state = 'cancelled' + else: + raise exceptions.UserError("Sold properties cannot be cancelled.") return True diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py index 9507cff2133..7a66c2e72e8 100644 --- a/estate/models/estate_property_offer.py +++ b/estate/models/estate_property_offer.py @@ -23,6 +23,15 @@ class EstatePropertyOffer(models.Model): 'The offer price must be strictly positive', ) + @api.depends('validity') + def _compute_deadline(self): + for record in self: + record.date_deadline = (record.create_date if record.create_date else date.today()) + timedelta(days=record.validity) + + def _inverse_deadline(self): + for record in self: + record.validity = (record.date_deadline - (record.create_date.date() if record.create_date else date.today())).days + @api.model def create(self, vals_list): for vals in vals_list: @@ -34,26 +43,17 @@ def create(self, vals_list): current_property.state = 'offer_received' return super().create(vals_list) - @api.depends('validity') - def _compute_deadline(self): - for record in self: - record.date_deadline = (record.create_date if record.create_date else date.today()) + timedelta(days=record.validity) - - def _inverse_deadline(self): - for record in self: - record.validity = (record.date_deadline - (record.create_date.date() if record.create_date else date.today())).days - def action_accept(self): - for record in self: - if record.property_id.state in ('new', 'offer_received'): - record.property_id.state = 'offer_accepted' - record.status = 'accepted' - record.property_id.selling_price = record.price - record.property_id.buyer_id = record.partner_id + self.ensure_one() + if self.property_id.state in ('new', 'offer_received'): + self.property_id.state = 'offer_accepted' + self.status = 'accepted' + self.property_id.selling_price = self.price + self.property_id.buyer_id = self.partner_id return True def action_refuse(self): - for record in self: - if not record.status: - record.status = 'refused' + self.ensure_one() + if not self.status: + self.status = 'refused' return True diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index e4728228487..54fd0d5f05b 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -32,8 +32,8 @@
-
diff --git a/estate_account/models/estate_property.py b/estate_account/models/estate_property.py index 1ed1b57fe27..6ad9adb496b 100644 --- a/estate_account/models/estate_property.py +++ b/estate_account/models/estate_property.py @@ -6,7 +6,7 @@ class EstateProperty(models.Model): _inherit = 'estate.property' - def property_sold(self): + def action_property_sold(self): invoice_vals_list = [] for record in self: if record.state == 'offer_accepted' and record.buyer_id: @@ -28,4 +28,4 @@ def property_sold(self): } invoice_vals_list.append(invoice_vals) self.env['account.move'].sudo().create(invoice_vals_list) - return super().property_sold() + return super().action_property_sold()