[IMP] estate: Added Inheritance to module and created estate account module

Applied model and view inheritance patterns to extend core functionality

Enabled interactions between modules for enhanced business workflows

Updated business logic to support new integration features

Author: zadh-odoo

awesome_gallery/models/ir_action.py

@@ -7,4 +7,4 @@ class ActWindowView(models.Model):
 
     view_mode = fields.Selection(selection_add=[
         ('gallery', "Awesome Gallery")
-    ],  ondelete={'gallery': 'cascade'})
+    ], ondelete={'gallery': 'cascade'})

estate/__init__.py

@@ -1 +1 @@
-from . import models
+from . import models

estate/__manifest__.py

@@ -1,13 +1,11 @@
 {
-    'name':"Real Estate",
-    'summary':"This is real estate module",
-    'category':"Tutorials",
-    'description':"This is real estate module",
-    'author':"Dhruvrajsinh Zala (zadh)",
+    'name': "Real Estate",
+    'summary': "This is real estate module",
+    'category': "Tutorials",
+    'description': "This is real estate module",
+    'author': "Dhruvrajsinh Zala (zadh)",
     'installable': True,
     'application': True,
-    'data':['security/ir.model.access.csv', 'views/estate_property_offers.xml',
-    'views/estate_property_views.xml',
-    'views/estate_menus.xml',
-    'views/estate_property_type_views.xml','views/estate_property_tags.xml']
-}
+    'data': ['security/ir.model.access.csv', 'views/estate_property_views.xml', 'views/estate_property_offers.xml', 'views/estate_property_type_views.xml', 'views/estate_property_tags.xml', 'views/res_users_views.xml', 'views/estate_menus.xml'],
+    'license': 'AGPL-3'
+}

estate/models/__init__.py

@@ -2,3 +2,4 @@
 from . import esatate_property_type
 from . import estate_property_tags
 from . import estate_property_offer
+from . import inherited_res_users

estate/models/esatate_property_type.py

@@ -1,31 +1,30 @@
-from odoo import models,fields,api
+from odoo import models, fields, api
 
 
 class EstatePropertyTyeps(models.Model):
-    _name="estate.property.types"
-    _description="Types of Estate Property"
-    _order="sequence,name"
+    _name = "estate.property.types"
+    _description = "Types of Estate Property"
+    _order = "sequence, name"
 
-    _unique_type_name = models.Constraint('UNIQUE(name)','Property type name must be unique.')
+    _sql_constraints = [
+        ("_unique_type_name", "UNIQUE(name)", "Property type name must be unique.")
+    ]
 
-    name=fields.Char(required=True)
-    property_ids=fields.One2many('estate.property','property_type_id',string="Properties")
+    name = fields.Char(required=True)
+    property_ids = fields.One2many(
+        "estate.property", "property_type_id", string="Properties"
+    )
     sequence = fields.Integer(default=1)
 
     offer_ids = fields.One2many(
-        'estate.property.offer',
-        'property_type_id',
-        string="Offers"
+        "estate.property.offer", "property_type_id", string="Offers"
     )
 
-
     offer_count = fields.Integer(
-        string="Offer Count",
-        compute='_compute_offer_count',
-        store=True
+        string="Offer Count", compute="_compute_offer_count", store=True
     )
 
-    @api.depends('offer_ids')
+    @api.depends("offer_ids")
     def _compute_offer_count(self):
         for record in self:
-            record.offer_count = len(record.offer_ids)
+            record.offer_count = len(record.offer_ids)

estate/models/estate_property.py

@@ -1,110 +1,131 @@
 from dateutil.relativedelta import relativedelta
-from odoo import models,fields,api,_
-from odoo.tools.float_utils import float_compare,float_is_zero
-from odoo.exceptions import UserError,ValidationError
+from odoo import models, fields, api, _
+from odoo.tools.float_utils import float_compare, float_is_zero
+from odoo.exceptions import UserError, ValidationError
 
-class EstateProperty(models.Model):
-    _name = 'estate.property'
-    _description = 'Real Estate Property'
-    _order = 'id desc'
 
+class EstateProperty(models.Model):
+    _name = "estate.property"
+    _description = "Real Estate Property"
+    _order = "id desc"
 
-    _check_expected_price = models.Constraint('CHECK(expected_price > 0)','The expected price must be positive.')
-    _check_selling_price = models.Constraint('CHECK(selling_price >= 0)','The selling price must be positive.')
+    _sql_constraints = [
+        (
+            "_check_expected_price",
+            "CHECK(expected_price > 0)",
+            "The expected price must be positive.",
+        ),
+        (
+            "_check_selling_price",
+            "CHECK(selling_price >= 0)",
+            "The selling price must be positive.",
+        ),
+    ]
 
