From e93df221737b59a0388c4ff8942a690e8ece4953 Mon Sep 17 00:00:00 2001
From: rodh-odoo <rodh@odoo.com>
Date: Thu, 3 Jul 2025 14:18:15 +0530
Subject: [PATCH 1/7] [ADD] real_estate: one model table is created
 estate_property

one model created real estate and display in app. one table created for estate
property in database with dome database .
variable of table like name, description, date etc.
---
 real_estate/__init__.py               |  1 +
 real_estate/__manifest__.py           |  8 ++++++++
 real_estate/models/__init__.py        |  1 +
 real_estate/models/estate_property.py | 28 +++++++++++++++++++++++++++
 4 files changed, 38 insertions(+)
 create mode 100644 real_estate/__init__.py
 create mode 100644 real_estate/__manifest__.py
 create mode 100644 real_estate/models/__init__.py
 create mode 100644 real_estate/models/estate_property.py

diff --git a/real_estate/__init__.py b/real_estate/__init__.py
new file mode 100644
index 00000000000..9a7e03eded3
--- /dev/null
+++ b/real_estate/__init__.py
@@ -0,0 +1 @@
+from . import models
\ No newline at end of file
diff --git a/real_estate/__manifest__.py b/real_estate/__manifest__.py
new file mode 100644
index 00000000000..2dc80df9815
--- /dev/null
+++ b/real_estate/__manifest__.py
@@ -0,0 +1,8 @@
+{
+    'name': 'real_estate',
+     'depends': [
+        'base_setup',
+        ],
+    'application': True,
+    'installable': True,
+}
\ No newline at end of file
diff --git a/real_estate/models/__init__.py b/real_estate/models/__init__.py
new file mode 100644
index 00000000000..f4c8fd6db6d
--- /dev/null
+++ b/real_estate/models/__init__.py
@@ -0,0 +1 @@
+from . import estate_property
\ No newline at end of file
diff --git a/real_estate/models/estate_property.py b/real_estate/models/estate_property.py
new file mode 100644
index 00000000000..d0c2670c842
--- /dev/null
+++ b/real_estate/models/estate_property.py
@@ -0,0 +1,28 @@
+
+from odoo import fields, models
+
+class EstateProperty(models.Model):
+    _name = "estate.property"
+    _description = "Estate"
+
+    name = fields.Char(string="Title", required=True)
+    description = fields.Text(string="Description")
+    postcode = fields.Char(string="Postcode")
+    date_availability = fields.Date(string="Available From")
+    expected_price = fields.Float(string="Expected Price",required=True)
+    selling_price = fields.Float(string="Selling Price")
+    bedrooms = fields.Integer(string="Bedrooms")
+    living_area = fields.Integer(string="Living Area")
+    facades = fields.Integer(string="Facades")
+    garage = fields.Boolean(string="Garage")
+    garden = fields.Boolean(string="Garden")
+    garden_area = fields.Integer(string="Garden Area")
+    garden_orientation = fields.Selection(
+        [
+            ('north', 'North'),
+            ('south', 'South'),
+            ('east', 'East'),
+            ('west', 'West')
+        ],
+        string="Garden Orientation"
+    )

From 82348a415de93a783d5640c64ab1feed6d0b69f0 Mon Sep 17 00:00:00 2001
From: rodh-odoo <rodh@odoo.com>
Date: Fri, 4 Jul 2025 10:59:28 +0530
Subject: [PATCH 2/7] [ADD] real_estate: add data through csv and xml in list
 and form view

first i add data with csv file in module and after that with xml file code
 i add data that display on frontend in list view and form view both
also disable selling price so anyone cannot change it
---
 real_estate/__manifest__.py                |  9 ++-
 real_estate/data/estate_menus.xml          | 11 +++
 real_estate/data/estate_property_views.xml | 91 ++++++++++++++++++++++
 real_estate/data/ir.model.access.csv       |  2 +
 real_estate/models/estate_property.py      | 24 ++++--
 5 files changed, 131 insertions(+), 6 deletions(-)
 create mode 100644 real_estate/data/estate_menus.xml
 create mode 100644 real_estate/data/estate_property_views.xml
 create mode 100644 real_estate/data/ir.model.access.csv

diff --git a/real_estate/__manifest__.py b/real_estate/__manifest__.py
index 2dc80df9815..619aa7a9281 100644
--- a/real_estate/__manifest__.py
+++ b/real_estate/__manifest__.py
@@ -1,8 +1,15 @@
 {
     'name': 'real_estate',
      'depends': [
-        'base_setup',
+        'base',
         ],
     'application': True,
     'installable': True,
+    'license':'LGPL-3',
+    'data': [
+    'data/ir.model.access.csv',
+    'data/estate_property_views.xml',
+    'data/estate_menus.xml',
+],
+
 }
\ No newline at end of file
diff --git a/real_estate/data/estate_menus.xml b/real_estate/data/estate_menus.xml
new file mode 100644
index 00000000000..fecfeb47368
--- /dev/null
+++ b/real_estate/data/estate_menus.xml
@@ -0,0 +1,11 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<odoo>
+    <!-- Root Menu -->
+    <menuitem id="menu_real_estate_root" name="Real Estate"/>
+
+    <!-- First-Level Menu -->
+    <menuitem id="menu_real_estate_properties" name="Properties" parent="menu_real_estate_root"/>
+
+    <!-- Action Menu  -->
+    <menuitem id="menu_estate_property_action" action="action_estate_property" parent="menu_real_estate_properties"/>
+</odoo>
diff --git a/real_estate/data/estate_property_views.xml b/real_estate/data/estate_property_views.xml
new file mode 100644
index 00000000000..686f0972f4c
--- /dev/null
+++ b/real_estate/data/estate_property_views.xml
@@ -0,0 +1,91 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="view_estate_property_list" model="ir.ui.view">
+        <field name="name">estate.property.list</field>
+        <field name="model">estate.property</field>
+         <field name="type">list</field> 
+        <field name="arch" type="xml">
+            <list string="Properties">
+                <field name="name"/>
+                <field name="postcode"/>
+                <field name="state"/>
+                <field name="expected_price"/>
+                <field name="selling_price"/>
+                <field name="bedrooms"/>
+                <field name="availability_date"/>
+                <field name="active"/>
+            </list>
+        </field>
+    </record>
+
+    <record id="view_estate_property_form" model="ir.ui.view">
+        <field name="name">estate.property.form</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <form string="Estate Property">
+                <sheet>
+                <h1>
+                    <field name="name"/>
+                    </h1>
+                    <group>
+                        <group>
+                            <field name="postcode"/>
+                            <field name="availability_date"/>
+                        </group>
+                        <group>
+                            <field name="expected_price"/>
+                            <field name="selling_price" readonly="1"/>
+                        </group>
+                    </group>
+
+                    <notebook>
+                        <page string="Description">
+                            <!-- <field name="description"/> -->
+                            <group>
+                              <field name="description"/>
+                                <field name="bedrooms"/>
+                                <field name="living_area"/>
+                                <field name="facades"/>
+                                <field name="garage"/>
+                                <field name="garden"/>
+                                <field name="garden_area"/>
+                                <field name="garden_orientation"/>
+                            </group>
+                        </page>
+                    </notebook>
+                </sheet>
+            </form>
+        </field>
+    </record>
+
+   <record id="view_estate_property_search" model="ir.ui.view">
+    <field name="name">estate.property.search</field>
+    <field name="model">estate.property</field>
+    <field name="arch" type="xml">
+        <search string="Search Properties">
+            <!-- Quick search fields -->
+            <field name="name"/>
+            <field name="postcode"/>
+<separator/>
+            <!-- Filters -->
+            <filter name="filter_available"
+                    string="Available"
+                    domain="[ '|', ('state', '=', 'new'), ('state', '=', 'offer_received') ]"/>
+
+            <!-- Group By -->
+            <group expand="1" string="Group By">
+                <filter name="group_by_postcode"
+                        string="Postcode"
+                        context="{'group_by': 'postcode'}"/>
+            </group>
+        </search>
+    </field>
+</record>
+
+
+    <record id="action_estate_property" model="ir.actions.act_window">
+    <field name="name">Real estate action</field>
+    <field name="res_model">estate.property</field>
+    <field name="view_mode">list,form</field>
+   </record>
+</odoo>
diff --git a/real_estate/data/ir.model.access.csv b/real_estate/data/ir.model.access.csv
new file mode 100644
index 00000000000..976b61e8cb3
--- /dev/null
+++ b/real_estate/data/ir.model.access.csv
@@ -0,0 +1,2 @@
+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
\ No newline at end of file
diff --git a/real_estate/models/estate_property.py b/real_estate/models/estate_property.py
index d0c2670c842..d28af49a28e 100644
--- a/real_estate/models/estate_property.py
+++ b/real_estate/models/estate_property.py
@@ -1,17 +1,17 @@
-
 from odoo import fields, models
+from datetime import date, timedelta
 
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Estate"
 
-    name = fields.Char(string="Title", required=True)
+    name = fields.Char(default="Unknown", required=True)
     description = fields.Text(string="Description")
     postcode = fields.Char(string="Postcode")
-    date_availability = fields.Date(string="Available From")
+    availability_date = fields.Date(default=lambda self: date.today() + timedelta(days=90), copy=False)
     expected_price = fields.Float(string="Expected Price",required=True)
-    selling_price = fields.Float(string="Selling Price")
-    bedrooms = fields.Integer(string="Bedrooms")
+    selling_price = fields.Float(readonly=True, copy=False)
+    bedrooms = fields.Integer(default=2)
     living_area = fields.Integer(string="Living Area")
     facades = fields.Integer(string="Facades")
     garage = fields.Boolean(string="Garage")
