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..a436ecaca90 --- /dev/null +++ b/estate/__manifest__.py @@ -0,0 +1,28 @@ +{ + 'name': "Estate", + 'summary': """ + The Real Estate Advertisement module + """, + + 'description': """ + The Real Estate Advertisement module + """, + 'author': "Odoo", + 'website': "https://www.odoo.com/", + 'category': 'Tutorials', + 'version': '19.0.0.1.0', + 'application': True, + 'depends': [ + 'base', + ], + 'data': [ + 'data/ir.model.access.csv', + 'views/estate_property_views.xml', + 'views/estate_property_type_views.xml', + 'views/estate_property_tag_views.xml', + 'views/estate_property_offer_views.xml', + 'views/estate_menus.xml', + 'views/res_users.xml', + ], + 'license': 'LGPL-3', +} diff --git a/estate/data/ir.model.access.csv b/estate/data/ir.model.access.csv new file mode 100644 index 00000000000..b2f2cb31b05 --- /dev/null +++ b/estate/data/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_access_user,estate.property.user,model_estate_property,base.group_user,1,1,1,1 +estate_property_type_access_user,estate.property.type.user,model_estate_property_type,base.group_user,1,1,1,1 +estate_property_tag_access_user,estate.property.tag.user,model_estate_property_tag,base.group_user,1,1,1,1 +estate_property_offer_access_user,estate.property.offer.user,model_estate_property_offer,base.group_user,1,1,1,1 diff --git a/estate/models/__init__.py b/estate/models/__init__.py new file mode 100644 index 00000000000..79238789c22 --- /dev/null +++ b/estate/models/__init__.py @@ -0,0 +1 @@ +from . import estate_property, estate_property_type, estate_property_tag, estate_property_offer, res_users diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py new file mode 100644 index 00000000000..396356f5eb0 --- /dev/null +++ b/estate/models/estate_property.py @@ -0,0 +1,121 @@ +from odoo import api, fields, models, exceptions, tools + + +class EstateProperty(models.Model): + _name = "estate.property" + _description = "Estate Property" + _order = "id desc" + + name = fields.Char('Property Name', required=True) + description = fields.Text() + property_type_id = fields.Many2one( + 'estate.property.type') + buyer_id = fields.Many2one('res.partner', copy=False) + salesperson_id = fields.Many2one( + 'res.users', default=lambda self: self.env.uid) + tag_ids = fields.Many2many( + 'estate.property.tag', string="Property Tag") + offer_ids = fields.One2many('estate.property.offer', 'property_id') + postcode = fields.Char() + date_availability = fields.Date('Availability Date', copy=False, + default=lambda self: fields.Date.add(fields.Date.today(), months=3)) + expected_price = fields.Float(required=True) + bedrooms = fields.Integer(default=2) + living_area = fields.Integer() + facades = fields.Integer() + garage = fields.Boolean() + garden = fields.Boolean() + garden_area = fields.Integer('Garden Area') + garden_orientation = fields.Selection( + selection=[ + ('north', 'North'), + ('south', 'South'), + ('east', 'East'), + ('west', 'West'), + ], + ) + total_area = fields.Integer(compute="_compute_total_area") + best_price = fields.Float('Best Offer', compute="_compute_best_price") + selling_price = fields.Float( + compute="_compute_selling_price", readonly=True, copy=False) + state = fields.Selection( + selection=[ + ('new', 'New'), + ('offer received', 'Offer Received'), + ('offer accepted', 'Offer Accepted'), + ('sold', 'Sold'), + ('cancelled', 'Cancelled'), + ], + default='new', + required=True, + ) + + _check_expected_price = models.Constraint( + 'CHECK(expected_price > 0)', + 'The expected price must be strictly positive.', + ) + + @api.depends('living_area', 'garden_area') + def _compute_total_area(self): + for line in self: + line.total_area = line.living_area + line.garden_area + + @api.depends('offer_ids.price') + def _compute_best_price(self): + for line in self: + line.best_price = max(line.mapped('offer_ids.price'), default=0) + + @api.depends('offer_ids.status', 'offer_ids.price') + def _compute_selling_price(self): + for line in self: + accepted_offer = line.offer_ids.filtered( + lambda o: o.status == 'accepted') + if accepted_offer: + line.selling_price = accepted_offer[0].price + line.buyer_id = accepted_offer[0].partner_id + else: + line.selling_price = 0 + + def action_cancel(self): + for record in self: + if record.state == 'sold': + raise exceptions.UserError( + "You cannot cancel a sold property.") + else: + record.state = 'cancelled' + return True + + def action_sold(self): + for record in self: + if record.state == 'cancelled': + raise exceptions.UserError( + "You cannot mark a cancelled property as sold.") + else: + record.state = 'sold' + return True + + @api.constrains('selling_price', 'expected_price') + def _check_selling_price(self): + for record in self: + if tools.float_is_zero(record.selling_price, precision_digits=2): + continue + + if tools.float_compare(record.selling_price, record.expected_price * 0.9, precision_digits=2) == -1: + raise exceptions.ValidationError( + "The selling price cannot be less than 90% of the expected price.") + + @api.onchange('garden') + def _onchange_garden(self): + if not self.garden: + self.garden_area = 0 + self.garden_orientation = False + else: + self.garden_area = 10 + self.garden_orientation = 'north' + + @api.ondelete(at_uninstall=False) + def _unlink_if_new_or_cancelled(self): + for record in self: + if record.state not in ['new', 'cancelled']: + raise exceptions.UserError( + "You cannot delete a property that is not '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..a4e403040fc --- /dev/null +++ b/estate/models/estate_property_offer.py @@ -0,0 +1,75 @@ +from datetime import timedelta +from odoo import api, fields, models, exceptions + + +class EstatePropertyOffer(models.Model): + _name = "estate.property.offer" + _description = "Estate Property Offer" + _order = "price desc" + + price = fields.Float() + partner_id = fields.Many2one("res.partner", required=True) + property_id = fields.Many2one("estate.property", required=True) + property_type_id = fields.Many2one( + related="property_id.property_type_id", store=True) + validity = fields.Integer("Validity (days)", default=7) + date_deadline = fields.Date( + "Deadline", compute="_compute_date_deadline", inverse="_inverse_date_deadline", store=True) + status = fields.Selection( + selection=[ + ('accepted', 'Accepted'), + ('refused', 'Refused'), + ], + copy=False, + ) + + _check_price = models.Constraint( + 'CHECK(price > 0)', + 'The offer price must be strictly positive.', + ) + + @api.depends('create_date', 'validity') + def _compute_date_deadline(self): + for offer in self: + start_date = fields.Date.to_date( + offer.create_date) or fields.Date.today() + offer.date_deadline = start_date + timedelta(days=offer.validity) + + def _inverse_date_deadline(self): + for offer in self: + if offer.date_deadline: + start_date = fields.Date.to_date( + offer.create_date) or fields.Date.today() + offer.validity = (offer.date_deadline - start_date).days + else: + offer.validity = 0 + + @api.model_create_multi + def create(self, vals_list): + for vals in vals_list: + prop = self.env['estate.property'].browse(vals['property_id']) + if prop.offer_ids: + max_offer = max(prop.offer_ids.mapped('price')) + if vals.get('price') < max_offer: + raise exceptions.UserError( + ("The offer must be higher than %.2f") % max_offer) + prop.state = 'offer received' + + return super().create(vals_list) + + def action_accept(self): + for offer in self: + existing_accepted = self.env['estate.property.offer'].search([ + ('property_id', '=', offer.property_id.id), + ('status', '=', 'accepted') + ]) + if existing_accepted: + raise exceptions.UserError( + "An offer for this property has already been accepted.") + offer.status = 'accepted' + offer.property_id.selling_price = offer.price + offer.property_id.state = 'offer accepted' + + def action_refuse(self): + for offer in self: + offer.status = 'refused' diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py new file mode 100644 index 00000000000..c04e3757483 --- /dev/null +++ b/estate/models/estate_property_tag.py @@ -0,0 +1,15 @@ +from odoo import fields, models + + +class EstatePropertyTag(models.Model): + _name = "estate.property.tag" + _description = "Estate Property Tag" + _order = "name" + + name = fields.Char('Property Tag Name', required=True) + color = fields.Integer('Color') + + _unique_name = models.Constraint( + "UNIQUE(name)", + "Property tag names 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..7258cbbd550 --- /dev/null +++ b/estate/models/estate_property_type.py @@ -0,0 +1,25 @@ +from odoo import fields, models + + +class EstatePropertyType(models.Model): + _name = "estate.property.type" + _description = "Estate Property Type" + _order = "sequence, name" + + name = fields.Char('Property Type Name', required=True) + property_ids = fields.One2many( + 'estate.property', 'property_type_id', string='Properties') + offer_ids = fields.One2many( + 'estate.property.offer', 'property_type_id', string='Offers') + offer_count = fields.Integer( + compute="_compute_offer_count", string="Number of Offers") + sequence = fields.Integer(default=1) + + _unique_name = models.Constraint( + "UNIQUE(name)", + "Property type names must be unique.", + ) + + def _compute_offer_count(self): + for record in self: + record.offer_count = len(record.offer_ids) diff --git a/estate/models/res_users.py b/estate/models/res_users.py new file mode 100644 index 00000000000..8b30c3a5567 --- /dev/null +++ b/estate/models/res_users.py @@ -0,0 +1,8 @@ +from odoo import fields, models + + +class ResUsers(models.Model): + _inherit = "res.users" + + property_ids = fields.One2many( + 'estate.property', 'salesperson_id') diff --git a/estate/tests/__init__.py b/estate/tests/__init__.py new file mode 100644 index 00000000000..576617cccff --- /dev/null +++ b/estate/tests/__init__.py @@ -0,0 +1 @@ +from . import test_estate_property diff --git a/estate/tests/test_estate_property.py b/estate/tests/test_estate_property.py new file mode 100644 index 00000000000..7202f8b67a0 --- /dev/null +++ b/estate/tests/test_estate_property.py @@ -0,0 +1,42 @@ +from odoo.exceptions import ValidationError +from odoo.tests import TransactionCase +from odoo import Command + + +class TestEstateProperty(TransactionCase): + + @classmethod + def setUpClass(cls): + super().setUpClass() + cls.estate = cls.env['estate.property'].create({ + 'name': 'Super test estate', + 'expected_price': 100000.0, + 'state': 'new', + }) + cls.test_partner = cls.env['res.partner'].create({ + 'name': 'Maman ours', + }) + + def test_estate_best_price(self): + ''' + Ensure best price is correctly updated when an offer is received. + ''' + self.assertEqual(self.estate.best_price, 0.0) + self.estate.offer_ids = [Command.create({ + 'price': 125000.0, + 'partner_id': self.test_partner.id, + })] + self.assertEqual(self.estate.best_price, 125000.0) + + def test_accept_offer_south_facing_garden(self): + ''' + Ensure offers for estates with south-facing gardens can only be accepted if above expected + price. + ''' + self.estate.expected_price = 500000 + self.estate.offer_ids = [Command.create({ + 'price': 475000.0, + 'partner_id': self.test_partner.id, + })] + with self.assertRaises(ValidationError): + self.estate.offer_ids.accept_offer() diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml new file mode 100644 index 00000000000..a55cc952013 --- /dev/null +++ b/estate/views/estate_menus.xml @@ -0,0 +1,12 @@ + + + + + + + + + + + + diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml new file mode 100644 index 00000000000..a3f2f23e040 --- /dev/null +++ b/estate/views/estate_property_offer_views.xml @@ -0,0 +1,35 @@ + + + estate.property.offer.form + estate.property.offer + +
+ + + + + + + + + +
+ + + + + estate.property.offer.list + estate.property.offer + + + + + + + + +

+ +

+ + + + + + + + + + + + + + + + + + estate.property.type.list + estate.property.type + + + + + + + + + + Property Types + estate.property.type + list,form + + diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml new file mode 100644 index 00000000000..095dad59bff --- /dev/null +++ b/estate/views/estate_property_views.xml @@ -0,0 +1,141 @@ + + + estate.property.search + estate.property + + + + + + + + + + + + + + + + + estate.property.form + estate.property + +
+

+ +

+
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+ + + + + estate.property.list + estate.property + + + + + + + + + + + + + + + + estate.property.kanban + estate.property + + + + + +
+
+ + + +
+
+ Expected Price: +
+
+ Best Offer: +
+
+ Selling Price: +
+
+ +
+
+ + + + + + + + Properties + estate.property + kanban,list,form + + {'search_default_available_properties': True} + + diff --git a/estate/views/res_users.xml b/estate/views/res_users.xml new file mode 100644 index 00000000000..b4fa40491d1 --- /dev/null +++ b/estate/views/res_users.xml @@ -0,0 +1,14 @@ + + + res.users.form + 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..ce27f3d4b7c --- /dev/null +++ b/estate_account/__manifest__.py @@ -0,0 +1,17 @@ +{ + 'name': "Estate Account", + 'author': "Odoo", + 'website': "https://www.odoo.com/", + 'category': 'Tutorials', + 'version': '19.0.0.1.0', + 'application': True, + 'depends': [ + 'base', + 'account', + 'estate', + ], + 'license': 'LGPL-3', + 'data': [ + 'data/ir.model.access.csv', + ], +} diff --git a/estate_account/data/ir.model.access.csv b/estate_account/data/ir.model.access.csv new file mode 100644 index 00000000000..97dd8b917b8 --- /dev/null +++ b/estate_account/data/ir.model.access.csv @@ -0,0 +1 @@ +id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink 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..a599f20886f --- /dev/null +++ b/estate_account/models/estate_property.py @@ -0,0 +1,30 @@ +from odoo import models, Command + + +class InheritedModel(models.Model): + _inherit = "estate.property" + + def action_sold(self): + res = super().action_sold() + for record in self: + commission_amount = record.selling_price * 0.06 + self.env['account.move'].create({ + 'partner_id': record.buyer_id.id, + 'move_type': 'out_invoice', + 'invoice_line_ids': [ + # Line 1: 6% Commission + Command.create({ + "name": f"6% Commission for {record.name}", + "quantity": 1.0, + "price_unit": commission_amount, + }), + # Line 2: Administrative Fees + Command.create({ + "name": "Administrative Fees", + "quantity": 1.0, + "price_unit": 100.0, + }), + ], + }) + + return res