From a3aebbe98bb8599a465c5e1dbd65de75a4a543ae Mon Sep 17 00:00:00 2001 From: farah Date: Thu, 18 Dec 2025 17:20:04 +0100 Subject: [PATCH 1/3] Created estate module with property model and view --- estate/__init__.py | 2 + estate/__manifest__.py | 22 +++++++ estate/models/__init__.py | 3 + estate/models/estate_property.py | 49 +++++++++++++++ estate/security/ir.model.access.csv | 2 + estate/views/estate_menus.xml | 8 +++ estate/views/estate_property_views.xml | 83 ++++++++++++++++++++++++++ 7 files changed, 169 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/estate_property_views.xml diff --git a/estate/__init__.py b/estate/__init__.py new file mode 100644 index 00000000000..24c19d687df --- /dev/null +++ b/estate/__init__.py @@ -0,0 +1,2 @@ + +from . import models \ No newline at end of file diff --git a/estate/__manifest__.py b/estate/__manifest__.py new file mode 100644 index 00000000000..9f847e608c9 --- /dev/null +++ b/estate/__manifest__.py @@ -0,0 +1,22 @@ +{ + 'name': 'Estate', + 'version': '1.0', + 'category': 'Sales', + 'sequence': 1, + 'summary': 'Sell and bid on the hottest real estate properties.', + 'website': 'https://www.odoo.com/app/estate', + 'depends': [ + 'base_setup', + 'mail', + 'calendar', + 'contacts', + 'phone_validation', + ], + 'data': [ + 'security/ir.model.access.csv', + 'views/estate_property_views.xml', + 'views/estate_menus.xml', + ], + 'installable': True, + 'application': True +} \ No newline at end of file diff --git a/estate/models/__init__.py b/estate/models/__init__.py new file mode 100644 index 00000000000..04b41fc345e --- /dev/null +++ b/estate/models/__init__.py @@ -0,0 +1,3 @@ + +from . import estate_property +#from . import estate_property_offer \ 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..cf0ba856001 --- /dev/null +++ b/estate/models/estate_property.py @@ -0,0 +1,49 @@ +from odoo import fields, models +from dateutil.relativedelta import relativedelta + +class EstateProperty(models.Model): + _name = 'estate.property' + _description = "Real Estate Property" + + name= fields.Char('Title', required = True, translate = True) + property_type = fields.Selection( + selection=[('office', 'Office'), ('apartment', 'Apartment'), ('house', 'House'), ('studio', 'Studio')], + string='Property Type', + ) + postcode= fields.Char('Postcode', required=True) + availability = fields.Date( + 'Available From', + required=True, + copy = False, + default= lambda self: fields.Date.today()+ relativedelta(months=3), + ) + description= fields.Text('Description') + bedrooms= fields.Integer('Bedrooms', required=True, default= 2) + living_area= fields.Integer('Living Area (sqm)', required=True) + currency_id = fields.Many2one('res.currency', string='Currency', default=lambda self: self.env.ref('base.EUR')) + expected_price= fields.Monetary('Expected Price', required = True, currency_field='currency_id') + selling_price= fields.Monetary('Selling Price', currency_field='currency_id', readonly=True, copy=False) + # best_offer_id = fields.Many2one('estate.property.offer', string='Best Offer', readonly=True) + facades= fields.Integer('Facades', default=False) + garage= fields.Boolean('Garage', default=False) + garden= fields.Boolean('Garden', default=False) + garden_area= fields.Integer('Garden Area (sqm)', required=False) + garden_orientation= fields.Selection( + selection=[('north', 'North'), ('south', 'South'), ('east', 'East'), ('west', 'West')], + string='Garden Orientation' + ) + total_area= fields.Integer('Total Area (sqm)') + + state = fields.Selection( + string = 'Status', + selection =[('new', 'New'), ('offer_received', 'Offer Received'), ('offer_accepted', 'Offer Accepted'), + ('sold', 'Sold'), ('canceled', 'Canceled')], + required= True, + copy= False, + default= 'new', + ) + active = fields.Boolean('Active', default= True) + # _check_expected_price = models.Constraint( + # 'CHECK(expected_price) >= 0', + # "The expected price can't be negative", + # ) \ No newline at end of file diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv new file mode 100644 index 00000000000..0e525a41a26 --- /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,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..16a09cfc70e --- /dev/null +++ b/estate/views/estate_menus.xml @@ -0,0 +1,8 @@ + + + + + + + + \ 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..14f0ded27c6 --- /dev/null +++ b/estate/views/estate_property_views.xml @@ -0,0 +1,83 @@ + + + + + estate.property.list.view + estate.property + + + + + + + + + + + + + + + estate.property.form.view + estate.property + +
+ +
+
+