-    name = fields.Char(required=True,string="Title")
+    name = fields.Char(required=True, string="Title")
     description = fields.Text()
     postcode = fields.Char()
     date_availability = fields.Date(
-    default=lambda self: fields.Date.today() + relativedelta(months=3),
-    copy=False,string="Available From")
+        default=lambda self: fields.Date.today() + relativedelta(months=3),
+        copy=False,
+        string="Available From",
+    )
     expected_price = fields.Float(required=True)
-    selling_price = fields.Float(readonly=True,copy=False)
+    selling_price = fields.Float(readonly=True, copy=False)
     bedrooms = fields.Integer(default=2)
     living_area = fields.Integer(string="Living Area (sqm)")
-    total_area = fields.Float(compute='_compute_total_area',store=True)
+    total_area = fields.Float(compute="_compute_total_area", store=True)
     facades = fields.Integer()
-    garage  = fields.Boolean()
+    garage = fields.Boolean()
     garden = fields.Boolean()
     garden_area = fields.Integer(string="Garden Area (sqm)")
     active = fields.Boolean(default=True)
-    garden_orientation = fields.Selection([
-        ('north','North'),
-        ('south','South'),
-        ('east','East'),
-        ('west','West')
-    ],string="Garden Orientation")
-    state = fields.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.types", string="Property Type"
+    garden_orientation = fields.Selection(
+        [("north", "North"), ("south", "South"), ("east", "East"), ("west", "West")],
+        string="Garden Orientation",
     )
-    buyer_id = fields.Many2one(
-        "res.partner", string="Buyer", copy=False
+    state = fields.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.types", string="Property Type")
+    buyer_id = fields.Many2one("res.partner", string="Buyer", copy=False)
     salesperson_id = fields.Many2one(
         "res.users", string="Salesperson", default=lambda self: self.env.user
     )
-    tag_ids = fields.Many2many('estate.property.tags',string="Tags")
+    tag_ids = fields.Many2many("estate.property.tags", string="Tags")
 
-    offer_ids = fields.One2many('estate.property.offer','property_id',string="Offers")
+    offer_ids = fields.One2many("estate.property.offer", "property_id", string="Offers")
 
-    best_price = fields.Float(compute='_get_best_offer_price',store=True)
+    best_price = fields.Float(compute="_get_best_offer_price", store=True)
 
-    @api.depends("living_area","garden_area")
+    @api.ondelete(at_uninstall=False)
+    def _unlink_except_state_new_or_cancelled(self):
+        for record in self:
+            if record.state not in ["new", "cancelled"]:
+                raise UserError(
+                    _("You can only delete properties in 'New' or 'Cancelled' state.")
+                )
+
+    @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')
+
+    @api.depends("offer_ids.price")
     def _get_best_offer_price(self):
         for record in self:
-            prices=record.offer_ids.mapped('price')
+            prices = record.offer_ids.mapped("price")
             record.best_price = max(prices) if prices else 0.0
 
-    @api.onchange('garden')
+    @api.onchange("garden")
     def _onchange_garden(self):
         for record in self:
             if record.garden:
                 record.garden_area = 10
-                record.garden_orientation = 'north'
+                record.garden_orientation = "north"
             else:
                 record.garden_area = 0
-                record.garden_orientation = ''
+                record.garden_orientation = ""
 
     def action_set_state_sold(self):
         for record in self:
-            if(record.state == 'cancelled'):
+            if record.state == "cancelled":
                 raise UserError(_("Cancelled property cannot be sold."))
             else:
-                record.state = 'sold'
+                record.state = "sold"
         return True
 
     def action_set_state_cancel(self):
         for record in self:
-            if(record.state == 'sold'):
+            if record.state == "sold":
                 raise UserError(_("Sold property cannot be cancelled."))
             else:
-                record.state = 'cancelled'
+                record.state = "cancelled"
         return True
-        
-    @api.constrains('selling_price','expected_price')
+
+    @api.constrains("selling_price", "expected_price")
     def _check_selling_price(self):
         for record in self:
-            if not float_is_zero(record.selling_price,precision_digits=2):
-                if float_compare(
-                    record.selling_price,
-                    record.expected_price * 0.9, 
-                    precision_digits=2) < 0 :
-                    raise ValidationError(_("The selling price cannot be lower than 90% of the expected price")) 
-
-    
+            if not float_is_zero(record.selling_price, precision_digits=2):
+                if (
+                    float_compare(
+                        record.selling_price,
+                        record.expected_price * 0.9,
+                        precision_digits=2,
+                    )
+                    < 0
+                ):
+                    raise ValidationError(
+                        _(
+                            "The selling price cannot be lower than 90% of the expected price"
+                        )
+                    )

estate/models/estate_property_offer.py

@@ -1,54 +1,93 @@
-from odoo import fields,models,api,_
+from odoo import fields, models, api, _
 from odoo.exceptions import UserError
-from datetime import timedelta,datetime
+from datetime import timedelta, datetime
 
-class EstatePropertyOffer(models.Model):
-    _name="estate.property.offer"
-    _description="Offers of Estate Property"
-    _order = 'price desc'
 
-    _check_offer_price = models.Constraint('CHECK(price > 0)','The Offer price must be positive.')
+class EstatePropertyOffer(models.Model):
+    _name = "estate.property.offer"
+    _description = "Offers of Estate Property"
+    _order = "price desc"
+    _sql_constraints = [
+        ("_check_offer_price", "CHECK(price > 0)", "The Offer price must be positive.")
+    ]
 
     price = fields.Float()
-    status = fields.Selection([('accepted','Accepted'),('refused','Refused')],copy=False)
-    partner_id = fields.Many2one('res.partner',string="Partner",required=True)
-    property_id = fields.Many2one('estate.property',string="Property",required=True)
+    status = fields.Selection(
+        [("accepted", "Accepted"), ("refused", "Refused")], copy=False
+    )
+    partner_id = fields.Many2one("res.partner", string="Partner", required=True)
+    property_id = fields.Many2one("estate.property", string="Property", required=True)
     property_type_id = fields.Many2one(
-    'estate.property.types',
-    related="property_id.property_type_id",
-    string="Property Type",
-    required=True)
-
+        "estate.property.types",
+        related="property_id.property_type_id",
+        string="Property Type",
+        required=True,
+    )
     validity = fields.Integer(default=7)
