diff --git a/.gitignore b/.gitignore index b6e47617de1..627f35a9e7e 100644 --- a/.gitignore +++ b/.gitignore @@ -127,3 +127,6 @@ dmypy.json # Pyre type checker .pyre/ + +# Pycharm IDE +/.idea 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..a243f2ba22e --- /dev/null +++ b/estate/__manifest__.py @@ -0,0 +1,22 @@ +{ + "name": "estate", + "version": 1.0, + "author": "gicha", + "license": "LGPL-3", + "depends": [ + "base", + ], + "installable": True, + "application": True, + "data": [ + # Access Rights + "security/ir.model.access.csv", + # Views + "views/estate_property_offer_views.xml", + "views/estate_property_views.xml", + "views/estate_property_type_views.xml", + "views/estate_property_tag_views.xml", + "views/estate_users_views.xml", + "views/estate_menus.xml", + ], +} diff --git a/estate/models/__init__.py b/estate/models/__init__.py new file mode 100644 index 00000000000..8ada7efccc7 --- /dev/null +++ b/estate/models/__init__.py @@ -0,0 +1,5 @@ +from . import estate_property +from . import estate_property_type +from . import estate_property_tag +from . import estate_property_offer +from . import users diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py new file mode 100644 index 00000000000..0b1a3a3d430 --- /dev/null +++ b/estate/models/estate_property.py @@ -0,0 +1,133 @@ +from odoo import models, fields, api +from odoo.exceptions import UserError, ValidationError +from odoo.tools.float_utils import float_compare + + +class EstateProperty(models.Model): + _name = "estate.property" + _description = "Estate property" + _order = "id desc" + + name = fields.Char(required=True, string="Title") + description = fields.Text() + + postcode = fields.Char() + date_availability = fields.Datetime( + copy=False, + default=lambda self: fields.Datetime.add(fields.Datetime.today(), months=3), + string="Available From", + ) + + expected_price = fields.Float(required=True) + selling_price = fields.Float(readonly=True, copy=False) + best_price = fields.Float(compute="_compute_best_price") + + bedrooms = fields.Integer(default=2) + living_area = fields.Integer(string="Living Area (sqm)") + facades = fields.Integer() + garage = fields.Boolean() + + garden = fields.Boolean() + garden_area = fields.Integer(string="Garden Area (sqm)") + garden_orientation = fields.Selection( + string="Garden Orientation", + selection=[ + ("north", "North"), + ("south", "South"), + ("east", "East"), + ("west", "West"), + ], + ) + + total_area = fields.Integer(compute="_compute_total_area", readonly=True) + + active = fields.Boolean(default=True) + state = fields.Selection( + string="Status", + selection=[ + ("new", "New"), + ("offer_received", "Offer Received"), + ("offer_accepted", "Offer Accepted"), + ("sold", "Sold"), + ("cancelled", "Cancelled"), + ], + default="new", + ) + + # Relations + property_type_id = fields.Many2one("estate.property.type", string="Type") + buyer_id = fields.Many2one("res.partner", string="Buyer", copy=False) + salesperson_id = fields.Many2one( + "res.users", string="Salesperson", default=lambda self: self.env.uid + ) + tag_ids = fields.Many2many("estate.property.tag", string="Tags") + offer_ids = fields.One2many("estate.property.offer", "property_id", string="Offers") + + # Functions + @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")) + if record.state == "new": + record.state = "offer_received" + else: + record.best_price = 0 + record.state = "new" + + @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 = "" + + def action_sold_property(self): + for property in self: + if property.state == "cancelled": + raise UserError("Cancelled properties cannot be sold") + property.state = "sold" + return True + + def action_cancel_property(self): + for property in self: + if property.state == "sold": + raise UserError("Sold properties cannot be cancelled") + property.state = "cancelled" + return True + + @api.ondelete(at_uninstall=False) + def _check_unlink(self): + for property in self: + if property.state != "new" and property.state != "cancelled": + raise UserError("Only new and cancelled properties can be deleted!") + + # Constraints + _check_positive_expected_price = models.Constraint( + "CHECK(expected_price > 0)", + "The expected price of a property must be strictly positive.", + ) + + _check_positive_selling_price = models.Constraint( + "CHECK(selling_price >= 0)", "The selling price of a property must be positive." + ) + + @api.constrains("selling_price") + def _check_selling_price(self): + for property in self: + if ( + float_compare( + property.selling_price, (0.9 * property.expected_price), 2 + ) + == -1 + ): + raise ValidationError( + "The selling price must be at least 90% of the expected price. You may decrease the expected price if you want to accept this offer." + ) diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py new file mode 100644 index 00000000000..09cdeaa2b67 --- /dev/null +++ b/estate/models/estate_property_offer.py @@ -0,0 +1,80 @@ +from odoo import models, fields, api +from odoo.exceptions import UserError + + +class EstatePropertyOffer(models.Model): + _name = "estate.property.offer" + _description = "Offer made for a property by a customer. The offer can be lower or higher than the expected price." + _order = "price desc" + + price = fields.Float() + status = fields.Selection( + string="Status", + selection=[("accepted", "Accepted"), ("refused", "Refused")], + copy=False, + ) + validity = fields.Integer(default=7, string="Validity (days)") + date_deadline = fields.Date( + compute="_compute_deadline", inverse="_inverse_deadline", string="Deadline" + ) + + # Relations + partner_id = fields.Many2one("res.partner", string="Partner") + property_id = fields.Many2one("estate.property", string="Property") + property_type_id = fields.Many2one( + related="property_id.property_type_id", store=True + ) + + # Methods + @api.depends("validity") + def _compute_deadline(self): + for offer in self: + if offer.create_date: + offer.date_deadline = fields.Date.add( + offer.create_date, days=offer.validity + ) + else: + offer.date_deadline = fields.Date.add( + fields.Date.today(), days=offer.validity + ) # Use today() instead of create_date to avoid crashing at record creation + + def _inverse_deadline(self): + for offer in self: + offer.validity = ( + offer.date_deadline - fields.Date.to_date(offer.create_date) + ).days + + def action_accept_offer(self): + for offer in self: + if offer.property_id.state == "offer_accepted": + offer.status = "refused" + return True + offer.status = "accepted" + offer.property_id.selling_price = offer.price + offer.property_id.buyer_id = offer.partner_id + offer.property_id.state = "offer_accepted" + return True + + def action_refuse_offer(self): + for offer in self: + if offer.status == "accepted": + offer.property_id.selling_price = 0 + offer.property_id.buyer_id = "" + offer.status = "refused" + return True + + @api.model + def create(self, vals): + for record in vals: + best_price = ( + self.env["estate.property"].browse(record["property_id"]).best_price + ) + if record["price"] < best_price: + raise UserError(f"The offer must be greater than {best_price}€") + return super().create(vals) + + # Constraints + _check_positive_price = models.Constraint( + "CHECK(price > 0)", + "The expected price of a offer must be positive.", + ) diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py new file mode 100644 index 00000000000..c1ca441b6f8 --- /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 = "Tag to add to properties" + _order = "name asc" + + name = fields.Char(required=True) + color = fields.Integer() + + # Constraints + _check_unique_name = models.Constraint( + "UNIQUE(name)", "A property tag name should be unique" + ) diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py new file mode 100644 index 00000000000..8039b816f40 --- /dev/null +++ b/estate/models/estate_property_type.py @@ -0,0 +1,26 @@ +from odoo import models, fields, api + + +class EstatePropertyType(models.Model): + _name = "estate.property.type" + _description = "Type of a property" + _order = "name asc" + + name = fields.Char(required=True) + sequence = fields.Integer( + "Sequence", default=1, help="Use to order property types. Lower is better." + ) + + property_ids = fields.One2many("estate.property", "property_type_id") + offer_ids = fields.One2many("estate.property.offer", "property_type_id") + offer_count = fields.Integer(compute="_compute_offer_count") + + @api.depends("offer_ids") + def _compute_offer_count(self): + for type in self: + type.offer_count = len(type.offer_ids) + + # Constraints + _check_unique_name = models.Constraint( + "UNIQUE(name)", "A property type name should be unique" + ) diff --git a/estate/models/users.py b/estate/models/users.py new file mode 100644 index 00000000000..ee8df81e744 --- /dev/null +++ b/estate/models/users.py @@ -0,0 +1,11 @@ +from odoo import fields, models + + +class User(models.Model): + _inherit = "res.users" + + property_ids = fields.One2many( + "estate.property", + "salesperson_id", + domain=["|", ("state", "=", "new"), ("state", "=", "offer_received")], + ) diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv new file mode 100644 index 00000000000..4c593ed42e4 --- /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,access_estate_property,model_estate_property,base.group_user,1,1,1,1 +access_estate_property_type,access_estate_property_type,model_estate_property_type,base.group_user,1,1,1,1 +access_estate_property_tag,access_estate_property_tag,model_estate_property_tag,base.group_user,1,1,1,1 +access_estate_property_offer,access_estate_property_offer,model_estate_property_offer,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..3851b734f85 --- /dev/null +++ b/estate/views/estate_menus.xml @@ -0,0 +1,11 @@ + + + + + + + + + + + \ No newline at end of file diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml new file mode 100644 index 00000000000..66184f46521 --- /dev/null +++ b/estate/views/estate_property_offer_views.xml @@ -0,0 +1,30 @@ + + + + Offers + estate.property.offer + + + + + + + + + +
+

+ +

+
+ + + + + + + + + + + + + + + + + + Property Types + estate.property.type + + + + + + + + + + + Property Types + estate.property.type + list,form + + \ 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..169178e9cdd --- /dev/null +++ b/estate/views/estate_property_views.xml @@ -0,0 +1,133 @@ + + + + estate.property.search + estate.property + + + + + + + + + + + + + + + + + + + Properties + estate.property + + + + + + + + + + + + + + + + + + Properties + estate.property + + +
+ + +
+
+ +
+

+ +

+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+ + + + + + + Properties + estate.property + list,form + {'search_default_available': True} + + \ No newline at end of file diff --git a/estate/views/estate_users_views.xml b/estate/views/estate_users_views.xml new file mode 100644 index 00000000000..cc36ab117ea --- /dev/null +++ b/estate/views/estate_users_views.xml @@ -0,0 +1,14 @@ + + + res.users.view.form.inherit.estate + res.users + + + + + + + + + + \ No newline at end of file