From 24e63d0a972b00641e43120464ef84bb697d9e03 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Mon, 20 Oct 2025 13:53:50 +0200
Subject: [PATCH 01/33] [ADD] estate: create the app

---
 estate/__init__.py     |  4 ++++
 estate/__manifest__.py | 24 ++++++++++++++++++++++++
 2 files changed, 28 insertions(+)
 create mode 100644 estate/__init__.py
 create mode 100644 estate/__manifest__.py

diff --git a/estate/__init__.py b/estate/__init__.py
new file mode 100644
index 00000000000..f2d9c576fbd
--- /dev/null
+++ b/estate/__init__.py
@@ -0,0 +1,4 @@
+from . import models
+
+
+from odoo import api, SUPERUSER_ID
\ No newline at end of file
diff --git a/estate/__manifest__.py b/estate/__manifest__.py
new file mode 100644
index 00000000000..73ac57e9439
--- /dev/null
+++ b/estate/__manifest__.py
@@ -0,0 +1,24 @@
+{
+    'name': "Real Estate",
+
+    'summary': """
+        Real Estate Tuto"
+    """,
+
+    'description': """
+        Starting tutorial real estate"
+    """,
+
+    'author': "Odoo",
+    'website': "https://www.odoo.com/",
+    'category': 'Tutorials',
+    'version': '0.1',
+    'application': True,
+    'depends': ['base'],
+
+    'data': [],
+    'assets': {
+
+    },
+    'license': 'AGPL-3'
+}

From 8d65340930dce9a305a801bf542d55e5ee9cae48 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Mon, 20 Oct 2025 14:51:16 +0200
Subject: [PATCH 02/33] [ADD] estate: added models and basic field

---
 estate/models/__init__.py        |  1 +
 estate/models/estate_proprety.py | 22 ++++++++++++++++++++++
 2 files changed, 23 insertions(+)
 create mode 100644 estate/models/__init__.py
 create mode 100644 estate/models/estate_proprety.py

diff --git a/estate/models/__init__.py b/estate/models/__init__.py
new file mode 100644
index 00000000000..75cf03f8521
--- /dev/null
+++ b/estate/models/__init__.py
@@ -0,0 +1 @@
+from . import estate_proprety
\ No newline at end of file
diff --git a/estate/models/estate_proprety.py b/estate/models/estate_proprety.py
new file mode 100644
index 00000000000..3b91debfb8c
--- /dev/null
+++ b/estate/models/estate_proprety.py
@@ -0,0 +1,22 @@
+from odoo import fields, models
+
+class EstateProprety(models.Model):
+    _name = "estate.proprety"
+    _description = "Estate Proprety"
+
+    name = fields.Char('Estate Name', required=True)
+    description = fields.Text('Estate Description', required=True)
+    postcode = fields.Char('Estate Postcode', required=True)
+    date_availablity = fields.Date('Estate Availability Date', required=True)
+    expected_price = fields.Float('Estate expected price', required=True)
+    selling_price = fields.Float('Estate selling price', required=True)
+    bedrooms = fields.Integer('Number of bedrooms',required=True)
+    facades = fields.Integer('Number of facades')
+    garage = fields.Boolean('Has a garage ?')
+    garden = fields.Boolean('Has a garden ?')
+    garden_area = fields.Integer('Garden area')
+    garden_orientation = fields.Selection(
+        string='Type',
+        selection=[('north','North'),('south','South'),('west','West'),('east','East')],
+        help="Garden orientation selection"
+    )
\ No newline at end of file

From a1f33148390e5cce765781a078485f1adc266cf8 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Mon, 20 Oct 2025 15:09:50 +0200
Subject: [PATCH 03/33] [FIX] estate: fixed style

---
 estate/__init__.py               | 2 +-
 estate/models/__init__.py        | 2 +-
 estate/models/estate_proprety.py | 8 +++++---
 3 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/estate/__init__.py b/estate/__init__.py
index f2d9c576fbd..1ccd6b6fffa 100644
--- a/estate/__init__.py
+++ b/estate/__init__.py
@@ -1,4 +1,4 @@
 from . import models
 
 
-from odoo import api, SUPERUSER_ID
\ No newline at end of file
+from odoo import api, SUPERUSER_ID
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 75cf03f8521..2367d7d23ba 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1 +1 @@
-from . import estate_proprety
\ No newline at end of file
+from . import estate_proprety
diff --git a/estate/models/estate_proprety.py b/estate/models/estate_proprety.py
index 3b91debfb8c..d28e1a02af7 100644
--- a/estate/models/estate_proprety.py
+++ b/estate/models/estate_proprety.py
@@ -4,19 +4,21 @@ class EstateProprety(models.Model):
     _name = "estate.proprety"
     _description = "Estate Proprety"
 
+
     name = fields.Char('Estate Name', required=True)
     description = fields.Text('Estate Description', required=True)
     postcode = fields.Char('Estate Postcode', required=True)
     date_availablity = fields.Date('Estate Availability Date', required=True)
     expected_price = fields.Float('Estate expected price', required=True)
     selling_price = fields.Float('Estate selling price', required=True)
-    bedrooms = fields.Integer('Number of bedrooms',required=True)
+    bedrooms = fields.Integer('Number of bedrooms', required=True)
     facades = fields.Integer('Number of facades')
     garage = fields.Boolean('Has a garage ?')
     garden = fields.Boolean('Has a garden ?')
     garden_area = fields.Integer('Garden area')
     garden_orientation = fields.Selection(
         string='Type',
-        selection=[('north','North'),('south','South'),('west','West'),('east','East')],
+        selection=[('north', 'North'), ('south', 'South'), ('west', 'West'), ('east', 'East')],
         help="Garden orientation selection"
-    )
\ No newline at end of file
+    )
+    
\ No newline at end of file

From 2bdaaf843ea8479b66d658b1be8aa74466225983 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Mon, 20 Oct 2025 15:14:44 +0200
Subject: [PATCH 04/33] [FIX] estate: fixed style

---
 estate/models/estate_proprety.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/estate/models/estate_proprety.py b/estate/models/estate_proprety.py
index d28e1a02af7..a7cddfccefe 100644
--- a/estate/models/estate_proprety.py
+++ b/estate/models/estate_proprety.py
@@ -1,10 +1,10 @@
 from odoo import fields, models
 
+
 class EstateProprety(models.Model):
     _name = "estate.proprety"
     _description = "Estate Proprety"
 
-
     name = fields.Char('Estate Name', required=True)
     description = fields.Text('Estate Description', required=True)
     postcode = fields.Char('Estate Postcode', required=True)
@@ -21,4 +21,3 @@ class EstateProprety(models.Model):
         selection=[('north', 'North'), ('south', 'South'), ('west', 'West'), ('east', 'East')],
         help="Garden orientation selection"
     )
-    
\ No newline at end of file

From 8f770020f7c29159b75af2efeedbc1f80192319e Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Mon, 20 Oct 2025 15:38:17 +0200
Subject: [PATCH 05/33] [ADD] estate: added security

---
 estate/__manifest__.py              | 4 +++-
 estate/security/ir.model.access.csv | 2 ++
 2 files changed, 5 insertions(+), 1 deletion(-)
 create mode 100644 estate/security/ir.model.access.csv

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 73ac57e9439..6d3221e89be 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -16,7 +16,9 @@
     'application': True,
     'depends': ['base'],
 
-    'data': [],
+    'data': [
+        'security/ir.model.access.csv',
+    ],
     'assets': {
 
     },
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
new file mode 100644
index 00000000000..daf139e1020
--- /dev/null
+++ b/estate/security/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_proprety,estate.proprety,model_estate_proprety,base.group_user,1,1,1,1
\ No newline at end of file

From a032b7d1f4584ce1a57162f0faf007bdd44e2658 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Tue, 21 Oct 2025 10:30:33 +0200
Subject: [PATCH 06/33] [FIX] estate: fixed typos

---
 estate/__manifest__.py              |  4 ++++
 estate/models/__init__.py           |  2 +-
 estate/models/estate_property.py    | 31 +++++++++++++++++++++++++++++
 estate/models/estate_proprety.py    | 23 ---------------------
 estate/security/ir.model.access.csv |  2 +-
 5 files changed, 37 insertions(+), 25 deletions(-)
 create mode 100644 estate/models/estate_property.py
 delete mode 100644 estate/models/estate_proprety.py

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 6d3221e89be..78e757df6cb 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -18,9 +18,13 @@
 
     'data': [
         'security/ir.model.access.csv',
+        'views/estate_property_views.xml',
+        'views/estate_menus.xml',
+
     ],
     'assets': {
 
     },
     'license': 'AGPL-3'
 }
+
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 2367d7d23ba..5e1963c9d2f 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1 +1 @@
-from . import estate_proprety
+from . import estate_property
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
new file mode 100644
index 00000000000..0ec6f4fda83
--- /dev/null
+++ b/estate/models/estate_property.py
@@ -0,0 +1,31 @@
+from odoo import fields, models
+from dateutil.relativedelta import relativedelta
+
+
+class Estateproperty(models.Model):
+    _name = "estate.property"
+    _description = "Estate property"
+
+    name = fields.Char('Title', required=True)
+    description = fields.Text('Description', required=True)
+    postcode = fields.Char('Postcode', required=True)
+    date_availablity = fields.Date('Available From', copy=False, default=fields.Datetime.now()+relativedelta(months=3))
+    expected_price = fields.Float('Expected Price', required=True)
+    selling_price = fields.Float('Selling Price', readonly=True, copy=False)
+    bedrooms = fields.Integer('Bedrooms', default=2)
+    facades = fields.Integer('Facades')
+    garage = fields.Boolean('Garage')
+    garden = fields.Boolean('Garden')
+    garden_area = fields.Integer('Garden area (sqm)')
+    garden_orientation = fields.Selection(
+        string='Type',
+        selection=[('north', 'North'), ('south', 'South'), ('west', 'West'), ('east', 'East')],
+        help="Garden orientation selection"
+    )
+    active = fields.Boolean('Active', default=True)
+    state = fields.Selection(
+        string='State',
+        selection=[('new', 'New'), ('offer received', 'Offer Received'), ('offer accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')],
+        help="Estate state",
+        default=('new')
+    )
diff --git a/estate/models/estate_proprety.py b/estate/models/estate_proprety.py
deleted file mode 100644
index a7cddfccefe..00000000000
--- a/estate/models/estate_proprety.py
+++ /dev/null
@@ -1,23 +0,0 @@
-from odoo import fields, models
-
-
-class EstateProprety(models.Model):
-    _name = "estate.proprety"
-    _description = "Estate Proprety"
-
-    name = fields.Char('Estate Name', required=True)
-    description = fields.Text('Estate Description', required=True)
-    postcode = fields.Char('Estate Postcode', required=True)
-    date_availablity = fields.Date('Estate Availability Date', required=True)
-    expected_price = fields.Float('Estate expected price', required=True)
-    selling_price = fields.Float('Estate selling price', required=True)
-    bedrooms = fields.Integer('Number of bedrooms', required=True)
-    facades = fields.Integer('Number of facades')
-    garage = fields.Boolean('Has a garage ?')
-    garden = fields.Boolean('Has a garden ?')
-    garden_area = fields.Integer('Garden area')
-    garden_orientation = fields.Selection(
-        string='Type',
-        selection=[('north', 'North'), ('south', 'South'), ('west', 'West'), ('east', 'East')],
-        help="Garden orientation selection"
-    )
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index daf139e1020..9ccf73e191e 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -1,2 +1,2 @@
 id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink
-access_estate_proprety,estate.proprety,model_estate_proprety,base.group_user,1,1,1,1
\ No newline at end of file
+access_estate_property,estate.property,model_estate_property,base.group_user,1,1,1,1
\ No newline at end of file

From f2f4addaceaf603fbbb7be68e6c4d479f3aa7c1a Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Tue, 21 Oct 2025 10:30:58 +0200
Subject: [PATCH 07/33] [ADD] estate: added views

---
 estate/views/estate_menus.xml          | 8 ++++++++
 estate/views/estate_property_views.xml | 8 ++++++++
 2 files changed, 16 insertions(+)
 create mode 100644 estate/views/estate_menus.xml
 create mode 100644 estate/views/estate_property_views.xml

diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
new file mode 100644
index 00000000000..d691e7dd1b1
--- /dev/null
+++ b/estate/views/estate_menus.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <menuitem id="estate_menu_root" name="Real Estate">
+        <menuitem id="estate_first_level_menu" name="Advertisements">
+            <menuitem id="estate_model_menu_action" action="estate_property_model_action"/>
+        </menuitem>
+    </menuitem>
+</odoo>
\ No newline at end of file
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
new file mode 100644
index 00000000000..5e0ecf1cbd6
--- /dev/null
+++ b/estate/views/estate_property_views.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <record id="estate_property_model_action" model="ir.actions.act_window">
+        <field name="name">Properties</field>
+        <field name="res_model">estate.property</field>
+        <field name="view_mode">list,form</field>
+    </record>
+</odoo>
\ No newline at end of file

From 520584545b37100d7e16b2e486274831b1201fdd Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Tue, 21 Oct 2025 10:55:15 +0200
Subject: [PATCH 08/33] [FIX] estate: styling fix

---
 estate/__manifest__.py           | 1 -
 estate/models/estate_property.py | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 78e757df6cb..2b5085abcdb 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -27,4 +27,3 @@
     },
     'license': 'AGPL-3'
 }
-
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 0ec6f4fda83..ea950af8cc5 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -9,7 +9,7 @@ class Estateproperty(models.Model):
     name = fields.Char('Title', required=True)
     description = fields.Text('Description', required=True)
     postcode = fields.Char('Postcode', required=True)
-    date_availablity = fields.Date('Available From', copy=False, default=fields.Datetime.now()+relativedelta(months=3))
+    date_availablity = fields.Date('Available From', copy=False, default=fields.Datetime.now() + relativedelta(months=3))
     expected_price = fields.Float('Expected Price', required=True)
     selling_price = fields.Float('Selling Price', readonly=True, copy=False)
     bedrooms = fields.Integer('Bedrooms', default=2)

From ee692d41c844cdd0f8a11edd2189b8d8fef35a38 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Tue, 21 Oct 2025 14:44:27 +0200
Subject: [PATCH 09/33] [ADD] estate: added basic views

---
 estate/models/estate_property.py       |  6 +--
 estate/views/estate_property_views.xml | 75 ++++++++++++++++++++++++++
 2 files changed, 78 insertions(+), 3 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index ea950af8cc5..e9badcdc11c 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -9,18 +9,18 @@ class Estateproperty(models.Model):
     name = fields.Char('Title', required=True)
     description = fields.Text('Description', required=True)
     postcode = fields.Char('Postcode', required=True)
-    date_availablity = fields.Date('Available From', copy=False, default=fields.Datetime.now() + relativedelta(months=3))
+    date_availability = fields.Date('Available From', copy=False, default=fields.Datetime.now() + relativedelta(months=3))
     expected_price = fields.Float('Expected Price', required=True)
     selling_price = fields.Float('Selling Price', readonly=True, copy=False)
     bedrooms = fields.Integer('Bedrooms', default=2)
+    living_area = fields.Integer('Living Area (sqm)')
     facades = fields.Integer('Facades')
     garage = fields.Boolean('Garage')
     garden = fields.Boolean('Garden')
     garden_area = fields.Integer('Garden area (sqm)')
     garden_orientation = fields.Selection(
-        string='Type',
+        string='Garden Orientation',
         selection=[('north', 'North'), ('south', 'South'), ('west', 'West'), ('east', 'East')],
-        help="Garden orientation selection"
     )
     active = fields.Boolean('Active', default=True)
     state = fields.Selection(
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 5e0ecf1cbd6..106625ed0d3 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -5,4 +5,79 @@
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
     </record>
+
+    <record id="estate_property_view_search" model="ir.ui.view">
+        <field name="name">estate.property.view.search</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <search string="property">
+                <field name="name" string="Title"/>
+                <field name="postcode"/>
+                <field name="expected_price"/>
+                <field name="bedrooms"/>
+                <field name="living_area"/>
+                <field name="facades"/>
+                <filter string="Available" name="available properties" domain="[ ('state', 'in', ['new', 'offer received']) ]"/>
+                <filter string="Postcode" name="postcode" context="{'group_by':'postcode', 'residual_visible':True}"/>
+            </search>
+        </field>
+    </record>
+
+    <record id="estate_property_view_list" model="ir.ui.view">
+        <field name="name">estate.property.list</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <list string="Channel">
+                <field name="name"/>
+                <field name="postcode"/>
+                <field name="bedrooms"/>
+                <field name="living_area"/>
+                <field name="expected_price"/>
+                <field name="selling_price"/>
+                <field name="date_availability"/>
+            </list>
+        </field>
+    </record>
+
+    <record id="estate_property_view_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="property">
+                <sheet>
+                    <group>
+                        <h1>
+                            <field name="name"/>
+                        </h1>
+                    </group>
+                    <group>
+                        <group>
+                            <field name="postcode"/>
+                            <field name="date_availability"/>
+                        </group>
+                    <group>
+                        <field name="expected_price"/>
+                        <field name="selling_price"/>
+                    </group>
+                    </group>
+                    <notebook>
+                        <page string="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>
+
+    
 </odoo>
\ No newline at end of file

From 8ed93682e2cd0c41d4941be10a8a26f14ee31556 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Tue, 21 Oct 2025 16:24:22 +0200
Subject: [PATCH 10/33] [ADD] estate: added buyer, salesperson and estate type

---
 estate/__manifest__.py                      |  1 +
 estate/models/__init__.py                   |  3 +++
 estate/models/buyer.py                      |  8 ++++++++
 estate/models/estate_property.py            |  3 +++
 estate/models/estate_property_type.py       | 10 ++++++++++
 estate/models/salesperson.py                |  8 ++++++++
 estate/security/ir.model.access.csv         |  5 ++++-
 estate/views/estate_menus.xml               |  3 +++
 estate/views/estate_property_type_views.xml | 10 ++++++++++
 estate/views/estate_property_views.xml      |  6 ++++++
 10 files changed, 56 insertions(+), 1 deletion(-)
 create mode 100644 estate/models/buyer.py
 create mode 100644 estate/models/estate_property_type.py
 create mode 100644 estate/models/salesperson.py
 create mode 100644 estate/views/estate_property_type_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 2b5085abcdb..58ced11253c 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -20,6 +20,7 @@
         'security/ir.model.access.csv',
         'views/estate_property_views.xml',
         'views/estate_menus.xml',
+        'views/estate_property_type_views.xml',
 
     ],
     'assets': {
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 5e1963c9d2f..9f5291914ec 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1 +1,4 @@
 from . import estate_property
+from . import estate_property_type
+from . import buyer
+from . import salesperson
diff --git a/estate/models/buyer.py b/estate/models/buyer.py
new file mode 100644
index 00000000000..03e8a490085
--- /dev/null
+++ b/estate/models/buyer.py
@@ -0,0 +1,8 @@
+from odoo import fields, models
+
+
+class Buyer(models.Model):
+    _name = 'buyer'
+    _description = 'The entity to buy the estate'
+
+    name = fields.Char('Buyer')
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index e9badcdc11c..8b66f5faaf6 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -29,3 +29,6 @@ class Estateproperty(models.Model):
         help="Estate state",
         default=('new')
     )
+    property_type_id = fields.Many2one("estate.property.type", string="Type")
+    salesperson = fields.Many2one('salesperson', string='Salesperson')
+    buyer = fields.Many2one('buyer', string='Buyer')
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
new file mode 100644
index 00000000000..3245fae7753
--- /dev/null
+++ b/estate/models/estate_property_type.py
@@ -0,0 +1,10 @@
+from odoo import fields, models
+
+
+class EstatePropertyType(models.Model):
+    _name = "estate.property.type"
+    _description = "Estate Property Type"
+
+    name = fields.Char('Type', required=True)
+    description = fields.Text('Description')
+    
\ No newline at end of file
diff --git a/estate/models/salesperson.py b/estate/models/salesperson.py
new file mode 100644
index 00000000000..c8e07714e66
--- /dev/null
+++ b/estate/models/salesperson.py
@@ -0,0 +1,8 @@
+from odoo import fields, models
+
+
+class Salesperson(models.Model):
+    _name = 'salesperson'
+    _description = 'The individual handling the sales for the estate'
+
+    name = fields.Char('Salesperson')
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index 9ccf73e191e..ee5f44fe72b 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -1,2 +1,5 @@
 id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink
-access_estate_property,estate.property,model_estate_property,base.group_user,1,1,1,1
\ No newline at end of file
+access_estate_property,estate.property,model_estate_property,base.group_user,1,1,1,1
+access_estate_property_type,estate.property.type,model_estate_property_type,base.group_user,1,1,1,1
+access_buyer,buyer,model_buyer,base.group_user,1,1,1,1
+access_salesperson,sales,model_salesperson,base.group_user,1,1,1,1
\ No newline at end of file
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
index d691e7dd1b1..d987332cb9f 100644
--- a/estate/views/estate_menus.xml
+++ b/estate/views/estate_menus.xml
@@ -4,5 +4,8 @@
         <menuitem id="estate_first_level_menu" name="Advertisements">
             <menuitem id="estate_model_menu_action" action="estate_property_model_action"/>
         </menuitem>
+        <menuitem id="estate_type_first_level_menu" name="Settings">
+            <menuitem id="estate_type_model_menu_action" action="estate_property_type_model_action"/>
+        </menuitem>
     </menuitem>
 </odoo>
\ No newline at end of file
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
new file mode 100644
index 00000000000..57586e76f3d
--- /dev/null
+++ b/estate/views/estate_property_type_views.xml
@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<odoo>
+    
+    <record id="estate_property_type_model_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>
\ No newline at end of file
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 106625ed0d3..850a85f3065 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -73,6 +73,12 @@
                                 <field name="garden_orientation"/>
                             </group>
                         </page>
+                        <page string="Other Info">
+                            <group>
+                                <field name="salesperson"/>
+                                <field name="buyer"/>
+                            </group>
+                        </page>
                     </notebook>
                 </sheet> 
             </form>

From a150ae91b448c059c6f4e540c68b9d7ec82d023a Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Wed, 22 Oct 2025 10:34:28 +0200
Subject: [PATCH 11/33] [FIX] estate: commit to save progress before debugging

---
 estate/__init__.py                           |  3 --
 estate/__manifest__.py                       |  1 +
 estate/models/__init__.py                    |  3 +-
 estate/models/buyer.py                       |  8 ----
 estate/models/estate_property.py             |  5 ++-
 estate/models/estate_property_tag.py         |  9 +++++
 estate/models/estate_property_type.py        |  1 -
 estate/models/salesperson.py                 |  8 ----
 estate/security/ir.model.access.csv          |  3 +-
 estate/views/estate_menus.xml                |  3 +-
 estate/views/estate_property_tag_views.xml   | 38 ++++++++++++++++++
 estate/views/estate_property_tag_views_2.xml | 41 ++++++++++++++++++++
 estate/views/estate_property_type_views.xml  | 27 +++++++++++++
 estate/views/estate_property_views.xml       |  4 ++
 14 files changed, 127 insertions(+), 27 deletions(-)
 delete mode 100644 estate/models/buyer.py
 create mode 100644 estate/models/estate_property_tag.py
 delete mode 100644 estate/models/salesperson.py
 create mode 100644 estate/views/estate_property_tag_views.xml
 create mode 100644 estate/views/estate_property_tag_views_2.xml

diff --git a/estate/__init__.py b/estate/__init__.py
index 1ccd6b6fffa..0650744f6bc 100644
--- a/estate/__init__.py
+++ b/estate/__init__.py
@@ -1,4 +1 @@
 from . import models
-
-
-from odoo import api, SUPERUSER_ID
diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 58ced11253c..edbbe92ef58 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -19,6 +19,7 @@
     'data': [
         'security/ir.model.access.csv',
         'views/estate_property_views.xml',
+        'views/estate_property_tag_views_2.xml',
         'views/estate_menus.xml',
         'views/estate_property_type_views.xml',
 
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 9f5291914ec..fdab6f99948 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1,4 +1,3 @@
 from . import estate_property
 from . import estate_property_type
-from . import buyer
-from . import salesperson
+from . import estate_property_tag
\ No newline at end of file
diff --git a/estate/models/buyer.py b/estate/models/buyer.py
deleted file mode 100644
index 03e8a490085..00000000000
--- a/estate/models/buyer.py
+++ /dev/null
@@ -1,8 +0,0 @@
-from odoo import fields, models
-
-
-class Buyer(models.Model):
-    _name = 'buyer'
-    _description = 'The entity to buy the estate'
-
-    name = fields.Char('Buyer')
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 8b66f5faaf6..70a54258732 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -30,5 +30,6 @@ class Estateproperty(models.Model):
         default=('new')
     )
     property_type_id = fields.Many2one("estate.property.type", string="Type")