-    date_deadline = fields.Date(compute="_compute_deadline",inverse="_inverse_deadline",store=True)
-    @api.depends('validity')
+    date_deadline = fields.Date(
+        compute="_compute_deadline", inverse="_inverse_deadline", store=True
+    )
+
+    @api.model_create_multi
+    def create(self, vals):
+        for val in vals:
+            property_id = val["property_id"]
+            offer_price = val["price"]
+            is_state_new = (
+                self.env["estate.property"].browse(property_id).state == "new"
+            )
+
+            result = self.read_group(
+                domain=[("property_id", "=", property_id)],
+                fields=["price:max"],
+                groupby=[],
+            )
+
+            max_price = result[0]["price"] if result else 0
+
+            if max_price >= offer_price:
+                raise UserError(
+                    _(
+                        "You cannot create an offer with a lower or equal amount than an existing offer for this property."
+                    )
+                )
+
+            if is_state_new:
+                self.env["estate.property"].browse(property_id).state = "offer_received"
+
+        return super().create(vals)
+
+    @api.depends("validity")
     def _compute_deadline(self):
         for record in self:
             create_date = record.create_date or datetime.now()
             record.date_deadline = create_date + timedelta(days=record.validity)
+
     def _inverse_deadline(self):
         for record in self:
-            create_date = record.create_date or datetime.now()    
+            create_date = record.create_date or datetime.now()
             if record.date_deadline:
                 delta = record.date_deadline - create_date.date()
                 record.validity = delta.days
-    
+
     def action_accept_offer(self):
         for record in self:
-            existing = self.search([
-                ('property_id','=',record.property_id.id),
-                ('status','=','accepted')
-            ])
+            existing = self.search(
+                [
+                    ("property_id", "=", record.property_id.id),
+                    ("status", "=", "accepted"),
+                ]
+            )
             if existing:
                 raise UserError(_("Another offer has been already accepted."))
             record.status = "accepted"
             record.property_id.state = "offer_accepted"
             record.property_id.selling_price = record.price
             record.property_id.buyer_id = record.partner_id.id
+
     def action_refuse_offer(self):
         for record in self:
-            if(record.status == 'accepted'):
+            if record.status == "accepted":
                 raise UserError(_("Accepted Offer cannot be Refused"))
             else:
                 record.status = "refused"
-