@@ -26,3 +26,17 @@ class EstateProperty(models.Model):
         ],
         string="Garden Orientation"
     )
+    last_seen = fields.Datetime("Last Seen", default=fields.Datetime.now)
+    active = fields.Boolean(default=False)
+    state = fields.Selection(
+        selection=[
+            ('new', 'New'),
+            ('offer_received', 'Offer Received'),
+            ('offer_accepted', 'Offer Accepted'),
+            ('sold', 'Sold'),
+            ('cancelled', 'Cancelled'),
+        ],
+        required=True,
+        copy=False,
+        default='new'
+    )

From b027653faac2114b4d855814881fafcf411b2740 Mon Sep 17 00:00:00 2001
From: rodh-odoo <rodh@odoo.com>
Date: Mon, 7 Jul 2025 10:12:24 +0530
Subject: [PATCH 3/7] [ADD] real_estate: add business login in estate property
 and offers for property

implement Sold and Cancel button in estate property with validation
add offer option in property with accept and refused button
automatic update selling price when buyer accept offer
only one buyer can accept one offer through validation
---
 real_estate/__manifest__.py                   | 10 ++--
 real_estate/data/estate_menus.xml             | 11 -----
 real_estate/data/ir.model.access.csv          |  2 -
 real_estate/models/__init__.py                |  5 +-
 real_estate/models/estate_property.py         | 48 +++++++++++++++++--
 real_estate/models/estate_property_offer.py   | 48 +++++++++++++++++++
 real_estate/models/estate_property_tag.py     |  7 +++
 real_estate/models/estate_property_type.py    |  7 +++
 real_estate/security/ir.model.access.csv      |  5 ++
 real_estate/view/estate_menus.xml             | 25 ++++++++++
 .../view/estate_property_offer_views.xml      | 43 +++++++++++++++++
 .../view/estate_property_tag_views.xml        | 10 ++++
 .../view/estate_property_type_views.xml       | 11 +++++
 .../{data => view}/estate_property_views.xml  | 39 ++++++++++++++-
 14 files changed, 249 insertions(+), 22 deletions(-)
 delete mode 100644 real_estate/data/estate_menus.xml
 delete mode 100644 real_estate/data/ir.model.access.csv
 create mode 100644 real_estate/models/estate_property_offer.py
 create mode 100644 real_estate/models/estate_property_tag.py
 create mode 100644 real_estate/models/estate_property_type.py
 create mode 100644 real_estate/security/ir.model.access.csv
 create mode 100644 real_estate/view/estate_menus.xml
 create mode 100644 real_estate/view/estate_property_offer_views.xml
 create mode 100644 real_estate/view/estate_property_tag_views.xml
 create mode 100644 real_estate/view/estate_property_type_views.xml
 rename real_estate/{data => view}/estate_property_views.xml (64%)

diff --git a/real_estate/__manifest__.py b/real_estate/__manifest__.py
index 619aa7a9281..2e90223426a 100644
--- a/real_estate/__manifest__.py
+++ b/real_estate/__manifest__.py
@@ -7,9 +7,13 @@
     'installable': True,
     'license':'LGPL-3',
     'data': [
-    'data/ir.model.access.csv',
-    'data/estate_property_views.xml',
-    'data/estate_menus.xml',
+    'security/ir.model.access.csv',
+    'view/estate_property_views.xml',
+    
+    'view/estate_property_type_views.xml',
+    'view/estate_property_tag_views.xml',
+    'view/estate_property_offer_views.xml',
+    'view/estate_menus.xml',
 ],
 
 }
\ No newline at end of file
diff --git a/real_estate/data/estate_menus.xml b/real_estate/data/estate_menus.xml
deleted file mode 100644
index fecfeb47368..00000000000
--- a/real_estate/data/estate_menus.xml
+++ /dev/null
@@ -1,11 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<odoo>
-    <!-- Root Menu -->
-    <menuitem id="menu_real_estate_root" name="Real Estate"/>
-
-    <!-- First-Level Menu -->
-    <menuitem id="menu_real_estate_properties" name="Properties" parent="menu_real_estate_root"/>
-
-    <!-- Action Menu  -->
-    <menuitem id="menu_estate_property_action" action="action_estate_property" parent="menu_real_estate_properties"/>
-</odoo>
diff --git a/real_estate/data/ir.model.access.csv b/real_estate/data/ir.model.access.csv
deleted file mode 100644
index 976b61e8cb3..00000000000
--- a/real_estate/data/ir.model.access.csv
+++ /dev/null
@@ -1,2 +0,0 @@
-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
\ No newline at end of file
diff --git a/real_estate/models/__init__.py b/real_estate/models/__init__.py
index f4c8fd6db6d..09b2099fe84 100644
--- a/real_estate/models/__init__.py
+++ b/real_estate/models/__init__.py
@@ -1 +1,4 @@
-from . import estate_property
\ No newline at end of file
+from . import estate_property
+from . import estate_property_type
+from . import estate_property_tag
+from . import estate_property_offer
\ No newline at end of file
diff --git a/real_estate/models/estate_property.py b/real_estate/models/estate_property.py
index d28af49a28e..aab0cd57b8f 100644
--- a/real_estate/models/estate_property.py
+++ b/real_estate/models/estate_property.py
@@ -1,5 +1,6 @@
-from odoo import fields, models
+from odoo import fields, models, api
 from datetime import date, timedelta
+from odoo.exceptions import UserError
 
 class EstateProperty(models.Model):
     _name = "estate.property"
@@ -12,11 +13,12 @@ class EstateProperty(models.Model):
     expected_price = fields.Float(string="Expected Price",required=True)
     selling_price = fields.Float(readonly=True, copy=False)
     bedrooms = fields.Integer(default=2)
-    living_area = fields.Integer(string="Living Area")
+    living_area = fields.Float(string="Living Area")
     facades = fields.Integer(string="Facades")
     garage = fields.Boolean(string="Garage")
     garden = fields.Boolean(string="Garden")