-    salesperson = fields.Many2one('salesperson', string='Salesperson')
-    buyer = fields.Many2one('buyer', string='Buyer')
+    salesperson = fields.Many2one('res.partner', string='Salesperson')
+    buyer = fields.Many2one('res.users', string='Buyer', default=lambda self: self.env.user)
+    #tag_ids = fields.Many2many('estate.property.tag', string='Tag')
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
new file mode 100644
index 00000000000..8ca5a69f1fc
--- /dev/null
+++ b/estate/models/estate_property_tag.py
@@ -0,0 +1,9 @@
+from odoo import fields, models 
+
+
+class EstatePropertyTag(models.Model):
+    _name = "estate.property.tag"
+    _description = "Tags for the estate"
+
+    name = fields.Char('Tag', required=True)
+    description = fields.Char('Description')
\ No newline at end of file
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index 3245fae7753..34fcfe87f7c 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -7,4 +7,3 @@ class EstatePropertyType(models.Model):
 
     name = fields.Char('Type', required=True)
     description = fields.Text('Description')
-    
\ No newline at end of file
diff --git a/estate/models/salesperson.py b/estate/models/salesperson.py
deleted file mode 100644
index c8e07714e66..00000000000
--- a/estate/models/salesperson.py
+++ /dev/null
@@ -1,8 +0,0 @@
-from odoo import fields, models
-
-
-class Salesperson(models.Model):
-    _name = 'salesperson'
-    _description = 'The individual handling the sales for the estate'
-
-    name = fields.Char('Salesperson')
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index ee5f44fe72b..70a94e12e73 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -1,5 +1,4 @@
 id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink
+access_estate_property_tag,estate.property.tag,model_estate_property_tag,base.group_user,1,1,1,1
 access_estate_property,estate.property,model_estate_property,base.group_user,1,1,1,1
 access_estate_property_type,estate.property.type,model_estate_property_type,base.group_user,1,1,1,1
-access_buyer,buyer,model_buyer,base.group_user,1,1,1,1
-access_salesperson,sales,model_salesperson,base.group_user,1,1,1,1
\ No newline at end of file
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
index d987332cb9f..ae7967382b3 100644
--- a/estate/views/estate_menus.xml
+++ b/estate/views/estate_menus.xml
@@ -4,8 +4,9 @@
         <menuitem id="estate_first_level_menu" name="Advertisements">
             <menuitem id="estate_model_menu_action" action="estate_property_model_action"/>
         </menuitem>
-        <menuitem id="estate_type_first_level_menu" name="Settings">
+        <menuitem id="estate_attributes_first_level_menu" name="Settings">
             <menuitem id="estate_type_model_menu_action" action="estate_property_type_model_action"/>
+           <menuitem id="estate_tag_model_menu_action" action="estate_property_tag_model_action"/>
         </menuitem>
     </menuitem>
 </odoo>
\ No newline at end of file
diff --git a/estate/views/estate_property_tag_views.xml b/estate/views/estate_property_tag_views.xml
new file mode 100644
index 00000000000..b90afe4be05
--- /dev/null
+++ b/estate/views/estate_property_tag_views.xml
@@ -0,0 +1,38 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<odoo>
+<data>
+    
+    <record id="estate_property_tag_model_action" 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>
+
+    <record id="estate_property_tag_view_list" model="ir.ui.view">
+        <field name="name">estate.property.tag.list</field>
+        <field name="model">estate.property.tag</field>
+        <field name="arch" tag="xml">
+            <list>
+                <field name="name"/>
+                <field name="description"/>
+            </list>
+        </field>
+    </record>
+
+    <record id="estate_property_tag_view_form" model="ir.ui.view">
+        <field name="name">estate.property.tag.form</field>
+        <field name="model">estate.property.tag</field>
+        <field name="arch" tag="xml">
+            <form>
+                <sheet>
+                    <group>
+                        <h1>
+                            <field name="name"/>
+                        </h1>
+                    </group>
+                </sheet>
+            </form>
+        </field>
+    </record>
+</data>
+</odoo>
\ No newline at end of file
diff --git a/estate/views/estate_property_tag_views_2.xml b/estate/views/estate_property_tag_views_2.xml
new file mode 100644
index 00000000000..1e910748628
--- /dev/null
+++ b/estate/views/estate_property_tag_views_2.xml
@@ -0,0 +1,41 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <data>
+
+    <record id="estate_property_tag_model_action" 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>
+
+    <record id="estate_property_tag_view_list" model="ir.ui.view">
+        <field name="name">estate.property.tag.list</field>
+        <field name="model">estate.property.tag</field>
+        <field name="arch" tag="xml">
+            <list>
+                <field name="name"/>
+                <field name="description"/>
+            </list>
+        </field>
+    </record>
+
+    <record id="estate_property_tag_view_form" model="ir.ui.view">
+        <field name="name">estate.property.tag.form</field>
+        <field name="model">estate.property.tag</field>
+        <field name="arch" tag="xml">
+            <form>
+                <sheet>
+                    <group>
+                        <h1>
+                            <field name="name"/>
+                        </h1>
+                    </group>
+                </sheet>
+            </form>
+        </field>
+    </record>
+    </data>
+    
+
+    
+</odoo>
\ No newline at end of file
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
index 57586e76f3d..61890ff816f 100644
--- a/estate/views/estate_property_type_views.xml
+++ b/estate/views/estate_property_type_views.xml
@@ -7,4 +7,31 @@
         <field name="view_mode">list,form</field>
     </record>
 
+    <record id="estate_property_type_view_list" model="ir.ui.view">
+        <field name="name">estate.property.type.list</field>
+        <field name="model">estate.property.type</field>
+        <field name="arch" type="xml">
+            <list>
+                <field name="name"/>
+                <field name="description"/>
+            </list>
+        </field>
+    </record>
+
+    <record id="estate_property_type_view_form" 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>
+                <sheet>
+                    <group>
+                        <h1>
+                            <field name="name"/>
+                        </h1>
+                    </group>
+                </sheet>
+            </form>
+        </field>
+    </record>
+
 </odoo>
\ No newline at end of file
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 850a85f3065..18677a955d4 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -35,6 +35,7 @@
                 <field name="expected_price"/>
                 <field name="selling_price"/>
                 <field name="date_availability"/>
+                <!-- <field name="tag_ids" widget="many2many_tags"/>-->
             </list>
         </field>
     </record>
@@ -50,6 +51,9 @@
                             <field name="name"/>
                         </h1>
                     </group>
+                    <!-- <group>
+                        <field name="tag_ids" widget="many2many_tags"/>
+                    </group>-->
                     <group>
                         <group>
                             <field name="postcode"/>

From 047a922e66d6b3ae8f9fca6e2627a62da170c067 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Wed, 22 Oct 2025 13:33:14 +0200
Subject: [PATCH 12/33] [ADD] estate: added offers for property

