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

zadh-odoo · zadh-odoo · commit 824ce8f838f9 · 2025-07-09T19:06:09.000+05:30
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
diff --git a/awesome_gallery/models/ir_action.py b/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'})
diff --git a/estate/__init__.py b/estate/__init__.py
@@ -1 +1 @@
-from . import models
+from . import models
diff --git a/estate/__manifest__.py b/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'
+}
diff --git a/estate/models/__init__.py b/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
diff --git a/estate/models/esatate_property_type.py b/estate/models/esatate_property_type.py
@@ -1,15 +1,16 @@
-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"
 
-    _unique_type_name = models.Constraint('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 = "estate.property.types"
+    _description = "Types of Estate Property"
+    _order = "sequence, name"
+
+    _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")
     sequence = fields.Integer(default=1)
 
     offer_ids = fields.One2many(
@@ -18,7 +19,6 @@ class EstatePropertyTyeps(models.Model):
         string="Offers"
     )
 
-
     offer_count = fields.Integer(
         string="Offer Count",
         compute='_compute_offer_count',
@@ -28,4 +28,4 @@ class EstatePropertyTyeps(models.Model):
     @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)
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
@@ -1,38 +1,37 @@
 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'
 
+    _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.')]
 
-    _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.')
-
-    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')
+        ('north', 'North'),
+        ('south', 'South'),
+        ('east', 'East'),
+        ('west', 'West')
     ],string="Garden Orientation")
     state = fields.Selection(
     [
@@ -54,21 +53,27 @@ class EstateProperty(models.Model):
     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')
     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')
@@ -83,28 +88,23 @@ def _onchange_garden(self):
 
     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'
         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'
         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"))
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
@@ -1,31 +1,56 @@
-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.')
+    _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)
-
     validity = fields.Integer(default=7)
-    date_deadline = fields.Date(compute="_compute_deadline",inverse="_inverse_deadline",store=True)
+    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()    
@@ -36,19 +61,19 @@ def _inverse_deadline(self):
     def action_accept_offer(self):
         for record in self:
             existing = self.search([
-                ('property_id','=',record.property_id.id),
-                ('status','=','accepted')
+                ('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"
-
diff --git a/estate/models/estate_property_tags.py b/estate/models/estate_property_tags.py
@@ -1,11 +1,13 @@
-from odoo import fields,models
+from odoo import fields, models
 
 class EstatePropertyTags(models.Model):
-    _name="estate.property.tags"
-    _description="Estate Property Tags"
+
+
+    _name = "estate.property.tags"
+    _description = "Estate Property Tags"
     _order = "name"
 
-    _unique_tag_name = models.Constraint('UNIQUE(name)','Tag name must be unique.')
+    _sql_constraints = [('_unique_tag_name', 'UNIQUE(name)', 'Tag name must be unique.')]
 
-    name=fields.Char(required=True)
-    color=fields.Integer(default=3)
+    name = fields.Char(required=True)
+    color = fields.Integer(default=3)
diff --git a/estate/models/inherited_res_users.py b/estate/models/inherited_res_users.py
@@ -0,0 +1,10 @@
+from odoo import models, fields
+
+class InheritedResUsers(models.Model):
+
+    _inherit = 'res.users'
+
+    property_ids = fields.One2many('estate.property',
+    'salesperson_id',
+    string='Available Properties',
+    domain=[('state','in',['new','offer_received'])])
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
@@ -1,5 +1,5 @@
 id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
-estate.access_estate_property,access_estate_property,estate.model_estate_property,,1,1,1,1
+estate.access_estate_property,access_estate_property,estate.model_estate_property,base.group_user,1,1,1,1
 estate.access_estate_property_types,access_estate_property_types,estate.model_estate_property_types,base.group_user,1,1,1,1
 estate.access_estate_property_tags,access_estate_property_tags,estate.model_estate_property_tags,base.group_user,1,1,1,1
 estate.access_estate_property_offer,access_estate_property_offer,estate.model_estate_property_offer,base.group_user,1,1,1,1
diff --git a/estate/views/res_users_views.xml b/estate/views/res_users_views.xml
@@ -0,0 +1,14 @@
+<odoo>
+    <record id='view_users_form_inherit_estate' model="ir.ui.view">
+        <field name="name">res.users.form.inherit.estate</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="Properties">
+                    <field name="property_ids"/>
+                </page>
+            </notebook>
+        </field>
+    </record>
+</odoo>
diff --git a/estate_account/__init__.py b/estate_account/__init__.py
@@ -0,0 +1 @@
+from . import models
diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py
@@ -0,0 +1,7 @@
+{
+    "name": "Estate Account",
+    "depends": ["estate", "account"],
+    'category': "Tutorials",
+    "installable": True,
+    "license": "AGPL-3"
+}
diff --git a/estate_account/models/__init__.py 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
@@ -0,0 +1,35 @@
+from odoo import models, Command
+from odoo.exceptions import UserError
+
+class EstateProperty(models.Model):
+
+
+    _inherit = "estate.property"
+
+    def action_set_state_sold(self):
+        res = super().action_set_state_sold()
+        for property in self:
+            if not property.buyer_id:
+                continue
+
+            journal = self.env['account.journal'].search([
+                ('type', '=', 'sale')
+            ], limit=1)
+
+            if not journal:
+                raise UserError("No sales journal found!")
+
+            commission = property.selling_price * 0.06
+
+            admin_fee = 100.00
+
+            self.env['account.move'].create({
+                'partner_id': property.buyer_id.id,
+                'move_type': 'out_invoice',
+                'journal_id': journal.id,
+                'invoice_line_ids': [
+                    Command.create({'name': "Commission (6%)", 'quantity': 1, 'price_unit': commission }),
+                    Command.create({'name': "Administrative fees", 'quantity': 1, 'price_unit': admin_fee })
+                ]
+            })
+        return res

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-from . import models`
	`1`	`+from . import models`