-    garden_area = fields.Integer(string="Garden Area")
+    garden_area = fields.Float(string="Garden Area")
+    total_area = fields.Float(string="Total Area", compute="_compute_total_area")
     garden_orientation = fields.Selection(
         [
             ('north', 'North'),
@@ -40,3 +42,43 @@ class EstateProperty(models.Model):
         copy=False,
         default='new'
     )
+    property_type_id = fields.Many2one('estate.property.type', 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.tag', string="Tags")
+    offer_ids = fields.One2many('estate.property.offer', 'property_id', string="Offers")
+    best_price = fields.Float(string="Best Offer", compute="_compute_best_price")
+
+    @api.depends('living_area', 'garden_area')
+    def _compute_total_area(self):
+        for record in self:
+            record.total_area = (record.living_area or 0.0) + (record.garden_area or 0.0)
+
+    @api.depends('offer_ids.price')
+    def _compute_best_price(self):
+        for record in self:
+            prices = record.offer_ids.mapped('price')
+            record.best_price = max(prices) if prices else 0.0
+
+    @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 = False
+    
+    def action_cancel(self):
+        for record in self:
+            if record.state == 'sold':
+                raise UserError("Sold properties cannot be cancelled.")
+            record.state = 'cancelled'
+        return True
+
+    def action_sold(self):
+        for record in self:
+            if record.state == 'cancelled':
+                raise UserError("Cancelled properties cannot be sold.")
+            record.state = 'sold'
+        return True
\ No newline at end of file
diff --git a/real_estate/models/estate_property_offer.py b/real_estate/models/estate_property_offer.py
new file mode 100644
index 00000000000..d10328f6d3c
--- /dev/null
+++ b/real_estate/models/estate_property_offer.py
@@ -0,0 +1,48 @@
+from odoo import models, fields, api
+from datetime import timedelta
+from odoo.exceptions import UserError
+
+class EstatePropertyOffer(models.Model):
+    _name = "estate.property.offer"
+    _description = "Property Offer"
+
+    price = fields.Float(string="Offer Price")
+    status = fields.Selection(
+        [('accepted', 'Accepted'), ('refused', 'Refused')],
+        string="Status",
+        copy=False
+    )
+    partner_id = fields.Many2one('res.partner', string="Partner", required=True)
+    property_id = fields.Many2one('estate.property', string="Property", required=True)
+    
+    validity = fields.Integer(default=7)
+    date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline", store=True)
+
+    @api.depends('create_date', 'validity')
+    def _compute_date_deadline(self):
+        for record in self:
+            create_date = record.create_date or fields.Datetime.now()
+            record.date_deadline = create_date.date() + timedelta(days=record.validity)
+
+    def _inverse_date_deadline(self):
+        for record in self:
+            create_date = record.create_date or fields.Datetime.now()
+            delta = record.date_deadline - create_date.date()
+            record.validity = delta.days
+        
+    def action_accept(self):
+        for offer in self:
+            if offer.property_id.buyer_id:
+                raise UserError("An offer has already been accepted.")
+            offer.status = 'accepted'
+            offer.property_id.write({
+                'selling_price': offer.price,
+                'buyer_id': offer.partner_id.id,
+                'state': 'offer_accepted'
+            })
+        return True
+
+    def action_refuse(self):
+        for offer in self:
+            offer.status = 'refused'
+        return True
\ No newline at end of file
diff --git a/real_estate/models/estate_property_tag.py b/real_estate/models/estate_property_tag.py
new file mode 100644
index 00000000000..72126d2f3ea
--- /dev/null
+++ b/real_estate/models/estate_property_tag.py
@@ -0,0 +1,7 @@
+from odoo import models, fields
+
+class EstatePropertyTag(models.Model):
+    _name = "estate.property.tag"
+    _description = "Property Tag"
+
+    name = fields.Char(string="Name", required=True)
diff --git a/real_estate/models/estate_property_type.py b/real_estate/models/estate_property_type.py
new file mode 100644
index 00000000000..d3a011880d2
--- /dev/null
+++ b/real_estate/models/estate_property_type.py
@@ -0,0 +1,7 @@
+from odoo import models, fields
+
+class EstatePropertyType(models.Model):
+    _name = 'estate.property.type'
+    _description = 'Property Type'
+
+    name = fields.Char(required=True)
\ No newline at end of file
diff --git a/real_estate/security/ir.model.access.csv b/real_estate/security/ir.model.access.csv
new file mode 100644
index 00000000000..8fab5609adf
--- /dev/null
+++ b/real_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_user,access.estate.property.type.user,model_estate_property_type,base.group_user,1,1,1,1
+access_estate_property_tag_user,access.estate.property.tag.user,model_estate_property_tag,base.group_user,1,1,1,1
+access_estate_property_offer_user,access.estate.property.offer.user,model_estate_property_offer,base.group_user,1,1,1,1
diff --git a/real_estate/view/estate_menus.xml b/real_estate/view/estate_menus.xml
new file mode 100644
index 00000000000..541225da8da
--- /dev/null
+++ b/real_estate/view/estate_menus.xml
@@ -0,0 +1,25 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<odoo>
+    <!-- Root Menu -->
+    <menuitem id="menu_real_estate_root" name="Real Estate"/>
+
+    <!-- First-Level Menu -->
+    <menuitem id="menu_real_estate_properties" name="Properties" parent="menu_real_estate_root"/>
+
+    <!-- Action Menu  -->
+    <menuitem id="menu_estate_property_action" action="action_estate_property" parent="menu_real_estate_properties"/>
+
+<menuitem id="setting"
+           name="Settings"
+            parent="menu_real_estate_root"/>
+
+    <menuitem id="menu_property_type_root"
+              name="Property Types"
+              parent="setting"
+              action="estate_property_type_action"/>
+
+    <menuitem id="menu_estate_property_tag"
+              name="Property Tags"
+              parent="setting"
+              action="action_estate_property_tag"/>
+</odoo>
diff --git a/real_estate/view/estate_property_offer_views.xml b/real_estate/view/estate_property_offer_views.xml
new file mode 100644
index 00000000000..85edca2a5fb
--- /dev/null
+++ b/real_estate/view/estate_property_offer_views.xml
@@ -0,0 +1,43 @@
+<odoo>
+
+ <record id="action_estate_property_offer" model="ir.actions.act_window">
+    <field name="name">Real estate offer</field>
+    <field name="res_model">estate.property.offer</field>
+    <field name="view_mode">list,form</field>
+   </record>
+    <!-- Form View for Offer -->
+    <record id="view_estate_property_offer_form" model="ir.ui.view">
+        <field name="name">estate.property.offer.form</field>
+        <field name="model">estate.property.offer</field>
+        <field name="arch" type="xml">
+            <form string="Property Offer">
+            <header>
+    <button name="action_accept" type="object" string="Accept" class="btn-primary"/>
+    <button name="action_refuse" type="object" string="Refuse" class="btn-secondary"/>
+            </header>
+                <sheet>
+                    <group>
+                        <field name="price"/>
+                        <field name="partner_id"/>
+                        <field name="status"/>
+                    </group>
+                </sheet>
+            </form>
+        </field>
+    </record>
+
+    <!-- List View for Offer -->
+    <record id="view_estate_property_offer_list" model="ir.ui.view">
+        <field name="name">estate.property.offer.list</field>
+        <field name="model">estate.property.offer</field>
+        <field name="arch" type="xml">
+            <list string="Offers">
+                <field name="price"/>
+                <field name="partner_id"/>
+                <field name="status"/>
+                 <button name="action_accept" type="object" string="✓" icon="fa-check"/>
+            <button name="action_refuse" type="object" string="✗" icon="fa-times"/>
+            </list>
+        </field>
+    </record>
+</odoo>
diff --git a/real_estate/view/estate_property_tag_views.xml b/real_estate/view/estate_property_tag_views.xml
new file mode 100644
index 00000000000..3d43e287a63
--- /dev/null
+++ b/real_estate/view/estate_property_tag_views.xml
@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<odoo>
+    <!-- Action for Property Tags -->
+    <record id="action_estate_property_tag" model="ir.actions.act_window">
+        <field name="name">Property Tags</field>
+        <field name="res_model">estate.property.tag</field>
+        <field name="view_mode">list,form</field>
+    </record>
+</odoo>
diff --git a/real_estate/view/estate_property_type_views.xml b/real_estate/view/estate_property_type_views.xml
new file mode 100644
index 00000000000..051effcab3d
--- /dev/null
+++ b/real_estate/view/estate_property_type_views.xml
@@ -0,0 +1,11 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<odoo>
+    <record id="estate_property_type_action" model="ir.actions.act_window">
+        <field name="name">Property Types</field>
+        <field name="res_model">estate.property.type</field>
+        <field name="view_mode">list,form</field>
+    </record>
+
+    
+</odoo>
diff --git a/real_estate/data/estate_property_views.xml b/real_estate/view/estate_property_views.xml
similarity index 64%
rename from real_estate/data/estate_property_views.xml
rename to real_estate/view/estate_property_views.xml
index 686f0972f4c..b0de68d3d34 100644
--- a/real_estate/data/estate_property_views.xml
+++ b/real_estate/view/estate_property_views.xml
@@ -14,6 +14,9 @@
                 <field name="bedrooms"/>
                 <field name="availability_date"/>
                 <field name="active"/>
+                 <field name="property_type_id"/>
+                 <field name="tag_ids"/>
+          
             </list>
         </field>
     </record>
@@ -23,21 +26,28 @@
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
             <form string="Estate Property">
+             <header>
+        <button name="action_sold" type="object" string="Sold" class="btn-primary"/>
+        <button name="action_cancel" type="object" string="Cancel" class="btn-secondary"/>
+        <field name="state" widget="statusbar" statusbar_visible="new,offer_received,offer_accepted,sold,cancelled"/>
+              </header>
                 <sheet>
                 <h1>
                     <field name="name"/>
                     </h1>
                     <group>
                         <group>
+                        <field name="tag_ids" widget="many2many_tags"/>
                             <field name="postcode"/>
                             <field name="availability_date"/>
+                             <field name="property_type_id"/>
                         </group>
                         <group>
                             <field name="expected_price"/>
                             <field name="selling_price" readonly="1"/>
                         </group>
                     </group>
-
+                   
                     <notebook>
                         <page string="Description">
                             <!-- <field name="description"/> -->
@@ -50,8 +60,32 @@
                                 <field name="garden"/>
                                 <field name="garden_area"/>
                                 <field name="garden_orientation"/>
+                                <field name="total_area" readonly="1"/>
+                                <field name="best_price" readonly="1"/>
                             </group>
                         </page>
+                        
+                        <page string="Sales Info">
+                            <group>
+                                <field name="salesperson_id"/>
+                                <field name="buyer_id"/>
+                            </group>
+                        </page>
+
+                        <page string="Offers">
+                          <field name="offer_ids">
+                                <list editable="bottom">
+                                     <field name="price"/>
+                                     <field name="partner_id"/>
+                                    <field name="status"/>
+                                    <field name="validity"/>
+                                    <field name="date_deadline"/>
+                                     <button name="action_accept" type="object" icon="fa-check"/>
+                                     <button name="action_refuse" type="object" icon="fa-times"/>
+                                </list>
+                           </field>
+                         </page>
+                        
                     </notebook>
                 </sheet>
             </form>
@@ -66,7 +100,8 @@
             <!-- Quick search fields -->
             <field name="name"/>
             <field name="postcode"/>
-<separator/>
+            <field name="property_type_id"/>
+          <separator/>
             <!-- Filters -->
             <filter name="filter_available"
                     string="Available"

From f47ee0efd46325e1b717a5123063a3dd33e896b3 Mon Sep 17 00:00:00 2001
From: rodh-odoo <rodh@odoo.com>
Date: Tue, 8 Jul 2025 11:01:16 +0530
Subject: [PATCH 4/7] [IMP] real_estate: add stat button, decorations, and
 search filters for offers

Added stat button on property type to show related offers with domain filtering
Applied decorations on offer and property list views for status indication
Made 'offer' and 'tag' list views editable and added optional availability field
Set default filter for 'Available' properties, improved living area search logic
---
 real_estate/__init__.py                       |  2 +-
 real_estate/__manifest__.py                   |  5 +-
 real_estate/models/__init__.py                |  2 +-
 real_estate/models/estate_property.py         | 28 +++++--
 real_estate/models/estate_property_offer.py   | 30 +++++---
 real_estate/models/estate_property_tag.py     |  6 ++
 real_estate/models/estate_property_type.py    | 24 +++++-
 .../view/estate_property_offer_views.xml      | 28 ++++---
 .../view/estate_property_tag_views.xml        | 20 +++--
 .../view/estate_property_type_views.xml       | 48 +++++++++++-
 real_estate/view/estate_property_views.xml    | 75 ++++++++++++-------
 11 files changed, 197 insertions(+), 71 deletions(-)

diff --git a/real_estate/__init__.py b/real_estate/__init__.py
index 9a7e03eded3..0650744f6bc 100644
--- a/real_estate/__init__.py
+++ b/real_estate/__init__.py
@@ -1 +1 @@
-from . import models
\ No newline at end of file
+from . import models
diff --git a/real_estate/__manifest__.py b/real_estate/__manifest__.py
index 2e90223426a..b4b5e510e3a 100644
--- a/real_estate/__manifest__.py
+++ b/real_estate/__manifest__.py
@@ -5,7 +5,7 @@
         ],
     'application': True,
     'installable': True,
-    'license':'LGPL-3',
+    'license': 'LGPL-3',
     'data': [
     'security/ir.model.access.csv',
     'view/estate_property_views.xml',
@@ -15,5 +15,4 @@
     'view/estate_property_offer_views.xml',
     'view/estate_menus.xml',
 ],
-
-}
\ No newline at end of file
+}
diff --git a/real_estate/models/__init__.py b/real_estate/models/__init__.py
index 09b2099fe84..2f1821a39c1 100644
--- a/real_estate/models/__init__.py
+++ b/real_estate/models/__init__.py
@@ -1,4 +1,4 @@
 from . import estate_property
 from . import estate_property_type
 from . import estate_property_tag