---
 estate/__manifest__.py                       |  6 +--
 estate/models/__init__.py                    |  3 +-
 estate/models/estate_property.py             |  3 +-
 estate/models/estate_property_offer.py       | 15 +++++++
 estate/models/estate_property_tag.py         |  2 +-
 estate/security/ir.model.access.csv          |  1 +
 estate/views/estate_property_offer_views.xml | 32 +++++++++++++++
 estate/views/estate_property_tag_views.xml   |  6 +--
 estate/views/estate_property_tag_views_2.xml | 41 --------------------
 estate/views/estate_property_views.xml       | 11 ++++--
 10 files changed, 66 insertions(+), 54 deletions(-)
 create mode 100644 estate/models/estate_property_offer.py
 create mode 100644 estate/views/estate_property_offer_views.xml
 delete mode 100644 estate/views/estate_property_tag_views_2.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index edbbe92ef58..024f8806cf4 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -19,10 +19,10 @@
     'data': [
         'security/ir.model.access.csv',
         'views/estate_property_views.xml',
-        'views/estate_property_tag_views_2.xml',
-        'views/estate_menus.xml',
         'views/estate_property_type_views.xml',
-
+        'views/estate_property_tag_views.xml',
+        'views/estate_property_offer_views.xml',
+        'views/estate_menus.xml',
     ],
     'assets': {
 
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index fdab6f99948..09b2099fe84 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -1,3 +1,4 @@
 from . import estate_property
 from . import estate_property_type
-from . import estate_property_tag
\ No newline at end of file
+from . import estate_property_tag
+from . import estate_property_offer
\ No newline at end of file
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 70a54258732..b461d66b64f 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -32,4 +32,5 @@ class Estateproperty(models.Model):
     property_type_id = fields.Many2one("estate.property.type", string="Type")
     salesperson = fields.Many2one('res.partner', string='Salesperson')
     buyer = fields.Many2one('res.users', string='Buyer', default=lambda self: self.env.user)
-    #tag_ids = fields.Many2many('estate.property.tag', string='Tag')
+    tag_ids = fields.Many2many('estate.property.tag', string='Tag')
+    offer_ids = fields.One2many('estate.property.offer','property_id', string='Offer')
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
new file mode 100644
index 00000000000..11abf6a65e1
--- /dev/null
+++ b/estate/models/estate_property_offer.py
@@ -0,0 +1,15 @@
+from odoo import fields, models
+
+
+class EstatePropertyOffer(models.Model):
+    _name='estate.property.offer'
+    _description='Offer for the property'
+
+    price = fields.Float()
+    status = fields.Selection(
+        string='Status',
+        selection=[('accepted', 'Accepted'), ('refused', 'Refused')],
+        copy=False
+    )
+    partner_id = fields.Many2one('res.partner', required=True)
+    property_id = fields.Many2one('estate.property', required=True)
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index 8ca5a69f1fc..d4d30cb7ced 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -6,4 +6,4 @@ class EstatePropertyTag(models.Model):
     _description = "Tags for the estate"
 
     name = fields.Char('Tag', required=True)
-    description = fields.Char('Description')
\ No newline at end of file
+    description = fields.Char('Description')
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index 70a94e12e73..9ef81f819e6 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -2,3 +2,4 @@ id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink
 access_estate_property_tag,estate.property.tag,model_estate_property_tag,base.group_user,1,1,1,1
 access_estate_property,estate.property,model_estate_property,base.group_user,1,1,1,1
 access_estate_property_type,estate.property.type,model_estate_property_type,base.group_user,1,1,1,1
+access_estate_property_offer,estate.property.offer,model_estate_property_offer,base.group_user,1,1,1,1
\ No newline at end of file
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
new file mode 100644
index 00000000000..178092755dc
--- /dev/null
+++ b/estate/views/estate_property_offer_views.xml
@@ -0,0 +1,32 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<odoo>
+
+    <record id="estate_property_offer_view_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>
+                <sheet>
+                    <group>
+                        <h1>
+                            <field name="price"/>
+                        </h1>
+                    </group>
+                </sheet>
+            </form>
+        </field>
+    </record>
+
+    <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">
+            <list>
+                <field name="price"/>
+                <field name="partner_id"/>
+                <field name="status"/>
+            </list>
+        </field>
+    </record>
+
+</odoo>
\ No newline at end of file
diff --git a/estate/views/estate_property_tag_views.xml b/estate/views/estate_property_tag_views.xml
index b90afe4be05..587472e0b2a 100644
--- a/estate/views/estate_property_tag_views.xml
+++ b/estate/views/estate_property_tag_views.xml
@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <odoo>
-<data>
     
     <record id="estate_property_tag_model_action" model="ir.actions.act_window">
         <field name="name">Property tags</field>
@@ -11,7 +10,7 @@
     <record id="estate_property_tag_view_list" model="ir.ui.view">
         <field name="name">estate.property.tag.list</field>
         <field name="model">estate.property.tag</field>
-        <field name="arch" tag="xml">
+        <field name="arch" type="xml">
             <list>
                 <field name="name"/>
                 <field name="description"/>
@@ -22,7 +21,7 @@
     <record id="estate_property_tag_view_form" model="ir.ui.view">
         <field name="name">estate.property.tag.form</field>
         <field name="model">estate.property.tag</field>
-        <field name="arch" tag="xml">
+        <field name="arch" type="xml">
             <form>
                 <sheet>
                     <group>
@@ -34,5 +33,4 @@
             </form>
         </field>
     </record>
-</data>
 </odoo>
\ No newline at end of file
diff --git a/estate/views/estate_property_tag_views_2.xml b/estate/views/estate_property_tag_views_2.xml
deleted file mode 100644
index 1e910748628..00000000000
--- a/estate/views/estate_property_tag_views_2.xml
+++ /dev/null
@@ -1,41 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<odoo>
-    <data>
-
-    <record id="estate_property_tag_model_action" 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>
-
-    <record id="estate_property_tag_view_list" model="ir.ui.view">
-        <field name="name">estate.property.tag.list</field>
-        <field name="model">estate.property.tag</field>
-        <field name="arch" tag="xml">
-            <list>
-                <field name="name"/>
-                <field name="description"/>
-            </list>
-        </field>
-    </record>
-
-    <record id="estate_property_tag_view_form" model="ir.ui.view">
-        <field name="name">estate.property.tag.form</field>
-        <field name="model">estate.property.tag</field>
-        <field name="arch" tag="xml">
-            <form>
-                <sheet>
-                    <group>
-                        <h1>
-                            <field name="name"/>
-                        </h1>
-                    </group>
-                </sheet>
-            </form>
-        </field>
-    </record>
-    </data>
-    
-
-    
-</odoo>
\ No newline at end of file
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 18677a955d4..571d5af5624 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -35,7 +35,7 @@
                 <field name="expected_price"/>
                 <field name="selling_price"/>
                 <field name="date_availability"/>
-                <!-- <field name="tag_ids" widget="many2many_tags"/>-->
+                <field name="tag_ids" widget="many2many_tags"/>
             </list>
         </field>
     </record>
@@ -51,9 +51,9 @@
                             <field name="name"/>
                         </h1>
                     </group>
-                    <!-- <group>
+                    <group>
                         <field name="tag_ids" widget="many2many_tags"/>
-                    </group>-->
+                    </group>
                     <group>
                         <group>
                             <field name="postcode"/>
@@ -77,6 +77,11 @@
                                 <field name="garden_orientation"/>
                             </group>
                         </page>
+                        <page string="Offers">
+                            <group>
+                                <field name="offer_ids"/>
+                            </group>
+                        </page>
                         <page string="Other Info">
                             <group>
                                 <field name="salesperson"/>

From 9734e7f8cf6679186ea48eac942399aa09b5417b Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Wed, 22 Oct 2025 13:40:23 +0200
Subject: [PATCH 13/33] [FIX] estate: runbot ci fixes

---
 estate/models/__init__.py                    | 2 +-
 estate/models/estate_property.py             | 2 +-
 estate/models/estate_property_offer.py       | 4 ++--
 estate/models/estate_property_tag.py         | 2 +-
 estate/security/ir.model.access.csv          | 2 +-
 estate/views/estate_menus.xml                | 2 +-
 estate/views/estate_property_offer_views.xml | 2 +-
 estate/views/estate_property_tag_views.xml   | 2 +-
 estate/views/estate_property_type_views.xml  | 2 +-
 estate/views/estate_property_views.xml       | 2 +-
 10 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 09b2099fe84..2f1821a39c1 100644
--- a/estate/models/__init__.py
+++ b/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/estate/models/estate_property.py b/estate/models/estate_property.py
index b461d66b64f..cdab9f05fc8 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -33,4 +33,4 @@ class Estateproperty(models.Model):
     salesperson = fields.Many2one('res.partner', string='Salesperson')
     buyer = fields.Many2one('res.users', string='Buyer', default=lambda self: self.env.user)
     tag_ids = fields.Many2many('estate.property.tag', string='Tag')
-    offer_ids = fields.One2many('estate.property.offer','property_id', string='Offer')
+    offer_ids = fields.One2many('estate.property.offer', 'property_id', string='Offer')
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 11abf6a65e1..69ae2253f51 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -2,8 +2,8 @@
 
 
 class EstatePropertyOffer(models.Model):
-    _name='estate.property.offer'
-    _description='Offer for the property'
+    _name = 'estate.property.offer'
+    _description = 'Offer for the property'
 
     price = fields.Float()
     status = fields.Selection(
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index d4d30cb7ced..96c6fa22081 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -1,4 +1,4 @@
-from odoo import fields, models 
+from odoo import fields, models
 
 
 class EstatePropertyTag(models.Model):
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index 9ef81f819e6..d5be899ec6e 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -2,4 +2,4 @@ id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink
 access_estate_property_tag,estate.property.tag,model_estate_property_tag,base.group_user,1,1,1,1
 access_estate_property,estate.property,model_estate_property,base.group_user,1,1,1,1
 access_estate_property_type,estate.property.type,model_estate_property_type,base.group_user,1,1,1,1
-access_estate_property_offer,estate.property.offer,model_estate_property_offer,base.group_user,1,1,1,1
\ No newline at end of file
+access_estate_property_offer,estate.property.offer,model_estate_property_offer,base.group_user,1,1,1,1
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
index ae7967382b3..e8e185599d3 100644
--- a/estate/views/estate_menus.xml
+++ b/estate/views/estate_menus.xml
@@ -9,4 +9,4 @@
            <menuitem id="estate_tag_model_menu_action" action="estate_property_tag_model_action"/>
         </menuitem>
     </menuitem>
-</odoo>
\ No newline at end of file
+</odoo>
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index 178092755dc..4692168f168 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -29,4 +29,4 @@
         </field>
     </record>
 
-</odoo>
\ No newline at end of file
+</odoo>
diff --git a/estate/views/estate_property_tag_views.xml b/estate/views/estate_property_tag_views.xml
index 587472e0b2a..ec40a2a2426 100644
--- a/estate/views/estate_property_tag_views.xml
+++ b/estate/views/estate_property_tag_views.xml
@@ -33,4 +33,4 @@
             </form>
         </field>
     </record>
-</odoo>
\ No newline at end of file
+</odoo>
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
index 61890ff816f..a09543e49c3 100644
--- a/estate/views/estate_property_type_views.xml
+++ b/estate/views/estate_property_type_views.xml
@@ -34,4 +34,4 @@
         </field>
     </record>
 
-</odoo>
\ No newline at end of file
+</odoo>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 571d5af5624..2896b8635fc 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -95,4 +95,4 @@
     </record>
 
     
-</odoo>
\ No newline at end of file
+</odoo>

From 1d50e6a457ba493533d60a65621b4d9329106e98 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Wed, 22 Oct 2025 15:09:27 +0200
Subject: [PATCH 14/33] [ADD] estate: added computed fields and onchanges

---
 estate/models/estate_property.py             | 24 +++++++++++++++++++-
 estate/models/estate_property_offer.py       | 17 +++++++++++++-
 estate/models/estate_property_type.py        |  2 +-
 estate/views/estate_property_offer_views.xml |  6 +++++
 estate/views/estate_property_views.xml       |  4 ++++
 5 files changed, 50 insertions(+), 3 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index cdab9f05fc8..dc83f33cb96 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,4 @@
-from odoo import fields, models
+from odoo import fields, models, api
 from dateutil.relativedelta import relativedelta
 
 
@@ -34,3 +34,25 @@ class Estateproperty(models.Model):
     buyer = fields.Many2one('res.users', string='Buyer', default=lambda self: self.env.user)
     tag_ids = fields.Many2many('estate.property.tag', string='Tag')
     offer_ids = fields.One2many('estate.property.offer', 'property_id', string='Offer')
+    total_area = fields.Float(compute="_compute_total_area")
+    best_offer = fields.Float(compute="_compute_best_offer")
+
+
+    @api.depends("garden_area","living_area")
+    def _compute_total_area(self):
+        for record in self:
+            record.total_area = record.garden_area + record.living_area
+
+    @api.depends("offer_ids")
+    def _compute_best_offer(self):
+        for record in self:
+            record.best_offer = max(record.offer_ids.mapped("price") or [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 = None
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 69ae2253f51..97176817754 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -1,4 +1,6 @@
-from odoo import fields, models
+from odoo import fields, models, api
+from dateutil.relativedelta import relativedelta
+from datetime import datetime
 
 
 class EstatePropertyOffer(models.Model):
@@ -13,3 +15,16 @@ class EstatePropertyOffer(models.Model):
     )
     partner_id = fields.Many2one('res.partner', required=True)
     property_id = fields.Many2one('estate.property', required=True)
+    validity = fields.Integer(string="Validity (days)", default=7)
+    date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline")
+
+
+    @api.depends("validity")
+    def _compute_date_deadline(self):
+        for record in self:
+            record.date_deadline = fields.Datetime.now() + relativedelta(days=record.validity)
+
+    def _inverse_date_deadline(self):
+        for record in self:
+            deadline = datetime(record.date_deadline.year, record.date_deadline.month, record.date_deadline.day)
+            record.validity = int((deadline - fields.Datetime.now()).days)
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index 34fcfe87f7c..d6b21c2dbe9 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -6,4 +6,4 @@ class EstatePropertyType(models.Model):
     _description = "Estate Property Type"
 
     name = fields.Char('Type', required=True)
-    description = fields.Text('Description')
+    description = fields.Text()
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index 4692168f168..d7989ef3738 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -10,6 +10,10 @@
                     <group>
                         <h1>
                             <field name="price"/>
+                            <field name="partner_id"/>
+                            <field name="validity"/>
+                            <field name="date_deadline"/>
+                            <field name="status"/>
                         </h1>
                     </group>
                 </sheet>
@@ -24,6 +28,8 @@
             <list>
                 <field name="price"/>
                 <field name="partner_id"/>
+                <field name="validity"/>
+                <field name="date_deadline"/>
                 <field name="status"/>
             </list>
         </field>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 2896b8635fc..69288542153 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -36,6 +36,7 @@
                 <field name="selling_price"/>
                 <field name="date_availability"/>
                 <field name="tag_ids" widget="many2many_tags"/>
+                <field name="total_area"/>
             </list>
         </field>
     </record>
@@ -56,10 +57,12 @@
                     </group>
                     <group>
                         <group>
+                            <field name="property_type_id"/>
                             <field name="postcode"/>
                             <field name="date_availability"/>
                         </group>
                     <group>
+                        <field name="best_offer"/>
                         <field name="expected_price"/>
                         <field name="selling_price"/>
                     </group>
@@ -75,6 +78,7 @@
                                 <field name="garden"/>
                                 <field name="garden_area"/>
                                 <field name="garden_orientation"/>
+                                <field name="total_area"/>
                             </group>
                         </page>
                         <page string="Offers">

From 3b6035d45aadd94592e4070e421f7e5c05906f47 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Wed, 22 Oct 2025 16:28:51 +0200
Subject: [PATCH 15/33] [ADD] estate: added action buttons

---
 estate/models/estate_property.py             | 21 +++++++++++++++++---
 estate/models/estate_property_offer.py       | 18 +++++++++++++++--
 estate/views/estate_property_offer_views.xml | 12 ++++++-----
 estate/views/estate_property_views.xml       |  4 ++++
 4 files changed, 45 insertions(+), 10 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index dc83f33cb96..750a61aaca7 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,5 @@
 from odoo import fields, models, api
+from odoo.exceptions import UserError
 from dateutil.relativedelta import relativedelta
 
 
@@ -31,14 +32,14 @@ class Estateproperty(models.Model):
     )
     property_type_id = fields.Many2one("estate.property.type", string="Type")
     salesperson = fields.Many2one('res.partner', string='Salesperson')
-    buyer = fields.Many2one('res.users', string='Buyer', default=lambda self: self.env.user)
+    buyer = fields.Many2one('res.users', string='Buyer')
     tag_ids = fields.Many2many('estate.property.tag', string='Tag')
     offer_ids = fields.One2many('estate.property.offer', 'property_id', string='Offer')
     total_area = fields.Float(compute="_compute_total_area")
     best_offer = fields.Float(compute="_compute_best_offer")
+    offer_accepted = fields.Boolean()
 
-
-    @api.depends("garden_area","living_area")
+    @api.depends("garden_area", "living_area")
     def _compute_total_area(self):
         for record in self:
             record.total_area = record.garden_area + record.living_area
@@ -56,3 +57,17 @@ def _onchange_garden(self):
         else:
             self.garden_area = 0
             self.garden_orientation = None
+
+    def set_property_sold(self):
+        for record in self:
+            if record.state != 'cancelled':
+                record.state = 'sold'
+            else:
+                raise(UserError("Can not cancel sold property"))
+    
+    def set_property_cancelled(self):
+        for record in self:
+            if record.state != 'sold':
+                record.state = 'cancelled'
+            else:
+                raise(UserError("Can not cancel sold property"))
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 97176817754..252840e7936 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -1,4 +1,5 @@
 from odoo import fields, models, api
+from odoo.exceptions import UserError
 from dateutil.relativedelta import relativedelta
 from datetime import datetime
 
@@ -13,12 +14,11 @@ class EstatePropertyOffer(models.Model):
         selection=[('accepted', 'Accepted'), ('refused', 'Refused')],
         copy=False
     )
-    partner_id = fields.Many2one('res.partner', required=True)
+    partner_id = fields.Many2one('res.users', required=True, default=lambda self: self.env.user)
     property_id = fields.Many2one('estate.property', required=True)
     validity = fields.Integer(string="Validity (days)", default=7)
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline")
 
-
     @api.depends("validity")
     def _compute_date_deadline(self):
         for record in self:
@@ -28,3 +28,17 @@ def _inverse_date_deadline(self):
         for record in self:
             deadline = datetime(record.date_deadline.year, record.date_deadline.month, record.date_deadline.day)
             record.validity = int((deadline - fields.Datetime.now()).days)
+
+    def accept_offer(self):
+        for record in self:
+            if not record.property_id.offer_accepted:
+                record.status = 'accepted'
+                record.property_id.selling_price = record.price
+                record.property_id.buyer = record.partner_id
+                record.property_id.offer_accepted = True 
+            else:
+                raise(UserError("Can not accept more than one offer"))
+    
+    def reject_offer(self):
+        for record in self:
+            record.status = 'refused'
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index d7989ef3738..ca842e5cbb4 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -9,11 +9,11 @@
                 <sheet>
                     <group>
                         <h1>
-                            <field name="price"/>
-                            <field name="partner_id"/>
-                            <field name="validity"/>
-                            <field name="date_deadline"/>
-                            <field name="status"/>
+                            <field name="price" string="Price"/>
+                            <field name="partner_id" string="Partner id"/>
+                            <field name="validity" string="Valid until"/>
+                            <field name="date_deadline" string="Deadline"/>
+                            <field name="status" string="Status"/>
                         </h1>
                     </group>
                 </sheet>
@@ -30,6 +30,8 @@
                 <field name="partner_id"/>
                 <field name="validity"/>
                 <field name="date_deadline"/>
+                <button name="accept_offer" string="Accept" type="object" icon="fa-check"/>
+                <button name="reject_offer" string="Reject" type="object" icon="fa-times"/>
                 <field name="status"/>
             </list>
         </field>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 69288542153..37fb3d68516 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -46,6 +46,10 @@
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
             <form string="property">
+                <header>
+                    <button name="set_property_sold" type="object" string="SOLD"/>
+                    <button name="set_property_cancelled" type="object" string="CANCEL"/>
+                </header>
                 <sheet>
                     <group>
                         <h1>

From 2ff796784d0f7f8e1e03409cb7f95e8c970ae64e Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Wed, 22 Oct 2025 17:04:18 +0200
Subject: [PATCH 16/33] [ADD] estate: added constraints

---
 estate/models/estate_property.py       | 16 ++++++++++++++++
 estate/models/estate_property_offer.py |  4 ++++
 estate/models/estate_property_tag.py   |  5 +++++
 estate/models/estate_property_type.py  |  5 +++++
 4 files changed, 30 insertions(+)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 750a61aaca7..386899e3d6d 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,5 +1,6 @@
 from odoo import fields, models, api
 from odoo.exceptions import UserError
+from odoo.tools.float_utils import float_is_zero, float_compare
 from dateutil.relativedelta import relativedelta
 
 
@@ -39,6 +40,14 @@ class Estateproperty(models.Model):
     best_offer = fields.Float(compute="_compute_best_offer")
     offer_accepted = fields.Boolean()
 
+    _expected_property_price_strictly_positive = models.Constraint(
+        'CHECK(expected_price > 0)',
+    )
+
+    _property_selling_price_positive = models.Constraint(
+        'CHECK(selling_price) >= 0',
+    )
+
     @api.depends("garden_area", "living_area")
     def _compute_total_area(self):
         for record in self:
@@ -71,3 +80,10 @@ def set_property_cancelled(self):
                 record.state = 'cancelled'
             else:
                 raise(UserError("Can not cancel sold property"))
+
+    @api.onchange('selling_price', 'expected_price')
+    @api.constrains('selling_price')
+    def _check_selling_price(self):
+        for record in self:
+            if not(float_is_zero(record.selling_price,2)) and record.selling_price < 0.9 * record.expected_price:
+                raise(UserError("Selling price can not be less than 90'%' of Excpected price"))
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 252840e7936..de4ef8a4c8f 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -19,6 +19,10 @@ class EstatePropertyOffer(models.Model):
     validity = fields.Integer(string="Validity (days)", default=7)
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline")
 
+    _offer_price_strictly_positive = models.Constraint(
+        'CHECK(price > 0)',
+    )
+
     @api.depends("validity")
     def _compute_date_deadline(self):
         for record in self:
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index 96c6fa22081..55617c285eb 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -7,3 +7,8 @@ class EstatePropertyTag(models.Model):
 
     name = fields.Char('Tag', required=True)
     description = fields.Char('Description')
+
+    _unique_tag = models.Constraint(
+        'UNIQUE(name)',
+        'Tag already exists'
+    )
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index d6b21c2dbe9..c9aea42997e 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -7,3 +7,8 @@ class EstatePropertyType(models.Model):
 
     name = fields.Char('Type', required=True)
     description = fields.Text()
+
+    _unique_type = models.Constraint(
+        'UNIQUE(name)',
+        'Property type name exists'
+    )

From 98c5ac758ae217bdd02c08dadcc89dc5442d14e2 Mon Sep 17 00:00:00 2001
From: Denis Rygaert <dery@odoo.com>
Date: Wed, 22 Oct 2025 15:28:17 +0200
Subject: [PATCH 17/33] [IMP] estate: Add notes.

---
 estate/models/estate_property.py       | 1 +
 estate/views/estate_property_views.xml | 3 +++
 2 files changed, 4 insertions(+)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 386899e3d6d..cfcbce9004d 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -10,6 +10,7 @@ class Estateproperty(models.Model):
 
     name = fields.Char('Title', required=True)
     description = fields.Text('Description', required=True)
+    notes = fields.Html()
     postcode = fields.Char('Postcode', required=True)
     date_availability = fields.Date('Available From', copy=False, default=fields.Datetime.now() + relativedelta(months=3))
     expected_price = fields.Float('Expected Price', required=True)
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 37fb3d68516..3a1de7ceca8 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -90,6 +90,9 @@
                                 <field name="offer_ids"/>
                             </group>
                         </page>
+                        <page name="notes" string="Notes">
+                            <field name="notes"/>
+                        </page>
                         <page string="Other Info">
                             <group>
                                 <field name="salesperson"/>

From 716e3267c3e1976f6f886cd1559581776f1856bd Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Thu, 23 Oct 2025 10:24:20 +0200
Subject: [PATCH 18/33] [ADD] estate: midway in adding sprinkles

---
 estate/models/estate_property.py            |  3 ++-
 estate/models/estate_property_offer.py      |  1 +
 estate/models/estate_property_tag.py        |  1 +
 estate/models/estate_property_type.py       | 11 +++++++++++
 estate/security/ir.model.access.csv         |  1 +
 estate/views/estate_property_tag_views.xml  |  2 +-
 estate/views/estate_property_type_views.xml | 11 +++++++++++
 estate/views/estate_property_views.xml      |  1 +
 8 files changed, 29 insertions(+), 2 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index cfcbce9004d..d69ba28cd9d 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -7,6 +7,7 @@
 class Estateproperty(models.Model):
     _name = "estate.property"
     _description = "Estate property"
+    _order = "id desc"
 
     name = fields.Char('Title', required=True)
     description = fields.Text('Description', required=True)
@@ -46,7 +47,7 @@ class Estateproperty(models.Model):
     )
 
     _property_selling_price_positive = models.Constraint(
-        'CHECK(selling_price) >= 0',
+        'CHECK(selling_price >= 0)',
     )
 
     @api.depends("garden_area", "living_area")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index de4ef8a4c8f..48f854442d4 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -7,6 +7,7 @@
 class EstatePropertyOffer(models.Model):
     _name = 'estate.property.offer'
     _description = 'Offer for the property'
