[IMP] estate: add inheritance and module interaction features

Added model and view inheritance for extending existing functionality.,
Implemented interaction with external modules using dependencies.,
Updated estate module to demonstrate cross-module field access and method calls.

Author: kums-odoo

estate/__manifest__.py

@@ -9,6 +9,7 @@
         "views/estate_property_offer_views.xml",
         "views/estate_property_type_views.xml",
         "views/estate_menus.xml",
+        "views/inherit_res_users_view.xml",
         "security/ir.model.access.csv",
     ],
     "license": "AGPL-3",

estate/models/__init__.py

@@ -2,3 +2,4 @@
 from . import estate_property_type
 from . import estate_property_tag
 from . import estate_property_offer
+from . import inherited_res_users

estate/models/estate_property.py

@@ -1,9 +1,8 @@
 from odoo import fields, models, api
-from odoo.exceptions import UserError,ValidationError
+from odoo.exceptions import UserError, ValidationError
 from odoo.tools.float_utils import float_compare
 
 
-
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Estate Model"
@@ -30,7 +29,7 @@ class EstateProperty(models.Model):
         ],
     )
 
-    active = fields.Boolean(default=False, string="Active")
+    active = fields.Boolean(default=True, string="Active")
 
     state = fields.Selection(
         selection=[
@@ -44,7 +43,7 @@ class EstateProperty(models.Model):
         copy=False,
     )
 
-    # property will have Many to one relation with property since many properties can belong to one property type
+    # property will have Many to one relation with property type since many properties can belong to one property type
 
     property_type_id = fields.Many2one("estate.property.type", "Property Type")
 
@@ -71,17 +70,22 @@ class EstateProperty(models.Model):
 
     best_price = fields.Integer(compute="_compute_best_price", string="Best Price")
 
-
     status = fields.Char(default="new", string="Status")
 
     _order = "id desc"
 
-
-    _sql_constraints = [('check_expected_price', 'CHECK(expected_price > 0)', 'Expected price must be strictly positive'),
-        ('check_selling_price', 'CHECK(selling_price >= 0)', 'Selling price should be positive')
+    _sql_constraints = [
+        (
+            "check_expected_price",
+            "CHECK(expected_price > 0)",
+            "Expected price must be strictly positive",
+        ),
+        (
+            "check_selling_price",
+            "CHECK(selling_price >= 0)",
+            "Selling price should be positive",
+        ),
     ]
-   
-
 
     @api.depends("garden_area", "living_area")
     def _compute_total_property_area(self):
@@ -135,22 +139,29 @@ def action_cancel_property_selling(self):
             else:
                 raise UserError("Sold  property cannot be cancelled.")
 
-    # constrains for the selling price 
-
-    @api.constrains('selling_price', 'expected_price')
+    # constrains for the selling price
 
+    @api.constrains("selling_price", "expected_price")
     def _check_selling_price(self):
-        
         for data in self:
             # if call will come after selling price change than it will allow updated price to work
-            if data.selling_price <= 0 :
+            if data.selling_price <= 0:
                 return
 
-            price_float_ratio = (data.selling_price/self.expected_price)
-            ratio_diffrence = float_compare(price_float_ratio,0.9, precision_digits=2)
-            if ratio_diffrence == -1 :
+            price_float_ratio = data.selling_price / self.expected_price
+            ratio_diffrence = float_compare(price_float_ratio, 0.9, precision_digits=2)
+            if ratio_diffrence == -1:
                 data.selling_price = 0
-                raise  ValidationError('The selling price cannot be lower than 90% of the expected price')
-                
-            
+                raise ValidationError(
+                    "The selling price cannot be lower than 90% of the expected price"
+                )
+
+    # delete opration for the process
 
+    @api.ondelete(at_uninstall=False)
+    def _unlink_if_state_new_or_cancelled(self):
+        for data in self:
+            if not bool(self.state == "new" or self.state == "cancelled"):
+                raise UserError(
+                    "Can't delete property which is not in new or cancelled state!"
+                )

estate/models/estate_property_offer.py

@@ -10,6 +10,8 @@
 from datetime import datetime
 from dateutil.relativedelta import relativedelta
 from odoo.exceptions import UserError
+from odoo.tools.float_utils import float_compare
+
 
 class EstatePropertyOffer(models.Model):
     _name = 'estate.property.offer'
@@ -84,4 +86,23 @@ def action_offer_refuse(self):
             record.status = 'refused'
 
 
-    
+    
+    # now in case of offer creation CRUD
+    # self will be a proxy object , 
+    # property_id feilds is available in vals 
+    @api.model_create_multi
+    def create(self, vals):
+
+        # will check the offer value and does property has other offers which are max thw\an this one
+        for value in vals: 
+            property_details = self.env['estate.property'].browse(value.get('property_id'))
+            for property_data in property_details:
+                offers_list = property_data.mapped("offer_ids.price")
+                max_offer = max(offers_list, default=0)
+                comparison_result = float_compare(value.get('price'),max_offer, precision_digits=2)
+                
+                if comparison_result == -1 :
+                    raise UserError('Offer with a lower amount than an existing offer')
+
+        return super().create(vals)
+       

estate/models/inherited_res_users.py

@@ -0,0 +1,12 @@
+
+
+# This model inherits from res.users model 
+
+from odoo import models, fields
+
+class InheritedResUsers(models.Model):
+    _inherit = 'res.users'
+    property_ids = fields.One2many('estate.property', 'user_id', "Estate Property",
+        domain=[('state', 'in', ['new','offer_received'])]
+    )
+

estate/views/estate_property_views.xml

@@ -43,9 +43,9 @@
             <field name="arch" type="xml">
                 <form string="Property Info">
                     <header>
-                        <button name="action_sell_property" type="object" string="Sold"/>
-                         <button name="action_cancel_property_selling" type="object" string="Cancel"/>
-                          <field name="state" widget="statusbar" statusbar_visible="new,offer_received,offer_accepted, sold"/>
+                        <button name="action_sell_property" type="object" string="Mark As Sold" class="oe_highlight" invisible="state in ('sold','cancelled')"/>
+                        <button name="action_cancel_property_selling" type="object" string="Cancel" invisible="state in ('sold','cancelled')"/>
+                        <field name="state" widget="statusbar" statusbar_visible="new,offer_received,offer_accepted, sold"/>
                     </header>
 
                     <sheet>

estate/views/inherit_res_users_view.xml

@@ -0,0 +1,20 @@
+
+<odoo>
+    <data>
+
+        <record id="res_users_view_form" model="ir.ui.view">
+            <field name="name">res.users.view.form.inherit.estate.property</field>
+            <field name="model">res.users</field>
+            <field name="inherit_id" ref="base.view_users_form"/>
+            <field name="arch" type="xml">
+                <notebook position="inside">
+                    <page name="estate.properties" string="Real Estate Properties">
+                        <field name="property_ids"/>
+                        
+                    </page>
+                </notebook>
+            </field>
+        </record>
+
+    </data>
+</odoo>

estate_account/__init__.py

@@ -0,0 +1,7 @@
+{
+    "name": "Real Estate Invoicing",
+    "category": "All",
+    "application": True,
+    "installable": True,
+    "depends": ["base", "account", "estate"],
+}