From f3b1efd89a0c5fa3f453535efc941b65b905731e Mon Sep 17 00:00:00 2001 From: niyp-odoo Date: Wed, 2 Jul 2025 17:42:54 +0530 Subject: [PATCH 01/15] [ADD] Estate module/app added (Chapter2 Tasks done) --- estate/__init__.py | 0 estate/__manifest__.py | 6 ++++++ 2 files changed, 6 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..95f73109275 --- /dev/null +++ b/estate/__manifest__.py @@ -0,0 +1,6 @@ +{ + 'name': 'Estate', + 'installable': True, + 'application': True, + 'auto_install': False +} \ No newline at end of file From cb3fa3c472bbac6d386fa5cca858d65c79bd10f0 Mon Sep 17 00:00:00 2001 From: niyp-odoo Date: Thu, 3 Jul 2025 10:53:42 +0530 Subject: [PATCH 02/15] [ADD] Estate : Created model estate_property and added fields in it (Chapter 3) --- estate/__init__.py | 1 + estate/models/__init__.py | 1 + estate/models/estate_property.py | 24 ++++++++++++++++++++++++ 3 files changed, 26 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..9a7e03eded3 100644 --- a/estate/__init__.py +++ b/estate/__init__.py @@ -0,0 +1 @@ +from . import models \ No newline at end of file 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..befdaed698f --- /dev/null +++ b/estate/models/estate_property.py @@ -0,0 +1,24 @@ +from odoo import fields,models + + +class EstateProperty(models.Model): + _name = "estate.property" + _description = "Estate Property is defined" + + name = fields.Char(required=True) + description = fields.Text() + postcode = fields.Char() + date_avaiblity = 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( + string = 'Type', + selection=[('north','North'), ('south','South'), ('east','East'), ('west','West')], + help = "Orientation is used to locate garden's direction" + ) \ No newline at end of file From 72ecf834a9bde8bb625425f7d58e4d24e9fcf1f4 Mon Sep 17 00:00:00 2001 From: niyp-odoo Date: Fri, 4 Jul 2025 10:06:27 +0530 Subject: [PATCH 03/15] [ADD] added security and enhanced UI --- estate/__manifest__.py | 8 ++- estate/models/estate_property.py | 20 ++++-- estate/security/ir.model.access.csv | 2 + estate/views/estate_menus.xml | 9 +++ estate/views/estate_property_views.xml | 95 ++++++++++++++++++++++++++ 5 files changed, 127 insertions(+), 7 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/__manifest__.py b/estate/__manifest__.py index 95f73109275..a3b414e742a 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -2,5 +2,11 @@ 'name': 'Estate', 'installable': True, 'application': True, - 'auto_install': False + 'auto_install': False, + 'data':[ + 'security/ir.model.access.csv', + 'views/estate_property_views.xml', + 'views/estate_menus.xml', + ], + 'license': 'LGPL-3', } \ No newline at end of file diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py index befdaed698f..83f3985896d 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -1,5 +1,6 @@ from odoo import fields,models - +from datetime import date +from dateutil.relativedelta import relativedelta class EstateProperty(models.Model): _name = "estate.property" @@ -8,17 +9,24 @@ class EstateProperty(models.Model): name = fields.Char(required=True) description = fields.Text() postcode = fields.Char() - date_avaiblity = fields.Date() + date_avaiblity = fields.Date(copy = False, default = 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() garden_orientation = fields.Selection( - string = 'Type', + string = 'Direction', selection=[('north','North'), ('south','South'), ('east','East'), ('west','West')], - help = "Orientation is used to locate garden's direction" + help = "This is used to locate garden's direction" + ) + active = fields.Boolean(default = True) + state = fields.Selection( + selection=[('new','New'), ('offer received','Offer Received'), ('offer acceptedt','Offer Accepted'), ('sold','Sold'), ('cancelled', 'Cancelled')], + default = 'new', + required = True, + copy = False, ) \ 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..ab63520e22b --- /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 +estate.access_estate_property,access_estate_property,estate.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..3abf7a00c5d --- /dev/null +++ b/estate/views/estate_menus.xml @@ -0,0 +1,9 @@ + + + + + + + + + \ No newline at end of file diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml new file mode 100644 index 00000000000..038a369cf61 --- /dev/null +++ b/estate/views/estate_property_views.xml @@ -0,0 +1,95 @@ + + + + Properties + estate.property + list,form + + + + estate.property + estate.property + + + + + + + + + + + + + + + + + estate.property.form + estate.property + +
+ +

+ +

+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+ + + + + + estate.property.search + estate.property + + + + + + + + + + + + + + + + + + + + + \ No newline at end of file From bba51c0dfe4302e7c87f0f5937f3d084d351d78a Mon Sep 17 00:00:00 2001 From: niyp-odoo Date: Mon, 7 Jul 2025 10:09:49 +0530 Subject: [PATCH 04/15] [ADD] Chapter 7,8,9,10 Done improved app and added functionalities --- estate/__manifest__.py | 3 + estate/models/__init__.py | 5 +- estate/models/estate_property.py | 70 +++++++++++- estate/models/estate_property_offer.py | 66 ++++++++++++ estate/models/estate_property_tag.py | 11 ++ estate/models/estate_property_type.py | 10 ++ estate/security/ir.model.access.csv | 5 +- estate/views/estate_menus.xml | 17 +-- estate/views/estate_property_tag_views.xml | 18 ++++ estate/views/estate_property_type_views.xml | 18 ++++ estate/views/estate_property_views.xml | 113 ++++++++++++++------ 11 files changed, 295 insertions(+), 41 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 a3b414e742a..50a1bc8539c 100644 --- a/estate/__manifest__.py +++ b/estate/__manifest__.py @@ -6,7 +6,10 @@ 'data':[ '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', + ], 'license': 'LGPL-3', } \ No newline at end of file diff --git a/estate/models/__init__.py b/estate/models/__init__.py index f4c8fd6db6d..09b2099fe84 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 +from . import estate_property_type +from . import estate_property_tag +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 index 83f3985896d..9d7ead8880f 100644 --- a/estate/models/estate_property.py +++ b/estate/models/estate_property.py @@ -1,6 +1,7 @@ -from odoo import fields,models +from odoo import fields,models, api from datetime import date from dateutil.relativedelta import relativedelta +from odoo.exceptions import UserError, ValidationError class EstateProperty(models.Model): _name = "estate.property" @@ -25,8 +26,71 @@ class EstateProperty(models.Model): ) active = fields.Boolean(default = True) state = fields.Selection( - selection=[('new','New'), ('offer received','Offer Received'), ('offer acceptedt','Offer Accepted'), ('sold','Sold'), ('cancelled', 'Cancelled')], + selection=[('new','New'), ('offer received','Offer Received'), ('offer accepted','Offer Accepted'), ('sold','Sold'), ('cancelled', 'Cancelled')], default = 'new', required = True, copy = False, - ) \ No newline at end of file + ) + property_type_id = fields.Many2one("estate.property.type", string="Property Type") + salesman_id = fields.Many2one('res.users', string='Salesman', index=True, default=lambda self: self.env.user) + buyer_id = fields.Many2one( + 'res.partner', + string='Buyer', + index=True, + tracking=True, + default=lambda self: self.env.user.partner_id.id +) + _sql_constraints = [ + ('check_expectep_price', 'CHECK(expected_price > 0 AND selling_price > 0)', + 'The Price must be positve.') + ] + + property_tag_ids = fields.Many2many("estate.property.tag") + offer_ids = fields.One2many("estate.property.offer", "property_id", string="Offers") + + total_area = fields.Float(compute="_compute_total") + + @api.depends("garden_area","living_area") + def _compute_total(self): + for record in self: + record.total_area = record.garden_area + record.living_area + + best_price = fields.Float(compute="_compute_best_price", string="Best Offer Price" ,readonly = True) + + @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 self.garden: + self.garden_area = 10 + self.garden_orientation = "north" + else: + self.garden_area = 0 + self.garden_orientation = False + + def action_cancel(self): + for record in self: + if record.state == 'sold': + raise UserError("Sold properties cannot be cancelled.") + record.state = 'cancelled' + + # SOLD button logic + def action_sold(self): + for record in self: + if record.state == 'cancelled': + raise UserError("Cancelled properties cannot be marked as sold.") + record.state = 'sold' + + + @api.constrains('selling_price') + def _check_price(self): + for record in self: + if record.selling_price >= record.expected_price *0.9: + raise ValidationError("The selling price cannot be lower than 90% of the expected price.") + \ No newline at end of file diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py new file mode 100644 index 00000000000..062467735fa --- /dev/null +++ b/estate/models/estate_property_offer.py @@ -0,0 +1,66 @@ +from datetime import timedelta +from odoo import api, fields, models +from odoo.exceptions import UserError + + +class EstatePropertyOffer(models.Model): + _name = "estate.property.offer" + _description = "Offers related to property are made" + + price = fields.Float() + status = fields.Selection( + selection=[('accepted', 'Accepted'), ('refused', 'Refuse')], + copy = False, + ) + partner_id = fields.Many2one( + "res.partner",string='Partner',index = True, default = lambda self: self.env.user.partner_id.id +) + property_id = fields.Many2one("estate.property",index = True, required = True) + validity = fields.Integer(default = 7) + date_deadline = fields.Date( + string="Deadline", + compute="_compute_date_deadline", + inverse="_inverse_date_deadline", + store=True + ) + + _sql_constraints = [ + ('check_price', 'CHECK(price > 0)', 'The offer price must be greater than 0') + ] + + @api.depends('create_date', 'validity') + def _compute_date_deadline(self): + for record in self: + # Use create_date if available, otherwise fallback to today + create_date = record.create_date or 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 or fields.Date.today() + if record.date_deadline: + record.validity = (record.date_deadline.day - create_date.day) + + def action_confirm(self): + for offer in self: + # Check if already accepted offer exists for the property + existing_offer = offer.property_id.offer_ids.filtered(lambda o: o.status == 'accepted' and o.id != offer.id) + if existing_offer: + raise UserError("Only one offer can be accepted per property.") + + # Set accepted + offer.status = 'accepted' + # Set buyer and selling price on the property + offer.property_id.buyer_id = offer.partner_id + offer.property_id.selling_price = offer.price + offer.property_id.state = 'offer accepted' + + + + def action_refuse(self): + for record in self: + record.status = 'refused' + + + + \ No newline at end of file diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py new file mode 100644 index 00000000000..46957f77a31 --- /dev/null +++ b/estate/models/estate_property_tag.py @@ -0,0 +1,11 @@ +from odoo import fields,models + +class EstatePropertyTag(models.Model): + _name = "estate.property.tag" + _description = "Tags of estate are defined" + + name = fields.Char(required = True) + + _sql_constraints = [ + ('check_unique_name', 'UNIQUE(name)', 'The name of the tag should be unique') + ] \ 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..02ebe0c5c61 --- /dev/null +++ b/estate/models/estate_property_type.py @@ -0,0 +1,10 @@ +from odoo import fields,models + +class EstatePropertyType(models.Model): + _name = "estate.property.type" + _description = "Type of estate is defined" + + name = fields.Char(required = True) + _sql_constraints = [ + ('check_unique_property_type', 'UNIQUE(name)', 'The Property type should be unique') + ] \ No newline at end of file diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv index ab63520e22b..0c0b62b7fee 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 -estate.access_estate_property,access_estate_property,estate.model_estate_property,base.group_user,1,1,1,1 \ No newline at end of file +estate.access_estate_property,access_estate_property,estate.model_estate_property,base.group_user,1,1,1,1 +estate.access_estate_property_type,access_estate_property_type,estate.model_estate_property_type,base.group_user,1,1,1,1 +estate.access_estate_property_tag,access_estate_property_tag,estate.model_estate_property_tag,base.group_user,1,1,1,1 +estate.access_estate_property_offer,access_estate_property_offer,estate.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 3abf7a00c5d..e56576d4cbd 100644 --- a/estate/views/estate_menus.xml +++ b/estate/views/estate_menus.xml @@ -1,9 +1,14 @@ - - - - - - \ No newline at end of file + + + + + + + + + + + diff --git a/estate/views/estate_property_tag_views.xml b/estate/views/estate_property_tag_views.xml new file mode 100644 index 00000000000..a1591e9f00a --- /dev/null +++ b/estate/views/estate_property_tag_views.xml @@ -0,0 +1,18 @@ + + + + Property Tags + estate.property.tag + list,form + + + + estate.property.tag.search + estate.property.tag + + + + + + + \ No newline at end of file diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml new file mode 100644 index 00000000000..e7bdf5db8c8 --- /dev/null +++ b/estate/views/estate_property_type_views.xml @@ -0,0 +1,18 @@ + + + + Property Types + estate.property.type + list,form + + + + estate.property.type.search + estate.property.type + + + + + + + \ No newline at end of file diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 038a369cf61..3e045dab5fd 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -1,16 +1,19 @@ + + - Properties - estate.property - list,form + Properties + estate.property + list,form + estate.property estate.property - + @@ -22,20 +25,42 @@ - - + estate.property.form estate.property
+
+
+

- +

- - - + + + + + + + + + + + + @@ -46,21 +71,48 @@ + + + + - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + +

+ +

+ + + + + + + + + + + + + + + +
+ @@ -15,4 +71,22 @@ + + + + + + estate.property.offer.tree + estate.property.offer + + + + + + + + + + + \ No newline at end of file diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml index 3a7d9d85bfc..febc6191443 100644 --- a/estate/views/estate_property_views.xml +++ b/estate/views/estate_property_views.xml @@ -6,6 +6,7 @@ Properties estate.property list,form + {'search_default_state': 1} @@ -13,14 +14,16 @@ estate.property estate.property - - - - - + + + + + + + - - + + @@ -36,13 +39,15 @@ string="Cancel" type="object" class="btn btn-secondary" - xoptions="{'invisible': [('state', 'in', ['cancelled', 'sold'])]}"/> + invisible= "state == 'cancelled' or state == 'sold'"/>