-from . import estate_property_offer
\ No newline at end of file
+from . import estate_property_offer
diff --git a/real_estate/models/estate_property.py b/real_estate/models/estate_property.py
index aab0cd57b8f..3d7660de6a8 100644
--- a/real_estate/models/estate_property.py
+++ b/real_estate/models/estate_property.py
@@ -1,17 +1,25 @@
-from odoo import fields, models, api
+from odoo import fields, models, api, _
 from datetime import date, timedelta
-from odoo.exceptions import UserError
+from odoo.exceptions import UserError,ValidationError
+from odoo.tools.float_utils import float_compare, float_is_zero
+
 
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Estate"
+    _order = "id desc"
 
     name = fields.Char(default="Unknown", required=True)
     description = fields.Text(string="Description")
     postcode = fields.Char(string="Postcode")
     availability_date = fields.Date(default=lambda self: date.today() + timedelta(days=90), copy=False)
-    expected_price = fields.Float(string="Expected Price",required=True)
+    expected_price = fields.Float(string="Expected Price", required=True)
     selling_price = fields.Float(readonly=True, copy=False)
+    _sql_constraints = [
+    ('check_expected_price_positive', 'CHECK(expected_price > 0)', 'The expected price must be strictly positive.'),
+    ('check_selling_price_positive', 'CHECK(selling_price >= 0)', 'The selling price must be positive.')
+]
+
     bedrooms = fields.Integer(default=2)
     living_area = fields.Float(string="Living Area")
     facades = fields.Integer(string="Facades")
@@ -49,17 +57,23 @@ class EstateProperty(models.Model):
     offer_ids = fields.One2many('estate.property.offer', 'property_id', string="Offers")
     best_price = fields.Float(string="Best Offer", compute="_compute_best_price")
 
+    @api.constrains('selling_price', 'expected_price')
+    def _check_selling_price(self):
+        for record in self:
+            if float_is_zero(record.selling_price, precision_digits=2):
+                continue
+            min_acceptable_price = 0.9 * record.expected_price
+            if float_compare(record.selling_price, min_acceptable_price, precision_digits=2) < 0:
+                raise ValidationError("Selling price cannot be lower than 90% of the expected price.")
     @api.depends('living_area', 'garden_area')
     def _compute_total_area(self):
         for record in self:
             record.total_area = (record.living_area or 0.0) + (record.garden_area or 0.0)
-
     @api.depends('offer_ids.price')
     def _compute_best_price(self):
         for record in self:
             prices = record.offer_ids.mapped('price')
             record.best_price = max(prices) if prices else 0.0
-
     @api.onchange('garden')
     def _onchange_garden(self):
         if self.garden:
@@ -68,17 +82,15 @@ def _onchange_garden(self):
         else:
             self.garden_area = 0
             self.garden_orientation = False
-    
     def action_cancel(self):
         for record in self:
             if record.state == 'sold':
                 raise UserError("Sold properties cannot be cancelled.")
             record.state = 'cancelled'
         return True
-
     def action_sold(self):
         for record in self:
             if record.state == 'cancelled':
                 raise UserError("Cancelled properties cannot be sold.")
             record.state = 'sold'
-        return True
\ No newline at end of file
+        return True
diff --git a/real_estate/models/estate_property_offer.py b/real_estate/models/estate_property_offer.py
index d10328f6d3c..5d0237e746c 100644
--- a/real_estate/models/estate_property_offer.py
+++ b/real_estate/models/estate_property_offer.py
@@ -2,28 +2,35 @@
 from datetime import timedelta
 from odoo.exceptions import UserError
 
+
 class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
     _description = "Property Offer"
+    _order = "price desc"
 
     price = fields.Float(string="Offer Price")
-    status = fields.Selection(
-        [('accepted', 'Accepted'), ('refused', 'Refused')],
-        string="Status",
-        copy=False
-    )
+    state = fields.Selection([
+        ('new', 'New'),
+        ('accepted', 'Accepted'),
+        ('refused', 'Refused')
+    ], string="State",default='new', required=True)
+    _sql_constraints = [
+    ('check_price', 'CHECK(price > 0)', 'The offer price must be strictly positive.'),
+]
     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(
+        related="property_id.property_type_id",
+        store=True,
+        readonly=True
+    )
     validity = fields.Integer(default=7)
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline", store=True)
-
     @api.depends('create_date', 'validity')
     def _compute_date_deadline(self):
         for record in self:
             create_date = record.create_date or fields.Datetime.now()
             record.date_deadline = create_date.date() + timedelta(days=record.validity)
-
     def _inverse_date_deadline(self):
         for record in self:
             create_date = record.create_date or fields.Datetime.now()
@@ -34,15 +41,14 @@ def action_accept(self):
         for offer in self:
             if offer.property_id.buyer_id:
                 raise UserError("An offer has already been accepted.")
-            offer.status = 'accepted'
+            offer.state = 'accepted'
             offer.property_id.write({
                 'selling_price': offer.price,
                 'buyer_id': offer.partner_id.id,
                 'state': 'offer_accepted'
             })
         return True
-
     def action_refuse(self):
         for offer in self:
-            offer.status = 'refused'
-        return True
\ No newline at end of file
+            offer.state = 'refused'
+        return True
diff --git a/real_estate/models/estate_property_tag.py b/real_estate/models/estate_property_tag.py
index 72126d2f3ea..ccad0243921 100644
--- a/real_estate/models/estate_property_tag.py
+++ b/real_estate/models/estate_property_tag.py
@@ -1,7 +1,13 @@
 from odoo import models, fields
 
+
 class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "Property Tag"
+    _order = "name"
 
     name = fields.Char(string="Name", required=True)
+    color = fields.Integer('Color')
+    _sql_constraints = [
+    ('unique_tag_name', 'UNIQUE(name)', 'A property tag name must be unique')
+]
diff --git a/real_estate/models/estate_property_type.py b/real_estate/models/estate_property_type.py
index d3a011880d2..d4ede99d3cf 100644
--- a/real_estate/models/estate_property_type.py
+++ b/real_estate/models/estate_property_type.py
@@ -1,7 +1,27 @@
-from odoo import models, fields
+from odoo import models, fields, api
+
 
 class EstatePropertyType(models.Model):
     _name = 'estate.property.type'
     _description = 'Property Type'
+    _order = "sequence, name"
 
-    name = fields.Char(required=True)
\ No newline at end of file
+    name = fields.Char(required=True)
+    _sql_constraints = [
+    ('unique_property_type', 'UNIQUE(name)', 'A property type name must be unique')
+]
+    property_ids = fields.One2many("estate.property", "property_type_id", string="Properties")
+    sequence = fields.Integer(string="Sequence", default=10)
+    offer_ids = fields.One2many(
+        "estate.property.offer",
+        "property_type_id",
+        string="Offers"
+    )
+    offer_count = fields.Integer(
+        string="Offers Count",
+        compute="_compute_offer_count"
+    )
+    @api.depends("offer_ids")
+    def _compute_offer_count(self):
+        for record in self:
+            record.offer_count = len(record.offer_ids)
diff --git a/real_estate/view/estate_property_offer_views.xml b/real_estate/view/estate_property_offer_views.xml
index 85edca2a5fb..342aef5bcea 100644
--- a/real_estate/view/estate_property_offer_views.xml
+++ b/real_estate/view/estate_property_offer_views.xml
@@ -1,43 +1,53 @@
 <odoo>
-
+<data>
  <record id="action_estate_property_offer" model="ir.actions.act_window">
     <field name="name">Real estate offer</field>
     <field name="res_model">estate.property.offer</field>
     <field name="view_mode">list,form</field>
+    <field name="domain">[('property_type_id', '=', active_id)]</field>
    </record>
-    <!-- Form View for Offer -->
+
     <record id="view_estate_property_offer_form" model="ir.ui.view">
         <field name="name">estate.property.offer.form</field>
         <field name="model">estate.property.offer</field>
         <field name="arch" type="xml">
             <form string="Property Offer">
-            <header>
-    <button name="action_accept" type="object" string="Accept" class="btn-primary"/>
-    <button name="action_refuse" type="object" string="Refuse" class="btn-secondary"/>
+           <header>
+    <button name="action_accept" string="Accept" type="object"
+            class="btn-primary"
+            invisible="state != 'accepted'"/>
+
+    <button name="action_refuse" string="Refuse" type="object"
+            class="btn-secondary"
+            invisible="state == 'accepted'"/>
             </header>
+
                 <sheet>
                     <group>
                         <field name="price"/>
                         <field name="partner_id"/>
-                        <field name="status"/>
+                        <field name="state"/>
                     </group>
                 </sheet>
             </form>
         </field>
     </record>
 
-    <!-- List View for Offer -->
+
     <record id="view_estate_property_offer_list" model="ir.ui.view">
         <field name="name">estate.property.offer.list</field>
         <field name="model">estate.property.offer</field>
         <field name="arch" type="xml">