+    _order = 'price desc'
 
     price = fields.Float()
     status = fields.Selection(
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index 55617c285eb..f39fed29008 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -4,6 +4,7 @@
 class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "Tags for the estate"
+    _order = 'name'
 
     name = fields.Char('Tag', required=True)
     description = fields.Char('Description')
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index c9aea42997e..998d05597d2 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -4,11 +4,22 @@
 class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "Estate Property Type"
+    _order = 'name'
 
     name = fields.Char('Type', required=True)
     description = fields.Text()
+    property_type_line_ids = fields.One2many('estate.property.type.line', 'name')
 
     _unique_type = models.Constraint(
         'UNIQUE(name)',
         'Property type name exists'
     )
+
+class EstatePropertyTypeLine(models.Model):
+    _name = 'estate.property.type.line'
+    _description = 'estate property view per type'
+
+    name = fields.Many2one('estate.property.type')
+    title = fields.Char()
+    expected_price = fields.Char()
+    state = fields.Char()
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index d5be899ec6e..6feb88ab0b1 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -3,3 +3,4 @@ access_estate_property_tag,estate.property.tag,model_estate_property_tag,base.gr
 access_estate_property,estate.property,model_estate_property,base.group_user,1,1,1,1
 access_estate_property_type,estate.property.type,model_estate_property_type,base.group_user,1,1,1,1
 access_estate_property_offer,estate.property.offer,model_estate_property_offer,base.group_user,1,1,1,1
+access_estate_property_type_line,estate.property.type.line,model_estate_property_type_line,base.group_user,1,1,1,1
diff --git a/estate/views/estate_property_tag_views.xml b/estate/views/estate_property_tag_views.xml
index ec40a2a2426..6cbd1b83bd1 100644
--- a/estate/views/estate_property_tag_views.xml
+++ b/estate/views/estate_property_tag_views.xml
@@ -2,7 +2,7 @@
 <odoo>
     
     <record id="estate_property_tag_model_action" model="ir.actions.act_window">
-        <field name="name">Property tags</field>
+        <field name="name">Property Tags</field>
         <field name="res_model">estate.property.tag</field>
         <field name="view_mode">list,form</field>
     </record>
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
index a09543e49c3..1b60d23fab5 100644
--- a/estate/views/estate_property_type_views.xml
+++ b/estate/views/estate_property_type_views.xml
@@ -29,6 +29,17 @@
                             <field name="name"/>
                         </h1>
                     </group>
+                    <notebook>
+                        <page string="Properties">
+                            <field name="property_type_line_ids">
+                                <list>
+                                    <field name="title"/>
+                                    <field name="expected_price"/>
+                                    <field name="state"/>
+                                </list>
+                            </field>
+                        </page>
+                    </notebook>
                 </sheet>
             </form>
         </field>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 3a1de7ceca8..b522f27c216 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -49,6 +49,7 @@
                 <header>
                     <button name="set_property_sold" type="object" string="SOLD"/>
                     <button name="set_property_cancelled" type="object" string="CANCEL"/>
+                    <field name="state" widget="statusbar" statusbar_visible="new, offer received, offer accepted, sold"/>
                 </header>
                 <sheet>
                     <group>

From fef3a693cfb8cfe45a25e8812bb451fad1a26921 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Thu, 23 Oct 2025 15:16:01 +0200
Subject: [PATCH 19/33] [ADD] estate: added all sprinkles

---
 estate/__manifest__.py                       |  2 +-
 estate/models/estate_property_offer.py       |  2 ++
 estate/models/estate_property_tag.py         |  1 +
 estate/models/estate_property_type.py        | 21 ++++++++--------
 estate/security/ir.model.access.csv          |  1 -
 estate/views/estate_property_offer_views.xml | 14 +++++++----
 estate/views/estate_property_tag_views.xml   |  2 +-
 estate/views/estate_property_type_views.xml  |  9 +++++--
 estate/views/estate_property_views.xml       | 25 ++++++++++----------
 9 files changed, 44 insertions(+), 33 deletions(-)

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 024f8806cf4..417cd7d130b 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -19,9 +19,9 @@
     'data': [
         'security/ir.model.access.csv',
         'views/estate_property_views.xml',
+        'views/estate_property_offer_views.xml',
         'views/estate_property_type_views.xml',
         'views/estate_property_tag_views.xml',
-        'views/estate_property_offer_views.xml',
         'views/estate_menus.xml',
     ],
     'assets': {
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 48f854442d4..bf6e117fbde 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -19,6 +19,7 @@ class EstatePropertyOffer(models.Model):
     property_id = fields.Many2one('estate.property', required=True)
     validity = fields.Integer(string="Validity (days)", default=7)
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline")
+    property_type_id = fields.Many2one(related="property_id.property_type_id")
 
     _offer_price_strictly_positive = models.Constraint(
         'CHECK(price > 0)',
@@ -40,6 +41,7 @@ def accept_offer(self):
                 record.status = 'accepted'
                 record.property_id.selling_price = record.price
                 record.property_id.buyer = record.partner_id
+                record.property_id.state = 'offer accepted'
                 record.property_id.offer_accepted = True 
             else:
                 raise(UserError("Can not accept more than one offer"))
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index f39fed29008..d7da9553776 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -8,6 +8,7 @@ class EstatePropertyTag(models.Model):
 
     name = fields.Char('Tag', required=True)
     description = fields.Char('Description')
+    color = fields.Integer()
 
     _unique_tag = models.Constraint(
         'UNIQUE(name)',
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index 998d05597d2..320dd618d5b 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -1,25 +1,24 @@
-from odoo import fields, models
+from odoo import fields, models, api
 
 
 class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "Estate Property Type"
-    _order = 'name'
+    _order = 'sequence, name, id'
 
     name = fields.Char('Type', required=True)
     description = fields.Text()
-    property_type_line_ids = fields.One2many('estate.property.type.line', 'name')
+    property_ids = fields.One2many('estate.property', 'property_type_id')
+    sequence = fields.Integer('Sequence', default=1)
+    offer_ids = fields.One2many('estate.property.offer', 'property_type_id')
+    offer_count = fields.Integer(compute="_computer_offers_count")
 
     _unique_type = models.Constraint(
         'UNIQUE(name)',
         'Property type name exists'
     )
 
-class EstatePropertyTypeLine(models.Model):
-    _name = 'estate.property.type.line'
-    _description = 'estate property view per type'
-
-    name = fields.Many2one('estate.property.type')
-    title = fields.Char()
-    expected_price = fields.Char()
-    state = fields.Char()
+    @api.depends("offer_ids")
+    def _compute_offers_count(self):
+        for record in self:
+            record.offer_count = len(record.offer_ids)
\ No newline at end of file
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index 6feb88ab0b1..d5be899ec6e 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -3,4 +3,3 @@ access_estate_property_tag,estate.property.tag,model_estate_property_tag,base.gr
 access_estate_property,estate.property,model_estate_property,base.group_user,1,1,1,1
 access_estate_property_type,estate.property.type,model_estate_property_type,base.group_user,1,1,1,1
 access_estate_property_offer,estate.property.offer,model_estate_property_offer,base.group_user,1,1,1,1
-access_estate_property_type_line,estate.property.type.line,model_estate_property_type_line,base.group_user,1,1,1,1
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index ca842e5cbb4..38fd4fe3f78 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -13,7 +13,6 @@
                             <field name="partner_id" string="Partner id"/>
                             <field name="validity" string="Valid until"/>
                             <field name="date_deadline" string="Deadline"/>
-                            <field name="status" string="Status"/>
                         </h1>
                     </group>
                 </sheet>
@@ -25,16 +24,21 @@
         <field name="name">estate.property.offer.list</field>
         <field name="model">estate.property.offer</field>
         <field name="arch" type="xml">
-            <list>
+            <list editable="top" decoration-success="status == 'accepted'" decoration-danger="status == 'refused'">
                 <field name="price"/>
                 <field name="partner_id"/>
                 <field name="validity"/>
                 <field name="date_deadline"/>
-                <button name="accept_offer" string="Accept" type="object" icon="fa-check"/>
-                <button name="reject_offer" string="Reject" type="object" icon="fa-times"/>
-                <field name="status"/>
+                <button name="accept_offer" string="Accept" type="object" invisible="status in ('accepted', 'refused')" icon="fa-check"/>
+                <button name="reject_offer" string="Reject" type="object" invisible="status in ('accepted', 'refused')" icon="fa-times"/>
             </list>
         </field>
     </record>
 
+    <record id="estate_property_offer_action" model="ir.actions.act_window">
+        <field name="name">Offers</field>
+        <field name="res_model">estate.property.offer</field>
+        <field name="view_mode">list</field>
+    </record>
+    
 </odoo>
diff --git a/estate/views/estate_property_tag_views.xml b/estate/views/estate_property_tag_views.xml
index 6cbd1b83bd1..e7a0cc07029 100644
--- a/estate/views/estate_property_tag_views.xml
+++ b/estate/views/estate_property_tag_views.xml
@@ -11,7 +11,7 @@
         <field name="name">estate.property.tag.list</field>
         <field name="model">estate.property.tag</field>
         <field name="arch" type="xml">
-            <list>
+            <list editable="top">
                 <field name="name"/>
                 <field name="description"/>
             </list>
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
index 1b60d23fab5..659baae8869 100644
--- a/estate/views/estate_property_type_views.xml
+++ b/estate/views/estate_property_type_views.xml
@@ -12,6 +12,7 @@
         <field name="model">estate.property.type</field>
         <field name="arch" type="xml">
             <list>
+                <field name="sequence" widget="handle"/>
                 <field name="name"/>
                 <field name="description"/>
             </list>
@@ -24,6 +25,10 @@
         <field name="arch" type="xml">
             <form>
                 <sheet>
+                    <div class="oe_button_box" name="button_box">
+                        <button name="%(estate.estate_property_offer_action)d" class="oe_stat_button" 
+                        icon="fa-money" string="offers" type="action" context="{'search_default_property_type_id': id}"/>
+                    </div>
                     <group>
                         <h1>
                             <field name="name"/>
@@ -31,9 +36,9 @@
                     </group>
                     <notebook>
                         <page string="Properties">
-                            <field name="property_type_line_ids">
+                            <field name="property_ids">
                                 <list>
-                                    <field name="title"/>
+                                    <field name="name"/>
                                     <field name="expected_price"/>
                                     <field name="state"/>
                                 </list>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index b522f27c216..f52dcb012ac 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -4,6 +4,7 @@
         <field name="name">Properties</field>
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
+        <field name="context">{'search_default_available': True, 'search_default_current': True}</field>
     </record>
 
     <record id="estate_property_view_search" model="ir.ui.view">
@@ -15,9 +16,9 @@
                 <field name="postcode"/>
                 <field name="expected_price"/>
                 <field name="bedrooms"/>
-                <field name="living_area"/>
+                <field name="living_area" filter_domain="[('living_area', '>=', self)]"/>
                 <field name="facades"/>
-                <filter string="Available" name="available properties" domain="[ ('state', 'in', ['new', 'offer received']) ]"/>
+                <filter string="Available" name="available" domain="[ ('state', 'in', ['new', 'offer received']) ]"/>
                 <filter string="Postcode" name="postcode" context="{'group_by':'postcode', 'residual_visible':True}"/>
             </search>
         </field>
@@ -27,15 +28,15 @@
         <field name="name">estate.property.list</field>
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
-            <list string="Channel">
+            <list string="Channel" decoration-muted="state == 'sold'" decoration-success="state in ('offer received', 'offer accepted')" decoration-bf="state == 'offer accepted'">
                 <field name="name"/>
                 <field name="postcode"/>
                 <field name="bedrooms"/>
                 <field name="living_area"/>
                 <field name="expected_price"/>
                 <field name="selling_price"/>
-                <field name="date_availability"/>
-                <field name="tag_ids" widget="many2many_tags"/>
+                <field name="date_availability" optional="hide"/>
+                <field name="tag_ids" widget="many2many_tags" options="{'color_field': 'color'}"/>
                 <field name="total_area"/>
             </list>
         </field>
@@ -47,8 +48,8 @@
         <field name="arch" type="xml">
             <form string="property">
                 <header>
-                    <button name="set_property_sold" type="object" string="SOLD"/>
-                    <button name="set_property_cancelled" type="object" string="CANCEL"/>
+                    <button name="set_property_sold" type="object" string="SOLD" invisible="state in ('sold', 'cancelled')"/>
+                    <button name="set_property_cancelled" type="object" string="CANCEL" invisible="state in ('sold', 'cancelled')"/>
                     <field name="state" widget="statusbar" statusbar_visible="new, offer received, offer accepted, sold"/>
                 </header>
                 <sheet>
@@ -58,11 +59,11 @@
                         </h1>
                     </group>
                     <group>
-                        <field name="tag_ids" widget="many2many_tags"/>
+                        <field name="tag_ids" widget="many2many_tags" options="{'color_field': 'color'}"/>
                     </group>
                     <group>
                         <group>
-                            <field name="property_type_id"/>
+                            <field name="property_type_id" options="{'no_create': true, 'no_create_edit': true}"/>
                             <field name="postcode"/>
                             <field name="date_availability"/>
                         </group>
@@ -81,14 +82,14 @@
                                 <field name="facades"/>
                                 <field name="garage"/>
                                 <field name="garden"/>
-                                <field name="garden_area"/>
-                                <field name="garden_orientation"/>
+                                <field name="garden_area" invisible="not garden"/>
+                                <field name="garden_orientation" invisible="not garden"/>
                                 <field name="total_area"/>
                             </group>
                         </page>
                         <page string="Offers">
                             <group>
-                                <field name="offer_ids"/>
+                                <field name="offer_ids" readonly="state in ('sold', 'offer received', 'cancelled')"/>
                             </group>
                         </page>
                         <page name="notes" string="Notes">

From 5cb4327a0805dd1fafce485e8c1b469d55754358 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Thu, 23 Oct 2025 16:45:13 +0200
Subject: [PATCH 20/33] [ADD] estate: added inheritance

---
 estate/__manifest__.py                 |  1 +
 estate/models/__init__.py              |  1 +
 estate/models/estate_property.py       |  9 ++++++++-
 estate/models/estate_property_offer.py |  9 ++++++++-
 estate/models/res_users.py             |  7 +++++++
 estate/security/ir.model.access.csv    |  1 +
 estate/views/users_views.xml           | 17 +++++++++++++++++
 7 files changed, 43 insertions(+), 2 deletions(-)
 create mode 100644 estate/models/res_users.py
 create mode 100644 estate/views/users_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 417cd7d130b..07cd665019f 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -18,6 +18,7 @@
 
     'data': [
         'security/ir.model.access.csv',
+        'views/users_views.xml',
         'views/estate_property_views.xml',
         'views/estate_property_offer_views.xml',
         'views/estate_property_type_views.xml',
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 2f1821a39c1..a9459ed5906 100644
--- a/estate/models/__init__.py
+++ b/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 res_users
\ No newline at end of file
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index d69ba28cd9d..de7b35d10d2 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -34,7 +34,7 @@ class Estateproperty(models.Model):
         default=('new')
     )
     property_type_id = fields.Many2one("estate.property.type", string="Type")
-    salesperson = fields.Many2one('res.partner', string='Salesperson')
+    salesperson = fields.Many2one('res.users', string='Salesperson')
     buyer = fields.Many2one('res.users', string='Buyer')
     tag_ids = fields.Many2many('estate.property.tag', string='Tag')
     offer_ids = fields.One2many('estate.property.offer', 'property_id', string='Offer')
@@ -89,3 +89,10 @@ def _check_selling_price(self):
         for record in self:
             if not(float_is_zero(record.selling_price,2)) and record.selling_price < 0.9 * record.expected_price:
                 raise(UserError("Selling price can not be less than 90'%' of Excpected price"))
+
+    @api.ondelete(at_uninstall=False)
+    def unlink(self):
+        for record in self:
+            if record.state not in {'new', 'cancelled'}:
+                raise(UserError("Can not delete properties at this state"))
+        return super.unlink()
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index bf6e117fbde..014ec5a56d5 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -19,7 +19,7 @@ class EstatePropertyOffer(models.Model):
     property_id = fields.Many2one('estate.property', required=True)
     validity = fields.Integer(string="Validity (days)", default=7)
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline")
-    property_type_id = fields.Many2one(related="property_id.property_type_id")
+    property_type_id = fields.Many2one(related="property_id.property_type_id", store=True)
 
     _offer_price_strictly_positive = models.Constraint(
         'CHECK(price > 0)',
@@ -49,3 +49,10 @@ def accept_offer(self):
     def reject_offer(self):
         for record in self:
             record.status = 'refused'
+    
+    @api.model
+    def create(self, vals_list):
+        if vals_list[0].get('price') < min(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')):
+            raise(UserError("Can not have an offer that is less the minimum offer"))
+        self.env['estate.property'].browse(vals_list[0]['property_id']).state = 'offer received'
+        return super().create(vals_list)
diff --git a/estate/models/res_users.py b/estate/models/res_users.py
new file mode 100644
index 00000000000..e86e9d241d1
--- /dev/null
+++ b/estate/models/res_users.py
@@ -0,0 +1,7 @@
+from odoo import models, fields
+
+
+class ResUsers(models.Model):
+    _inherit = ['res.users']
+
+    property_ids = fields.One2many('estate.property', 'salesperson')
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index d5be899ec6e..b26774ee0ce 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -3,3 +3,4 @@ access_estate_property_tag,estate.property.tag,model_estate_property_tag,base.gr
 access_estate_property,estate.property,model_estate_property,base.group_user,1,1,1,1
 access_estate_property_type,estate.property.type,model_estate_property_type,base.group_user,1,1,1,1
 access_estate_property_offer,estate.property.offer,model_estate_property_offer,base.group_user,1,1,1,1
+access_res_users,res.users,model_res_users,base.group_user,1,1,1,1
diff --git a/estate/views/users_views.xml b/estate/views/users_views.xml
new file mode 100644
index 00000000000..a3bb737c89c
--- /dev/null
+++ b/estate/views/users_views.xml
@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<odoo>
+    <record id="res_users_view_form" model="ir.ui.view">
+        <field name="name">res.users.view.form</field>
+        <field name="model">res.users</field>
+        <field name="inherit_id" ref="base.view_users_form"/>
+        <field name="arch" type="xml">
+            <notebook>
+                <page string="Real Estate Properties">
+                    <group>
+                        <field name="property_ids"/>
+                    </group>
+                </page>
+            </notebook>
+        </field>
+    </record>
+</odoo>
\ No newline at end of file

From 7f8e1818a57e5f8028249a48582b05a9f9c679c1 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Fri, 24 Oct 2025 11:04:11 +0200
Subject: [PATCH 21/33] [ADD] estate: added inheritance

---
 estate/models/estate_property.py       | 2 +-
 estate/models/estate_property_offer.py | 2 +-
 estate/models/res_users.py             | 4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index de7b35d10d2..6cda5c59749 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -35,7 +35,7 @@ class Estateproperty(models.Model):
     )
     property_type_id = fields.Many2one("estate.property.type", string="Type")
     salesperson = fields.Many2one('res.users', string='Salesperson')
-    buyer = fields.Many2one('res.users', string='Buyer')
+    buyer = fields.Many2one('res.partner', string='Buyer')
     tag_ids = fields.Many2many('estate.property.tag', string='Tag')
     offer_ids = fields.One2many('estate.property.offer', 'property_id', string='Offer')
     total_area = fields.Float(compute="_compute_total_area")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 014ec5a56d5..eaafd73d2e3 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -15,7 +15,7 @@ class EstatePropertyOffer(models.Model):
         selection=[('accepted', 'Accepted'), ('refused', 'Refused')],
         copy=False
     )
-    partner_id = fields.Many2one('res.users', required=True, default=lambda self: self.env.user)
+    partner_id = fields.Many2one('res.partner', required=True, default=lambda self: self.env.user)
     property_id = fields.Many2one('estate.property', required=True)
     validity = fields.Integer(string="Validity (days)", default=7)
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline")
diff --git a/estate/models/res_users.py b/estate/models/res_users.py
index e86e9d241d1..31b3f9f56ca 100644
--- a/estate/models/res_users.py
+++ b/estate/models/res_users.py
@@ -2,6 +2,6 @@
 
 
 class ResUsers(models.Model):
-    _inherit = ['res.users']
+    _inherit = 'res.users'
 
-    property_ids = fields.One2many('estate.property', 'salesperson')
+    property_ids = fields.One2many('estate.property', 'salesperson', domain=[('state', 'in', ['new', 'offer received'])] )

From 73f89e15660217d9823da822d101877049d3ba99 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Fri, 24 Oct 2025 11:04:57 +0200
Subject: [PATCH 22/33] [ADD] estate_account: added interact with other modules

---
 estate_account/__init__.py                  |  1 +
 estate_account/__manifest__.py              | 26 +++++++++++++++++++++
 estate_account/models/__init__.py           |  1 +
 estate_account/models/estate_property.py    | 15 ++++++++++++
 estate_account/security/ir.model.access.csv |  2 ++
 5 files changed, 45 insertions(+)
 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 estate_account/security/ir.model.access.csv

diff --git a/estate_account/__init__.py b/estate_account/__init__.py
new file mode 100644
index 00000000000..9a7e03eded3
--- /dev/null
+++ b/estate_account/__init__.py
@@ -0,0 +1 @@
+from . import models
\ No newline at end of file
diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py
new file mode 100644
index 00000000000..6f9593cd199
--- /dev/null
+++ b/estate_account/__manifest__.py
@@ -0,0 +1,26 @@
+{
+    'name': "Estate Accounting",
+
+    'summary': """
+        Real Estate Tuto"
+    """,
+
+    'description': """
+        Estate accounting module"
+    """,
+
+    'author': "Odoo",
+    'website': "https://www.odoo.com/",
+    'category': 'Tutorials',
+    'version': '0.1',
+    'application': True,
+    'depends': ['estate','account'],
+
+    'data': [
+        'security/ir.model.access.csv',
+    ],
+    'assets': {
+
+    },
+    'license': 'AGPL-3'
+}
diff --git a/estate_account/models/__init__.py b/estate_account/models/__init__.py
new file mode 100644
index 00000000000..f4c8fd6db6d
--- /dev/null
+++ b/estate_account/models/__init__.py
@@ -0,0 +1 @@
+from . import estate_property
\ No newline at end of file
diff --git a/estate_account/models/estate_property.py b/estate_account/models/estate_property.py
new file mode 100644
index 00000000000..27b88386a77
--- /dev/null
+++ b/estate_account/models/estate_property.py
@@ -0,0 +1,15 @@
+from odoo import models, fields, Command
+from odoo.exceptions import AccessError
+
+
+class EstateProperty(models.Model):
+    _inherit = 'estate.property'
+
+    def set_property_sold(self):
+        for val in self:
+            self.env['account.move'].create({
+                'partner_id': val.buyer.id, 'move_type':'out_invoice',
+                'invoice_line_ids': [Command.create({'name': val.name, 'quantity': 1, 'price_unit': val.selling_price * 0.06})],
+                'invoice_line_ids': [Command.create({'name': 'administrative_fees', 'quantity':1, 'price_unit': 100.00})]
+                })
+        return super().set_property_sold()
diff --git a/estate_account/security/ir.model.access.csv b/estate_account/security/ir.model.access.csv
new file mode 100644
index 00000000000..9ccf73e191e
--- /dev/null
+++ b/estate_account/security/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,estate.property,model_estate_property,base.group_user,1,1,1,1
\ No newline at end of file

From 33752380ac34a789916e01ba432b1af976235f15 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Fri, 24 Oct 2025 11:42:56 +0200
Subject: [PATCH 23/33] [ADD] estate: added kanban view

---
 estate/models/estate_property_offer.py |  2 +-
 estate/views/estate_property_views.xml | 33 ++++++++++++++++++++++++--
 2 files changed, 32 insertions(+), 3 deletions(-)

diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index eaafd73d2e3..33100610fb3 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -52,7 +52,7 @@ def reject_offer(self):
     
     @api.model
     def create(self, vals_list):
-        if vals_list[0].get('price') < min(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')):
+        if len(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')) and vals_list[0].get('price') < min(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')):
             raise(UserError("Can not have an offer that is less the minimum offer"))
         self.env['estate.property'].browse(vals_list[0]['property_id']).state = 'offer received'
         return super().create(vals_list)
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index f52dcb012ac..81758f1346d 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -3,7 +3,7 @@
     <record id="estate_property_model_action" model="ir.actions.act_window">
         <field name="name">Properties</field>
         <field name="res_model">estate.property</field>
-        <field name="view_mode">list,form</field>
+        <field name="view_mode">list,form,kanban</field>
         <field name="context">{'search_default_available': True, 'search_default_current': True}</field>
     </record>
 
@@ -89,7 +89,7 @@
                         </page>
                         <page string="Offers">
                             <group>
-                                <field name="offer_ids" readonly="state in ('sold', 'offer received', 'cancelled')"/>
+                                <field name="offer_ids" readonly="state in ('sold', 'offer accepted', 'cancelled')"/>
                             </group>
                         </page>
                         <page name="notes" string="Notes">
@@ -107,5 +107,34 @@
         </field>
     </record>
 
+    <record id="estate_property_view_kanban" model="ir.ui.view">
+        <field name="name">estate.properties.kanban.view</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <kanban default_group_by="property_type_id" records_draggable="0">
+                <field name="state"/>
+                <templates>
+                    <t t-name="card">
+                        <div>
+                            <field name="name"/>
+                            <div>Expected Price:
+                                <field name="expected_price"/>
+                            </div>
+                            <div t-if="record.state.raw_value == 'offer received'">
+                                Best Price:
+                                <field name="best_offer"/>
+                            </div>
+                            <div t-if="record.state.raw_value == 'offer accepted'">
+                                Selling Price:
+                                <field name="selling_price"/>
+                            </div>
+                            <field name="tag_ids" string="Property Tags" widget="many2many_tags"
+                                options="{'color_field': 'color'}"/>
+                        </div>
+                    </t>
+                </templates>
+            </kanban>
+        </field>
+    </record>
     
 </odoo>

From b65cb6d4d0e6b99187ee76b1374c310c00b9e10c Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Fri, 24 Oct 2025 13:50:17 +0200
Subject: [PATCH 24/33] [IMP] estate: improve styles for estate module

---
 estate/models/__init__.py              |  2 +-
 estate/models/estate_property.py       | 12 ++++++------
 estate/models/estate_property_offer.py |  8 ++++----
 estate/models/estate_property_type.py  |  2 +-
 estate/models/res_users.py             |  2 +-
 5 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index a9459ed5906..9a2189b6382 100644
--- a/estate/models/__init__.py
+++ b/estate/models/__init__.py
@@ -2,4 +2,4 @@
 from . import estate_property_type
 from . import estate_property_tag
 from . import estate_property_offer
-from . import res_users
\ No newline at end of file
+from . import res_users
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 6cda5c59749..19e0f5d2c74 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,6 +1,6 @@
 from odoo import fields, models, api
 from odoo.exceptions import UserError
-from odoo.tools.float_utils import float_is_zero, float_compare
+from odoo.tools.float_utils import float_is_zero
 from dateutil.relativedelta import relativedelta
 
 
@@ -74,25 +74,25 @@ def set_property_sold(self):
             if record.state != 'cancelled':
                 record.state = 'sold'
             else:
-                raise(UserError("Can not cancel sold property"))
+                raise UserError("Can not cancel sold property")
     
     def set_property_cancelled(self):
         for record in self:
             if record.state != 'sold':
                 record.state = 'cancelled'
             else:
-                raise(UserError("Can not cancel sold property"))
+                raise UserError("Can not cancel sold property")
 
     @api.onchange('selling_price', 'expected_price')
     @api.constrains('selling_price')
     def _check_selling_price(self):
         for record in self:
-            if not(float_is_zero(record.selling_price,2)) and record.selling_price < 0.9 * record.expected_price:
-                raise(UserError("Selling price can not be less than 90'%' of Excpected price"))
+            if not float_is_zero(record.selling_price,2) and record.selling_price < 0.9 * record.expected_price:
+                raise UserError("Selling price can not be less than 90'%' of Excpected price")
 
     @api.ondelete(at_uninstall=False)
     def unlink(self):
         for record in self:
             if record.state not in {'new', 'cancelled'}:
                 raise(UserError("Can not delete properties at this state"))
-        return super.unlink()
+        return super().unlink()
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 33100610fb3..7737277c538 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -44,15 +44,15 @@ def accept_offer(self):
                 record.property_id.state = 'offer accepted'
                 record.property_id.offer_accepted = True 
             else:
-                raise(UserError("Can not accept more than one offer"))
-    
+                raise UserError("Can not accept more than one offer")
+
     def reject_offer(self):
         for record in self:
             record.status = 'refused'
-    
+
     @api.model
     def create(self, vals_list):
         if len(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')) and vals_list[0].get('price') < min(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')):
-            raise(UserError("Can not have an offer that is less the minimum offer"))
+            raise UserError("Can not have an offer that is less the minimum offer")
         self.env['estate.property'].browse(vals_list[0]['property_id']).state = 'offer received'
         return super().create(vals_list)
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index 320dd618d5b..198dbd2a3a4 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -21,4 +21,4 @@ class EstatePropertyType(models.Model):
     @api.depends("offer_ids")
     def _compute_offers_count(self):
         for record in self:
-            record.offer_count = len(record.offer_ids)
\ No newline at end of file
+            record.offer_count = len(record.offer_ids)
diff --git a/estate/models/res_users.py b/estate/models/res_users.py
index 31b3f9f56ca..df42bf7ff3c 100644
--- a/estate/models/res_users.py
+++ b/estate/models/res_users.py
@@ -4,4 +4,4 @@
 class ResUsers(models.Model):
     _inherit = 'res.users'
 
-    property_ids = fields.One2many('estate.property', 'salesperson', domain=[('state', 'in', ['new', 'offer received'])] )
+    property_ids = fields.One2many('estate.property', 'salesperson', domain=[('state', 'in', ['new', 'offer received'])])

From 3a9b3fa6065d66aa692b6b654389ed1f989674ae Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Fri, 24 Oct 2025 13:53:05 +0200
Subject: [PATCH 25/33] [IMP] estate_account: improve style for estate account

---
 estate_account/__init__.py               | 2 +-
 estate_account/models/__init__.py        | 2 +-
 estate_account/models/estate_property.py | 9 ++++-----
 3 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/estate_account/__init__.py b/estate_account/__init__.py
index 9a7e03eded3..0650744f6bc 100644
--- a/estate_account/__init__.py
+++ b/estate_account/__init__.py
@@ -1 +1 @@
-from . import models
\ No newline at end of file
+from . import models
diff --git a/estate_account/models/__init__.py b/estate_account/models/__init__.py
index f4c8fd6db6d..5e1963c9d2f 100644
--- a/estate_account/models/__init__.py
+++ b/estate_account/models/__init__.py
@@ -1 +1 @@
-from . import estate_property
\ No newline at end of file
+from . import estate_property
diff --git a/estate_account/models/estate_property.py b/estate_account/models/estate_property.py
index 27b88386a77..17399332f84 100644
--- a/estate_account/models/estate_property.py
+++ b/estate_account/models/estate_property.py
@@ -1,5 +1,4 @@
-from odoo import models, fields, Command
-from odoo.exceptions import AccessError
+from odoo import models, Command
 
 
 class EstateProperty(models.Model):
@@ -8,8 +7,8 @@ class EstateProperty(models.Model):
     def set_property_sold(self):
         for val in self:
             self.env['account.move'].create({
-                'partner_id': val.buyer.id, 'move_type':'out_invoice',
-                'invoice_line_ids': [Command.create({'name': val.name, 'quantity': 1, 'price_unit': val.selling_price * 0.06})],
-                'invoice_line_ids': [Command.create({'name': 'administrative_fees', 'quantity':1, 'price_unit': 100.00})]
+                'partner_id': val.buyer.id, 'move_type': 'out_invoice',
+                'invoice_line_ids': [Command.create({'name': val.name, 'quantity': 1, 'price_unit': val.selling_price * 0.06}), 
+                                    Command.create({'name': 'administrative_fees', 'quantity': 1, 'price_unit': 100.00})]
                 })
         return super().set_property_sold()

From 8cd1bc6538b6d7da8c5a4d4c082a1ba4da5ceb64 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Fri, 24 Oct 2025 13:56:03 +0200
Subject: [PATCH 26/33] [FIX] estate: fixing runbot test failures

---
 estate/models/estate_property_offer.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 7737277c538..e745a02d768 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -52,7 +52,7 @@ def reject_offer(self):
 
     @api.model
     def create(self, vals_list):
-        if len(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')) and vals_list[0].get('price') < min(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')):
+        if len(vals_list) > 0 and len(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')) and vals_list[0].get('price') < min(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')):
             raise UserError("Can not have an offer that is less the minimum offer")
         self.env['estate.property'].browse(vals_list[0]['property_id']).state = 'offer received'
         return super().create(vals_list)

From f5487415694f09dcac16c764e7df51315ed4c82a Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Fri, 24 Oct 2025 14:03:01 +0200
Subject: [PATCH 27/33] [IMP] estate: improve styles for estate module

---
 estate/models/estate_property.py       | 4 ++--
 estate/models/estate_property_offer.py | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 19e0f5d2c74..6c4e73f33c5 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -87,12 +87,12 @@ def set_property_cancelled(self):
     @api.constrains('selling_price')
     def _check_selling_price(self):
         for record in self:
-            if not float_is_zero(record.selling_price,2) and record.selling_price < 0.9 * record.expected_price:
+            if not float_is_zero(record.selling_price, 2) and record.selling_price < 0.9 * record.expected_price:
                 raise UserError("Selling price can not be less than 90'%' of Excpected price")
 
     @api.ondelete(at_uninstall=False)
     def unlink(self):
         for record in self:
             if record.state not in {'new', 'cancelled'}:
-                raise(UserError("Can not delete properties at this state"))
+                raise UserError("Can not delete properties at this state")
         return super().unlink()
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index e745a02d768..96877dcbb95 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -42,7 +42,7 @@ def accept_offer(self):
                 record.property_id.selling_price = record.price
                 record.property_id.buyer = record.partner_id
                 record.property_id.state = 'offer accepted'
-                record.property_id.offer_accepted = True 
+                record.property_id.offer_accepted = True
             else:
                 raise UserError("Can not accept more than one offer")
 

From 60a38013e6be43240f6ae9955b5e9f79712aa293 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Fri, 24 Oct 2025 14:03:11 +0200
Subject: [PATCH 28/33] [IMP] estate_account: improve style for estate account

---
 estate_account/__manifest__.py           | 2 +-
 estate_account/models/estate_property.py | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py
index 6f9593cd199..15b3d5f96d9 100644
--- a/estate_account/__manifest__.py
+++ b/estate_account/__manifest__.py
@@ -14,7 +14,7 @@
     'category': 'Tutorials',
     'version': '0.1',
     'application': True,
-    'depends': ['estate','account'],
+    'depends': ['estate','account'], 
 
     'data': [
         'security/ir.model.access.csv',
diff --git a/estate_account/models/estate_property.py b/estate_account/models/estate_property.py
index 17399332f84..ca0bbbc0969 100644
--- a/estate_account/models/estate_property.py
+++ b/estate_account/models/estate_property.py
@@ -8,7 +8,7 @@ def set_property_sold(self):
         for val in self:
             self.env['account.move'].create({
                 'partner_id': val.buyer.id, 'move_type': 'out_invoice',
-                'invoice_line_ids': [Command.create({'name': val.name, 'quantity': 1, 'price_unit': val.selling_price * 0.06}), 
+                'invoice_line_ids': [Command.create({'name': val.name, 'quantity': 1, 'price_unit': val.selling_price * 0.06}),
                                     Command.create({'name': 'administrative_fees', 'quantity': 1, 'price_unit': 100.00})]
                 })
         return super().set_property_sold()

From 704e468c4ee7ffcdba37650f08cf4bfa7b733e29 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Fri, 24 Oct 2025 14:04:45 +0200
Subject: [PATCH 29/33] [FIX] estate: fixing runbot test failures

---
 estate/models/estate_property_offer.py | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 96877dcbb95..86dffb22020 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -52,7 +52,8 @@ def reject_offer(self):
 
     @api.model
     def create(self, vals_list):
-        if len(vals_list) > 0 and len(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')) and vals_list[0].get('price') < min(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')):
-            raise UserError("Can not have an offer that is less the minimum offer")
-        self.env['estate.property'].browse(vals_list[0]['property_id']).state = 'offer received'
+        if len(vals_list) > 0:
+            if len(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')) and vals_list[0].get('price') < min(self.env['estate.property'].browse(vals_list[0].get('property_id')).offer_ids.mapped('price')):
+                raise UserError("Can not have an offer that is less the minimum offer")
+            self.env['estate.property'].browse(vals_list[0]['property_id']).state = 'offer received'
         return super().create(vals_list)

From e33fb323770f077dff753e89345104a34bfc6504 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Fri, 24 Oct 2025 14:08:16 +0200
Subject: [PATCH 30/33] [FIX] estate: fixing runbot test failures

---
 estate/models/estate_property.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 6c4e73f33c5..200d22c27b2 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -75,7 +75,7 @@ def set_property_sold(self):
                 record.state = 'sold'
             else:
                 raise UserError("Can not cancel sold property")
-    
+
     def set_property_cancelled(self):
         for record in self:
             if record.state != 'sold':

From 32f1597cb7fb6d7d3b1c426a0e93439493ba69d2 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Fri, 24 Oct 2025 14:08:28 +0200
Subject: [PATCH 31/33] [IMP] estate_account: improve style for estate account

---
 estate_account/__manifest__.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py
index 15b3d5f96d9..5308f6b9770 100644
--- a/estate_account/__manifest__.py
+++ b/estate_account/__manifest__.py
@@ -14,8 +14,7 @@
     'category': 'Tutorials',
     'version': '0.1',
     'application': True,
-    'depends': ['estate','account'], 
-
+    'depends': ['estate','account'],
     'data': [
         'security/ir.model.access.csv',
     ],

From cdf8a1231cc3bd503431c2546c96bd4539e986b8 Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Fri, 24 Oct 2025 14:10:00 +0200
Subject: [PATCH 32/33] [IMP] estate_account: improve style for estate account

---
 estate_account/__manifest__.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py
index 5308f6b9770..6499fb0f8ae 100644
--- a/estate_account/__manifest__.py
+++ b/estate_account/__manifest__.py
@@ -14,7 +14,7 @@
     'category': 'Tutorials',
     'version': '0.1',
     'application': True,
-    'depends': ['estate','account'],
+    'depends': ['estate', 'account'],
     'data': [
         'security/ir.model.access.csv',
     ],

From b9f9f94492849c60a0f7d512348c536c76aaf30f Mon Sep 17 00:00:00 2001
From: Wajih-Wanis <wajihwanis2000@gmail.com>
Date: Fri, 24 Oct 2025 14:26:39 +0200
Subject: [PATCH 33/33] [FIX] estate: fixing runbot test failures

---
 estate/models/estate_property.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 200d22c27b2..5fa197c6069 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -91,7 +91,7 @@ def _check_selling_price(self):
                 raise UserError("Selling price can not be less than 90'%' of Excpected price")
 
     @api.ondelete(at_uninstall=False)
-    def unlink(self):
+    def unlink_if_not_set(self):
         for record in self:
             if record.state not in {'new', 'cancelled'}:
                 raise UserError("Can not delete properties at this state")