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..4344f144632 --- /dev/null +++ b/estate/__manifest__.py @@ -0,0 +1,17 @@ +{ + 'name': 'Real Estate', + 'version': '1.0', + 'depends': ['base'], + 'application': True, + 'installable': True, + '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_property_offer_views.xml', + 'views/estate_menu.xml', + ], + 'author': 'macai', + 'license': 'LGPL-3', +} diff --git a/estate/models/__init__.py b/estate/models/__init__.py new file mode 100644 index 00000000000..2f1821a39c1 --- /dev/null +++ b/estate/models/__init__.py @@ -0,0 +1,4 @@ +from . import estate_property +from . import estate_property_type +from . import estate_property_tag +from . import estate_property_offer diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py new file mode 100644 index 00000000000..4716b505a59 --- /dev/null +++ b/estate/models/estate_property.py @@ -0,0 +1,134 @@ +from odoo import models, fields, api +from odoo.exceptions import UserError +from odoo.tools import float_is_zero +from odoo.tools.float_utils import float_compare + + +class EstateProperty(models.Model): + _name = 'estate.property' + _description = 'Real Estate property model' + _order = 'id desc' + +# Property information + name = fields.Char('Name', required=True) + description = fields.Text('Description') + postcode = fields.Char('Postcode', required=True) + date_availability = fields.Date('Availability', copy=False, default=lambda self: fields.Date.add(fields.Date.today(), 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') + facades = fields.Integer('Facades') + has_garage = fields.Boolean('Garage') + has_garden = fields.Boolean('Garden') + garden_area = fields.Integer('Garden Area') + garden_orientation = fields.Selection( + string='Garden Orientation', + selection=[('north', 'North'), + ('south', 'South'), + ('east', 'East'), + ('west', 'West'), + ] + ) + total_area = fields.Integer(compute='_compute_total_area', string='Total Area') + active = fields.Boolean('Active', default=True) + 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' + ) + property_type_id = fields.Many2one("estate.property.type", string='Property Type') + +# Other Information + salesman_id = fields.Many2one('res.users', string='Salesman', default=lambda self: self.env.user) + buyer_id = fields.Many2one('res.partner', string='Buyer', copy=False) + +# Tags + tag_ids = fields.Many2many('estate.property.tag', string='Tags') + +# Offers + offer_ids = fields.One2many('estate.property.offer', 'property_id', string='Offer') + best_price = fields.Float(compute='_compute_best_price', string='Best Offer') + +# Constraints: +# SQL: + _check_expected_price = models.Constraint( + 'CHECK(expected_price > 0)', + 'The expected price should be stricly positive.' + ) + _check_selling_price = models.Constraint( + 'CHECK(selling_price > 0)', + 'The proporty selling price should be stricly positive.' + ) + +# Python: + @api.constrains('selling_price') + def _check_selling_price(self): + for record in self: + if not float_is_zero(record.selling_price, 2): + if float_compare(record.selling_price, record.expected_price, 2) == -1: + raise UserError('The Selling Price must be at least 90% of the Expected Price,' + ' you must change the expected price to accepted this offer') + +# Computation Fields: + + @api.depends('living_area', 'garden_area') + def _compute_total_area(self): + for record in self: # QUESTION is the for each loop here necessary ? or can I just use 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 len(record.offer_ids) > 0: + record.best_price = max(record.offer_ids.mapped('price')) + else: + record.best_price = 0. + + @api.onchange('has_garden') + def _onchange_has_garden(self): + if self.has_garden: + self.garden_area = 10 + self.garden_orientation = 'north' + else: + self.garden_area = None + self.garden_orientation = None + +# Actions buttons + def action_cancel_property_state(self): + for record in self: + if record.state == 'cancelled': + raise UserError('Cancelled properties cannot be cancelled again !') + elif record.state == 'sold': + raise UserError('Sold Properties cannot be cancelled !') + record.state = 'cancelled' + return True + return True + + def action_sell_property_state(self): + for record in self: + if record.state == 'sold': + raise UserError('Sold Properties cannot be sold again !') + elif record.state == 'cancelled': + raise UserError('Cancelled properties cannot be sold !') + record.state = 'sold' + return True + return True + +# Actions : + def action_accept_selling_offer(self, buyer, price): + for record in self: + if record.buyer_id: + raise UserError('An offer is already accepted for this house') + record.buyer_id = buyer + record.selling_price = price + record.state = 'offer_accepted' + return True + return True diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py new file mode 100644 index 00000000000..fa7af167c10 --- /dev/null +++ b/estate/models/estate_property_offer.py @@ -0,0 +1,55 @@ +from odoo import models, fields, api +from odoo.exceptions import UserError + + +class EstatePropertyOffer(models.Model): + _name = 'estate.property.offer' + _description = 'Estate Property Offer' + _order = 'price desc' + + price = fields.Float(string='Price') + status = fields.Selection([ + ('accepted', 'Accepted'), + ('refused', 'Refused') + ], copy=False, string='Status') + validity = fields.Integer(string='Validity (Days)', default=7) + date_deadline = fields.Date(compute='_compute_date_deadline', inverse='_inverse_date_deadline', string='Deadline') + partner_id = fields.Many2one('res.partner', required=True, string='Partner') + property_id = fields.Many2one('estate.property', required=True) + property_type_id = fields.Many2one(related='property_id.property_type_id') + +# Constraints: + _check_price = models.Constraint( + 'CHECK(price > 0)', + 'The offer price should be stricty positive' + ) + +# computed fields + + @api.depends('validity', 'create_date') + def _compute_date_deadline(self): + for record in self: + base_date = record.create_date or fields.Date.today() # Fallback when create_date is not set yet: + record.date_deadline = fields.Date.add(base_date, days=record.validity) + + def _inverse_date_deadline(self): + for record in self: + record.validity = (record.date_deadline - record.create_date.date()).days + +# Action buttons + + def action_accept_property_offer(self): + for record in self: + for properties in record.property_id: + properties.action_accept_selling_offer(record.partner_id, record.price) + record.status = 'accepted' + return True + return True + + def action_refuse_property_offer(self): + for record in self: + if record.status == 'accepted': + raise UserError('Accepted offer cannot be refused !') + record.status = 'refused' + return True + return True diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py new file mode 100644 index 00000000000..598ae7d3840 --- /dev/null +++ b/estate/models/estate_property_tag.py @@ -0,0 +1,15 @@ +from odoo import models, fields + + +class EstatePropertyTag(models.Model): + _name = 'estate.property.tag' + _description = 'Estate Property Tag' + _order = 'name' + + name = fields.Char(string='Name', required=True) + color = fields.Integer('Color') +# Constraints: + _check_unique = models.Constraint( + 'UNIQUE (name)', + 'The tag name 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..bc8499ad247 --- /dev/null +++ b/estate/models/estate_property_type.py @@ -0,0 +1,23 @@ +from odoo import models, fields, api + + +class EstatePropertyType(models.Model): + _name = 'estate.property.type' + _description = 'Define the Real Estate Property Type' + _order = 'sequence, name' + + name = fields.Char(string='Name', required=True) + property_ids = fields.One2many('estate.property', 'property_type_id', string='Properties') + sequence = fields.Integer(string='Sequence', default=1) +# Offers handling + offer_ids = fields.One2many('estate.property.offer','property_type_id','Property Type') + offer_count = fields.Integer(compute='_compute_offer_count', string='Offer Count') + +# Compute fields: + + @api.depends('offer_ids') + def _compute_offer_count(self): + for record in self: + record.offer_count = len(record.offer_ids) + return True + return True diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv new file mode 100644 index 00000000000..2d9ca473a31 --- /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 +access_estate_property_user,access.estate.property,model_estate_property,base.group_user,1,1,1,1 +access_estate_property_type_user,access.estate.property.type,model_estate_property_type,base.group_user,1,1,1,1 +access_estate_property_tag_user,access.estate.property.tag,model_estate_property_tag,base.group_user,1,1,1,1 +access_estate_property_offer_user,access.estate.property.offer,model_estate_property_offer,base.group_user,1,1,1,1 diff --git a/estate/views/estate_menu.xml b/estate/views/estate_menu.xml new file mode 100644 index 00000000000..0ec713c1c9a --- /dev/null +++ b/estate/views/estate_menu.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..109799df3f7 --- /dev/null +++ b/estate/views/estate_property_offer_views.xml @@ -0,0 +1,46 @@ + + + + + Property Offers + estate.property.offer + list,form + [('property_type_id', '=', active_id)] + + + + estate.property.offer.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..812208f5910 --- /dev/null +++ b/estate/views/estate_property_views.xml @@ -0,0 +1,118 @@ + + + + + Properties + estate.property + list,form,search + {'search_default_active': True, 'search_default_current': True} + + + + estate.properties.list + estate.property + + + + + + + + + + + + + + + + estate.properties.form + estate.property + +
+
+
+ +

+ +

+

+ +

+
+
+ + + + + +
+
+ + + + + +
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + +
+ + + + + estate.properties.search + estate.property + + + + + + + + + + + + + + + + + + + +