-            <list string="Offers">
+            <list editable="bottom" string="Offers"
+              decoration-danger="state == 'refused'"
+              decoration-success="state == 'accepted'">
                 <field name="price"/>
                 <field name="partner_id"/>
-                <field name="status"/>
+                <field name="state" invisible="1"/>
                  <button name="action_accept" type="object" string="✓" icon="fa-check"/>
             <button name="action_refuse" type="object" string="✗" icon="fa-times"/>
             </list>
         </field>
     </record>
+    </data>
 </odoo>
diff --git a/real_estate/view/estate_property_tag_views.xml b/real_estate/view/estate_property_tag_views.xml
index 3d43e287a63..12ae069fac0 100644
--- a/real_estate/view/estate_property_tag_views.xml
+++ b/real_estate/view/estate_property_tag_views.xml
@@ -1,10 +1,20 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
 <odoo>
-    <!-- Action for Property Tags -->
+   <record id="view_estate_property_tag_list" model="ir.ui.view">
+    <field name="name">estate.property.tag.list</field>
+    <field name="model">estate.property.tag</field>
+    <field name="arch" type="xml">
+        <list editable="bottom">
+            <field name="name"/>
+            <field name="color" widget="color_picker"/>
+        </list>
+    </field>
+</record>
+
     <record id="action_estate_property_tag" model="ir.actions.act_window">
-        <field name="name">Property Tags</field>
-        <field name="res_model">estate.property.tag</field>
-        <field name="view_mode">list,form</field>
-    </record>
+    <field name="name">Property Tags</field>
+    <field name="res_model">estate.property.tag</field>
+    <field name="view_mode">list,form</field>
+</record>
 </odoo>
diff --git a/real_estate/view/estate_property_type_views.xml b/real_estate/view/estate_property_type_views.xml
index 051effcab3d..b83b6c8d4a0 100644
--- a/real_estate/view/estate_property_type_views.xml
+++ b/real_estate/view/estate_property_type_views.xml
@@ -1,11 +1,53 @@
 <?xml version="1.0" encoding="UTF-8"?>
-
 <odoo>
+<data>
     <record id="estate_property_type_action" model="ir.actions.act_window">
         <field name="name">Property Types</field>
         <field name="res_model">estate.property.type</field>
         <field name="view_mode">list,form</field>
     </record>
-
-    
+    <record id="estate_property_type_form_view" model="ir.ui.view">
+        <field name="name">estate.property.type.form</field>
+        <field name="model">estate.property.type</field>
+        
+        <field name="arch" type="xml">
+            <form string="Property Type">
+            <div class="oe_button_box" name="button_box">
+             <button name="%(action_estate_property_offer)d"
+            type="action"
+            class="oe_stat_button"
+            icon="fa-money" title="Action">
+        <field name="offer_count" widget="statinfo" string="Offers"/>
+             </button>
+             </div>
+                <sheet>
+                    <group>
+                        <field name="name"/>
+                    </group>
+                    <notebook>
+                        <page string="Properties">
+                            <field name="property_ids">
+                                <list string="Properties">
+                                    <field name="name"/>
+                                    <field name="expected_price"/>
+                                    <field name="state"/>
+                                </list>
+                            </field>
+                        </page>
+                    </notebook>
+                </sheet>
+            </form>
+        </field>
+    </record>
+    <record id="estate_property_type_tree_view" model="ir.ui.view">
+        <field name="name">estate.property.type.tree</field>
+        <field name="model">estate.property.type</field>
+        <field name="arch" type="xml">
+            <list string="Property Types">
+                 <field name="sequence" widget="handle"/>
+                 <field name="name"/>
+            </list>
+        </field>
+    </record>
+    </data>
 </odoo>
diff --git a/real_estate/view/estate_property_views.xml b/real_estate/view/estate_property_views.xml
index b0de68d3d34..81ea2327878 100644
--- a/real_estate/view/estate_property_views.xml
+++ b/real_estate/view/estate_property_views.xml
@@ -1,16 +1,21 @@
 <?xml version="1.0"?>
 <odoo>
+<data>
     <record id="view_estate_property_list" model="ir.ui.view">
         <field name="name">estate.property.list</field>
         <field name="model">estate.property</field>
-         <field name="type">list</field> 
         <field name="arch" type="xml">
-            <list string="Properties">
+            <list string="Properties"
+              decoration-success="state == 'offer_received'"
+              decoration-info="state == 'offer_accepted'"
+              decoration-muted="state == 'sold'">
                 <field name="name"/>
                 <field name="postcode"/>
                 <field name="state"/>
                 <field name="expected_price"/>
                 <field name="selling_price"/>
+                <field name="availability_date" optional="show" invisible="1"/>
+                <field name="state"/>
                 <field name="bedrooms"/>
                 <field name="availability_date"/>
                 <field name="active"/>
@@ -27,20 +32,26 @@
         <field name="arch" type="xml">
             <form string="Estate Property">
              <header>
-        <button name="action_sold" type="object" string="Sold" class="btn-primary"/>
-        <button name="action_cancel" type="object" string="Cancel" class="btn-secondary"/>
-        <field name="state" widget="statusbar" statusbar_visible="new,offer_received,offer_accepted,sold,cancelled"/>
-              </header>
+    <button name="action_sold" string="Sold" type="object"
+            class="btn-primary"
+            invisible="state != 'offer_accepted'"/>
+
+    <button name="action_cancel" string="Cancel" type="object"
+            class="btn-secondary"
+            invisible="state in ('sold', 'cancelled')"/>
+            </header>
                 <sheet>
                 <h1>
                     <field name="name"/>
                     </h1>
                     <group>
                         <group>
-                        <field name="tag_ids" widget="many2many_tags"/>
+                        <field name="tag_ids" widget="many2many_tags" options="{'color_field': 'color'}"/>
+
                             <field name="postcode"/>
                             <field name="availability_date"/>
-                             <field name="property_type_id"/>
+                             <field name="property_type_id" options="{'no_create': True, 'no_open': True}"/>
+
                         </group>
                         <group>
                             <field name="expected_price"/>
@@ -50,16 +61,16 @@
                    
                     <notebook>
                         <page string="Description">
-                            <!-- <field name="description"/> -->
                             <group>
                               <field name="description"/>
                                 <field name="bedrooms"/>
                                 <field name="living_area"/>
                                 <field name="facades"/>
                                 <field name="garage"/>
-                                <field name="garden"/>
-                                <field name="garden_area"/>
-                                <field name="garden_orientation"/>
+                                <field name="active"/>
+                               <field name="garden"/>
+                               <field name="garden_area" invisible="not garden"/>
+                               <field name="garden_orientation" invisible="not garden"/>
                                 <field name="total_area" readonly="1"/>
                                 <field name="best_price" readonly="1"/>
                             </group>
@@ -74,18 +85,24 @@
 
                         <page string="Offers">
                           <field name="offer_ids">
-                                <list editable="bottom">
-                                     <field name="price"/>
-                                     <field name="partner_id"/>
-                                    <field name="status"/>
-                                    <field name="validity"/>
-                                    <field name="date_deadline"/>
-                                     <button name="action_accept" type="object" icon="fa-check"/>
-                                     <button name="action_refuse" type="object" icon="fa-times"/>
-                                </list>
-                           </field>
+                         <list editable="bottom" 
+                         decoration-success="state == 'accepted'"
+              decoration-danger="state == 'refused'">
+                           <field name="price"/>
+                           <field name="partner_id"/>
+                           <field name="state"/>
+                           <field name="validity"/>
+                           <field name="date_deadline"/>
+                          <button name="action_accept" string="Accept" type="object"
+                            class="btn-primary" title="Accept Offer"
+                            invisible="state != 'accepted'"/>
+                          <button name="action_refuse" string="Refuse" type="object"
+                             class="btn-secondary" title="Refuce Offer"
+                             invisible="state == 'accepted'"/>
+                       </list>
+                   </field>
+
                          </page>
-                        
                     </notebook>
                 </sheet>
             </form>
@@ -97,17 +114,20 @@
     <field name="model">estate.property</field>
     <field name="arch" type="xml">
         <search string="Search Properties">
-            <!-- Quick search fields -->
+          <filter name="available" string="Available" domain="[('state', '=', 'new')]"/>
+
             <field name="name"/>
+            <field name="living_area" string="Living Area"
+       filter_domain="[('living_area', '>=', self)]"/>
+
             <field name="postcode"/>
             <field name="property_type_id"/>
           <separator/>
-            <!-- Filters -->
+        
             <filter name="filter_available"
                     string="Available"
                     domain="[ '|', ('state', '=', 'new'), ('state', '=', 'offer_received') ]"/>
 
-            <!-- Group By -->
             <group expand="1" string="Group By">
                 <filter name="group_by_postcode"
                         string="Postcode"
@@ -117,10 +137,11 @@
     </field>
 </record>
 
-
     <record id="action_estate_property" model="ir.actions.act_window">
     <field name="name">Real estate action</field>
     <field name="res_model">estate.property</field>
     <field name="view_mode">list,form</field>
+    <field name="context">{'search_default_available': 1}</field>
    </record>
+   </data>
 </odoo>