+ +

+
+
+ + + + + + + + + + + + + + + + + + + + + +
+ + + + + Property + estate.property + list,form + + + + estate.property.search + estate.property + + + + + + + + + + + + + + + + + \ No newline at end of file From 19206b1c231b79522c1f5294e803809813433554 Mon Sep 17 00:00:00 2001 From: farah Date: Fri, 19 Dec 2025 15:03:09 +0100 Subject: [PATCH 2/3] [ADD] estate_property_type: model and corresponding views (framework 101) --- estate/__init__.py | 2 +- estate/__manifest__.py | 9 +--- estate/models/__init__.py | 2 +- estate/models/estate_property.py | 61 ++++++++++++------------ estate/models/estate_property_type.py | 11 +++++ estate/security/ir.model.access.csv | 3 +- estate/views/estate_menus.xml | 9 ++-- estate/views/estate_property_views.xml | 65 ++++++++++++++++---------- 8 files changed, 95 insertions(+), 67 deletions(-) create mode 100644 estate/models/estate_property_type.py diff --git a/estate/__init__.py b/estate/__init__.py index 24c19d687df..a9e3372262c 100644 --- a/estate/__init__.py +++ b/estate/__init__.py @@ -1,2 +1,2 @@ -from . import models \ No newline at end of file +from . import models diff --git a/estate/__manifest__.py b/estate/__manifest__.py index 9f847e608c9..e5838f68794 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -1,22 +1,17 @@ { 'name': 'Estate', - 'version': '1.0', 'category': 'Sales', 'sequence': 1, 'summary': 'Sell and bid on the hottest real estate properties.', 'website': 'https://www.odoo.com/app/estate', 'depends': [ 'base_setup', - 'mail', - 'calendar', - 'contacts', - 'phone_validation', + 'web', ], 'data': [ 'security/ir.model.access.csv', 'views/estate_property_views.xml', 'views/estate_menus.xml', ], - 'installable': True, 'application': True -} \ No newline at end of file +} diff --git a/estate/models/__init__.py b/estate/models/__init__.py index 04b41fc345e..a544207d8f5 100644 --- a/estate/models/__init__.py +++ b/estate/models/__init__.py @@ -1,3 +1,3 @@ from . import estate_property -#from . import estate_property_offer \ No newline at end of file +from . import estate_property_type diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index cf0ba856001..8c06012015f 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -2,48 +2,51 @@ from dateutil.relativedelta import relativedelta class EstateProperty(models.Model): - _name = 'estate.property' - _description = "Real Estate Property" + _name='estate.property' + _description="Real Estate Property" - name= fields.Char('Title', required = True, translate = True) - property_type = fields.Selection( - selection=[('office', 'Office'), ('apartment', 'Apartment'), ('house', 'House'), ('studio', 'Studio')], + name=fields.Char('Title', required=True, translate=True) + property_type_id=fields.Many2one( + 'estate.property.type', string='Property Type', ) - postcode= fields.Char('Postcode', required=True) - availability = fields.Date( + postcode=fields.Char('Postcode', required=True) + availability =fields.Date( 'Available From', required=True, - copy = False, - default= lambda self: fields.Date.today()+ relativedelta(months=3), - ) - description= fields.Text('Description') - bedrooms= fields.Integer('Bedrooms', required=True, default= 2) - living_area= fields.Integer('Living Area (sqm)', required=True) - currency_id = fields.Many2one('res.currency', string='Currency', default=lambda self: self.env.ref('base.EUR')) - expected_price= fields.Monetary('Expected Price', required = True, currency_field='currency_id') - selling_price= fields.Monetary('Selling Price', currency_field='currency_id', readonly=True, copy=False) + copy=False, + default=lambda self: fields.Date.today()+relativedelta(months=3), + ) + description=fields.Text('Description') + bedrooms=fields.Integer('Bedrooms', required=True, default=2) + living_area=fields.Integer('Living Area (sqm)', required=True) + currency_id=fields.Many2one('res.currency', string='Currency', default=lambda self: self.env.company.currency_id.id) + expected_price=fields.Monetary('Expected Price', required=True) + selling_price=fields.Monetary('Selling Price', readonly=True, copy=False) # best_offer_id = fields.Many2one('estate.property.offer', string='Best Offer', readonly=True) - facades= fields.Integer('Facades', default=False) - garage= fields.Boolean('Garage', default=False) - garden= fields.Boolean('Garden', default=False) - garden_area= fields.Integer('Garden Area (sqm)', required=False) - garden_orientation= fields.Selection( + facades=fields.Integer('Facades', default=False) + garage=fields.Boolean('Garage', default=False) + garden=fields.Boolean('Garden', default=False) + garden_area=fields.Integer('Garden Area (sqm)', required=False) + garden_orientation=fields.Selection( selection=[('north', 'North'), ('south', 'South'), ('east', 'East'), ('west', 'West')], string='Garden Orientation' ) - total_area= fields.Integer('Total Area (sqm)') + total_area=fields.Integer('Total Area (sqm)') - state = fields.Selection( - string = 'Status', + state=fields.Selection( + string ='Status', selection =[('new', 'New'), ('offer_received', 'Offer Received'), ('offer_accepted', 'Offer Accepted'), ('sold', 'Sold'), ('canceled', 'Canceled')], - required= True, - copy= False, - default= 'new', + required=True, + copy=False, + default='new', ) - active = fields.Boolean('Active', default= True) + active =fields.Boolean('Active', default=True) + buyer_id =fields.Many2one('res.partner', string='Buyer', copy=False) + seller_id=fields.Many2one('res.users', string='Salesperson', default=lambda self:self.env.user) + # _check_expected_price = models.Constraint( # 'CHECK(expected_price) >= 0', # "The expected price can't be negative", - # ) \ No newline at end of file + # ) diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py new file mode 100644 index 00000000000..9f457a5121d --- /dev/null +++ b/estate/models/estate_property_type.py @@ -0,0 +1,11 @@ +from odoo import fields, models +from dateutil.relativedelta import relativedelta + +class EstateProperty(models.Model): + _name = 'estate.property.type' + _description = "Real Estate Property Type" + + name=fields.Char( + "Name", + required=True, + ) diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv index 0e525a41a26..622378709a1 100644 --- a/estate/security/ir.model.access.csv +++ b/estate/security/ir.model.access.csv @@ -1,2 +1,3 @@ id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink -access_estate_property,estate.property,model_estate_property,base.group_user,1,1,1,1 \ No newline at end of file +access_estate_property,estate.property,model_estate_property,base.group_user,1,1,1,1 +access_estate_property_type,estate.property.type,model_estate_property_type,base.group_user,1,1,1,1 diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml index 16a09cfc70e..aa1c7bda34e 100644 --- a/estate/views/estate_menus.xml +++ b/estate/views/estate_menus.xml @@ -1,8 +1,11 @@ - - + + + + + - \ No newline at end of file + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 14f0ded27c6..6244436f750 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -2,17 +2,17 @@ - estate.property.list.view - estate.property - - - - - - - - - + estate.property.list.view + estate.property + + + + + + + + + @@ -20,7 +20,7 @@ estate.property.form.view estate.property - +
@@ -31,22 +31,29 @@
- - - - + + + + + - - - - - - - - + + + + + + + + + + + + + + @@ -72,12 +79,20 @@ + + - \ No newline at end of file + + + Property Types + estate.property.type + list + + From 34e81d61f96f73785b3a1c28c92b4daf5bd14bac Mon Sep 17 00:00:00 2001 From: farah Date: Fri, 19 Dec 2025 16:39:13 +0100 Subject: [PATCH 3/3] trying to pass runbot checks --- estate/models/estate_property.py | 65 +++++++++++++-------------- estate/models/estate_property_type.py | 4 +- ruff.toml | 44 ++++++++++++++++++ 3 files changed, 77 insertions(+), 36 deletions(-) create mode 100644 ruff.toml diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index 8c06012015f..d2b48001de5 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -1,50 +1,47 @@ from odoo import fields, models from dateutil.relativedelta import relativedelta + class EstateProperty(models.Model): - _name='estate.property' - _description="Real Estate Property" + _name = 'estate.property' + _description = "Real Estate Property" - name=fields.Char('Title', required=True, translate=True) - property_type_id=fields.Many2one( + name = fields.Char("Title", required=True, translate=True) + property_type_id = fields.Many2one( 'estate.property.type', - string='Property Type', - ) - postcode=fields.Char('Postcode', required=True) - availability =fields.Date( - 'Available From', - required=True, - copy=False, - default=lambda self: fields.Date.today()+relativedelta(months=3), + string="Property Type", ) - description=fields.Text('Description') - bedrooms=fields.Integer('Bedrooms', required=True, default=2) - living_area=fields.Integer('Living Area (sqm)', required=True) - currency_id=fields.Many2one('res.currency', string='Currency', default=lambda self: self.env.company.currency_id.id) - expected_price=fields.Monetary('Expected Price', required=True) - selling_price=fields.Monetary('Selling Price', readonly=True, copy=False) - # best_offer_id = fields.Many2one('estate.property.offer', string='Best Offer', readonly=True) - facades=fields.Integer('Facades', default=False) - garage=fields.Boolean('Garage', default=False) - garden=fields.Boolean('Garden', default=False) - garden_area=fields.Integer('Garden Area (sqm)', required=False) - garden_orientation=fields.Selection( - selection=[('north', 'North'), ('south', 'South'), ('east', 'East'), ('west', 'West')], - string='Garden Orientation' + postcode = fields.Char("Postcode", required=True) + availability = fields.Date( + "Available From", + required=True, + copy=False, + default=lambda self: fields.Date.today() + relativedelta(months=3), ) - total_area=fields.Integer('Total Area (sqm)') + description = fields.Text("Description") + bedrooms = fields.Integer("Bedrooms", required=True, default=2) + living_area = fields.Integer("Living Area (sqm)", required=True) + currency_id = fields.Many2one('res.currency', string="Currency", default=lambda self: self.env.company.currency_id.id) + expected_price = fields.Monetary("Expected Price", required=True) + selling_price = fields.Monetary("Selling Price", readonly=True, copy=False) + # best_offer_id = fields.Many2one('estate.property.offer', string='Best Offer', readonly=True) + facades = fields.Integer("Facades", default=False) + garage = fields.Boolean("Garage", default=False) + garden = fields.Boolean("Garden", default=False) + garden_area = fields.Integer("Garden Area (sqm)", required=False) + garden_orientation = fields.Selection(selection=[('north', "North"), ('south', "South"), ('east', "East"), ('west', "West")], string="Garden Orientation") + total_area = fields.Integer("Total Area (sqm)") - state=fields.Selection( - string ='Status', - selection =[('new', 'New'), ('offer_received', 'Offer Received'), ('offer_accepted', 'Offer Accepted'), - ('sold', 'Sold'), ('canceled', 'Canceled')], + state = fields.Selection( + string="Status", + selection=[('new', "New"), ('offer_received', "Offer Received"), ('offer_accepted', "Offer Accepted"), ('sold', "Sold"), ('canceled', "Canceled")], required=True, copy=False, default='new', ) - active =fields.Boolean('Active', default=True) - buyer_id =fields.Many2one('res.partner', string='Buyer', copy=False) - seller_id=fields.Many2one('res.users', string='Salesperson', default=lambda self:self.env.user) + active = fields.Boolean("Active", default=True) + buyer_id = fields.Many2one('res.partner', string="Buyer", copy=False) + seller_id = fields.Many2one('res.users', string="Salesperson", default=lambda self: self.env.user) # _check_expected_price = models.Constraint( # 'CHECK(expected_price) >= 0', diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py index 9f457a5121d..77bc4eb6ad0 100644 --- a/estate/models/estate_property_type.py +++ b/estate/models/estate_property_type.py @@ -1,11 +1,11 @@ from odoo import fields, models -from dateutil.relativedelta import relativedelta + class EstateProperty(models.Model): _name = 'estate.property.type' _description = "Real Estate Property Type" - name=fields.Char( + name = fields.Char( "Name", required=True, ) diff --git a/ruff.toml b/ruff.toml new file mode 100644 index 00000000000..e84deba95cb --- /dev/null +++ b/ruff.toml @@ -0,0 +1,44 @@ +# Exclude a variety of commonly ignored directories. +exclude = [ + ".bzr", + ".direnv", + ".eggs", + ".git", + ".git-rewrite", + ".hg", + ".ipynb_checkpoints", + ".mypy_cache", + ".nox", + ".pants.d", + ".pyenv", + ".pytest_cache", + ".pytype", + ".ruff_cache", + ".svn", + ".tox", + ".venv", + ".vscode", + "__pypackages__", + "_build", + "buck-out", + "build", + "dist", + "node_modules", + "site-packages", + "venv", +] + +# Assume Python 3.12 +target-version = "py312" + +line-length = 255 +[lint] +# Enable Pyflakes (`F`) and a subset of the pycodestyle (`E`) codes by default. +select = ["ALL"] +ignore = ["A","ARG","ANN","B","C901","D","DTZ","DOC","E501","ERA001","FBT","N","PD","PERF","PIE790","PLR","PT","Q","RSE102","RUF001","RUF012","S","SIM102","SIM108","SLF001","TID252","UP031","TRY003","TRY300","E713","SIM117","PGH003","RUF005","FIX","TD","TRY400","C408","PLW2901","PTH","EM102","INP001","CPY001","E266","PIE808","PLC2701","RUF100","FA100","FURB","C420","COM812","TRY002","B904","EM101","I001","UP006","UP007","RET","RUF021","E741","FAST","ASYNC","AIR","DJ","NPY","FA102","F401"] + +# Allow fix for all enabled rules (when `--fix`) is provided. +fixable = ["ALL"] +unfixable = ["A","ARG","ANN","B","C901","D","DTZ","DOC","E501","ERA001","FBT","N","PD","PERF","PIE790","PLR","PT","Q","RSE102","RUF001","RUF012","S","SIM102","SIM108","SLF001","TID252","UP031","TRY003","TRY300","E713","SIM117","PGH003","RUF005","FIX","TD","TRY400","C408","PLW2901","PTH","EM102","INP001","CPY001","E266","PIE808","PLC2701","RUF100","FA100","FURB","C420","COM812","TRY002","B904","EM101","I001","UP006","UP007","RET","RUF021","E741","FAST","ASYNC","AIR","DJ","NPY","FA102","F401"] +[format] +quote-style = "preserve"