diff --git a/.gitignore b/.gitignore index b6e47617de1..39994b9be5c 100644 --- a/.gitignore +++ b/.gitignore @@ -127,3 +127,16 @@ dmypy.json # Pyre type checker .pyre/ + +# Ruff linter +ruff.toml + +# Pre-commit config files +.pre-commit-config.yaml +.pre-commit-hooks.yaml + +# Checkstyle file +check-style + +# VS Code Config +.vscode/ 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..cf522e8102a --- /dev/null +++ b/estate/__manifest__.py @@ -0,0 +1,22 @@ +{ + 'name': 'Real Estate', + 'version': '1.9', + 'category': 'Real Estates', + 'summary': 'Manage real estate operations', + 'author': 'Haroune Hassine', + 'license': 'LGPL-3', + 'depends': [ + 'base_setup', + ], + 'application': True, + 'installable': True, + 'data': [ + 'security/ir.model.access.csv', + 'views/estate_property_views.xml', + 'views/estate_property_offer_views.xml', + 'views/estate_property_tag_views.xml', + 'views/estate_property_type_views.xml', + 'views/estate_menus.xml', + 'views/res_users_views.xml', + ], +} diff --git a/estate/models/__init__.py b/estate/models/__init__.py new file mode 100644 index 00000000000..8d1b0ed8937 --- /dev/null +++ b/estate/models/__init__.py @@ -0,0 +1,7 @@ +from . import ( + estate_property, + estate_property_offer, + estate_property_tag, + estate_property_type, + res_users, +) diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py new file mode 100644 index 00000000000..41e6fdbcdd9 --- /dev/null +++ b/estate/models/estate_property.py @@ -0,0 +1,120 @@ +from dateutil.relativedelta import relativedelta + +from odoo import api, fields, models +from odoo.exceptions import UserError, ValidationError +from odoo.tools.float_utils import float_compare +from odoo.tools.translate import _ + +DEFAULT_GARDEN_AREA = 10 + + +class EstateProperty(models.Model): + _name = "estate.property" + _description = "Real Estate Property." + _order = "id desc" + + active = fields.Boolean('Active', default=True) + name = fields.Char('Real Estate Name', required=True) + description = fields.Text('Description') + postcode = fields.Char('Postcode') + date_availability = fields.Date('Availability Date', copy=False, default=fields.Datetime.today() + relativedelta(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) + 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 (sqm)') + garden_orientation = fields.Selection( + string='Garden Orientation', + selection=[('north', 'North'), ('south', 'South'), ('east', 'East'), ('west', 'West')], + ) + state = fields.Selection( + string='State', + selection=[('new', 'New'), ('offer_received', 'Offer Received'), ('offer_accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')], + required=True, + copy=False, + default='new', + ) + total_area = fields.Integer(compute='_compute_total_area', string='Total Area (sqm)') + best_price = fields.Float(compute='_compute_best_price', string='Best Price') + sales_person_id = fields.Many2one('res.users', string='Salesman', index=True, default=lambda self: self.env.user) + buyer_id = fields.Many2one('res.partner', string='Buyer', copy=False, readonly=True, compute='_compute_buyer') + tag_ids = fields.Many2many('estate.property.tag', string='Tags') + offer_ids = fields.One2many('estate.property.offer', 'property_id', string='Offers') + type_id = fields.Many2one('estate.property.type', string='Type', required=True) + + ## CONSTRATINS ## + _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.constrains('expected_price', 'selling_price') + def _check_expected_vs_selling_price_ratio(self): + for property in self: + if any(offer.status == 'accepted' for offer in property.offer_ids): + if float_compare(property.selling_price, property.expected_price * 0.9, precision_digits=2) < 0: + raise ValidationError(_('The selling price cannot be lower than 90 precent of the expected price: \n Selling Price: %s, Expected Price: %s') % (property.selling_price, property.expected_price)) + + @api.depends('offer_ids.price') + def _compute_selling_price(self): + for property in self: + property.selling_price = self.offer_ids.price + + @api.depends('living_area', 'garden_area') + def _compute_total_area(self): + for record in self: + record.total_area = record.living_area + record.garden_area + + @api.depends('offer_ids.price') + def _compute_best_price(self): + for record in self: + if record.offer_ids: + record.best_price = max(record.offer_ids.mapped('price')) + else: + record.best_price = 0.0 + + @api.depends('offer_ids.status') + def _compute_buyer(self): + for record in self: + for offer in record.offer_ids: + if offer.status == 'accepted': + record.buyer_id = offer.partner_id + return + record.buyer_id = None + + @api.onchange('garden') + def _onchange_garden(self): + if self.garden: + self.garden_area = DEFAULT_GARDEN_AREA + self.garden_orientation = 'north' + else: + self.garden_area = 0 + self.garden_orientation = None + + def action_cancel_property(self): + if self.state == 'sold': + raise UserError(_('Sold properties cannot be canceled.')) + self.state = 'cancelled' + for offer in self.offer_ids: + offer.status = None + return True + + def action_sold_property(self): + if self.state == 'cancelled': + raise UserError(_('Cancelled properties cannot be sold.')) + self.state = 'sold' + return True + + @api.ondelete(at_uninstall=False) + def _unlink_except_new_or_cancelled(self): + for record in self: + if record.state not in ('new', 'cancelled'): + raise UserError(_('You cannot delete a property unless its state is `New` or `Cancelled`.')) diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py new file mode 100644 index 00000000000..15fc9a41f96 --- /dev/null +++ b/estate/models/estate_property_offer.py @@ -0,0 +1,72 @@ +from datetime import date + +from dateutil.relativedelta import relativedelta + +from odoo import api, fields, models +from odoo.exceptions import UserError +from odoo.tools.translate import _ + + +class EstateProperOffer(models.Model): + _name = "estate.property.offer" + _description = "Property Offer." + _order = "price desc" + + price = fields.Float('Price') + status = fields.Selection( + string='Status', + selection=[('accepted', 'Accepted'), ('refused', 'Refused')], + copy=False, + ) + validity = fields.Integer('Validity (days)', default=7) + date_deadline = fields.Date(compute='_compute_deadline', inverse='_inverse_deadline', string='Deadline') + partner_id = fields.Many2one('res.partner', required=True) + property_id = fields.Many2one('estate.property', required=True) + property_type_id = fields.Many2one(related='property_id.type_id') + + _check_offer_price = models.Constraint( + 'CHECK(price > 0)', + 'The offer price must be strictly positive.', + ) + + @api.depends('create_date', 'validity') + def _compute_deadline(self): + for record in self: + if record.create_date: + record.date_deadline = record.create_date + relativedelta(days=+record.validity) + else: + record.date_deadline = date.today() + relativedelta(days=+record.validity) + + def _inverse_deadline(self): + for record in self: + record.validity = (record.date_deadline - record.create_date.date()).days + + def action_accept(self): + if self.status != 'accepted': + for offer in self.property_id.offer_ids: + if offer.status == 'accepted': + raise UserError(_('Only one offer can be accepted.')) + self.status = 'accepted' + self.property_id.selling_price = self.price + self.property_id.state = 'offer_accepted' + return True + + def action_refuse(self): + self.status = 'refused' + if self.property_id.state != 'offer_accepted': + self.property_id.selling_price = None + self.property_id.state = 'offer_received' + return True + + @api.model + def create(self, vals_list): + for vals in vals_list: + property_id = self.env['estate.property'].browse(vals['property_id']) + max_offer = 0 + if property_id.offer_ids: + max_offer = max(offer for offer in property_id.offer_ids).price + if vals['price'] > max_offer: + self.env['estate.property'].browse(vals['property_id']).state = 'offer_received' + else: + raise UserError(_("You cannot add an offer with a price lower than the maximum existing price.")) + return super().create(vals_list) diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py new file mode 100644 index 00000000000..06c226a1b59 --- /dev/null +++ b/estate/models/estate_property_tag.py @@ -0,0 +1,15 @@ +from odoo import fields, models + + +class EstateProperTag(models.Model): + _name = "estate.property.tag" + _description = "Property Tag." + _order = "name" + + name = fields.Char('Property Tag', required=True) + color = fields.Integer('Color') + + _check_name = models.Constraint( + 'UNIQUE(name)', + 'The tag must be unique.', + ) diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py new file mode 100644 index 00000000000..eae7c14b6fd --- /dev/null +++ b/estate/models/estate_property_type.py @@ -0,0 +1,23 @@ +from odoo import api, fields, models + + +class EstatePropertyType(models.Model): + _name = "estate.property.type" + _description = "Property Type." + _order = "sequence, name" + + name = fields.Char('Property Type', required=True) + sequence = fields.Integer('Sequence', default=1, help="Used to order types. Lower is better.") + offer_count = fields.Integer('Offer Count', compute='_compute_offer_count') + property_ids = fields.One2many('estate.property', 'type_id') + offer_ids = fields.One2many('estate.property.offer', 'property_type_id') + + _check_name = models.Constraint( + 'UNIQUE(name)', + 'The type must be unique.', + ) + + @api.depends('offer_ids') + def _compute_offer_count(self): + for type in self: + type.offer_count = len(type.offer_ids) diff --git a/estate/models/res_users.py b/estate/models/res_users.py new file mode 100644 index 00000000000..090055324bd --- /dev/null +++ b/estate/models/res_users.py @@ -0,0 +1,7 @@ +from odoo import fields, models + + +class ResUsers(models.Model): + _inherit = 'res.users' + + property_ids = fields.One2many('estate.property', 'sales_person_id') diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv new file mode 100644 index 00000000000..992440e42e8 --- /dev/null +++ b/estate/security/ir.model.access.csv @@ -0,0 +1,5 @@ +"id","name","model_id:id","group_id:id","perm_read","perm_write","perm_create","perm_unlink" +"estate_property_model","estate_property_model","model_estate_property","base.group_user",1,1,1,1 +"estate_property_type_model","estate_property_type_model","model_estate_property_type","base.group_user",1,1,1,1 +"estate_property_tag_model","estate_property_tag_model","model_estate_property_tag","base.group_user",1,1,1,1 +"estate_property_offer_model","estate_property_offer_model","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 new file mode 100644 index 00000000000..605a8d68cb6 --- /dev/null +++ b/estate/views/estate_menus.xml @@ -0,0 +1,15 @@ + + + + + + + + + + + + + + + diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml new file mode 100644 index 00000000000..924bfd293e8 --- /dev/null +++ b/estate/views/estate_property_offer_views.xml @@ -0,0 +1,27 @@ + + + + + estate.property.offer.action + estate.property.offer + list + [('property_type_id', '=', active_id)] + + + + estate.property.offer.view.list + estate.property.offer + + + + + + + + + +

+ + + + + + + + + + + + + + + + + + + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml new file mode 100644 index 00000000000..cafecfa0bfd --- /dev/null +++ b/estate/views/estate_property_views.xml @@ -0,0 +1,139 @@ + + + + + estate.property.action + estate.property + list,form,kanban + {'search_default_available': True} + + + + estate.property.view.kanban + estate.property + + + + + +
+

+
+
+ Expected Price: $ +
+
+ Best Price: $ +
+
+ Selling Price: $ +
+ + + + + + + + estate.property.view.list + estate.property + + + + + + + + + + + + + + + + + + + + + + estate.property.view.search + estate.property + + + + + + + + + + + + + + + + + + + estate.property.view.form + estate.property + +
+
+
+ +

+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+ + + + diff --git a/estate/views/res_users_views.xml b/estate/views/res_users_views.xml new file mode 100644 index 00000000000..a25de2eaaea --- /dev/null +++ b/estate/views/res_users_views.xml @@ -0,0 +1,19 @@ + + + + + res.users.view.form.inherit.estate + res.users + + + + + + + + + + + + + diff --git a/estate_account/__init__.py b/estate_account/__init__.py new file mode 100644 index 00000000000..0650744f6bc --- /dev/null +++ b/estate_account/__init__.py @@ -0,0 +1 @@ +from . import models diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py new file mode 100644 index 00000000000..c0c336faddd --- /dev/null +++ b/estate_account/__manifest__.py @@ -0,0 +1,17 @@ +{ + 'name': 'Real Estate Account', + 'version': '1.9', + 'category': 'Real Estates', + 'summary': 'Manage real estate accounts', + 'author': 'Haroune Hassine', + 'license': 'LGPL-3', + 'depends': [ + 'estate', + 'account', + ], + 'application': True, + 'installable': True, + 'data': [ + 'security/ir.model.access.csv', + ], +} diff --git a/estate_account/models/__init__.py b/estate_account/models/__init__.py new file mode 100644 index 00000000000..5e1963c9d2f --- /dev/null +++ b/estate_account/models/__init__.py @@ -0,0 +1 @@ +from . import estate_property diff --git a/estate_account/models/estate_property.py b/estate_account/models/estate_property.py new file mode 100644 index 00000000000..c9d69c841a7 --- /dev/null +++ b/estate_account/models/estate_property.py @@ -0,0 +1,32 @@ +from odoo import Command, models + + +class EstateProperty(models.Model): + _inherit = 'estate.property' + + def action_sold_property(self): + self.env['account.move'].create( + { + "partner_id": self.buyer_id.id, + "move_type": "out_invoice", + "invoice_line_ids": [ + Command.create({ + "name": self.name, + "quantity": 1, + "price_unit": self.selling_price, + }), + Command.create({ + "name": "6% Discount", + "quantity": 1, + "price_unit": -0.6 * self.selling_price, + }), + Command.create({ + "name": "Administrative fees", + "quantity": 1, + "price_unit": 100, + }), + ], + }, + ) + + return super().action_sold_property() diff --git a/estate_account/security/ir.model.access.csv b/estate_account/security/ir.model.access.csv new file mode 100644 index 00000000000..bc90d1448fb --- /dev/null +++ b/estate_account/security/ir.model.access.csv @@ -0,0 +1,2 @@ +"id","name","model_id:id","group_id:id","perm_read","perm_write","perm_create","perm_unlink" +"estate_property_model","estate_property_model","model_estate_property","base.group_user",1,1,1,1