From 56ea6cc8a168c498d2ac8fc817e0f8c6cdef7296 Mon Sep 17 00:00:00 2001
From: rodh-odoo <rodh@odoo.com>
Date: Wed, 9 Jul 2025 10:25:32 +0530
Subject: [PATCH 5/7] [ADD] estate_account: generate invoice for property sold
 through inheritance
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Added logic to create a customer invoice when a property is marked as Sold
Invoice includes 6% commission and ₹100 administrative fees
Display property in user which buy by them
only can delete property if it is in new or cancelled state
---
 estate_account/__init__.py                    |  1 +
 estate_account/__manifest__.py                |  8 ++++
 estate_account/models/__init__.py             |  1 +
 estate_account/models/estate_property.py      | 40 +++++++++++++++++++
 real_estate/__manifest__.py                   |  4 +-
 real_estate/models/__init__.py                |  1 +
 real_estate/models/estate_property.py         | 25 ++++++++----
 real_estate/models/estate_property_offer.py   | 25 +++++++++---
 real_estate/models/estate_property_type.py    |  1 +
 real_estate/models/estate_res_users.py        | 12 ++++++
 .../view/estate_property_offer_views.xml      |  3 +-
 real_estate/view/estate_property_views.xml    | 13 +++---
 real_estate/view/estate_res_users_view.xml    | 21 ++++++++++
 13 files changed, 132 insertions(+), 23 deletions(-)
 create mode 100644 estate_account/__init__.py
 create mode 100644 estate_account/__manifest__.py
 create mode 100644 estate_account/models/__init__.py
 create mode 100644 estate_account/models/estate_property.py
 create mode 100644 real_estate/models/estate_res_users.py
 create mode 100644 real_estate/view/estate_res_users_view.xml

diff --git a/estate_account/__init__.py b/estate_account/__init__.py
new file mode 100644
index 00000000000..0650744f6bc
--- /dev/null
+++ b/estate_account/__init__.py
@@ -0,0 +1 @@
+from . import models
diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py
new file mode 100644
index 00000000000..2c2d43955c5
--- /dev/null
+++ b/estate_account/__manifest__.py
@@ -0,0 +1,8 @@
+{
+    'name': 'Estate Accounting',
+    'version': '1.0',
+    'depends': ['base','real_estate', 'account'],
+    'category': 'Real Estate',
+    'installable': True,
+    'application': False,
+}
diff --git a/estate_account/models/__init__.py b/estate_account/models/__init__.py
new file mode 100644
index 00000000000..5e1963c9d2f
--- /dev/null
+++ 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
new file mode 100644
index 00000000000..1e2ce0d61bd
--- /dev/null
+++ b/estate_account/models/estate_property.py
@@ -0,0 +1,40 @@
+from odoo import models, Command
+from odoo.exceptions import UserError
+
+
+class EstateProperty(models.Model):
+    _inherit = 'estate.property'
+
+    def action_sold(self):
+        res = super().action_sold()
+
+        for record in self:
+            if not record.buyer_id:
+                raise UserError("Please set a Buyer before generating an invoice.")
+            if not record.selling_price:
+                raise UserError("Please set a Selling Price before generating an invoice.")
+
+            journal = self.env['account.journal'].search([('type', '=', 'sale')], limit=1)
+            if not journal:
+                raise UserError("No sale journal found. Please configure at least one sale journal.")
+
+            invoice_vals = {
+                "partner_id": record.buyer_id.id,
+                "move_type": "out_invoice",
+                "journal_id": journal.id,
+                "invoice_line_ids": [
+                    Command.create({
+                        "name": "6% Commission",
+                        "quantity": 1,
+                        "price_unit": 0.06 * record.selling_price,
+                    }),
+                    Command.create({
+                        "name": "Administrative Fees",
+                        "quantity": 1,
+                        "price_unit": 100.0,
+                    }),
+                ]
+            }
+
+            self.env["account.move"].create(invoice_vals)
+        return res
diff --git a/real_estate/__manifest__.py b/real_estate/__manifest__.py
index b4b5e510e3a..f24523d1389 100644
--- a/real_estate/__manifest__.py
+++ b/real_estate/__manifest__.py
@@ -9,10 +9,10 @@
     'data': [
     'security/ir.model.access.csv',
     'view/estate_property_views.xml',
-    
+    'view/estate_property_offer_views.xml',
     'view/estate_property_type_views.xml',
     'view/estate_property_tag_views.xml',
-    'view/estate_property_offer_views.xml',
     'view/estate_menus.xml',
+    'view/estate_res_users_view.xml'
 ],
 }
diff --git a/real_estate/models/__init__.py b/real_estate/models/__init__.py
index 2f1821a39c1..06001dca391 100644
--- a/real_estate/models/__init__.py
+++ b/real_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 estate_res_users
diff --git a/real_estate/models/estate_property.py b/real_estate/models/estate_property.py
index 3d7660de6a8..2e2daa28ace 100644
--- a/real_estate/models/estate_property.py
+++ b/real_estate/models/estate_property.py
@@ -1,4 +1,4 @@
-from odoo import fields, models, api, _
+from odoo import fields, models, api
 from datetime import date, timedelta
 from odoo.exceptions import UserError,ValidationError
 from odoo.tools.float_utils import float_compare, float_is_zero
@@ -19,7 +19,6 @@ class EstateProperty(models.Model):
     ('check_expected_price_positive', 'CHECK(expected_price > 0)', 'The expected price must be strictly positive.'),
     ('check_selling_price_positive', 'CHECK(selling_price >= 0)', 'The selling price must be positive.')
 ]
-
     bedrooms = fields.Integer(default=2)
     living_area = fields.Float(string="Living Area")
     facades = fields.Integer(string="Facades")
@@ -33,7 +32,7 @@ class EstateProperty(models.Model):
             ('south', 'South'),
             ('east', 'East'),
             ('west', 'West')
-        ],
+        ], 
         string="Garden Orientation"
     )
     last_seen = fields.Datetime("Last Seen", default=fields.Datetime.now)
@@ -45,9 +44,9 @@ class EstateProperty(models.Model):
             ('offer_accepted', 'Offer Accepted'),
             ('sold', 'Sold'),
             ('cancelled', 'Cancelled'),
-        ],
-        required=True,
-        copy=False,
+        ], 
+        required=True, 
+        copy=False, 
         default='new'
     )
     property_type_id = fields.Many2one('estate.property.type', string="Property Type")
@@ -60,20 +59,24 @@ class EstateProperty(models.Model):
     @api.constrains('selling_price', 'expected_price')
     def _check_selling_price(self):
         for record in self:
-            if float_is_zero(record.selling_price, precision_digits=2):
+            if (float_is_zero(record.selling_price, precision_digits=2)
+            or record.state not in ['offer_accepted', 'sold']):
                 continue
             min_acceptable_price = 0.9 * record.expected_price
             if float_compare(record.selling_price, min_acceptable_price, precision_digits=2) < 0:
                 raise ValidationError("Selling price cannot be lower than 90% of the expected price.")
+
     @api.depends('living_area', 'garden_area')
     def _compute_total_area(self):
         for record in self:
             record.total_area = (record.living_area or 0.0) + (record.garden_area or 0.0)
+
     @api.depends('offer_ids.price')
     def _compute_best_price(self):
         for record in self:
             prices = record.offer_ids.mapped('price')
             record.best_price = max(prices) if prices else 0.0
+
     @api.onchange('garden')
     def _onchange_garden(self):
         if self.garden:
@@ -82,15 +85,23 @@ def _onchange_garden(self):
         else:
             self.garden_area = 0
             self.garden_orientation = False
+
     def action_cancel(self):
         for record in self:
             if record.state == 'sold':
                 raise UserError("Sold properties cannot be cancelled.")
             record.state = 'cancelled'
         return True
+
     def action_sold(self):
         for record in self:
             if record.state == 'cancelled':
                 raise UserError("Cancelled properties cannot be sold.")
             record.state = 'sold'
         return True
+
+    @api.ondelete(at_uninstall=False)
+    def _check_deletion(self):
+       for record in self:
+           if record.state not in ['new', 'cancelled']:
+            raise UserError("You can only delete a property if its state is 'New' or 'Cancelled'.")
diff --git a/real_estate/models/estate_property_offer.py b/real_estate/models/estate_property_offer.py
index 5d0237e746c..9795889ee2c 100644
--- a/real_estate/models/estate_property_offer.py
+++ b/real_estate/models/estate_property_offer.py
@@ -1,6 +1,6 @@
 from odoo import models, fields, api
 from datetime import timedelta
-from odoo.exceptions import UserError
+from odoo.exceptions import UserError, ValidationError
 
 
 class EstatePropertyOffer(models.Model):
@@ -13,7 +13,7 @@ class EstatePropertyOffer(models.Model):
         ('new', 'New'),
         ('accepted', 'Accepted'),
         ('refused', 'Refused')
-    ], string="State",default='new', required=True)
+    ], string="State", default='new', required=True)
     _sql_constraints = [
     ('check_price', 'CHECK(price > 0)', 'The offer price must be strictly positive.'),
 ]
@@ -26,29 +26,44 @@ class EstatePropertyOffer(models.Model):
     )
     validity = fields.Integer(default=7)
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline", store=True)
+
     @api.depends('create_date', 'validity')
     def _compute_date_deadline(self):
         for record in self:
             create_date = record.create_date or fields.Datetime.now()
             record.date_deadline = create_date.date() + timedelta(days=record.validity)
+
     def _inverse_date_deadline(self):
         for record in self:
             create_date = record.create_date or fields.Datetime.now()
             delta = record.date_deadline - create_date.date()
             record.validity = delta.days
-        
+
     def action_accept(self):
         for offer in self:
             if offer.property_id.buyer_id:
                 raise UserError("An offer has already been accepted.")
             offer.state = 'accepted'
             offer.property_id.write({
-                'selling_price': offer.price,
-                'buyer_id': offer.partner_id.id,
+                'selling_price': offer.price, 
+                'buyer_id': offer.partner_id.id, 
                 'state': 'offer_accepted'
             })
         return True
+
     def action_refuse(self):
         for offer in self:
             offer.state = 'refused'
         return True
