From 9a8f241210fa65f4376700f30772e464d9199f4b Mon Sep 17 00:00:00 2001 From: jupao-odoo Date: Mon, 16 Feb 2026 14:24:24 +0100 Subject: [PATCH 1/6] [IMP] awesomeclicker: add editor metadata in Python file Added a new line 'editor': "jupao" in the module code to indicate the editor used for this script. This helps with code clarity and consistency within the module. --- awesome_clicker/__manifest__.py | 1 + 1 file changed, 1 insertion(+) diff --git a/awesome_clicker/__manifest__.py b/awesome_clicker/__manifest__.py index 56dc2f779b9..155a2286e64 100644 --- a/awesome_clicker/__manifest__.py +++ b/awesome_clicker/__manifest__.py @@ -11,6 +11,7 @@ """, 'author': "Odoo", + 'editor': "jupao", 'website': "https://www.odoo.com/", 'category': 'Tutorials', 'version': '0.1', From 67c33509841ff7230beb33db82c654428e01e16a Mon Sep 17 00:00:00 2001 From: jupao-odoo Date: Mon, 16 Feb 2026 17:28:26 +0100 Subject: [PATCH 2/6] feat(estate): complete tutorial chapters 1-5 - Initialize module structure - Create estate.property model - Add basic fields and attributes - Add security access rules - Add action and menus for UI --- estate/__init__.py | 1 + estate/__manifest__.py | 29 ++++++++++++++ estate/models/__init__.py | 1 + estate/models/estate_property.py | 54 +++++++++++++++++++++++++++ estate/security/ir.model.access.csv | 2 + estate/views/estate_menus.xml | 21 +++++++++++ estate/views/state_property_views.xml | 11 ++++++ settings.json | 7 ++++ 8 files changed, 126 insertions(+) create mode 100644 estate/__init__.py create mode 100644 estate/__manifest__.py create mode 100644 estate/models/__init__.py create mode 100644 estate/models/estate_property.py create mode 100644 estate/security/ir.model.access.csv create mode 100644 estate/views/estate_menus.xml create mode 100644 estate/views/state_property_views.xml create mode 100644 settings.json 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..915e38ce961 --- /dev/null +++ b/estate/__manifest__.py @@ -0,0 +1,29 @@ +# -*- coding: utf-8 -*- +{ + 'name': "Real Estate", + + 'summary': """ + Starting module for "Master the Odoo web framework, chapter 1: Build a new application" + """, + + 'description': """ + Starting module for "Master the Odoo web framework, chapter 1: Build a new application" + """, + + 'author': "Odoo", + 'website': "https://www.odoo.com/", + 'category': 'Tutorials', + 'version': '0.1', + 'application': True, + 'installable': True, + 'depends': ['base'], + + 'data': [ + 'security/ir.model.access.csv', + 'views/state_property_views.xml', + 'views/estate_menus.xml' + ], + + 'assets': {}, + 'license': 'AGPL-3' +} diff --git a/estate/models/__init__.py b/estate/models/__init__.py new file mode 100644 index 00000000000..f4c8fd6db6d --- /dev/null +++ b/estate/models/__init__.py @@ -0,0 +1 @@ +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..5d40d94d069 --- /dev/null +++ b/estate/models/estate_property.py @@ -0,0 +1,54 @@ +from odoo import models, fields +from datetime import timedelta + + +class EstateProperty(models.Model): + _name = "estate.property" + _description = "Real 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() + timedelta(days=90) + ) + + expected_price = fields.Float(required=True) + + selling_price = fields.Float( + readonly=True, + copy=False + ) + + bedrooms = fields.Integer(default=2) + living_area = fields.Integer("Living Area (sqm)") + facades = fields.Integer() + garage = fields.Boolean() + garden = fields.Boolean() + garden_area = fields.Integer("Garden Area (sqm)") + + garden_orientation = fields.Selection( + [ + ('north', 'North'), + ('south', 'South'), + ('east', 'East'), + ('west', 'West'), + ], + ) + + active = fields.Boolean(default=True) + + state = fields.Selection( + [ + ('new', 'New'), + ('offer_received', 'Offer Received'), + ('offer_accepted', 'Offer Accepted'), + ('sold', 'Sold'), + ('cancelled', 'Cancelled'), + ], + required=True, + copy=False, + default='new' + ) 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 diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml new file mode 100644 index 00000000000..6cc40b0f2da --- /dev/null +++ b/estate/views/estate_menus.xml @@ -0,0 +1,21 @@ + + + + + + + + + + + + + diff --git a/estate/views/state_property_views.xml b/estate/views/state_property_views.xml new file mode 100644 index 00000000000..1eec051bbd7 --- /dev/null +++ b/estate/views/state_property_views.xml @@ -0,0 +1,11 @@ + + + + + + Properties + estate.property + list,form + + + diff --git a/settings.json b/settings.json new file mode 100644 index 00000000000..efb1648f8e2 --- /dev/null +++ b/settings.json @@ -0,0 +1,7 @@ +{ + "python.linting.flake8Enabled": true, + "python.linting.enabled": true, + "python.linting.flake8Args": [ + "--ignore=E501,E301,E302" + ] +} From d9470de9972a3ad48559df9207168ed3b3e671cd Mon Sep 17 00:00:00 2001 From: jupao-odoo Date: Tue, 17 Feb 2026 11:13:26 +0100 Subject: [PATCH 3/6] [ADD] estate: define list, form and search views for properties Add custom views for the `estate.property` model to improve user interface and data interaction. The commit introduces: - A list view showing key fields: name, postcode, bedrooms, living area, expected price, selling price, and available date. - A form view with structured groups and a notebook for detailed information including description, property features, and reserved fields like active and state. - A search view with filters and group-by options to easily find and categorize properties, including an "Available" filter for new and offer_received properties, and a group-by on postcode. This improves the usability of the real estate module, replacing default auto-generated views with tailored ones that match business requirements. Closes task-6 --- estate/__manifest__.py | 2 +- estate/models/estate_property.py | 2 +- estate/views/estate_menus.xml | 2 +- estate/views/estate_property_views.xml | 114 +++++++++++++++++++++++++ estate/views/state_property_views.xml | 11 --- 5 files changed, 117 insertions(+), 14 deletions(-) create mode 100644 estate/views/estate_property_views.xml delete mode 100644 estate/views/state_property_views.xml diff --git a/estate/__manifest__.py b/estate/__manifest__.py index 915e38ce961..c17cbfdb6fe 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -20,7 +20,7 @@ 'data': [ 'security/ir.model.access.csv', - 'views/state_property_views.xml', + 'views/estate_property_views.xml', 'views/estate_menus.xml' ], diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index 5d40d94d069..edb6541dbc2 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -10,7 +10,7 @@ class EstateProperty(models.Model): description = fields.Text() postcode = fields.Char() - date_availability = fields.Date( + available_from = fields.Date( copy=False, default=lambda self: fields.Date.today() + timedelta(days=90) ) diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml index 6cc40b0f2da..4dd506fd2f2 100644 --- a/estate/views/estate_menus.xml +++ b/estate/views/estate_menus.xml @@ -1,4 +1,4 @@ - + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml new file mode 100644 index 00000000000..b9e2f8c8a09 --- /dev/null +++ b/estate/views/estate_property_views.xml @@ -0,0 +1,114 @@ + + + + + + Properties + estate.property + list,form + + + + + + estate.property.list + estate.property + + + + + + + + + + + + + + + + estate.property.form + estate.property + +
+ + +
+

+ +

+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+
+
+
+ + + estate.property.search + estate.property + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
diff --git a/estate/views/state_property_views.xml b/estate/views/state_property_views.xml deleted file mode 100644 index 1eec051bbd7..00000000000 --- a/estate/views/state_property_views.xml +++ /dev/null @@ -1,11 +0,0 @@ - - - - - - Properties - estate.property - list,form - - - From 2710636a4018634161a11988e48916c75b813234 Mon Sep 17 00:00:00 2001 From: jupao-odoo Date: Tue, 17 Feb 2026 15:08:16 +0100 Subject: [PATCH 4/6] [IMP] estate: add property types, tags and offers models Introduce estate.property.type and estate.property.tag models. Extend estate.property with: - Many2one relation to property type - Many2many relation to property tags - One2many relation to property offers Add corresponding views and menus where required. Fix flake8 issues to satisfy ci/style checks. --- .flake8 | 4 ++ awesome_dashboard/__init__.py | 1 - awesome_dashboard/controllers/__init__.py | 2 +- awesome_dashboard/controllers/controllers.py | 1 - awesome_gallery/models/ir_action.py | 2 +- awesome_owl/__init__.py | 2 +- awesome_owl/controllers/__init__.py | 2 +- estate/__manifest__.py | 1 + estate/models/__init__.py | 5 ++- estate/models/estate_property.py | 27 ++++++++++++ estate/models/estate_property_offer.py | 24 +++++++++++ estate/models/estate_property_tag.py | 8 ++++ estate/models/estate_property_type.py | 8 ++++ estate/security/ir.model.access.csv | 5 ++- estate/views/estate_menus.xml | 38 +++++++++++++++++ estate/views/estate_property_offer_views.xml | 33 ++++++++++++++ estate/views/estate_property_views.xml | 45 ++++++++++++-------- 17 files changed, 183 insertions(+), 25 deletions(-) create mode 100644 .flake8 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_offer_views.xml diff --git a/.flake8 b/.flake8 new file mode 100644 index 00000000000..20e70b32c5c --- /dev/null +++ b/.flake8 @@ -0,0 +1,4 @@ +[flake8] +ignore = E501,E301,E302,F401 +max-line-length = 88 +exclude = __pycache__,.git \ No newline at end of file diff --git a/awesome_dashboard/__init__.py b/awesome_dashboard/__init__.py index b0f26a9a602..153a9e31ebb 100644 --- a/awesome_dashboard/__init__.py +++ b/awesome_dashboard/__init__.py @@ -1,3 +1,2 @@ # -*- coding: utf-8 -*- - from . import controllers diff --git a/awesome_dashboard/controllers/__init__.py b/awesome_dashboard/controllers/__init__.py index 457bae27e11..b0f26a9a602 100644 --- a/awesome_dashboard/controllers/__init__.py +++ b/awesome_dashboard/controllers/__init__.py @@ -1,3 +1,3 @@ # -*- coding: utf-8 -*- -from . import controllers \ No newline at end of file +from . import controllers diff --git a/awesome_dashboard/controllers/controllers.py b/awesome_dashboard/controllers/controllers.py index 05977d3bd7f..c46cfc63bfa 100644 --- a/awesome_dashboard/controllers/controllers.py +++ b/awesome_dashboard/controllers/controllers.py @@ -33,4 +33,3 @@ def get_statistics(self): }, 'total_amount': random.randint(100, 1000) } - diff --git a/awesome_gallery/models/ir_action.py b/awesome_gallery/models/ir_action.py index eae20acbf5c..d796bc8beae 100644 --- a/awesome_gallery/models/ir_action.py +++ b/awesome_gallery/models/ir_action.py @@ -7,4 +7,4 @@ class ActWindowView(models.Model): view_mode = fields.Selection(selection_add=[ ('gallery', "Awesome Gallery") - ], ondelete={'gallery': 'cascade'}) + ], ondelete={'gallery': 'cascade'}) diff --git a/awesome_owl/__init__.py b/awesome_owl/__init__.py index 457bae27e11..b0f26a9a602 100644 --- a/awesome_owl/__init__.py +++ b/awesome_owl/__init__.py @@ -1,3 +1,3 @@ # -*- coding: utf-8 -*- -from . import controllers \ No newline at end of file +from . import controllers diff --git a/awesome_owl/controllers/__init__.py b/awesome_owl/controllers/__init__.py index 457bae27e11..b0f26a9a602 100644 --- a/awesome_owl/controllers/__init__.py +++ b/awesome_owl/controllers/__init__.py @@ -1,3 +1,3 @@ # -*- coding: utf-8 -*- -from . import controllers \ No newline at end of file +from . import controllers diff --git a/estate/__manifest__.py b/estate/__manifest__.py index c17cbfdb6fe..a61ff76420b 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -20,6 +20,7 @@ 'data': [ 'security/ir.model.access.csv', + 'views/estate_property_offer_views.xml', 'views/estate_property_views.xml', 'views/estate_menus.xml' ], diff --git a/estate/models/__init__.py b/estate/models/__init__.py index f4c8fd6db6d..2a1d3a13388 100644 --- a/estate/models/__init__.py +++ b/estate/models/__init__.py @@ -1 +1,4 @@ -from . import estate_property \ No newline at end of file +from . import estate_property_type +from . import estate_property_tag +from . import estate_property +from . import estate_property_offer diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index edb6541dbc2..ad734658cf2 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -52,3 +52,30 @@ class EstateProperty(models.Model): copy=False, default='new' ) + # Many2one links + property_type_id = fields.Many2one( + 'estate.property.type', + string='Property Type' + ) + buyer_id = fields.Many2one( + 'res.partner', + string='Buyer', + copy=False + ) + salesperson_id = fields.Many2one( + 'res.users', + string='Salesperson', + default=lambda self: self.env.user + ) + # Many2many links + tag_ids = fields.Many2many( + 'estate.property.tag', + string="Tags" + ) + + # One2many links + offer_ids = fields.One2many( + '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..44a90e2e97f --- /dev/null +++ b/estate/models/estate_property_offer.py @@ -0,0 +1,24 @@ +from odoo import models, fields + + +class EstatePropertyOffer(models.Model): + _name = "estate.property.offer" + _description = "Property Offer" + + price = fields.Float(required=True) + status = fields.Selection( + [('accepted', 'Accepted'), ('refused', 'Refused')], + string="Status", + copy=False + ) + partner_id = fields.Many2one( + 'res.partner', + string="Buyer", + required=True + ) + property_id = fields.Many2one( + 'estate.property', + string="Property", + required=True, + ondelete='cascade' + ) diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py new file mode 100644 index 00000000000..7d6f197c1eb --- /dev/null +++ b/estate/models/estate_property_tag.py @@ -0,0 +1,8 @@ +from odoo import models, fields + + +class EstatePropertyTag(models.Model): + _name = "estate.property.tag" + _description = "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..ad121b33163 --- /dev/null +++ b/estate/models/estate_property_type.py @@ -0,0 +1,8 @@ +from odoo import models, fields + + +class EstatePropertyType(models.Model): + _name = "estate.property.type" + _description = "Real 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 0e11f47e58d..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 \ No newline at end of file +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 4dd506fd2f2..151ff098ebe 100644 --- a/estate/views/estate_menus.xml +++ b/estate/views/estate_menus.xml @@ -18,4 +18,42 @@ parent="estate_menu_properties" action="estate_property_action"/> + + + + + + + Property Type + estate.property.type + list,form + + + + + + + + + + + Property Tags + estate.property.tag + list,form + + +
+ diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml new file mode 100644 index 00000000000..64327ab0023 --- /dev/null +++ b/estate/views/estate_property_offer_views.xml @@ -0,0 +1,33 @@ + + + + + estate.property.offer.form + estate.property.offer + +
+ + + + + + + +
+
+
+ + + + estate.property.offer.list + estate.property.offer + + + + + + + + + +
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index b9e2f8c8a09..806b2d387ce 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -15,13 +15,15 @@ estate.property - - - - - - - + + + + + + + + + @@ -42,6 +44,7 @@ + @@ -66,6 +69,21 @@ + + + + + + + + + + + + + + + @@ -94,6 +112,9 @@ name="group_by_postcode" context="{'group_by':'postcode'}"/> + + + @@ -101,14 +122,4 @@ - - - - - - - - - - From d3932c074f987ce1a04852da9a7cf68f39f96c94 Mon Sep 17 00:00:00 2001 From: jupao-odoo Date: Tue, 17 Feb 2026 16:44:52 +0100 Subject: [PATCH 5/6] [IMP] estate: chapter 8 - computed fields and onchange - estate.property: * added total_area computed field (living_area + garden_area) * added best_price computed field (maximum of offer prices) * added @onchange on 'garden' to set default garden_area=10 and garden_orientation='north' - estate.property.offer: * added validity_date computed field (with inverse) --- .gitignore | 3 ++ estate/__manifest__.py | 1 - estate/models/estate_property.py | 41 +++++++++++++++-- estate/models/estate_property_offer.py | 25 ++++++++++- estate/views/estate_menus.xml | 47 +++++--------------- estate/views/estate_property_offer_views.xml | 2 + estate/views/estate_property_views.xml | 6 ++- 7 files changed, 82 insertions(+), 43 deletions(-) diff --git a/.gitignore b/.gitignore index b6e47617de1..e64747d9b18 100644 --- a/.gitignore +++ b/.gitignore @@ -127,3 +127,6 @@ dmypy.json # Pyre type checker .pyre/ + +.flake8 +settings.json diff --git a/estate/__manifest__.py b/estate/__manifest__.py index a61ff76420b..03b06879b35 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -1,4 +1,3 @@ -# -*- coding: utf-8 -*- { 'name': "Real Estate", diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index ad734658cf2..93255ce963b 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -1,5 +1,5 @@ -from odoo import models, fields -from datetime import timedelta +from odoo import models, fields, api +from dateutil.relativedelta import relativedelta class EstateProperty(models.Model): @@ -12,7 +12,7 @@ class EstateProperty(models.Model): available_from = fields.Date( copy=False, - default=lambda self: fields.Date.today() + timedelta(days=90) + default=lambda self: fields.Date.today() + relativedelta(months=3) ) expected_price = fields.Float(required=True) @@ -38,6 +38,15 @@ class EstateProperty(models.Model): ], ) + @api.onchange('garden') + def _onchange_garden(self): + if self.garden: + self.garden_area = 10 + self.garden_orientation = 'north' + else: + self.garden_area = 0 + self.garden_orientation = False + active = fields.Boolean(default=True) state = fields.Selection( @@ -79,3 +88,29 @@ class EstateProperty(models.Model): 'property_id', string="Offers" ) + + # Computed Fields + total_area = fields.Float( + string="Total Area (sqm)", + compute="_compute_total_area", + store=True + ) + + @api.depends('living_area', 'garden_area') + def _compute_total_area(self): + for record in self: + record.total_area = record.living_area + record.garden_area + + best_offer = fields.Float( + string="Best Offer", + compute="_compute_best_offer", + store=True + ) + + @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.0 diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py index 44a90e2e97f..ca926cd8102 100644 --- a/estate/models/estate_property_offer.py +++ b/estate/models/estate_property_offer.py @@ -1,4 +1,5 @@ -from odoo import models, fields +from odoo import models, fields, api +from datetime import timedelta class EstatePropertyOffer(models.Model): @@ -22,3 +23,25 @@ class EstatePropertyOffer(models.Model): required=True, ondelete='cascade' ) + + validity = fields.Integer( + string="Validity (days)", + default=7 + ) + + date_deadline = fields.Date( + string="Deadline", + compute="_compute_date_deadline", + inverse="_inverse_date_deadline", + store=True + ) + @api.depends('validity', 'create_date') + def _compute_date_deadline(self): + for record in self: + create_date = record.create_date.date() if record.create_date else fields.Date.today() + record.date_deadline = create_date + timedelta(days=record.validity) + + def _inverse_date_deadline(self): + for record in self: + create_date = record.create_date.date() if record.create_date else fields.Date.today() + record.validity = (record.date_deadline - create_date).days diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml index 151ff098ebe..48f0387f95d 100644 --- a/estate/views/estate_menus.xml +++ b/estate/views/estate_menus.xml @@ -2,28 +2,21 @@ - + - - + + + - - + + + + + + - - + @@ -32,28 +25,10 @@ list,form - - - - - - Property Tags estate.property.tag list,form - - - diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml index 64327ab0023..3adbd536c90 100644 --- a/estate/views/estate_property_offer_views.xml +++ b/estate/views/estate_property_offer_views.xml @@ -10,6 +10,8 @@ + + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 806b2d387ce..d76410daaa1 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -50,6 +50,7 @@ + @@ -65,8 +66,7 @@ - - + @@ -74,6 +74,8 @@ + + From ac9f85e0d629393d3d2c025bcac0d063dbc99b2e Mon Sep 17 00:00:00 2001 From: jupao-odoo Date: Tue, 17 Feb 2026 17:23:37 +0100 Subject: [PATCH 6/6] [REF] estate: add action buttons for properties and offers Chapter 9: link business logic to UI actions - estate.property: * Add "Cancel" and "Sold" buttons in form view header. * Enforce state constraints: a cancelled property cannot be sold and a sold property cannot be cancelled. * Raise UserError when state transitions are invalid. - estate.property.offer: * Add "Accept" and "Refuse" buttons. * Accepting an offer sets the buyer and selling price on the related property. * Only one offer can be accepted per property. - Updated views to include buttons in header sections. - Added Python methods implementing the business logic for these actions. --- estate/models/estate_property.py | 15 +++++++++++++++ estate/models/estate_property_offer.py | 18 ++++++++++++++++++ estate/views/estate_property_offer_views.xml | 4 ++++ estate/views/estate_property_views.xml | 6 ++++++ 4 files changed, 43 insertions(+) diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index 93255ce963b..00bfcee8952 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -1,4 +1,5 @@ from odoo import models, fields, api +from odoo.exceptions import UserError from dateutil.relativedelta import relativedelta @@ -114,3 +115,17 @@ def _compute_best_offer(self): record.best_offer = max(record.offer_ids.mapped('price')) else: record.best_offer = 0.0 + + def action_cancel(self): + for record in self: + if record.state == 'sold': + raise UserError("Sold properties cannot be cancelled.") + record.state = 'cancelled' + return True + + def action_sold(self): + for record in self: + if record.state == 'cancelled': + raise UserError("Cancelled properties cannot be sold.") + record.state = 'sold' + return True diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py index ca926cd8102..d8b7225d367 100644 --- a/estate/models/estate_property_offer.py +++ b/estate/models/estate_property_offer.py @@ -1,4 +1,5 @@ from odoo import models, fields, api +from odoo.exceptions import UserError from datetime import timedelta @@ -45,3 +46,20 @@ def _inverse_date_deadline(self): for record in self: create_date = record.create_date.date() if record.create_date else fields.Date.today() record.validity = (record.date_deadline - create_date).days + + def action_accept(self): + for offer in self: + property = offer.property_id + # Check if another offer was already accepted + if property.offer_ids.filtered(lambda o: o.status == 'accepted'): + raise UserError("Only one offer can be accepted per property.") + offer.status = 'accepted' + property.selling_price = offer.price + property.buyer_id = offer.partner_id + property.state = 'offer_accepted' + return True + + def action_refuse(self): + for offer in self: + offer.status = 'refused' + return True diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml index 3adbd536c90..4d9402cc0fd 100644 --- a/estate/views/estate_property_offer_views.xml +++ b/estate/views/estate_property_offer_views.xml @@ -6,6 +6,10 @@ estate.property.offer
+
+
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index d76410daaa1..92e1c2bf3a0 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -35,6 +35,10 @@ +
+

@@ -76,6 +80,8 @@ +