+
+    @api.model_create_multi
+    def create(self, vals):
+        property_id = vals.get('property_id')
+        price = vals.get('price')
+        property_obj = self.env['estate.property'].browse(property_id)
+        existing_offers = property_obj.offer_ids.filtered(lambda o: o.price >= price)
+        if existing_offers:
+            raise ValidationError("You cannot create an offer with a lower or equal price than existing offers.")
+        property_obj.state = 'offer_received'
+
+        return super().create(vals)
diff --git a/real_estate/models/estate_property_type.py b/real_estate/models/estate_property_type.py
index d4ede99d3cf..ba2be36598b 100644
--- a/real_estate/models/estate_property_type.py
+++ b/real_estate/models/estate_property_type.py
@@ -21,6 +21,7 @@ class EstatePropertyType(models.Model):
         string="Offers Count",
         compute="_compute_offer_count"
     )
+
     @api.depends("offer_ids")
     def _compute_offer_count(self):
         for record in self:
diff --git a/real_estate/models/estate_res_users.py b/real_estate/models/estate_res_users.py
new file mode 100644
index 00000000000..4a5c497fbc6
--- /dev/null
+++ b/real_estate/models/estate_res_users.py
@@ -0,0 +1,12 @@
+from odoo import models, fields
+
+
+class ResUsers(models.Model):
+    _inherit = 'res.users'
+
+    property_ids = fields.One2many(
+        'estate.property',
+        'salesperson_id',
+        string="Properties",
+        domain=[('state', '!=', 'cancelled')]
+    )
diff --git a/real_estate/view/estate_property_offer_views.xml b/real_estate/view/estate_property_offer_views.xml
index 342aef5bcea..1d73ef19abc 100644
--- a/real_estate/view/estate_property_offer_views.xml
+++ b/real_estate/view/estate_property_offer_views.xml
@@ -33,7 +33,6 @@
         </field>
     </record>
 
-
     <record id="view_estate_property_offer_list" model="ir.ui.view">
         <field name="name">estate.property.offer.list</field>
         <field name="model">estate.property.offer</field>
@@ -45,7 +44,7 @@
                 <field name="partner_id"/>
                 <field name="state" invisible="1"/>
                  <button name="action_accept" type="object" string="✓" icon="fa-check"/>
-            <button name="action_refuse" type="object" string="✗" icon="fa-times"/>
+                <button name="action_refuse" type="object" string="✗" icon="fa-times"/>
             </list>
         </field>
     </record>
diff --git a/real_estate/view/estate_property_views.xml b/real_estate/view/estate_property_views.xml
index 81ea2327878..01486ac51b0 100644
--- a/real_estate/view/estate_property_views.xml
+++ b/real_estate/view/estate_property_views.xml
@@ -39,6 +39,7 @@
     <button name="action_cancel" string="Cancel" type="object"
             class="btn-secondary"
             invisible="state in ('sold', 'cancelled')"/>
+             <field name="state" widget="statusbar"/>
             </header>
                 <sheet>
                 <h1>
@@ -58,7 +59,6 @@
                             <field name="selling_price" readonly="1"/>
                         </group>
                     </group>
-                   
                     <notebook>
                         <page string="Description">
                             <group>
@@ -75,7 +75,6 @@
                                 <field name="best_price" readonly="1"/>
                             </group>
                         </page>
-                        
                         <page string="Sales Info">
                             <group>
                                 <field name="salesperson_id"/>
@@ -94,11 +93,11 @@
                            <field name="validity"/>
                            <field name="date_deadline"/>
                           <button name="action_accept" string="Accept" type="object"
-                            class="btn-primary" title="Accept Offer"
-                            invisible="state != 'accepted'"/>
-                          <button name="action_refuse" string="Refuse" type="object"
-                             class="btn-secondary" title="Refuce Offer"
-                             invisible="state == 'accepted'"/>
+            class="btn-primary"
+            invisible="state in ('accepted','refused')"/>
+    <button name="action_refuse" string="Refuse" type="object"
+            class="btn-secondary"
+            invisible="state in ('accepted','refused')"/>
                        </list>
                    </field>
 
diff --git a/real_estate/view/estate_res_users_view.xml b/real_estate/view/estate_res_users_view.xml
new file mode 100644
index 00000000000..c8be904b8c9
--- /dev/null
+++ b/real_estate/view/estate_res_users_view.xml
@@ -0,0 +1,21 @@
+<odoo>
+    <record id="view_users_form_inherit_real_estate" model="ir.ui.view">
+        <field name="name">res.users.form.inherit.real.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 string="Properties">
+                    <field name="property_ids">
+                        <list>
+                            <field name="name"/>
+                            <field name="state"/>
+                            <field name="expected_price"/>
+                             <field name="selling_price"/>
+                        </list>
+                    </field>
+                </page>
+            </notebook>
+        </field>
+    </record>
+</odoo>

From 32ad5b78dab7c0099dc21038cdad33be7c68eb0a Mon Sep 17 00:00:00 2001
From: rodh-odoo <rodh@odoo.com>
Date: Wed, 9 Jul 2025 14:02:49 +0530
Subject: [PATCH 6/7] [FIX] estate_account: fixed check style and tutorial
 changes

 fixed check style and tutorial changes in estate_account and estate_property
---
 estate_account/__manifest__.py                |  3 ++-
 real_estate/models/estate_property.py         | 16 +++++++-------
 real_estate/models/estate_property_offer.py   | 22 +++++++++----------
 .../view/estate_property_offer_views.xml      |  3 ---
 real_estate/view/estate_property_views.xml    | 13 ++++-------
 5 files changed, 25 insertions(+), 32 deletions(-)

diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py
index 2c2d43955c5..8a9983d2198 100644
--- a/estate_account/__manifest__.py
+++ b/estate_account/__manifest__.py
@@ -1,8 +1,9 @@
 {
     'name': 'Estate Accounting',
     'version': '1.0',
-    'depends': ['base','real_estate', 'account'],
+    'depends': ['real_estate', 'account'],
     'category': 'Real Estate',
+    'license': 'LGPL-3',
     'installable': True,
     'application': False,
 }
diff --git a/real_estate/models/estate_property.py b/real_estate/models/estate_property.py
index 2e2daa28ace..016aa604f29 100644
--- a/real_estate/models/estate_property.py
+++ b/real_estate/models/estate_property.py
@@ -1,6 +1,6 @@
 from odoo import fields, models, api
 from datetime import date, timedelta
-from odoo.exceptions import UserError,ValidationError
+from odoo.exceptions import UserError, ValidationError
 from odoo.tools.float_utils import float_compare, float_is_zero
 
 
@@ -32,7 +32,7 @@ class EstateProperty(models.Model):
             ('south', 'South'),
             ('east', 'East'),
             ('west', 'West')
-        ], 
+        ],
         string="Garden Orientation"
     )
     last_seen = fields.Datetime("Last Seen", default=fields.Datetime.now)
@@ -44,9 +44,9 @@ class EstateProperty(models.Model):
             ('offer_accepted', 'Offer Accepted'),
             ('sold', 'Sold'),
             ('cancelled', 'Cancelled'),
-        ], 
-        required=True, 
-        copy=False, 
+        ],
+        required=True,
+        copy=False,
         default='new'
     )
     property_type_id = fields.Many2one('estate.property.type', string="Property Type")
@@ -102,6 +102,6 @@ def action_sold(self):
 
     @api.ondelete(at_uninstall=False)
     def _check_deletion(self):
-       for record in self:
-           if record.state not in ['new', 'cancelled']:
-            raise UserError("You can only delete a property if its state is 'New' or 'Cancelled'.")
+        for record in self:
+            if record.state not in ['new', 'cancelled']:
+                raise UserError("You can only delete a property if its state is 'New' or 'Cancelled'.")
diff --git a/real_estate/models/estate_property_offer.py b/real_estate/models/estate_property_offer.py
index 9795889ee2c..9facb3e00b8 100644
--- a/real_estate/models/estate_property_offer.py
+++ b/real_estate/models/estate_property_offer.py
@@ -45,8 +45,8 @@ def action_accept(self):
                 raise UserError("An offer has already been accepted.")
             offer.state = 'accepted'
             offer.property_id.write({
-                'selling_price': offer.price, 
-                'buyer_id': offer.partner_id.id, 
+                'selling_price': offer.price,
+                'buyer_id': offer.partner_id.id,
                 'state': 'offer_accepted'
             })
         return True
@@ -57,13 +57,13 @@ def action_refuse(self):
         return True
 
     @api.model_create_multi
-    def create(self, vals):
-        property_id = vals.get('property_id')
-        price = vals.get('price')
-        property_obj = self.env['estate.property'].browse(property_id)
-        existing_offers = property_obj.offer_ids.filtered(lambda o: o.price >= price)
-        if existing_offers:
-            raise ValidationError("You cannot create an offer with a lower or equal price than existing offers.")
-        property_obj.state = 'offer_received'
+    def create(self, vals_list):
+        for vals in vals_list:
+            price = vals.get('price')
+            property_obj = self.env['estate.property'].browse(vals.get('property_id'))
+            existing_offers = property_obj.offer_ids.filtered(lambda o: o.price >= price)
+            if existing_offers:
+                raise ValidationError("You cannot create an offer with a lower or equal price than existing offers.")
+            property_obj.state = 'offer_received'
 
-        return super().create(vals)
+        return super().create(vals_list)
diff --git a/real_estate/view/estate_property_offer_views.xml b/real_estate/view/estate_property_offer_views.xml
index 1d73ef19abc..66ce96d1d30 100644
--- a/real_estate/view/estate_property_offer_views.xml
+++ b/real_estate/view/estate_property_offer_views.xml
@@ -14,11 +14,9 @@
             <form string="Property Offer">
            <header>
     <button name="action_accept" string="Accept" type="object"
-            class="btn-primary"
             invisible="state != 'accepted'"/>
 
     <button name="action_refuse" string="Refuse" type="object"
-            class="btn-secondary"
             invisible="state == 'accepted'"/>
             </header>
 
@@ -42,7 +40,6 @@
               decoration-success="state == 'accepted'">
                 <field name="price"/>
                 <field name="partner_id"/>
-                <field name="state" invisible="1"/>
                  <button name="action_accept" type="object" string="✓" icon="fa-check"/>
                 <button name="action_refuse" type="object" string="✗" icon="fa-times"/>
             </list>
diff --git a/real_estate/view/estate_property_views.xml b/real_estate/view/estate_property_views.xml
index 01486ac51b0..5093ed78163 100644
--- a/real_estate/view/estate_property_views.xml
+++ b/real_estate/view/estate_property_views.xml
@@ -14,7 +14,6 @@
                 <field name="state"/>
                 <field name="expected_price"/>
                 <field name="selling_price"/>
-                <field name="availability_date" optional="show" invisible="1"/>
                 <field name="state"/>
                 <field name="bedrooms"/>
                 <field name="availability_date"/>
@@ -33,11 +32,9 @@
             <form string="Estate Property">
              <header>
     <button name="action_sold" string="Sold" type="object"
-            class="btn-primary"
             invisible="state != 'offer_accepted'"/>
 
     <button name="action_cancel" string="Cancel" type="object"
-            class="btn-secondary"
             invisible="state in ('sold', 'cancelled')"/>
              <field name="state" widget="statusbar"/>
             </header>
@@ -86,18 +83,16 @@
                           <field name="offer_ids">
                          <list editable="bottom" 
                          decoration-success="state == 'accepted'"
-              decoration-danger="state == 'refused'">
+                         decoration-danger="state == 'refused'">
                            <field name="price"/>
                            <field name="partner_id"/>
                            <field name="state"/>
                            <field name="validity"/>
                            <field name="date_deadline"/>
                           <button name="action_accept" string="Accept" type="object"
-            class="btn-primary"
-            invisible="state in ('accepted','refused')"/>
-    <button name="action_refuse" string="Refuse" type="object"
-            class="btn-secondary"
-            invisible="state in ('accepted','refused')"/>
+                           invisible="state in ('accepted','refused')"/>
+                         <button name="action_refuse" string="Refuse" type="object"
+                          invisible="state in ('accepted','refused')"/>
                        </list>
                    </field>
 

From 8ce84da3d5043345f7e8c2d16d779a08c96595ef Mon Sep 17 00:00:00 2001
From: rodh-odoo <rodh@odoo.com>
Date: Thu, 10 Jul 2025 10:14:43 +0530
Subject: [PATCH 7/7] [IMP] real_estate: improve code in estate property model

improve code and change file name according to guidelines
rename model name same as the record name
---
 real_estate/__manifest__.py                            | 4 ++--
 real_estate/models/estate_property.py                  | 2 +-
 real_estate/models/estate_property_offer.py            | 2 +-
 real_estate/view/{estate_menus.xml => estate_menu.xml} | 0
 real_estate/view/estate_property_offer_views.xml       | 4 ++--
 real_estate/view/estate_property_tag_views.xml         | 2 +-
 real_estate/view/estate_property_type_views.xml        | 4 ++--
 real_estate/view/estate_property_views.xml             | 6 +++---
 real_estate/view/estate_res_users_view.xml             | 2 +-
 9 files changed, 13 insertions(+), 13 deletions(-)
 rename real_estate/view/{estate_menus.xml => estate_menu.xml} (100%)

diff --git a/real_estate/__manifest__.py b/real_estate/__manifest__.py
index f24523d1389..6ea392103d1 100644
--- a/real_estate/__manifest__.py
+++ b/real_estate/__manifest__.py
@@ -12,7 +12,7 @@
     'view/estate_property_offer_views.xml',
     'view/estate_property_type_views.xml',
     'view/estate_property_tag_views.xml',
-    'view/estate_menus.xml',
-    'view/estate_res_users_view.xml'
+    'view/estate_res_users_view.xml',
+    'view/estate_menu.xml',
 ],
 }
diff --git a/real_estate/models/estate_property.py b/real_estate/models/estate_property.py
index 016aa604f29..8cc819c0573 100644
--- a/real_estate/models/estate_property.py
+++ b/real_estate/models/estate_property.py
@@ -1,5 +1,5 @@
-from odoo import fields, models, api
 from datetime import date, timedelta
+from odoo import fields, models, api
 from odoo.exceptions import UserError, ValidationError
 from odoo.tools.float_utils import float_compare, float_is_zero
 
diff --git a/real_estate/models/estate_property_offer.py b/real_estate/models/estate_property_offer.py
index 9facb3e00b8..a4eea0ac3d5 100644
--- a/real_estate/models/estate_property_offer.py
+++ b/real_estate/models/estate_property_offer.py
@@ -1,5 +1,5 @@
-from odoo import models, fields, api
 from datetime import timedelta
+from odoo import models, fields, api
 from odoo.exceptions import UserError, ValidationError
 
 
diff --git a/real_estate/view/estate_menus.xml b/real_estate/view/estate_menu.xml
similarity index 100%
rename from real_estate/view/estate_menus.xml
rename to real_estate/view/estate_menu.xml
diff --git a/real_estate/view/estate_property_offer_views.xml b/real_estate/view/estate_property_offer_views.xml
index 66ce96d1d30..1750622b0d7 100644
--- a/real_estate/view/estate_property_offer_views.xml
+++ b/real_estate/view/estate_property_offer_views.xml
@@ -7,7 +7,7 @@
     <field name="domain">[('property_type_id', '=', active_id)]</field>
    </record>
 
-    <record id="view_estate_property_offer_form" model="ir.ui.view">
+    <record id="estate_property_offer_form" model="ir.ui.view">
         <field name="name">estate.property.offer.form</field>
         <field name="model">estate.property.offer</field>
         <field name="arch" type="xml">
@@ -31,7 +31,7 @@
         </field>
     </record>
 
-    <record id="view_estate_property_offer_list" model="ir.ui.view">
+    <record id="estate_property_offer_list" model="ir.ui.view">
         <field name="name">estate.property.offer.list</field>
         <field name="model">estate.property.offer</field>
         <field name="arch" type="xml">
diff --git a/real_estate/view/estate_property_tag_views.xml b/real_estate/view/estate_property_tag_views.xml
index 12ae069fac0..8a20f9c5f69 100644
--- a/real_estate/view/estate_property_tag_views.xml
+++ b/real_estate/view/estate_property_tag_views.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
 <odoo>
-   <record id="view_estate_property_tag_list" model="ir.ui.view">
+   <record id="estate_property_tag_list" model="ir.ui.view">
     <field name="name">estate.property.tag.list</field>
     <field name="model">estate.property.tag</field>
     <field name="arch" type="xml">
diff --git a/real_estate/view/estate_property_type_views.xml b/real_estate/view/estate_property_type_views.xml
index b83b6c8d4a0..881ddcb9552 100644
--- a/real_estate/view/estate_property_type_views.xml
+++ b/real_estate/view/estate_property_type_views.xml
@@ -6,7 +6,7 @@
         <field name="res_model">estate.property.type</field>
         <field name="view_mode">list,form</field>
     </record>
-    <record id="estate_property_type_form_view" model="ir.ui.view">
+    <record id="estate_property_type_form" model="ir.ui.view">
         <field name="name">estate.property.type.form</field>
         <field name="model">estate.property.type</field>
         
@@ -39,7 +39,7 @@
             </form>
         </field>
     </record>
-    <record id="estate_property_type_tree_view" model="ir.ui.view">
+    <record id="estate_property_type_tree" model="ir.ui.view">
         <field name="name">estate.property.type.tree</field>
         <field name="model">estate.property.type</field>
         <field name="arch" type="xml">
diff --git a/real_estate/view/estate_property_views.xml b/real_estate/view/estate_property_views.xml
index 5093ed78163..0761131f06a 100644
--- a/real_estate/view/estate_property_views.xml
+++ b/real_estate/view/estate_property_views.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0"?>
 <odoo>
 <data>
-    <record id="view_estate_property_list" model="ir.ui.view">
+    <record id="estate_property_list" model="ir.ui.view">
         <field name="name">estate.property.list</field>
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
@@ -25,7 +25,7 @@
         </field>
     </record>
 
-    <record id="view_estate_property_form" model="ir.ui.view">
+    <record id="estate_property_form" model="ir.ui.view">
         <field name="name">estate.property.form</field>
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
@@ -103,7 +103,7 @@
         </field>
     </record>
 
-   <record id="view_estate_property_search" model="ir.ui.view">
+   <record id="estate_property_search" model="ir.ui.view">
     <field name="name">estate.property.search</field>
     <field name="model">estate.property</field>
     <field name="arch" type="xml">
diff --git a/real_estate/view/estate_res_users_view.xml b/real_estate/view/estate_res_users_view.xml
index c8be904b8c9..c0dbecd7237 100644
--- a/real_estate/view/estate_res_users_view.xml
+++ b/real_estate/view/estate_res_users_view.xml
@@ -1,5 +1,5 @@
 <odoo>
-    <record id="view_users_form_inherit_real_estate" model="ir.ui.view">
+    <record id="res_users_form_inherit_real_estate" model="ir.ui.view">
         <field name="name">res.users.form.inherit.real.estate</field>
         <field name="model">res.users</field>
         <field name="inherit_id" ref="base.view_users_form"/>