From 5c40c2382d147dae82d26a0f3742cbcb6a6d3775 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Mon, 19 May 2025 15:16:34 +0200
Subject: [PATCH 01/23] [ADD] created estate module - ch2

---
 estate/__init__.py     |  0
 estate/__manifest__.py | 10 ++++++++++
 2 files changed, 10 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..e69de29bb2d
diff --git a/estate/__manifest__.py b/estate/__manifest__.py
new file mode 100644
index 00000000000..e6dc01f1001
--- /dev/null
+++ b/estate/__manifest__.py
@@ -0,0 +1,10 @@
+{
+    'name': 'ESTATE',
+    'description': "aras estate tutorial module",
+    'website': 'https://www.odoo.com/page/estate',
+    'depends': [
+        'base',
+    ],
+    'installable': True,
+    'application': True,
+}
\ No newline at end of file

From 93035f1ff2c69e3c7a859cf64375b55a260ebc17 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Tue, 20 May 2025 11:37:45 +0200
Subject: [PATCH 02/23] [IMP] added estate property model and access csv - ch3
 & ch4

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

diff --git a/estate/__init__.py b/estate/__init__.py
index e69de29bb2d..0650744f6bc 100644
--- a/estate/__init__.py
+++ b/estate/__init__.py
@@ -0,0 +1 @@
+from . import models
diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index e6dc01f1001..ae59d923abe 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -5,6 +5,9 @@
     'depends': [
         'base',
     ],
+    'data': [
+        'security/ir.model.access.csv',
+    ],
     'installable': True,
     'application': True,
-}
\ No newline at end of file
+}
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
new file mode 100644
index 00000000000..5e1963c9d2f
--- /dev/null
+++ b/estate/models/__init__.py
@@ -0,0 +1 @@
+from . import estate_property
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
new file mode 100644
index 00000000000..09ae0531384
--- /dev/null
+++ b/estate/models/estate_property.py
@@ -0,0 +1,26 @@
+from odoo import fields, models
+
+
+class EstateProperty(models.Model):
+    _name = "estate.property"
+    _description = "ch3 exercise tutorial"
+
+    name = fields.Char(required=True)
+    description = fields.Text()
+    postcode = fields.Char()
+    date_availability = fields.Date()
+    expected_price = fields.Float()
+    selling_price = fields.Float()
+    bedrooms = fields.Integer()
+    living_area = fields.Integer()
+    facades = fields.Integer()
+    garage = fields.Boolean()
+    garden = fields.Boolean()
+    garden_area = fields.Integer()
+    garden_orientation = fields.Selection(
+        string = 'Type',
+        selection = [('north', 'North'), ('south', 'South'), ('east', 'East'), ('west', 'West')]
+    )
+
+
+    
\ No newline at end of file
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
new file mode 100644
index 00000000000..405022dbef4
--- /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_property","access_estate_property","model_estate_property","base.group_user",1,1,1,1

From f6bc65e6a4098a895f91891291e577dd6f5509a8 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Tue, 20 May 2025 15:28:18 +0200
Subject: [PATCH 03/23] [IMP] added estate views - ch5

---
 estate/__manifest__.py                 |  2 ++
 estate/models/estate_property.py       | 14 +++++++++-----
 estate/views/estate_menus.xml          |  9 +++++++++
 estate/views/estate_property_views.xml |  9 +++++++++
 4 files changed, 29 insertions(+), 5 deletions(-)
 create mode 100644 estate/views/estate_menus.xml
 create mode 100644 estate/views/estate_property_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index ae59d923abe..277803288d8 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -7,6 +7,8 @@
     ],
     'data': [
         'security/ir.model.access.csv',
+        'views/estate_property_views.xml',
+        'views/estate_menus.xml',
     ],
     'installable': True,
     'application': True,
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 09ae0531384..cc4459a1272 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,24 +1,28 @@
 from odoo import fields, models
 
-
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "ch3 exercise tutorial"
 
     name = fields.Char(required=True)
+    active = False
+    state = fields.Selection(
+        string='Property State',
+        selection= [('new', 'New'), ('offer_received', 'Offer Received'), ('offer_accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')]
+    )
     description = fields.Text()
     postcode = fields.Char()
-    date_availability = fields.Date()
+    date_availability = fields.Date(default=lambda _: fields.Date.add(fields.Date.today(), months=3) , copy=False)
     expected_price = fields.Float()
-    selling_price = fields.Float()
-    bedrooms = fields.Integer()
+    selling_price = fields.Float(readonly=True, copy=False)
+    bedrooms = fields.Integer(default="2")
     living_area = fields.Integer()
     facades = fields.Integer()
     garage = fields.Boolean()
     garden = fields.Boolean()
     garden_area = fields.Integer()
     garden_orientation = fields.Selection(
-        string = 'Type',
+        string = 'Garden Orientation',
         selection = [('north', 'North'), ('south', 'South'), ('east', 'East'), ('west', 'West')]
     )
 
diff --git a/estate/views/estate_menus.xml b/estate/views/estate_menus.xml
new file mode 100644
index 00000000000..15b9f086f35
--- /dev/null
+++ b/estate/views/estate_menus.xml
@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<odoo>
+    <menuitem id="estate_property_menu_action" name="Estate" >
+        <menuitem id="advertisements_menu" name="Advertisements">
+            <menuitem id="properties_menu" name="Properties" action="estate_property_action"/>
+        </menuitem>
+    </menuitem>
+</odoo>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
new file mode 100644
index 00000000000..de1dcecaa8f
--- /dev/null
+++ b/estate/views/estate_property_views.xml
@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<odoo>
+    <record id="estate_property_action" model="ir.actions.act_window">
+        <field name="name">Estate Property</field>
+        <field name="res_model">estate.property</field>
+        <field name="view_mode">list,form</field>
+    </record>
+</odoo>

From 69db2b7af5efda5c7665bd3e6bce7bc9423c7ca3 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Tue, 20 May 2025 17:12:27 +0200
Subject: [PATCH 04/23] [IMP] created view layouts for list, form, and search -
 ch6

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

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index cc4459a1272..596470f710f 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -5,7 +5,7 @@ class EstateProperty(models.Model):
     _description = "ch3 exercise tutorial"
 
     name = fields.Char(required=True)
-    active = False
+    active = fields.Boolean(default=False)
     state = fields.Selection(
         string='Property State',
         selection= [('new', 'New'), ('offer_received', 'Offer Received'), ('offer_accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')]
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index de1dcecaa8f..0a90d7b7957 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -6,4 +6,87 @@
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</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="Property">
+                <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>
+                        <group>
+                            <field name="name"/>
+                        </group>
+                        <group>
+                            <field name="postcode"/>
+                            <field name="expected_price"/>
+                            <field name="date_availability"/>
+                            <field name="selling_price"/>
+                        </group>
+                    </group>
+                    <notebook>
+                        <page string="Description">
+                            <group>
+                                <field name="bedrooms"/>
+                            </group>
+                            <group>
+                                <field name="living_area"/>
+                            </group>
+                            <group>
+                                <field name="facades"/>
+                            </group>
+                            <group>
+                                <field name="garage"/>
+                            </group>
+                            <group>
+                                <field name="garden"/>
+                            </group>
+                            <group>
+                                <field name="garden_area"/>
+                            </group>
+                            <group>
+                                <field name="garden_orientation"/>
+                            </group>
+                        </page>
+                    </notebook>
+                </sheet>
+            </form>
+        </field>
+    </record>
+       
+    <record id="estate_property_view_search" model="ir.ui.view">
+        <field name="name">estate.property.search</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <search string="Property">
+                <field name="name"/>
+                <field name="postcode"/>
+                <field name="expected_price"/>
+                <field name="bedrooms"/>
+                <field name="living_area"/>
+                <field name="facades"/>
+                <filter string="Available" name="state" domain="['|', ('state', '=', 'new'), ('state', '=', 'offer_recieved')]"/>
+                <group expand="1" string="Group By">
+                    <filter string="Postcode" name="postcode" context="{'group_by':'postcode'}"/>
+                </group>
+            </search>
+        </field>
+    </record>
+
 </odoo>

From 1c513a2d228a3c2961e76d6e2234989701734019 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Wed, 21 May 2025 15:21:33 +0200
Subject: [PATCH 05/23] [IMP] -Created the estate property type model with form
 and list views, and added ability to set property types for properties.
 -Created the estate property tag with action, to add tags to properties.
 -Added settings menu to contain the type and tag menus. -Created estate
 property offer model with list and form views, to provide ability for
 partners to set offers on available properties. -Gave security access to new
 models.

---
 estate/__manifest__.py                       |  3 ++
 estate/models/__init__.py                    |  3 ++
 estate/models/estate_property.py             | 10 +++--
 estate/models/estate_property_offer.py       | 14 +++++++
 estate/models/estate_property_tag.py         |  7 ++++
 estate/models/estate_property_type.py        |  7 ++++
 estate/security/ir.model.access.csv          |  3 ++
 estate/views/estate_menus.xml                |  7 +++-
 estate/views/estate_property_offer_views.xml | 31 ++++++++++++++
 estate/views/estate_property_tag_views.xml   |  8 ++++
 estate/views/estate_property_type_views.xml  | 43 ++++++++++++++++++++
 estate/views/estate_property_views.xml       | 32 +++++++++------
 12 files changed, 150 insertions(+), 18 deletions(-)
 create mode 100644 estate/models/estate_property_offer.py
 create mode 100644 estate/models/estate_property_tag.py
 create mode 100644 estate/models/estate_property_type.py
 create mode 100644 estate/views/estate_property_offer_views.xml
 create mode 100644 estate/views/estate_property_tag_views.xml
 create mode 100644 estate/views/estate_property_type_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 277803288d8..deee250ad12 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -8,6 +8,9 @@
     'data': [
         'security/ir.model.access.csv',
         'views/estate_property_views.xml',
+        'views/estate_property_type_views.xml',
+        'views/estate_property_tag_views.xml',
+        'views/estate_property_offer_views.xml',
         'views/estate_menus.xml',
     ],
     'installable': True,
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 5e1963c9d2f..2f1821a39c1 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 estate_property_tag
+from . import estate_property_offer
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 596470f710f..177c4a2a399 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -5,7 +5,7 @@ class EstateProperty(models.Model):
     _description = "ch3 exercise tutorial"
 
     name = fields.Char(required=True)
-    active = fields.Boolean(default=False)
+    # active = fields.Boolean(default=False)
     state = fields.Selection(
         string='Property State',
         selection= [('new', 'New'), ('offer_received', 'Offer Received'), ('offer_accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')]
@@ -25,6 +25,8 @@ class EstateProperty(models.Model):
         string = 'Garden Orientation',
         selection = [('north', 'North'), ('south', 'South'), ('east', 'East'), ('west', 'West')]
     )
-
-
-    
\ No newline at end of file
+    property_type_id = fields.Many2one("estate.property.type", string="Property Type")
+    buyer = fields.Many2one("res.partner", string="Buyer", copy=False)
+    sales_person = fields.Many2one("res.users", string="Sales Person", default=lambda self: self.env.user)
+    property_tag_ids = fields.Many2many("estate.property.tag")
+    offer_ids = fields.One2many("estate.property.offer", "property_id")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
new file mode 100644
index 00000000000..4d213b3e1e9
--- /dev/null
+++ b/estate/models/estate_property_offer.py
@@ -0,0 +1,14 @@
+from odoo import fields, models
+
+class EstatePropertyOffer(models.Model):
+    _name = "estate.property.offer"
+    _description = "ch7 exercise tutorial"
+
+    price = fields.Float()
+    status = fields.Selection(
+        copy=False,
+        string='Offer Status',
+        selection= [('accepted', 'Accepted'), ('refused', 'Refused')]
+    )
+    partner_id = fields.Many2one("res.partner", string="Partner", required=True)
+    property_id = fields.Many2one("estate.property", string="Property", required=True)
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
new file mode 100644
index 00000000000..8ee25409072
--- /dev/null
+++ b/estate/models/estate_property_tag.py
@@ -0,0 +1,7 @@
+from odoo import fields, models
+
+class EstatePropertyTag(models.Model):
+    _name = "estate.property.tag"
+    _description = "ch7 exercise tutorial"
+
+    name = fields.Char(required=True)
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
new file mode 100644
index 00000000000..f846de27936
--- /dev/null
+++ b/estate/models/estate_property_type.py
@@ -0,0 +1,7 @@
+from odoo import fields, models
+
+class EstatePropertyType(models.Model):
+    _name = "estate.property.type"
+    _description = "ch7 exercise tutorial"
+
+    name = fields.Char(required=True)
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index 405022dbef4..6587527c500 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","access_estate_property","model_estate_property","base.group_user",1,1,1,1
+"access_estate_property_type","access_estate_property_type","model_estate_property_type","base.group_user",1,1,1,1
+"access_estate_property_tag","access_estate_property_tag","model_estate_property_tag","base.group_user",1,1,1,1
+"access_estate_property_offer","access_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_menus.xml b/estate/views/estate_menus.xml
index 15b9f086f35..000f1996766 100644
--- a/estate/views/estate_menus.xml
+++ b/estate/views/estate_menus.xml
@@ -1,9 +1,14 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
 <odoo>
-    <menuitem id="estate_property_menu_action" name="Estate" >
+    <menuitem id="estate_property_menu" name="Estate" >
         <menuitem id="advertisements_menu" name="Advertisements">
             <menuitem id="properties_menu" name="Properties" action="estate_property_action"/>
         </menuitem>
+
+        <menuitem id="settings_menu" name="Settings">
+            <menuitem id="property_types_menu" name="Property Types" action="estate_property_type_action"/>
+            <menuitem id="property_tags_menu" name="Property Tags" action="estate_property_type_action"/>
+        </menuitem>
     </menuitem>
 </odoo>
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
new file mode 100644
index 00000000000..53ad6341902
--- /dev/null
+++ b/estate/views/estate_property_offer_views.xml
@@ -0,0 +1,31 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<odoo>
+    <record id="estate_property_offer_view_list" model="ir.ui.view">
+        <field name="name">estate.property.offer.list</field>
+        <field name="model">estate.property.offer</field>
+        <field name="arch" type="xml">
+            <list string="Offer">
+                <field name="price"/>
+                <field name="partner_id"/>
+                <field name="status"/>
+            </list>
+        </field>
+    </record>
+
+    <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 string="Offer">
+                <sheet>
+                    <group>
+                        <field name="price"/>
+                        <field name="partner_id"/>
+                        <field name="status"/>
+                    </group>
+                </sheet>
+            </form>
+        </field>
+    </record>
+</odoo>
diff --git a/estate/views/estate_property_tag_views.xml b/estate/views/estate_property_tag_views.xml
new file mode 100644
index 00000000000..968ba9bcde6
--- /dev/null
+++ b/estate/views/estate_property_tag_views.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<odoo>
+    <record id="estate_property_tag_action" model="ir.actions.act_window">
+        <field name="name"/>
+        <field name="res_model">estate.property.tag</field>
+    </record>
+</odoo>
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
new file mode 100644
index 00000000000..0431ed7b535
--- /dev/null
+++ b/estate/views/estate_property_type_views.xml
@@ -0,0 +1,43 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<odoo>
+    <record id="estate_property_type_action" model="ir.actions.act_window">
+        <field name="name">Property Types</field>
+        <field name="res_model">estate.property.type</field>
+        <field name="view_mode">list,form,search</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 string="Property">
+                <field name="name"/>
+            </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 string="Property">
+                <sheet>
+                    <group>
+                        <field name="name"/>
+                    </group>
+                </sheet>
+            </form>
+        </field>
+    </record>
+       
+    <record id="estate_property_type_view_search" model="ir.ui.view">
+        <field name="name">estate.property.type.search</field>
+        <field name="model">estate.property.type</field>
+        <field name="arch" type="xml">
+            <search string="Property">
+                <field name="name"/>
+            </search>
+        </field>
+    </record>
+</odoo>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 0a90d7b7957..66b593daa39 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -4,7 +4,7 @@
     <record id="estate_property_action" model="ir.actions.act_window">
         <field name="name">Estate Property</field>
         <field name="res_model">estate.property</field>
-        <field name="view_mode">list,form</field>
+        <field name="view_mode">list,form,search</field>
     </record>
     
     <record id="estate_property_view_list" model="ir.ui.view">
@@ -13,6 +13,8 @@
         <field name="arch" type="xml">
             <list string="Property">
                 <field name="name"/>
+                <field name="property_tag_ids" widget="many2many_tags"/>
+                <field name="property_type_id"/>
                 <field name="postcode"/>
                 <field name="bedrooms"/>
                 <field name="living_area"/>
@@ -32,36 +34,40 @@
                     <group>
                         <group>
                             <field name="name"/>
+                            <field name="property_tag_ids" widget="many2many_tags"/>
                         </group>
                         <group>
+                            <field name="property_type_id"/>
                             <field name="postcode"/>
                             <field name="expected_price"/>
                             <field name="date_availability"/>
                             <field name="selling_price"/>
                         </group>
                     </group>
+
                     <notebook>
                         <page string="Description">
                             <group>
                                 <field name="bedrooms"/>
-                            </group>
-                            <group>
                                 <field name="living_area"/>
-                            </group>
-                            <group>
                                 <field name="facades"/>
-                            </group>
-                            <group>
                                 <field name="garage"/>
-                            </group>
-                            <group>
                                 <field name="garden"/>
+                                <field name="garden_area"/>
+                                <field name="garden_orientation"/>
                             </group>
+                        </page>
+
+                        <page string="Offers">
                             <group>
-                                <field name="garden_area"/>
+                                <field name="offer_ids"/>
                             </group>
+                        </page>
+
+                        <page string="Other Info">
                             <group>
-                                <field name="garden_orientation"/>
+                                <field name="buyer"/>
+                                <field name="sales_person"/>
                             </group>
                         </page>
                     </notebook>
@@ -76,17 +82,17 @@
         <field name="arch" type="xml">
             <search string="Property">
                 <field name="name"/>
+                <field name="property_type_id"/>
                 <field name="postcode"/>
                 <field name="expected_price"/>
                 <field name="bedrooms"/>
                 <field name="living_area"/>
                 <field name="facades"/>
-                <filter string="Available" name="state" domain="['|', ('state', '=', 'new'), ('state', '=', 'offer_recieved')]"/>
+                <filter string="Available" name="state" domain="[('state', 'in', ('new', 'offer_recieved'))]"/>
                 <group expand="1" string="Group By">
                     <filter string="Postcode" name="postcode" context="{'group_by':'postcode'}"/>
                 </group>
             </search>
         </field>
     </record>
-
 </odoo>

From 52dd22078329669c0745f72a814e5fa8cf93141a Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Wed, 21 May 2025 17:34:41 +0200
Subject: [PATCH 06/23] [IMP]  - Created total area field in estate property
 model to present the total area of the property, and added a compute method
 for it. - Created a best price field in estate method to determine the best
 price in all offers, and a compute method for it. -Created deadline and
 validity fields for offers to set a maximum time for offer validities, and
 created compute methods to compute them from each other so only 1 value needs
 to be inserted. - (ch8 tutorial)

---
 estate/models/estate_property.py             | 26 +++++++++++++++++++-
 estate/models/estate_property_offer.py       | 14 ++++++++++-
 estate/views/estate_property_offer_views.xml |  4 +++
 estate/views/estate_property_tag_views.xml   |  2 +-
 estate/views/estate_property_views.xml       |  2 ++
 5 files changed, 45 insertions(+), 3 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 177c4a2a399..fc9a3cf8d4f 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 api, fields, models
 
 class EstateProperty(models.Model):
     _name = "estate.property"
@@ -30,3 +30,27 @@ class EstateProperty(models.Model):
     sales_person = fields.Many2one("res.users", string="Sales Person", default=lambda self: self.env.user)
     property_tag_ids = fields.Many2many("estate.property.tag")
     offer_ids = fields.One2many("estate.property.offer", "property_id")
+    total_area = fields.Integer(compute="_compute_total_area")
+    best_price = fields.Float(compute="_compute_best_price", string="Best Offer")
+
+    @api.depends("living_area", "garden_area")
+    def _compute_total_area(self):
+        for property in self:
+            property.total_area = property.living_area + property.garden_area
+
+    @api.depends("offer_ids.price")
+    def _compute_best_price(self):
+        self.best_price = 0
+        for id in self.offer_ids:
+            if self.best_price < id.price:
+                self.best_price = id.price
+
+    @api.onchange("garden")
+    def _onchange_partner_id(self):
+        if self.garden:
+            self.garden_area = 10
+            self.garden_orientation = 'north'
+        else:
+            self.garden_area = None
+            self.garden_orientation = None
+
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 4d213b3e1e9..f01ae674fc6 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -1,4 +1,4 @@
-from odoo import fields, models
+from odoo import fields, models, api
 
 class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
@@ -12,3 +12,15 @@ class EstatePropertyOffer(models.Model):
     )
     partner_id = fields.Many2one("res.partner", string="Partner", required=True)
     property_id = fields.Many2one("estate.property", string="Property", required=True)
+    validity = fields.Integer(default=7, string="Validity")
+    date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline", string="Deadline")
+    create_date = fields.Date(default=fields.Date.today)
+
+    @api.depends("create_date", "validity")
+    def _compute_date_deadline(self):
+        for offer in self:
+            offer.date_deadline = fields.Date.add(offer.create_date, days=offer.validity)
+
+    def _inverse_date_deadline(self):
+        for offer in self:
+            offer.validity = (offer.date_deadline - offer.create_date).days
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index 53ad6341902..d5e91059d3b 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -8,6 +8,8 @@
             <list string="Offer">
                 <field name="price"/>
                 <field name="partner_id"/>
+                <field name="validity"/>
+                <field name="date_deadline"/>
                 <field name="status"/>
             </list>
         </field>
@@ -22,6 +24,8 @@
                     <group>
                         <field name="price"/>
                         <field name="partner_id"/>
+                        <field name="validity"/>
+                        <field name="date_deadline"/>
                         <field name="status"/>
                     </group>
                 </sheet>
diff --git a/estate/views/estate_property_tag_views.xml b/estate/views/estate_property_tag_views.xml
index 968ba9bcde6..d0241b3e72b 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_action" model="ir.actions.act_window">
-        <field name="name"/>
+        <field name="name">Property Tags</field>
         <field name="res_model">estate.property.tag</field>
     </record>
 </odoo>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 66b593daa39..2e1924cc83a 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -41,6 +41,7 @@
                             <field name="postcode"/>
                             <field name="expected_price"/>
                             <field name="date_availability"/>
+                            <field name="best_price"/>
                             <field name="selling_price"/>
                         </group>
                     </group>
@@ -55,6 +56,7 @@
                                 <field name="garden"/>
                                 <field name="garden_area"/>
                                 <field name="garden_orientation"/>
+                                <field name="total_area"/>
                             </group>
                         </page>
 

From 9dcc21f3f570c8bd572ffa58ccdec2cb7634b4b1 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Thu, 22 May 2025 10:40:52 +0200
Subject: [PATCH 07/23] [IMP] -Created header buttons and actions to set the
 status(cancelled/sold) of estate.property. -Created buttons and actions to
 set the status(accepted/refused) of estate.property.offer. -Update the buyer
 and selling_price of estate.property when a correspondung
 estate.property.offer is accepted. -Set all action/compute loops to iterate
 on records for better practice and clarity. -(ch9)

---
 estate/models/estate_property.py             | 45 ++++++++++++++------
 estate/models/estate_property_offer.py       | 27 +++++++++---
 estate/views/estate_property_offer_views.xml |  2 +
 estate/views/estate_property_views.xml       |  5 +++
 4 files changed, 60 insertions(+), 19 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index fc9a3cf8d4f..1e13f7e0633 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,4 @@
-from odoo import api, fields, models
+from odoo import api, fields, models, exceptions
 
 class EstateProperty(models.Model):
     _name = "estate.property"
@@ -7,7 +7,7 @@ class EstateProperty(models.Model):
     name = fields.Char(required=True)
     # active = fields.Boolean(default=False)
     state = fields.Selection(
-        string='Property State',
+        string='Status',
         selection= [('new', 'New'), ('offer_received', 'Offer Received'), ('offer_accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')]
     )
     description = fields.Text()
@@ -35,22 +35,39 @@ class EstateProperty(models.Model):
 
     @api.depends("living_area", "garden_area")
     def _compute_total_area(self):
-        for property in self:
-            property.total_area = property.living_area + property.garden_area
+        for record in self:
+            record.total_area = record.living_area + record.garden_area
 
     @api.depends("offer_ids.price")
     def _compute_best_price(self):
-        self.best_price = 0
-        for id in self.offer_ids:
-            if self.best_price < id.price:
-                self.best_price = id.price
+        for record in self:
+            record.best_price = 0
+            for id in record.offer_ids:
+                if record.best_price < id.price:
+                    record.best_price = id.price
 
     @api.onchange("garden")
     def _onchange_partner_id(self):
-        if self.garden:
-            self.garden_area = 10
-            self.garden_orientation = 'north'
-        else:
-            self.garden_area = None
-            self.garden_orientation = None
+        for record in self:
+            if record.garden:
+                record.garden_area = 10
+                record.garden_orientation = 'north'
+            else:
+                record.garden_area = None
+                record.garden_orientation = None
 
+    def action_property_sold(self):
+        for record in self:
+            if record.state == 'cancelled':
+                raise exceptions.UserError("Cancelled properties cannot be sold.")
+            else:
+                record.state = 'sold'
+        return True
+
+    def action_property_cancelled(self):
+        for record in self:
+            if record.state == 'sold':
+                raise exceptions.UserError("Sold properties cannot be cancelled.")
+            else:
+                record.state = 'cancelled'
+        return True
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index f01ae674fc6..64ce7f84e61 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -1,4 +1,4 @@
-from odoo import fields, models, api
+from odoo import fields, models, api, exceptions
 
 class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
@@ -18,9 +18,26 @@ class EstatePropertyOffer(models.Model):
 
     @api.depends("create_date", "validity")
     def _compute_date_deadline(self):
-        for offer in self:
-            offer.date_deadline = fields.Date.add(offer.create_date, days=offer.validity)
+        for record in self:
+            record.date_deadline = fields.Date.add(record.create_date, days=record.validity)
 
     def _inverse_date_deadline(self):
-        for offer in self:
-            offer.validity = (offer.date_deadline - offer.create_date).days
+        for record in self:
+            record.validity = (record.date_deadline - record.create_date).days
+
+    def action_offer_accept(self):
+        for record in self:
+            if record.status == 'accepted':
+                raise exceptions.UserError("A offer has already been accepted.")
+            record.status = 'accepted'
+            for id in record.property_id:
+                id.selling_price = record.price
+                id.buyer = record.partner_id
+        return True
+
+    def action_offer_refuse(self):
+        for record in self:
+            if record.status == 'accepted':
+                raise exceptions.UserError("You cannot refuse an accepted offer.")
+            record.status = 'refused'
+        return True
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index d5e91059d3b..b3a9eddd6a1 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -10,6 +10,8 @@
                 <field name="partner_id"/>
                 <field name="validity"/>
                 <field name="date_deadline"/>
+                <button name="action_offer_accept" type="object" icon="fa-check" title="accept"/>
+                <button name="action_offer_refuse" type="object" icon="fa-close" title="refuse"/>
                 <field name="status"/>
             </list>
         </field>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 2e1924cc83a..3010ebafa59 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -30,6 +30,10 @@
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
             <form string="Property">
+                <header>
+                    <button type="object" name="action_property_sold" string="SOLD"/>
+                    <button type="object" name="action_property_cancelled" string="CANCEL"/>
+                </header>
                 <sheet>
                     <group>
                         <group>
@@ -37,6 +41,7 @@
                             <field name="property_tag_ids" widget="many2many_tags"/>
                         </group>
                         <group>
+                            <field name="state"/>
                             <field name="property_type_id"/>
                             <field name="postcode"/>
                             <field name="expected_price"/>

From 8489f245f254a937a2f2807d4fe2fd558599ae31 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Thu, 22 May 2025 11:56:04 +0200
Subject: [PATCH 08/23] [IMP] -Created constraint in estate.property to ensure
 expected_price is positive and in estate.property.offer to ensure price is
 positive. -Created constraints in estate.property.tag and
 estate.property.type to ensure all tags and types are unique -Changed menu
 names to comply with coding guidelines. -(ch10)

---
 estate/models/estate_property.py       | 34 +++++++++++++++++---------
 estate/models/estate_property_offer.py | 15 +++++++++---
 estate/models/estate_property_tag.py   |  6 +++++
 estate/models/estate_property_type.py  |  6 +++++
 estate/security/ir.model.access.csv    |  2 +-
 estate/views/estate_menus.xml          |  8 +++---
 6 files changed, 51 insertions(+), 20 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 1e13f7e0633..4b605bfa56e 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -1,4 +1,7 @@
-from odoo import api, fields, models, exceptions
+from odoo import api, fields, models
+from odoo.exceptions import UserError, ValidationError
+from odoo.tools import float_compare
+
 
 class EstateProperty(models.Model):
     _name = "estate.property"
@@ -8,11 +11,11 @@ class EstateProperty(models.Model):
     # active = fields.Boolean(default=False)
     state = fields.Selection(
         string='Status',
-        selection= [('new', 'New'), ('offer_received', 'Offer Received'), ('offer_accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')]
+        selection=[('new', 'New'), ('offer_received', 'Offer Received'), ('offer_accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')]
     )
     description = fields.Text()
     postcode = fields.Char()
-    date_availability = fields.Date(default=lambda _: fields.Date.add(fields.Date.today(), months=3) , copy=False)
+    date_availability = fields.Date(default=lambda _: fields.Date.add(fields.Date.today(), months=3), copy=False)
     expected_price = fields.Float()
     selling_price = fields.Float(readonly=True, copy=False)
     bedrooms = fields.Integer(default="2")
@@ -22,8 +25,8 @@ class EstateProperty(models.Model):
     garden = fields.Boolean()
     garden_area = fields.Integer()
     garden_orientation = fields.Selection(
-        string = 'Garden Orientation',
-        selection = [('north', 'North'), ('south', 'South'), ('east', 'East'), ('west', 'West')]
+        string='Garden Orientation',
+        selection=[('north', 'North'), ('south', 'South'), ('east', 'East'), ('west', 'West')]
     )
     property_type_id = fields.Many2one("estate.property.type", string="Property Type")
     buyer = fields.Many2one("res.partner", string="Buyer", copy=False)
@@ -33,6 +36,11 @@ class EstateProperty(models.Model):
     total_area = fields.Integer(compute="_compute_total_area")
     best_price = fields.Float(compute="_compute_best_price", string="Best Offer")
 
+    _sql_constraints = [
+        ('check_expected_price', 'CHECK(expected_price > 0)',
+         'The Expected price of an estate should be positive.')
+    ]
+
     @api.depends("living_area", "garden_area")
     def _compute_total_area(self):
         for record in self:
@@ -41,10 +49,8 @@ def _compute_total_area(self):
     @api.depends("offer_ids.price")
     def _compute_best_price(self):
         for record in self:
-            record.best_price = 0
-            for id in record.offer_ids:
-                if record.best_price < id.price:
-                    record.best_price = id.price
+            prices = record.offer_ids.mapped('price')
+            record.best_price = max(prices) if prices else 0
 
     @api.onchange("garden")
     def _onchange_partner_id(self):
@@ -59,7 +65,7 @@ def _onchange_partner_id(self):
     def action_property_sold(self):
         for record in self:
             if record.state == 'cancelled':
-                raise exceptions.UserError("Cancelled properties cannot be sold.")
+                raise UserError("Cancelled properties cannot be sold.")
             else:
                 record.state = 'sold'
         return True
@@ -67,7 +73,13 @@ def action_property_sold(self):
     def action_property_cancelled(self):
         for record in self:
             if record.state == 'sold':
-                raise exceptions.UserError("Sold properties cannot be cancelled.")
+                raise UserError("Sold properties cannot be cancelled.")
             else:
                 record.state = 'cancelled'
         return True
+    
+    @api.constrains('selling_price')
+    def _check_selling_price(self):
+        for record in self:
+            if record.selling_price is not None and float_compare(record.selling_price, 0.9 * record.expected_price, precision_digits=9) == -1:
+                raise ValidationError("The selling price cannot be less than 90% of the expected price")
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 64ce7f84e61..8a9eb2433e2 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -1,4 +1,6 @@
-from odoo import fields, models, api, exceptions
+from odoo import api, fields, models
+from odoo.exceptions import UserError
+
 
 class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
@@ -8,7 +10,7 @@ class EstatePropertyOffer(models.Model):
     status = fields.Selection(
         copy=False,
         string='Offer Status',
-        selection= [('accepted', 'Accepted'), ('refused', 'Refused')]
+        selection=[('accepted', 'Accepted'), ('refused', 'Refused')]
     )
     partner_id = fields.Many2one("res.partner", string="Partner", required=True)
     property_id = fields.Many2one("estate.property", string="Property", required=True)
@@ -16,6 +18,11 @@ class EstatePropertyOffer(models.Model):
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline", string="Deadline")
     create_date = fields.Date(default=fields.Date.today)
 
+    _sql_constraints = [
+        ('check_price', 'CHECK(price > 0)',
+         'The Price offered for an estate should be positive.'),
+    ]
+
     @api.depends("create_date", "validity")
     def _compute_date_deadline(self):
         for record in self:
@@ -28,7 +35,7 @@ def _inverse_date_deadline(self):
     def action_offer_accept(self):
         for record in self:
             if record.status == 'accepted':
-                raise exceptions.UserError("A offer has already been accepted.")
+                raise UserError("A offer has already been accepted.")
             record.status = 'accepted'
             for id in record.property_id:
                 id.selling_price = record.price
@@ -38,6 +45,6 @@ def action_offer_accept(self):
     def action_offer_refuse(self):
         for record in self:
             if record.status == 'accepted':
-                raise exceptions.UserError("You cannot refuse an accepted offer.")
+                raise UserError("You cannot refuse an accepted offer.")
             record.status = 'refused'
         return True
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index 8ee25409072..fa2b2ad36c3 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -1,7 +1,13 @@
 from odoo import fields, models
 
+
 class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "ch7 exercise tutorial"
 
     name = fields.Char(required=True)
+
+    _sql_constraints = [
+        ('unique_property_tag', 'UNIQUE(name)',
+         'Each property tag should have a unique name.')
+    ]
\ No newline at end of file
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index f846de27936..28b7df77b60 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -1,7 +1,13 @@
 from odoo import fields, models
 
+
 class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "ch7 exercise tutorial"
 
     name = fields.Char(required=True)
+
+    _sql_constraints = [
+        ('unique_property_type', 'UNIQUE(name)',
+         'Each property type should have a unique name.')
+    ]
\ No newline at end of file
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
index 6587527c500..72c60bbff73 100644
--- a/estate/security/ir.model.access.csv
+++ b/estate/security/ir.model.access.csv
@@ -2,4 +2,4 @@
 "access_estate_property","access_estate_property","model_estate_property","base.group_user",1,1,1,1
 "access_estate_property_type","access_estate_property_type","model_estate_property_type","base.group_user",1,1,1,1
 "access_estate_property_tag","access_estate_property_tag","model_estate_property_tag","base.group_user",1,1,1,1
-"access_estate_property_offer","access_estate_property_offer","model_estate_property_offer","base.group_user",1,1,1,1
\ No newline at end of file
+"access_estate_property_offer","access_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 000f1996766..5908b95ce1b 100644
--- a/estate/views/estate_menus.xml
+++ b/estate/views/estate_menus.xml
@@ -1,14 +1,14 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
 <odoo>
-    <menuitem id="estate_property_menu" name="Estate" >
+    <menuitem id="estate_property_menu_root" name="Estate" >
         <menuitem id="advertisements_menu" name="Advertisements">
-            <menuitem id="properties_menu" name="Properties" action="estate_property_action"/>
+            <menuitem id="properties_menu_action" name="Properties" action="estate_property_action"/>
         </menuitem>
 
         <menuitem id="settings_menu" name="Settings">
-            <menuitem id="property_types_menu" name="Property Types" action="estate_property_type_action"/>
-            <menuitem id="property_tags_menu" name="Property Tags" action="estate_property_type_action"/>
+            <menuitem id="property_types_menu_action" name="Property Types" action="estate_property_type_action"/>
+            <menuitem id="property_tags_menu_action" name="Property Tags" action="estate_property_tag_action"/>
         </menuitem>
     </menuitem>
 </odoo>

From 82af63f3b974307ce102f71c2a5e588471038c25 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Mon, 26 May 2025 09:12:57 +0200
Subject: [PATCH 09/23] [IMP] -Added a list view in property types to view all
 houses with such property. -Created a widget to display property status in
 property form view. -Added ordering for properties, offers, tags, and types
 for deterministic ordering. -Added ability to manually order property types.
 -Added ability to set/change colors for property tags. -Set garden fields in
 properties to conditionally appear only if a garden is set. -Made the offer
 accept and refuse to conditionally disappear once the choice is set. -Made it
 impossible to add new offers to a property once it is sold or cancelled.
 -Made it possible to change offers and tags from list views. -Made property
 date availability optional and hidden by default. -Added decorations to color
 code offer states. -Added decorations to color code offer states. -Added a
 filter to make available properties selected by default. -Added a filter
 domain to search for properties with a given area or more. -Added a state
 button to view for property types all offers for houses of that type.

---
 estate/models/estate_property.py             |  8 ++++--
 estate/models/estate_property_offer.py       |  5 +++-
 estate/models/estate_property_tag.py         |  4 ++-
 estate/models/estate_property_type.py        | 11 +++++++-
 estate/views/estate_property_offer_views.xml | 22 +++++++++++----
 estate/views/estate_property_tag_views.xml   | 11 ++++++++
 estate/views/estate_property_type_views.xml  | 19 ++++++++++++-
 estate/views/estate_property_views.xml       | 29 ++++++++++++--------
 8 files changed, 85 insertions(+), 24 deletions(-)

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 4b605bfa56e..6cd76351cda 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -6,12 +6,14 @@
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "ch3 exercise tutorial"
+    _order = "id desc"
 
     name = fields.Char(required=True)
     # active = fields.Boolean(default=False)
     state = fields.Selection(
         string='Status',
-        selection=[('new', 'New'), ('offer_received', 'Offer Received'), ('offer_accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')]
+        selection=[('new', 'New'), ('offer_received', 'Offer Received'), ('offer_accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')],
+        default='new'
     )
     description = fields.Text()
     postcode = fields.Char()
@@ -51,6 +53,8 @@ def _compute_best_price(self):
         for record in self:
             prices = record.offer_ids.mapped('price')
             record.best_price = max(prices) if prices else 0
+            if prices and record.state in [None, 'new']:
+                record.state = 'offer_received'
 
     @api.onchange("garden")
     def _onchange_partner_id(self):
@@ -77,7 +81,7 @@ def action_property_cancelled(self):
             else:
                 record.state = 'cancelled'
         return True
-    
+
     @api.constrains('selling_price')
     def _check_selling_price(self):
         for record in self:
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 8a9eb2433e2..6ac1285fa5a 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -5,6 +5,7 @@
 class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
     _description = "ch7 exercise tutorial"
+    _order = "price desc"
 
     price = fields.Float()
     status = fields.Selection(
@@ -16,7 +17,8 @@ class EstatePropertyOffer(models.Model):
     property_id = fields.Many2one("estate.property", string="Property", required=True)
     validity = fields.Integer(default=7, string="Validity")
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline", string="Deadline")
-    create_date = fields.Date(default=fields.Date.today)
+    create_date = fields.Date(default=fields.Date.today) 
+    property_type_id = fields.Many2one(related="property_id.property_type_id")
 
     _sql_constraints = [
         ('check_price', 'CHECK(price > 0)',
@@ -40,6 +42,7 @@ def action_offer_accept(self):
             for id in record.property_id:
                 id.selling_price = record.price
                 id.buyer = record.partner_id
+                id.state = 'offer_accepted'
         return True
 
     def action_offer_refuse(self):
diff --git a/estate/models/estate_property_tag.py b/estate/models/estate_property_tag.py
index fa2b2ad36c3..0b973fa9e56 100644
--- a/estate/models/estate_property_tag.py
+++ b/estate/models/estate_property_tag.py
@@ -4,10 +4,12 @@
 class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "ch7 exercise tutorial"
+    _order = "name"
 
     name = fields.Char(required=True)
+    color = fields.Integer()
 
     _sql_constraints = [
         ('unique_property_tag', 'UNIQUE(name)',
          'Each property tag should have a unique name.')
-    ]
\ No newline at end of file
+    ]
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index 28b7df77b60..23bbe68a7f2 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -4,10 +4,19 @@
 class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "ch7 exercise tutorial"
+    _order = "sequence, name"
 
     name = fields.Char(required=True)
+    property_ids = fields.One2many("estate.property", "property_type_id")
+    sequence = fields.Integer()
+    offer_ids = fields.One2many("estate.property.offer", "property_type_id")
+    offer_count= fields.Integer(compute="_compute_offer_count")
 
     _sql_constraints = [
         ('unique_property_type', 'UNIQUE(name)',
          'Each property type should have a unique name.')
-    ]
\ No newline at end of file
+    ]
+
+    def _compute_offer_count(self):
+        for record in self:
+            record.offer_count = len(record.offer_ids)
diff --git a/estate/views/estate_property_offer_views.xml b/estate/views/estate_property_offer_views.xml
index b3a9eddd6a1..d97ef3ae650 100644
--- a/estate/views/estate_property_offer_views.xml
+++ b/estate/views/estate_property_offer_views.xml
@@ -1,27 +1,37 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
 <odoo>
+    <record id="estate_property_offer_action" model="ir.actions.act_window">
+        <field name="name">Property Offers</field>
+        <field name="res_model">estate.property.offer</field>
+        <field name="domain">[('property_type_id', '=', active_id)]</field>
+        <field name="view_mode">list,form</field>
+    </record>
+
     <record id="estate_property_offer_view_list" model="ir.ui.view">
         <field name="name">estate.property.offer.list</field>
         <field name="model">estate.property.offer</field>
         <field name="arch" type="xml">
-            <list string="Offer">
+            <list string="Offer"
+                  decoration-success="status == 'accepted'"
+                  decoration-danger="status == 'refused'"
+                  editable="top">
                 <field name="price"/>
                 <field name="partner_id"/>
                 <field name="validity"/>
                 <field name="date_deadline"/>
-                <button name="action_offer_accept" type="object" icon="fa-check" title="accept"/>
-                <button name="action_offer_refuse" type="object" icon="fa-close" title="refuse"/>
-                <field name="status"/>
+                <button name="action_offer_accept" type="object" icon="fa-check" title="accept" invisible="status"/>
+                <button name="action_offer_refuse" type="object" icon="fa-close" title="refuse" invisible="status"/>
+                <field name="property_type_id"/>
             </list>
         </field>
     </record>
 
     <record id="estate_property_offer_view_form" model="ir.ui.view">
-        <field name="name">estate.property.offer.form</field>
+        <field name="name">estate.property.off string="Offer"er.form</field>
         <field name="model">estate.property.offer</field>
         <field name="arch" type="xml">
-            <form string="Offer">
+            <form>
                 <sheet>
                     <group>
                         <field name="price"/>
diff --git a/estate/views/estate_property_tag_views.xml b/estate/views/estate_property_tag_views.xml
index d0241b3e72b..e863683c616 100644
--- a/estate/views/estate_property_tag_views.xml
+++ b/estate/views/estate_property_tag_views.xml
@@ -4,5 +4,16 @@
     <record id="estate_property_tag_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</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" type="xml">
+            <list string="Property" editable="top">
+                <field name="name"/>
+            </list>
+        </field>
     </record>
 </odoo>
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
index 0431ed7b535..c84f2dd7de9 100644
--- a/estate/views/estate_property_type_views.xml
+++ b/estate/views/estate_property_type_views.xml
@@ -11,7 +11,8 @@
         <field name="name">estate.property.type.list</field>
         <field name="model">estate.property.type</field>
         <field name="arch" type="xml">
-            <list string="Property">
+            <list string="Property" editable="top">
+                <field name="sequence" widget="handle"/>
                 <field name="name"/>
             </list>
         </field>
@@ -23,9 +24,25 @@
         <field name="arch" type="xml">
             <form string="Property">
                 <sheet>
+                    <xpath expr=".">
+                        <button class="oe_stat_button" name="%(estate_property_offer_action)d" string="Offers" type="action"/>
+                    </xpath>
                     <group>
                         <field name="name"/>
                     </group>
+                    <notebook>
+                        <page string="Properties">
+                            <group>
+                                <field name="property_ids">
+                                    <list>
+                                        <field name="name"/>
+                                        <field name="expected_price"/>
+                                        <field name="state"/>
+                                    </list>
+                                </field>
+                            </group>
+                        </page>
+                    </notebook>
                 </sheet>
             </form>
         </field>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index 3010ebafa59..cc27510462e 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -4,6 +4,7 @@
     <record id="estate_property_action" model="ir.actions.act_window">
         <field name="name">Estate Property</field>
         <field name="res_model">estate.property</field>
+        <field name="context">{'search_default_state': True}</field>
         <field name="view_mode">list,form,search</field>
     </record>
     
@@ -11,16 +12,19 @@
         <field name="name">estate.property.list</field>
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
-            <list string="Property">
+            <list string="Property"
+                  decoration-bf="state == 'offer_accepted'"
+                  decoration-success="state in ('offer_received', 'offer_accepted')"
+                  decoration-muted="state == 'sold'">
                 <field name="name"/>
-                <field name="property_tag_ids" widget="many2many_tags"/>
+                <field name="property_tag_ids" widget="many2many_tags" options="{'color_field': 'color'}"/>
                 <field name="property_type_id"/>
                 <field name="postcode"/>
                 <field name="bedrooms"/>
                 <field name="living_area"/>
                 <field name="expected_price"/>
                 <field name="selling_price"/>
-                <field name="date_availability"/>
+                <field name="date_availability" optional="hide"/>
             </list>
         </field>
     </record>
@@ -31,18 +35,18 @@
         <field name="arch" type="xml">
             <form string="Property">
                 <header>
-                    <button type="object" name="action_property_sold" string="SOLD"/>
-                    <button type="object" name="action_property_cancelled" string="CANCEL"/>
+                    <button type="object" name="action_property_sold" string="SOLD" invisible="state in ('sold', 'cancelled')"/>
+                    <button type="object" name="action_property_cancelled" string="CANCEL" invisible="state in ('sold', 'cancelled')"/>
+                    <field name="state" widget="statusbar" statusbar_visible="new,offer_received,offer_accepted,sold"/>
                 </header>
                 <sheet>
                     <group>
                         <group>
                             <field name="name"/>
-                            <field name="property_tag_ids" widget="many2many_tags"/>
+                            <field name="property_tag_ids" widget="many2many_tags" options="{'color_field': 'color'}"/>
                         </group>
                         <group>
-                            <field name="state"/>
-                            <field name="property_type_id"/>
+                            <field name="property_type_id" options="{'no_create': true}"/>
                             <field name="postcode"/>
                             <field name="expected_price"/>
                             <field name="date_availability"/>
@@ -59,15 +63,15 @@
                                 <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 ('offer_accepted', 'sold', 'cancelled')"/>
                             </group>
                         </page>
 
@@ -93,7 +97,8 @@
                 <field name="postcode"/>
                 <field name="expected_price"/>
                 <field name="bedrooms"/>
-                <field name="living_area"/>
+                <field string="larger or equal area" name="living_area"
+                       filter_domain="[('living_area', '>=', self)]"/>
                 <field name="facades"/>
                 <filter string="Available" name="state" domain="[('state', 'in', ('new', 'offer_recieved'))]"/>
                 <group expand="1" string="Group By">

From b8a329490d621808d8d0ae4e7140395f20751992 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Tue, 27 May 2025 09:52:13 +0200
Subject: [PATCH 10/23] [IMP] -Allowed new offers to be created only if their
 pricing is higher than existing ones. -Allowed properties to be deleted iff
 they are new or cancelled. -Added a list view for user page for all
 properties they have an offer for. -(ch12)

---
 estate/__manifest__.py                 |  1 +
 estate/models/__init__.py              |  1 +
 estate/models/estate_property.py       |  8 ++++++++
 estate/models/estate_property_offer.py | 11 +++++++++++
 estate/models/res_user.py              | 12 ++++++++++++
 estate/views/estate_property_views.xml |  2 +-
 estate/views/res_users_views.xml       | 18 ++++++++++++++++++
 7 files changed, 52 insertions(+), 1 deletion(-)
 create mode 100644 estate/models/res_user.py
 create mode 100644 estate/views/res_users_views.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index deee250ad12..ab40a80f409 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -12,6 +12,7 @@
         'views/estate_property_tag_views.xml',
         'views/estate_property_offer_views.xml',
         'views/estate_menus.xml',
+        'views/res_users_views.xml'
     ],
     'installable': True,
     'application': True,
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 2f1821a39c1..4419cea2809 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_user
\ No newline at end of file
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 6cd76351cda..c19df633d6e 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -87,3 +87,11 @@ def _check_selling_price(self):
         for record in self:
             if record.selling_price is not None and float_compare(record.selling_price, 0.9 * record.expected_price, precision_digits=9) == -1:
                 raise ValidationError("The selling price cannot be less than 90% of the expected price")
+            
+    @api.ondelete(at_uninstall=False)
+    def unlink_if_new_or_cancelled(self):
+        for record in self:
+            if record.state not in ('new', 'cancelled'):
+                raise UserError("Only new and cancelled properties can be sold.")
+            if record.offer_ids:
+                record.offer_ids.unlink()
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 6ac1285fa5a..2495f34154b 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -51,3 +51,14 @@ def action_offer_refuse(self):
                 raise UserError("You cannot refuse an accepted offer.")
             record.status = 'refused'
         return True
+    
+    @api.model_create_multi
+    def create(self, vals_list):
+        for vals in vals_list:
+            price = vals.get('price')
+            estate_property = self.env['estate.property'].browse(vals.get('property_id'))
+            if estate_property.best_price and price <= estate_property.best_price:
+                raise UserError(f"The selling must be higher than {estate_property.best_price:.2f}")
+            estate_property.best_price = price
+        return super().create(vals_list)
+
diff --git a/estate/models/res_user.py b/estate/models/res_user.py
new file mode 100644
index 00000000000..0d31645bf92
--- /dev/null
+++ b/estate/models/res_user.py
@@ -0,0 +1,12 @@
+from odoo import fields, models
+
+class ResUsers(models.Model):
+    _inherit = "res.users"
+
+    property_ids = fields.One2many(
+        comodel_name="estate.property",
+        inverse_name="sales_person",
+        domain="""[
+            ('state', 'in', ('new', 'offer_received'))
+        ]"""
+    )
\ No newline at end of file
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index cc27510462e..f5d241b6eba 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -100,7 +100,7 @@
                 <field string="larger or equal area" name="living_area"
                        filter_domain="[('living_area', '>=', self)]"/>
                 <field name="facades"/>
-                <filter string="Available" name="state" domain="[('state', 'in', ('new', 'offer_recieved'))]"/>
+                <filter string="Available" name="state" domain="[('state', 'in', ('new', 'offer_received'))]"/>
                 <group expand="1" string="Group By">
                     <filter string="Postcode" name="postcode" context="{'group_by':'postcode'}"/>
                 </group>
diff --git a/estate/views/res_users_views.xml b/estate/views/res_users_views.xml
new file mode 100644
index 00000000000..771b96840ac
--- /dev/null
+++ b/estate/views/res_users_views.xml
@@ -0,0 +1,18 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<odoo>
+    <record id="res_users_view_form" model="ir.ui.view">
+        <field name="name">res.users.form.properties</field>
+        <field name="model">res.users</field>
+        <field name="inherit_id" ref="base.view_users_form"/>
+        <field name="arch" type="xml">
+            <xpath expr="//notebook" position="inside">
+                <page string="Real Estate Properties">
+                    <group>
+                        <field name="property_ids"/>
+                    </group>
+                </page>
+            </xpath>
+        </field>
+    </record>
+</odoo>

From 1b1277556093a65eaac57b836948303c2097d808 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Tue, 27 May 2025 15:25:52 +0200
Subject: [PATCH 11/23] [ADD] -Added estate_account module to invoice sold
 properties. -Added 100 administraction fees and 6% tax on property in
 invoices.

---
 estate_account/__init__.py               |  1 +
 estate_account/__manifest__.py           | 13 ++++++++++++
 estate_account/models/__init__.py        |  1 +
 estate_account/models/estate_property.py | 26 ++++++++++++++++++++++++
 4 files changed, 41 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

diff --git a/estate_account/__init__.py b/estate_account/__init__.py
new file mode 100644
index 00000000000..0650744f6bc
--- /dev/null
+++ b/estate_account/__init__.py
@@ -0,0 +1 @@
+from . import models
diff --git a/estate_account/__manifest__.py b/estate_account/__manifest__.py
new file mode 100644
index 00000000000..afdc575584b
--- /dev/null
+++ b/estate_account/__manifest__.py
@@ -0,0 +1,13 @@
+{
+    'name': 'ESTATE_ACCOUNT',
+    'description': "aras estate tutorial module",
+    'website': 'https://www.odoo.com/page/estate',
+    'depends': [
+        'estate',
+        'account'
+    ],
+    'data': [
+    ],
+    'installable': True,
+    'application': True,
+}
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..36af0bc268b
--- /dev/null
+++ b/estate_account/models/estate_property.py
@@ -0,0 +1,26 @@
+from odoo import Command, models
+
+class EstateProperty(models.Model):
+    _inherit = "estate.property"
+
+    def action_property_sold(self):
+        for record in self:
+            self.env["account.move"].create(
+                {
+                    "partner_id": record.buyer.id,
+                    "move_type": 'out_invoice',
+                    "line_ids": [
+                        Command.create({
+                            "name": "Selling price 6 percent",
+                            "price_unit": record.selling_price * 0.06,
+                            "quantity": 1,
+                        }),
+                        Command.create({
+                            "name": "Administrative fees",
+                            "price_unit": 100,
+                            "quantity": 1,
+                        })
+                    ],
+                }
+            )
+        return super().action_property_sold()
\ No newline at end of file

From 461b5d3de3d72a4313649af3c1983231355f5b8a Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Tue, 27 May 2025 16:43:23 +0200
Subject: [PATCH 12/23] [IMP] -Added kanban view for estate properties. -Set
 best price in kanban property view to appear iff property state is
 offer_received. -Set Selling price in kanban property view to appear iff
 property state is offer_accepted. -Group properties in kanban view by type.
 -(ch14)

---
 estate/views/estate_property_views.xml   | 31 +++++++++++++++++++++++-
 estate_account/models/__init__.py        |  2 +-
 estate_account/models/estate_property.py |  3 ++-
 3 files changed, 33 insertions(+), 3 deletions(-)

diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
index f5d241b6eba..5563fb9319e 100644
--- a/estate/views/estate_property_views.xml
+++ b/estate/views/estate_property_views.xml
@@ -5,7 +5,7 @@
         <field name="name">Estate Property</field>
         <field name="res_model">estate.property</field>
         <field name="context">{'search_default_state': True}</field>
-        <field name="view_mode">list,form,search</field>
+        <field name="view_mode">list,form,kanban,search</field>
     </record>
     
     <record id="estate_property_view_list" model="ir.ui.view">
@@ -86,6 +86,35 @@
             </form>
         </field>
     </record>
+
+    <record id="estate_property_view_kanban" model="ir.ui.view">
+        <field name="name">estate.property.kanban</field>
+        <field name="model">estate.property</field>
+        <field name="arch" type="xml">
+            <kanban default_group_by="property_type_id">
+                <field name="state"/>
+                <templates>
+                    <t t-name="card">
+                        <div>
+                            <b><field name="name"/></b>
+                            <br/>
+                            Expected Price: 
+                            <field name="expected_price"/>
+                            <div t-if="record.state.raw_value == 'offer_received'">
+                                Best Price: 
+                                <field name="best_price"/>
+                            </div>
+                            <div t-if="record.state.raw_value == 'offer_accepted'">
+                                Selling Price: 
+                                <field name="selling_price"/>
+                            </div>
+                            <field name="property_tag_ids" widget="many2many_tags" options="{'color_field': 'color'}"/>
+                        </div>
+                    </t>
+                </templates>
+            </kanban>
+        </field>
+    </record>
        
     <record id="estate_property_view_search" model="ir.ui.view">
         <field name="name">estate.property.search</field>
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 36af0bc268b..ecddbcd226c 100644
--- a/estate_account/models/estate_property.py
+++ b/estate_account/models/estate_property.py
@@ -23,4 +23,5 @@ def action_property_sold(self):
                     ],
                 }
             )
-        return super().action_property_sold()
\ No newline at end of file
+        return super().action_property_sold()
+        
\ No newline at end of file

From 6c38871228fbb9cfd620eefdcc88109d02cd87ec Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Tue, 27 May 2025 17:09:01 +0200
Subject: [PATCH 13/23] [IMP] -Refactored code accorting to runbot linting
 guidelines. -(ch15)

---
 estate/models/__init__.py                | 2 +-
 estate/models/estate_property.py         | 2 +-
 estate/models/estate_property_offer.py   | 5 ++---
 estate/models/estate_property_type.py    | 2 +-
 estate/models/res_user.py                | 3 ++-
 estate_account/models/estate_property.py | 2 +-
 6 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/estate/models/__init__.py b/estate/models/__init__.py
index 4419cea2809..8f914bbb526 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_user
\ No newline at end of file
+from . import res_user
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index c19df633d6e..a816fd33d3d 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -87,7 +87,7 @@ def _check_selling_price(self):
         for record in self:
             if record.selling_price is not None and float_compare(record.selling_price, 0.9 * record.expected_price, precision_digits=9) == -1:
                 raise ValidationError("The selling price cannot be less than 90% of the expected price")
-            
+
     @api.ondelete(at_uninstall=False)
     def unlink_if_new_or_cancelled(self):
         for record in self:
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 2495f34154b..8df062241a8 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -17,7 +17,7 @@ class EstatePropertyOffer(models.Model):
     property_id = fields.Many2one("estate.property", string="Property", required=True)
     validity = fields.Integer(default=7, string="Validity")
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline", string="Deadline")
-    create_date = fields.Date(default=fields.Date.today) 
+    create_date = fields.Date(default=fields.Date.today)
     property_type_id = fields.Many2one(related="property_id.property_type_id")
 
     _sql_constraints = [
@@ -51,7 +51,7 @@ def action_offer_refuse(self):
                 raise UserError("You cannot refuse an accepted offer.")
             record.status = 'refused'
         return True
-    
+
     @api.model_create_multi
     def create(self, vals_list):
         for vals in vals_list:
@@ -61,4 +61,3 @@ def create(self, vals_list):
                 raise UserError(f"The selling must be higher than {estate_property.best_price:.2f}")
             estate_property.best_price = price
         return super().create(vals_list)
-
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
index 23bbe68a7f2..bb9a7d75b99 100644
--- a/estate/models/estate_property_type.py
+++ b/estate/models/estate_property_type.py
@@ -10,7 +10,7 @@ class EstatePropertyType(models.Model):
     property_ids = fields.One2many("estate.property", "property_type_id")
     sequence = fields.Integer()
     offer_ids = fields.One2many("estate.property.offer", "property_type_id")
-    offer_count= fields.Integer(compute="_compute_offer_count")
+    offer_count = fields.Integer(compute="_compute_offer_count")
 
     _sql_constraints = [
         ('unique_property_type', 'UNIQUE(name)',
diff --git a/estate/models/res_user.py b/estate/models/res_user.py
index 0d31645bf92..80c83227d1f 100644
--- a/estate/models/res_user.py
+++ b/estate/models/res_user.py
@@ -1,5 +1,6 @@
 from odoo import fields, models
 
+
 class ResUsers(models.Model):
     _inherit = "res.users"
 
@@ -9,4 +10,4 @@ class ResUsers(models.Model):
         domain="""[
             ('state', 'in', ('new', 'offer_received'))
         ]"""
-    )
\ No newline at end of file
+    )
diff --git a/estate_account/models/estate_property.py b/estate_account/models/estate_property.py
index ecddbcd226c..f31c56ab369 100644
--- a/estate_account/models/estate_property.py
+++ b/estate_account/models/estate_property.py
@@ -1,5 +1,6 @@
 from odoo import Command, models
 
+
 class EstateProperty(models.Model):
     _inherit = "estate.property"
 
@@ -24,4 +25,3 @@ def action_property_sold(self):
                 }
             )
         return super().action_property_sold()
-        
\ No newline at end of file

From b219aad27a11be89ab2c4377a3a367585a57c63c Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Wed, 4 Jun 2025 13:04:39 +0200
Subject: [PATCH 14/23] [IMP] -added some estate property types that are
 available with estate module installation.

---
 estate/__manifest__.py               | 7 +++++--
 estate/demo/estate.property.type.csv | 5 +++++
 2 files changed, 10 insertions(+), 2 deletions(-)
 create mode 100644 estate/demo/estate.property.type.csv

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index ab40a80f409..4bf6e2446bb 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -8,11 +8,14 @@
     'data': [
         'security/ir.model.access.csv',
         'views/estate_property_views.xml',
-        'views/estate_property_type_views.xml',
         'views/estate_property_tag_views.xml',
         'views/estate_property_offer_views.xml',
+        'views/estate_property_type_views.xml',
         'views/estate_menus.xml',
-        'views/res_users_views.xml'
+        'views/res_users_views.xml',
+    ],
+    'demo': [
+        'demo/estate.property.type.csv'
     ],
     'installable': True,
     'application': True,
diff --git a/estate/demo/estate.property.type.csv b/estate/demo/estate.property.type.csv
new file mode 100644
index 00000000000..cf6c135c542
--- /dev/null
+++ b/estate/demo/estate.property.type.csv
@@ -0,0 +1,5 @@
+"id","name"
+"data_property_type_residential","Residential"
+"data_property_type_commerial","Commerial"
+"data_property_type_industrial","Industrial"
+"data_property_type_land","Land"
\ No newline at end of file

From e00585c7148a6c924858b3acaf3d66821c3aa3d0 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Wed, 4 Jun 2025 13:06:35 +0200
Subject: [PATCH 15/23] [IMP] -added some demo estate model data for estate
 module.

---
 estate/__manifest__.py          |  3 ++-
 estate/demo/estate_property.xml | 45 +++++++++++++++++++++++++++++++++
 2 files changed, 47 insertions(+), 1 deletion(-)
 create mode 100644 estate/demo/estate_property.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index 4bf6e2446bb..cb5c9063ac4 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -15,7 +15,8 @@
         'views/res_users_views.xml',
     ],
     'demo': [
-        'demo/estate.property.type.csv'
+        'demo/estate.property.type.csv',
+        'demo/estate_property.xml'
     ],
     'installable': True,
     'application': True,
diff --git a/estate/demo/estate_property.xml b/estate/demo/estate_property.xml
new file mode 100644
index 00000000000..ced7161e068
--- /dev/null
+++ b/estate/demo/estate_property.xml
@@ -0,0 +1,45 @@
+<odoo>
+    <record id="estate_property_1" model="estate.property">
+        <field name="name">Big Villa</field>
+        <field name="state">new</field>
+        <field name="description">A nice and big villa</field>
+        <field name="postcode">12345</field>
+        <field name="date_availability">2020-02-02</field>
+        <field name="expected_price">1600000</field>
+        <field name="bedrooms">6</field>
+        <field name="living_area">100</field>
+        <field name="facades">4</field>
+        <field name="garage">True</field>
+        <field name="garden">True</field>
+        <field name="garden_area">100000</field>
+        <field name="garden_orientation">south</field>
+    </record>
+
+    <record id="estate_property_2" model="estate.property">
+        <field name="name">Trailer home</field>
+        <field name="state">cancelled</field>
+        <field name="description">Home in a trailer park</field>
+        <field name="postcode">54321</field>
+        <field name="date_availability">1970-01-01</field>
+        <field name="expected_price">100000</field>
+        <field name="selling_price">120000</field>
+        <field name="bedrooms">1</field>
+        <field name="living_area">10</field>
+        <field name="facades">4</field>
+        <field name="garage">False</field>
+      </record>
+
+      <record id="estate_property_3" model="estate.property">
+        <field name="name">Tutorial House</field>
+        <field name="state">offer_received</field>
+        <field name="description">Home in a trailer park</field>
+        <field name="postcode">54321</field>
+        <field name="date_availability">1970-01-01</field>
+        <field name="expected_price">100000</field>
+        <field name="selling_price">120000</field>
+        <field name="bedrooms">1</field>
+        <field name="living_area">10</field>
+        <field name="facades">4</field>
+        <field name="garage">False</field>
+      </record>
+  </odoo>
\ No newline at end of file

From cc0c3ea4fe82b0dbd3b32f3538df2d6b0aa8086b Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Wed, 4 Jun 2025 13:08:03 +0200
Subject: [PATCH 16/23] [IMP] -Added some demo estate offer model data for
 estate module.

---
 estate/__manifest__.py                |  3 ++-
 estate/demo/estate_property_offer.xml | 22 ++++++++++++++++++++++
 2 files changed, 24 insertions(+), 1 deletion(-)
 create mode 100644 estate/demo/estate_property_offer.xml

diff --git a/estate/__manifest__.py b/estate/__manifest__.py
index cb5c9063ac4..bddb9a613a9 100644
--- a/estate/__manifest__.py
+++ b/estate/__manifest__.py
@@ -16,7 +16,8 @@
     ],
     'demo': [
         'demo/estate.property.type.csv',
-        'demo/estate_property.xml'
+        'demo/estate_property.xml',
+        'demo/estate_property_offer.xml'
     ],
     'installable': True,
     'application': True,
diff --git a/estate/demo/estate_property_offer.xml b/estate/demo/estate_property_offer.xml
new file mode 100644
index 00000000000..1ba2a59b98e
--- /dev/null
+++ b/estate/demo/estate_property_offer.xml
@@ -0,0 +1,22 @@
+<odoo>
+    <record id="estate_property_offer_1" model="estate.property.offer">
+        <field name="partner_id" ref="base.res_partner_12"/>
+        <field name="property_id" ref="estate_property_1"/>
+        <field name="price">10000</field>
+        <field name="validity">14</field>
+    </record>
+
+    <record id="estate_property_offer_2" model="estate.property.offer">
+        <field name="partner_id" ref="base.res_partner_12"/>
+        <field name="property_id" ref="estate_property_1"/>
+        <field name="price">1500000</field>
+        <field name="validity">14</field>
+    </record>
+
+    <record id="estate_property_offer_3" model="estate.property.offer">
+        <field name="partner_id" ref="base.res_partner_2"/>
+        <field name="property_id" ref="estate_property_1"/>
+        <field name="price">1500001</field>
+        <field name="validity">14</field>
+    </record>
+</odoo>
\ No newline at end of file

From b3f827738df5d54c595b8934321cd89284ef4fbc Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Wed, 4 Jun 2025 13:22:34 +0200
Subject: [PATCH 17/23] [IMP] -Set demo estate offers dates to be relative to
 module creation time.

---
 estate/demo/estate_property_offer.xml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/estate/demo/estate_property_offer.xml b/estate/demo/estate_property_offer.xml
index 1ba2a59b98e..b21c1dd9f9b 100644
--- a/estate/demo/estate_property_offer.xml
+++ b/estate/demo/estate_property_offer.xml
@@ -4,6 +4,7 @@
         <field name="property_id" ref="estate_property_1"/>
         <field name="price">10000</field>
         <field name="validity">14</field>
+        <field name="create_date" eval="datetime.now()"/>
     </record>
 
     <record id="estate_property_offer_2" model="estate.property.offer">
@@ -11,6 +12,7 @@
         <field name="property_id" ref="estate_property_1"/>
         <field name="price">1500000</field>
         <field name="validity">14</field>
+        <field name="create_date" eval="datetime.now()"/>
     </record>
 
     <record id="estate_property_offer_3" model="estate.property.offer">
@@ -18,5 +20,6 @@
         <field name="property_id" ref="estate_property_1"/>
         <field name="price">1500001</field>
         <field name="validity">14</field>
+        <field name="create_date" eval="datetime.now()"/>
     </record>
 </odoo>
\ No newline at end of file

From 05d3a45a3af6c2f6315de4f1f0fcc6cdcd2cf46d Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Wed, 4 Jun 2025 13:23:54 +0200
Subject: [PATCH 18/23] [IMP] -Set demo estate properties to be residential
 type.

---
 estate/demo/estate_property.xml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/estate/demo/estate_property.xml b/estate/demo/estate_property.xml
index ced7161e068..8de56ac09a8 100644
--- a/estate/demo/estate_property.xml
+++ b/estate/demo/estate_property.xml
@@ -13,6 +13,7 @@
         <field name="garden">True</field>
         <field name="garden_area">100000</field>
         <field name="garden_orientation">south</field>
+        <field name="property_type_id" ref="data_property_type_residential"/>
     </record>
 
     <record id="estate_property_2" model="estate.property">
@@ -27,6 +28,7 @@
         <field name="living_area">10</field>
         <field name="facades">4</field>
         <field name="garage">False</field>
+        <field name="property_type_id" ref="data_property_type_residential"/>
       </record>
 
       <record id="estate_property_3" model="estate.property">
@@ -41,5 +43,6 @@
         <field name="living_area">10</field>
         <field name="facades">4</field>
         <field name="garage">False</field>
+        <field name="property_type_id" ref="data_property_type_residential"/>
       </record>
   </odoo>
\ No newline at end of file

From a96dd13e6e5fff094d520bfba1826cc42dfddc92 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Wed, 4 Jun 2025 13:54:09 +0200
Subject: [PATCH 19/23] [IMP] -Set an offer to be accepted by default, and the
 rest to be refused.

---
 estate/demo/estate_property_offer.xml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/estate/demo/estate_property_offer.xml b/estate/demo/estate_property_offer.xml
index b21c1dd9f9b..f4409f25a14 100644
--- a/estate/demo/estate_property_offer.xml
+++ b/estate/demo/estate_property_offer.xml
@@ -22,4 +22,8 @@
         <field name="validity">14</field>
         <field name="create_date" eval="datetime.now()"/>
     </record>
+
+    <function model="estate.property.offer" name="action_offer_refuse" eval="[ref('estate_property_offer_1')]"/>
+    <function model="estate.property.offer" name="action_offer_refuse" eval="[ref('estate_property_offer_2')]"/>
+    <function model="estate.property.offer" name="action_offer_accept" eval="[ref('estate_property_offer_3')]"/>
 </odoo>
\ No newline at end of file

From 7958f6983ef0ba0c8d482cb0faa38d55f6395851 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Wed, 4 Jun 2025 14:36:05 +0200
Subject: [PATCH 20/23] [IMP] -Created demo estate property with offers inside.

---
 estate/demo/estate_property.xml | 27 ++++++++++++++++++++-------
 1 file changed, 20 insertions(+), 7 deletions(-)

diff --git a/estate/demo/estate_property.xml b/estate/demo/estate_property.xml
index 8de56ac09a8..20374c71513 100644
--- a/estate/demo/estate_property.xml
+++ b/estate/demo/estate_property.xml
@@ -34,15 +34,28 @@
       <record id="estate_property_3" model="estate.property">
         <field name="name">Tutorial House</field>
         <field name="state">offer_received</field>
-        <field name="description">Home in a trailer park</field>
-        <field name="postcode">54321</field>
-        <field name="date_availability">1970-01-01</field>
-        <field name="expected_price">100000</field>
-        <field name="selling_price">120000</field>
-        <field name="bedrooms">1</field>
-        <field name="living_area">10</field>
+        <field name="description">X2many tutorial</field>
+        <field name="postcode">13579</field>
+        <field name="date_availability">2011-11-11</field>
+        <field name="expected_price">123456</field>
+        <field name="selling_price">135791</field>
+        <field name="bedrooms">2</field>
+        <field name="living_area">30</field>
         <field name="facades">4</field>
         <field name="garage">False</field>
         <field name="property_type_id" ref="data_property_type_residential"/>
+        <field name="offer_ids" eval="[
+            Command.clear(),
+            Command.create({
+                'partner_id': ref('base.res_partner_12'),
+                'price': 10000,
+                'validity': 14,
+            }),
+            Command.create({
+                'partner_id': ref('base.res_partner_2'),
+                'price': 10002,
+                'validity': 14,
+            }),
+        ]"/>
       </record>
   </odoo>
\ No newline at end of file

From 94d04787612e550141b12c1b877ac3a686b24be5 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Tue, 10 Jun 2025 15:31:01 +0200
Subject: [PATCH 21/23] [ADD] -Added check to raise an error if an offer is
 created for a sold/cancelled property. -Added check to raise an error if a
 property is sold without any offer accepted. -Created tests cases to verify
 total area calculation, verify action_property_sold, and verify offer
 creation for sold property. -Added missing new lines in some files.

---
 estate/demo/estate.property.type.csv   |  2 +-
 estate/demo/estate_property.xml        |  3 +-
 estate/demo/estate_property_offer.xml  |  2 +-
 estate/models/estate_property.py       |  4 +-
 estate/models/estate_property_offer.py |  3 +
 estate/tests/__init__.py               |  1 +
 estate/tests/test_estate_property.py   | 84 ++++++++++++++++++++++++++
 7 files changed, 95 insertions(+), 4 deletions(-)
 create mode 100644 estate/tests/__init__.py
 create mode 100644 estate/tests/test_estate_property.py

diff --git a/estate/demo/estate.property.type.csv b/estate/demo/estate.property.type.csv
index cf6c135c542..17f4695bb64 100644
--- a/estate/demo/estate.property.type.csv
+++ b/estate/demo/estate.property.type.csv
@@ -2,4 +2,4 @@
 "data_property_type_residential","Residential"
 "data_property_type_commerial","Commerial"
 "data_property_type_industrial","Industrial"
-"data_property_type_land","Land"
\ No newline at end of file
+"data_property_type_land","Land"
diff --git a/estate/demo/estate_property.xml b/estate/demo/estate_property.xml
index 20374c71513..8359e8af293 100644
--- a/estate/demo/estate_property.xml
+++ b/estate/demo/estate_property.xml
@@ -58,4 +58,5 @@
             }),
         ]"/>
       </record>
-  </odoo>
\ No newline at end of file
+  </odoo>
+  
\ No newline at end of file
diff --git a/estate/demo/estate_property_offer.xml b/estate/demo/estate_property_offer.xml
index f4409f25a14..87c01d263e1 100644
--- a/estate/demo/estate_property_offer.xml
+++ b/estate/demo/estate_property_offer.xml
@@ -26,4 +26,4 @@
     <function model="estate.property.offer" name="action_offer_refuse" eval="[ref('estate_property_offer_1')]"/>
     <function model="estate.property.offer" name="action_offer_refuse" eval="[ref('estate_property_offer_2')]"/>
     <function model="estate.property.offer" name="action_offer_accept" eval="[ref('estate_property_offer_3')]"/>
-</odoo>
\ No newline at end of file
+</odoo>
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index a816fd33d3d..855223a92d0 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -70,6 +70,8 @@ def action_property_sold(self):
         for record in self:
             if record.state == 'cancelled':
                 raise UserError("Cancelled properties cannot be sold.")
+            elif record.state != 'offer_accepted':
+                raise UserError("You cannot sell a property without an accepted offer.")
             else:
                 record.state = 'sold'
         return True
@@ -92,6 +94,6 @@ def _check_selling_price(self):
     def unlink_if_new_or_cancelled(self):
         for record in self:
             if record.state not in ('new', 'cancelled'):
-                raise UserError("Only new and cancelled properties can be sold.")
+                raise UserError("Only new and cancelled properties can be deleted.")
             if record.offer_ids:
                 record.offer_ids.unlink()
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
index 8df062241a8..3334e5a0ce0 100644
--- a/estate/models/estate_property_offer.py
+++ b/estate/models/estate_property_offer.py
@@ -55,6 +55,9 @@ def action_offer_refuse(self):
     @api.model_create_multi
     def create(self, vals_list):
         for vals in vals_list:
+            property = self.env['estate.property'].browse(vals.get('property_id'))
+            if property.state in ('sold', 'cancelled'):
+                raise UserError(f"Cannot create an offer in a {property.state} property.")
             price = vals.get('price')
             estate_property = self.env['estate.property'].browse(vals.get('property_id'))
             if estate_property.best_price and price <= estate_property.best_price:
diff --git a/estate/tests/__init__.py b/estate/tests/__init__.py
new file mode 100644
index 00000000000..576617cccff
--- /dev/null
+++ b/estate/tests/__init__.py
@@ -0,0 +1 @@
+from . import test_estate_property
diff --git a/estate/tests/test_estate_property.py b/estate/tests/test_estate_property.py
new file mode 100644
index 00000000000..30ed9d64712
--- /dev/null
+++ b/estate/tests/test_estate_property.py
@@ -0,0 +1,84 @@
+from odoo.tests.common import TransactionCase
+from odoo.exceptions import UserError
+from odoo.tests import tagged
+from odoo import Command
+
+@tagged('post_install', '-at_install')
+class EstateTestCase(TransactionCase):
+
+    @classmethod
+    def setUpClass(cls):
+        # add env on cls and many other things
+        super().setUpClass()
+
+        cls.properties = cls.env['estate.property'].create([{'name': 'test_house'}])
+        cls.partner = cls.env['res.partner'].create([{
+            'id': 'test_partner',
+            'name': 'test_person',
+            'company_name': 'test_company',
+            'street': 'test_street',
+            'city': 'test_city',
+            'zip': '12345',
+            'country_id': cls.env.ref('base.us').id,
+            'state_id': cls.env.ref('base.state_us_39').id,
+            'phone': '+1 555-555-5555',
+            'email': 'test@testing.example.com',
+            'tz': 'Europe/Brussels',
+        }])
+
+    def test_creation_area(self):
+        """Test that the total_area is computed like it should."""
+        self.properties.living_area = 20
+        self.properties.garden = True
+        self.properties.garden_area = 15
+        self.properties.garden_orientation = 'north'
+
+        self.assertRecordValues(self.properties, [
+           {'total_area': 35},
+        ])
+
+    def test_action_sell_without_accepted_offer(self):
+        """Test that everything behaves like it should when selling an invalid property."""
+
+        self.assertRecordValues(self.properties, [
+           {'state': 'new'},
+        ])
+
+        with self.assertRaises(UserError):
+            self.properties.action_property_sold()
+
+    def test_action_sell_with_accepted_offer(self):
+        """Test that everything behaves like it should when selling a valid property."""
+
+        self.properties.offer_ids.create({
+            'property_id': self.properties.id,
+            'partner_id': self.partner.id,
+            'price': 124,
+            'validity': 14,
+        })
+        self.properties.offer_ids.action_offer_accept()
+        self.properties.action_property_sold()
+
+        self.assertRecordValues(self.properties, [
+           {'state': 'sold'},
+        ])
+
+    def test_creation_offer_for_sold_property(self):
+        """Test that everything behaves like it should when property is sold."""
+
+        self.properties.offer_ids.create({
+            'property_id': self.properties.id,
+            'partner_id': self.partner.id,
+            'price': 124,
+            'validity': 14,
+        })
+        self.properties.offer_ids.action_offer_accept()
+        self.properties.action_property_sold()
+
+        with self.assertRaises(UserError):
+            self.properties.offer_ids.create({
+                'property_id': self.properties.id,
+                'partner_id': self.partner.id,
+                'price': 130,
+                'validity': 14,
+            })

From 996ee6dfb08d00e31009e61dd4bede70a2173569 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Tue, 10 Jun 2025 16:19:59 +0200
Subject: [PATCH 22/23] [IMP] -Added tests to check garden values are set/reset
 correctly onchange.

---
 estate.zip                           | Bin 0 -> 20425 bytes
 estate/models/estate_property.py     |   6 +++---
 estate/tests/test_estate_property.py |  25 +++++++++++++++++++++++--
 3 files changed, 26 insertions(+), 5 deletions(-)
 create mode 100644 estate.zip

diff --git a/estate.zip b/estate.zip
new file mode 100644
index 0000000000000000000000000000000000000000..544623713813f3c439860a40f972b0cb5894c321
GIT binary patch
literal 20425
zcmbt+1z45K+BPjncZ#$$NOw0#qjYz72uOpZbc1xaG|~;yh;+AfBlWX%vtjdj_SyUV
zb2)2RuIqZ9nS1V;cb=G6S{xJ%1qe_UeLIwY{_)4h6CgYwJsVpsTRj>Dd03z)BEbBH
zqPG>$+Brf40fQmD00R2?A^lSbBoHJJopk$Ogdp7u(a<n5H?q~xptf}SDQ{kq*HEgE
z&k)c)k_QSTE&c=&&<~J5I&-gqGT^?OW}rYoXb&}*X_*`8-*)<5i#G(nq0emvw2!nz
zMX+16;-UDQUOw#&j6sWMtg8}Jt5t}y^Rgh2#YIC;RF=7Repwm8P(<zr?BCj1!vUAA
zfTp6wqf^=)5YFL9nG8`OC_EhhPP<Q?L_t8?LXCNi%n-J<&3SK3R64HGP8vO}1LD=I
zurm=pqz~1^m}=n%tYPyD%wrTda?RwmX#^*QU)MPZu?(<n8}ePg&C?;A1hUjkrK<4F
z(igT7_HBjN!fbyyK!Cpx-m@(Mhh~VUCG=*TRm#-XCD|dLY4s*x*<h!BCN9;@14*Rq
zImul!WQBF(a53vU^;Rd@8jnxvoP=F1dgY~~>0mz9`{LLymrPdvn5s^5;?3Pq@9>h1
zaY!4@wJ2?HNAfdx_&3DsqgK8jvhA#X{s9b_fK&C+AJ2e+fB-Q6ngGCTAZWDyWr~p>
zPJpG8j+Tz0o`%LR&J1>N8D_@t83Niz=-<!iLxGjI3MhUr@K65eSUTAnT9{MOzofh6
zlFm;u0Tdv{{#FrtUvF<a5EyjGxZZn^jZPR5cPKG8G$5oAOj!(B3>yq<%ybM)O!7E}
zI1M@wA4W`z(v{Whj8&8iBf1%FQf-KULftc%;geoPZbEKm1#WI)?&o~VBxo@iXd5w+
z(aY$1B&#Ujv(aLRqBAx-I}KL#qT4#Zqsm_|mH=bDiH|UT(+11{sw?>N{k}(BSlB+U
zmk1(u=r4w^_AMX9+>RJ9>U)5R9<3p`dpyi6boESaeqq~oyKbCQ_zVH<Z?U}<`hO-1
z>;K5o(D?tIGjdEIAlAQ?`)eD~u(Y<Y)U&pAqW+gN*siE1f&B(88Pgd88<dIL_mU?T
z$Tt=tf`oX2+-R7jnaBuaBRsr_e;=*^6b9pijX_HDD`f{%qi*wzrq>k1*+L4>m81OR
z^G3P!GsP}S>cZ9JCfGOg!cNGBxp=}|H+pw!zMWu{ByCM8kDXn+oc7nxIqa~m{h;~!
zH44UM$IU@|6ZA#vWyHkMf=#1T=X_}k9z>~@el%>!Csp@^3NT7aqotNHC9bA{r@$U#
zR0{!0Doq`9IP^_;->D$>%k*kP$bPsObzeh}qk~zr>&PA{!vvxzz-aM>wTjStE1`C4
z_4Q(&R54(l>J(_VvJ&{95^V%vrNXL@<s?JqhvUNJev*OeO$|?AF(*z4jROjd1@aqr
z2e$&IC<Aj03G>;AUj!%kPJTn2ppp@sCT0o>6#<iX7AaV%;H)K%GNY(HQZvV+P*WBE
z-Z;RJdcQ*q?`x^@F>wq(8=0(2ziPz_Lf2Dzg#`MqGO7kps(pDQ%W0`NJuV+FeBKKM
z>~TIRUGjOKMB~z1RF+r6Q!yzu@s-VHw~p)tow(#UN9(KEqH@n7mJD)9O>4O)r(PKQ
z5Ow(l+NViVZ56VU3Z!wgh|_u$n=CMWcw@$I-_&38TiF$}z7bXmH00mTf3|9hyBwzr
zn&iF{-yjb%<6nEASibw(!^vU-arP2R`jo_lx&1?A(=PKqXTl!mdJ`3iCD8f&M!HU?
zSkN|OC+;-E310h$m?rku8g#-qlMIW4En2(1La!RL#A{uk@?|i;L%+<H_}*aH(EKc(
zjk4#P9*!)7(&(BkDupb|>r_0%Zsm2lYI5x8;yyA(AzxY;uQdi5_?<qausKFbuY)@2
zBAFK?x;S9mBov2q0bgOlYR`IA>)k{7@LBWF5sKF~-Ajy(F)+up9Ff*A!wK+NaeQMY
zrPczg<5S_@4B8e5kigcF_^lx9=Lvy_zsRU5!PREA0^Nzg@Az>9!H<(`ZZBceE+Bc>
zZbpM4BQqVH{k$|^b+*e_tIz90H=n!2dw4`wu|wsTN=aIrpvF94yj}BlsG`t7T3%Qa
zK4*vD>*Y2Jrt=9D`ndFoU&UBis1p}5=j&TAtu7lkf39p;ZyBgu(lWBg5q+WkN`<O9
zX4laDjS;S6TU_yrhPZR%RqSJm<2rxtflDKj?Z)sU4bgMAC=7T`X*pr(L?<z(<x}fw
z|01poA#@(95(KE+ussm-+b0t~jW}>=Eie=zD*>FAV8}WH(Az><_YXOWAL-6`5BN9Z
z*@QP+m;ErjVAvT8uJ(v39Q0)C2qLiTTiasWaur*dbWS#)WmsP)&80bF=9M7P;jK26
zt$Io43+tNhWv`;?CVmVoVAX$dWrI*!q4>1~C7N*`De`L<nwRz=#7a7ZomU)nrF(c5
z-4~Z;xNv?#;Z;})C)jmU@WZ4ov0#;MK7q1OYqkpHq|IDO{~?WRVHs<bOyQ{I93+E1
zb#2v}ZRy4DfjMC$Iy>9JN77g!E}S`d>B(6{Hkv}UdEyIrL@$sEDhR&S=DOkNgR02!
zSr;6jL<+vm&_Z?9Mb#y<l*i_ivo<6l)%eiD8zlqRcz85%Jz}vnHA$X~0RL9Uk4CCn
z+pUQIi+v7q8Vcg4K=cYSJL;e|E2W^k_%tuKeaw&&I(Rq_>M=_R*@W~cs!gSXP1w~@
z_SVm!k#o8hC7HI~V@P5hhEjuDH6!|ZXc*JtRe_@OZN;JV(n=}2&~VQQUV)qmqY?%O
zq1uEuN@ST%kvKs?y>B++l)n-Vu1lsZxT36EA9$Z+v@b2oaimE!!yM{e%u=#`lbWa3
z$_F$`soh`jT>)X7JM~j};AMGJ_xVAhgi62^Lc#}b<{E4q`}His`sNj90?I{qM)6c(
zo!z5mc2nZjPhaQ<ogtL8yuU7=gDgz+!eepFfRj+Zt>MK&E{TwLY&ADWy(~Zp<~u6+
zBK9q~$FtBgA79n$wE*%ZIjZl8VZ6`N2)WfRn<z2~$5>LE)RdsF?-0DLb|vdLMXO$H
z?}*kqb2WxrpHls}nVmX~TWPqV@2sMs`tdDK87BKeeeDJ3^HZ01^A~Eu^o8}}Ezk3r
zwrD{W`Vq>aRVUxe@RnSyfVWD5aq`+gkn=Y?p5p0?@4g=4;vw@J7o=8=ZG%kd8(4nz
znc87+)YGQ36AC*4X4(+a6Fw=NAOaz#Y!-dlzrtdz0&E>4ZWA3`8=So)+NySU_Hw2#
zDBDC-(WXN0EgTILclGBF9J8udc!d`7pz+~~lan;)T_!YAxY`CM&HB%3jjFoJ*XK&Q
zKy>5A*>UlDpPu;pe+aRQ6%C9%EPCpVB4SLCC+UJNYVT5wmJ-v<8G4BUy)5(PqX04v
zea@5aE`n<@k00Igu91xrW3=p_E@!G;mu8*k*z4R(O<%pgmS5s&Vcg64${%F$EU4I~
zKHQ4&@?%$i=5iQO34?{c@_IUlMhCiBS)~m-{%ZAj1G=;U0S(%IRb$iK<*DSr>+9OX
z%WaxH19|@U@DW||TTVV4Np7TrPI(+jRwp6(JbJ7>sWZycxJyvy`O%ksuuA?6^Ew(s
zeJE)A;8yFU=nyY%PTm}xf9vkF_gyV2A{C$&B(%lXlO3w)yz!9QB7&Nz30@_z=cVn+
zV36_qMAcwD$WKrMr|Bf_;-?y`T1)E>F4Q-c0rVzvv<g4AkD_u=WlKTKeU?Bj_+{3c
zg{`OJdau~0B9^Bvj@$i`HslDq<p}+&ib*=8-%5zLoSK;>X+czfnx^;NVDIEzAz*If
z<(g)0yBc2?wUx}m%0?>6g-w=->+4cX7aR7nK_mu&hV~PGo_+Jmt*!sl3Y?tLS6&T^
zjpQ{oRmIi*xWp+x`4EUVn}aU=p|gROYhLm&s38lLa>lhxgXQz1>V)A?Hi*(%S&F@C
z6Co*dh~69c@uqAAvt|L&e(J;{C-ey|mQ_y5$lJmZ{pm;5>SBxyahCO>$L6nCRp^U4
z%EcvW+sgAPHZ}=TK|?H=qbvrU$4kJy5RG2LK?L_`A`~gvXvr2mg(&V5T~LLW!XFf&
z{-WqV44n&<&rjDsb08@tI$k0h>&oJ0lnH}5)_Y(#V`++O;`QY<d~*`qF3Edk0du?I
z&l%pCR>TGC&=9*XZg91=zZ`90pso?razJ%|HpAKJ&ST3TQXS2S4dp}nVkVy``?Z8p
z&`=g1csOY33<r3US}e?m-&lYon`7;1w_U(A)R~pF8eJZPHO*6@jry*V5#UdI;~aeP
z851#Pqy!^ed7LDvI3G!ht-vcxw7Syfc{D}+*Rd)G^$1wWg+ql7wc_7mR=S9}4@596
z2Ja(m<S+=EI1>4Je=mk%UILPD?3(p06IIo_=G~HJ2iJE9?3S-+yP!Xw*@uqX?5Ms8
zOSjtZn$fBHFjG~;%CS%XAuk^NY9nt|SHY%Z;>cTPb8dl-`t;LNlM000Cv)nJ2DvtR
zHwV`r;M(T8SRAPO=UjJBoEsZ4xf3Ra-btOc?Uxcxcp=dbRXqEK^xC1L-oZ#$&(`TE
zz4S~oi&yoaSXHrM&#BX^Y78r+okl;AinjTpYOe9QvW<N!jvc|59hH;J%}LE4@vH6r
zbzxIs@_or?JXvo~7rSOkr*v{RqpRIi%)VDA%{m8fB+V{MT69d9j|OutQPp{SS6Iz-
zY^^KDYn*GIdkK8YhyPlU73N)mFIV9_<mRw`GFTr7UyFNSqVSGSdTKZ<Dx&wr?sMn8
zch(|G`5z%#;x#R6b15c9qsC8-^ESXRw~W=0UMKeUdpjOYMMs+8y`z?!svWH^GjM`;
zA2XI}8!#|dNlA68VI6YXTJWiuWL{eP20nI;RcBth<!;9aRLE!Ma#r<*`*ZE6bt7B*
z1rtjMcQ023E#EF1Rm<>UsDPPDfW_Rr<24?zZ9&BFTUf$T-kWv5bny$>%N^JvPd9&C
z=HiIq2sRH3$3@T1+CW4sAeNJb@nPwJ-SK(K&wO*sO4)*CvC|T#t7DGvPBP3JyBV1U
zP71P57HA8WKVTZS9p&g)pTA7Z7<!@c!p*$dW30@)#8a7Odw7A?c*4C?gUD@Gmnuq?
z8hY&F)3)Q-Qex9{+6nIG3<oq#wLW?iA7ev^M)r|ymY+*>(l3zr+8gw3&B2h3h+abj
ztX3=s2|{uhARvHQ@_Qqb^wyr`{5J-RhOL&tzcpISWa}+Pcu*P--|#Ks;h8BQAnEh8
zK(=E;R`h^S=Yb)LY%i|po)?u;OvOx3@UZEv4@y4s@Gr`_#59$F^j${_jU6XIMI&*k
zBt(sRaeg|>c{95<lP#GI{*?849A1MD=$$)#+P5F&8cindz`id<Ql22U%i&2k>9|L^
zx+|n@Va+Ed=0u&8`|E$_&NQNE8TS_ns-Z;+5dGry32`%RUQ#I(2wr=h4C5{NYYy1_
z<RT#LEp-kJ7?bBm$cpHu(FGJYM!93^(AfOz6)#}x$f)Z$4(Shqh7h$TbbFug*okgz
z1{-V8P$c&9I?fRkLqZcs7Elh523{%}=6(#N!VVxIT;k9Mo61{XHDc(8QYPY|^Qhq*
zcWKz3jRroMRr?0dXTn?Z)U%yGzH|KP6?)r2A-NCnM7@Vu014l)kVrWKd1XxJFcTuy
z+B?;oA#t&VHt#2tSbB)Bd^|{7y<gU-r*gu$c3=&gnr2~QB9R=qcSQP&zyF|vG8k#_
z<->4L<CAP_6*tuv8BkWFS>>K>YnWR7+E1o)Pgdj|ls{f$HZaH2o(qW$27D++FM-!z
z?aO8VJ{!jsXIDECyt^Ic<t%LCXVN+O?WAxcJ<e%cbIH(><xE@sW33ISx+CqIuL?__
zD%+ybHy|ph87AJa;zvyQT+3I=MyZBNm%UkL=sPxca<#8u>0xet109=>&|)#)L6Z&}
z%-q3#Sa^<&qK|ow4xlSr5-B#(t(yx#%HPv9=k`~+zoo0So{ff`jh^+t=BUg|6BJ&Q
zcl34aH8>6e!HA_V1uQ?2R%vP>E`u;B8-g|K3;M3CKC9AlSWBE>uemjPd|n$zjlDiH
z7(ag!VuVt#1>zqZMPDZh+4khH&zb&k_OyX{u5N}&@#3hEyH*hBT{9+h$Z2bXk1GP6
z2pqVm2rsgjdkZLnExfm%Z9+5a#Ui8g;=m>8+4l5%ET%8C=#wRUPrn?`H+o|;;`8x=
z5y^+PrMq9-UvT)Rv>;!9n~x0YQ!Lk1mXhHf$vNsz<C{qs-9g$UFDzkxZGC&Tjth~r
z8J&huj%GjoSoLOL{H<amoJFy(l93fB<$hG)vgP{#3LH%)K}4Y;qshQ7`XrBnsC_!B
zLEAEG<9r?Dm-vAU6jiETpSc#N%UP?SBBsxu^=n8DU!G;Pf3o%_&yPelP?i$=ESHhF
zeIWom;C&6_CwVp(>_`M1lo?v)LGTqduR2~{R?nztN$A})3|FPif;n~lzC_Jjesh(0
zvC_EL`EbQIcsJ@|*Xp{)b1Rz$>)U@z?!(YC>EQNp^-dYoNknFX9{bPqX0ojrCHBTk
z8GLBiax`e_gj7zk>Rs*HecN8DS6I#?^1W+1Cae@0pW3*Xaj0`^s<m6?1`rBl(qnrF
z7zhYJo8J@43LFTC>u(A5$YG;l>tw0-Z^>mR$7?Ynf|6YNj9D~-BqNnCpHD4?Qk}}*
zI|Z>fPtX%0kIx~@-77On&nc@dN++8hjsP(Q=Pl{5Sb27fzq-sh=5#d}ssMxXc2S$Z
zN@^Xt<d<<#`AIrIW_k-Qx83?(&PK`X2>0UPH*!%SU`<a-OxkNN`cas_-+HFiq<tU?
z#C*X#z=}YKs6dHyjAXzs#<Fm>eTSuw#)Yj$iL<C##swpD6`pIwOx86@zXiU51J%~#
z0fT|!<rfN1#hZ993X~tnf8m9{qGyV158A;mzXKE*0Kt?=O^%r+1Sj&j4m)@Epv_@F
zqp{Nqi6-JI(RXlO%}`g`7!zYcNcp*T4JMW%@O9D|ql9Nip%5(dA<Ge}v6=xs!fdO3
zX2!xr>^NMzvNXKQqN2IY`^8MQ6VuC2t4azk;(nBp)xG!_jP%a2ij@v@;Mp4~s|)qb
zmp<SUinHZc#)Y_=^q&t^W}sP4dA=3)Qfi#y;7(rpmc`X_ayuWMd@%|On&qI^nLsV{
z8wBG^M{V38DAb;5jn5P$(%J#X-iZT!$-%y235V-WG5<jZ2)8BW!&@NqOi;Eb&c*sS
z#{)V~4S}BZAAZ(Dccw&)TG_+LnN~|}LkXAx&q6Z20F!A>G1BO1571~Ae185;HxCV#
zmh5DbCon9i-7uxD$|93&6pN^NBc6L_*J%N-<`73h%6bGODeO5HXe?=A<P8>gjn&#!
z+c`7xT2;r3H$A(uom}0Qdcu*c#m}{^Mo_flNyOtxq_TGcY<d`CHlwFp{e2L21Y#%K
za=}A(Uw&N%*WvSCm#9%Kg_Pq{ykv&|Sg5S-mH7i4Y5_vO9%yc*6=}B*50RU=8`85t
zG)7h3PZ(62Rh_4GQUi#N5^vFgz<A*O>k$&>-joQo+4v2Zo6*`?(dTASH_vdC;#{ov
z_uFAtw&w-Diy54qZ7%;fnzX%Wt2tS&C|pp2`+}oZbGkgVB&-rwI#wZRZ>pd+P@p%d
zoa?G(PT^e7IMr>GpFIX`;#0|W#pIC7SXJ24Ml)6BB49>wo|J(?A!wAF>iaV5J+2=g
zLKt@741!rO5-xg{MqA8Taaf0(fUQ^%E@w+$B@F)Mlc1~YRn3i!uEQaN^c&E3FpZZ_
z0qgq>Opl!W?ZX1FihsYpjc-@~-#;+^S6{kbXsKi?ErRdvB2gg%Dkv2;D6JkoJ0Gf4
z6R2Y;!HO6mF$)_zf3i1h=>e|kXWq;b+ERSG?`^DlJngM+AGyUH*cu1)h4I!_%xd06
z|A2CnJW{E0K$4Q?^JiqpK?D_U2@4T=-6EDBrn8{fN%*40UlQR5LxM|cdAFqvUq5=X
zL~kYWNqx?_2VH}O=;a2t2FH7K9OF7=#NO7(?zn!$IfD%d#crMuej@x~1Z4j=<G%Iw
z%;%F#kyFEyt_(hhZ8f`}SMqYy8lFcoR}$4=%5lD)js0FZ_sT-ho{{QYQQ7C~wCbV?
z{vyyCs4AT0%cieS60Ob_p965WqD28ayLG7p(EUB`b_hT~-2ZA{wb0l9_fPGl*SGHQ
z3DkzeHEv6ZCM!I5PJ9nqkJy<^SR-U&B49rf;Bb<J?Y9Xb5OdH>7*jR$`Z{7rl>xOv
zedKs{g0B@SmdVt7{5VdmC=`tIxJKa$OoJ;oU%O5+AIKS@9;cr6k9T;a<{2w}bLdZW
zcG)>iYHX~Yq2}SDaO1$d@<Voa^UV7o{lk@&FIo=>Sa(q~9*Ta%G&X6q1&3eHH1Q)^
z3kay9B^=KVwL8ZmiAi(ifKLA<>=eDH?nSgYvE*8$yhK%Sr!-Dptr;7XjA|6d%z073
z9$tqiPHv?ZMct7Fg!>17IY@6z$QDZ7798DHj^!2*=uk837$4BgxsukwoVg-kCsH2l
za;cL1kbpXB{waghZEtGDPs%Do1v4@uunzU98%zu;lnk6D=60t#l^JU<j|8CW-;M*-
z=R1?5IqYT_Q6+Iyt5t3~e~Bz$+;WVl&kshWqA`uvY;k^HWI<<`S_jV*V*#SFW4<^C
zsj};7iF_1c_Z%j)B2JN|a)u8rh+g{h_nbu2KF<V+aW`<0i(_A21vv&=s26gOCUHai
zR^A4uc>EX=Q5nq{WFZ0<0=CEtA`*gW(PQB(S#1VtWH^C%hy;vC@r8_(@85Ktmy?&)
z^7{)&i}6c4BvNm`9}O9DR3_%T_|OOIuf-KEp=X<rx1gxa=a5GlLZD0;7hBjY0-J?W
zK#VZ99rLA6!M~k>&F+a1^YF`{d6NOOVmsmU*K?DweI{ZY5IV}fY(PWR;UOkjupe<m
zC9{XI!c+OsGnyPJJ>GLf$MP!9kUPLc+P9fGZ{(QMY+#WPL&(h<sPy!Ay;QW47^5Q-
z03+guLa-8~-_bz!PwR>_b!Sy^us7V>@{vTIh*(q^Hr@S7ibR7s0D%n|up~-J4mI?-
zGDj$4g9W8CG*&F{B^8sGXB&x-*oJbgPL}Nrq#WwAa;~90w{vlx?;|)EK;NW*ECYv_
zS`#ZEUA5{=L*mMn#GXxM=1Z{VtYOq>f1ee_)pc(RK`?a8(UtG^b?QOj=CNoZL(t}S
z>HsFUZEWsAFX)d*!Te^1AsCL4H`+kj!1WFJm1KG{kpxMfp0W(1w{y>a{z)v*Nvz(d
zBceB5bc1kNbdEH{M<s#{V1=!IXe7s^N~_Jx4X!jmOP~97`8A%0;y!b8yd28)%TC6j
zI)Ynue)7@xn(fEhP?Vg>lXvm3t_l=fYdM?ZPrI<?t@QFbm~z=9atRPDgkwQ0c&90i
z6e2!HRen)0TxT-W6e7YMK{C*8XoTS~&t|4yXit(Ktfps(#uRv!Yk@t8^2$&MilLpy
z8GkSV{)h{c2IpB%goS3#{PXS$;j&aIAu{rGnCQy1mKYhQTK}HoBpHt{C{~!?45Oy_
zDpdpBnt80nii8G1$g~>+#|<1H9bOkD;$u;Z2ludcg}rbFKPBX-77j6R6vb$kd}=&p
zL_gD>lNKPBm_h#~6h|1UR8`UI2Q@!GHFS{}oU)J+aL3Vyqxd|63}~TOLo-AQkjTjT
z^+RfzKa357GU+phpF$)c`LUC>`sLs=K`8X6%q*)}NCzQNy<w+y?jnbyL<uI1pN0Xc
z`;^e;ZZL4A0$uf5b%4XK@dVs@u_@ky_6@V5=|-3D4lKGW6}X1M?6-3e%bgv;N|CaS
zwpQf8t+z>vrn4G`JN?Ya`^C@NSBnHs<T`EGWu`2VMzHNt`Y$xn);9HzaLyXj{qjSS
zowqG$o%XJDm4itA1!R%?PJ{y;;us=U@QU#B%B!BGrpTRC5>KHGzLL8EUC&xn8J#?3
z=3PsChL++t=CWpOw-!LPO?2qXyd_}LD$`tCn>XHbR)L7Zr3dK=!@IX8^sXf0tH>J!
z$|bwNgioeap=KcL3>YSXsuSAnbljaPD<nIKO9gWh+k_Sg3<mA1aA+xe*w<AGL2}Lt
zc86&2<zgK)kT&xXB+B4DKXez%g6DBJp^_3_K0DPHoE#UQAeGEk2(H{SG$jxVR}TzF
zh+Fa84HCCcO5lm_Pi+e_$8PP0R|v1r$xw>JG}W(7$%4A7?K8c|a2lTenv}7hZ_SZq
z%qCUk`=r)DJbfgo49A}$kGz+hkz{z_v(lWDtnjIlPwcsr>vH|ukCby^>T71Mz+eVa
z_9^RTs<_HnZ}(>7t2o&ekE&d8uPEc$Z7Fwec$?=wo}RjGT*)mx$NGq?#C{dQcylo^
zl-8-SCo;e>ygL}d?$gkF-q1VX8Dk_+nRI?2zp!<DIq|*bBUuf-4(rR?SWXR_Vx;H9
zP_Z#PlW~_uLms%DGa8p_^%Y*_>c$sVwqVC7kXrs|<Q|4_y&2-(jDhSWTMdd{=3%kV
zM?$pn??d6rSfbN%)2DRlMDa}F@cBrS<r$DU*P;$wj5G3fFkU0YIfQ3;H>l~quDT&j
zO@*?aG>kX1Wi=ch3fHZC8K0+KWk)*tUH4chQ33^u4vM1GJ4?Ht7DDDJaf)#ezEEpB
zL-0!<^<rjy^csfkR}sxj`j%OB;)y~cF5Pr-Vgu4yhAbxuU)Xw}Uoltsd_LcyCLP0#
zeNM@Qlia9i%iQPJkZbnEcSl3D`Ya!jz`jbq7<IbP$-!huP2;E>4ZGG_xPNu?iTcF3
zTIKnmFZAfH#u3$%Xb_4*DLSdfs)mc<%X5M-JIYha<K@Y>Zf;x-=QGDYOb*ez_P4%V
zO+i<^iF{FHH?V)6n#Np$q1;eni~nunBRh9=f>^P$!Kz>2apAJ#H`#AJXX&|qay}sq
zF2ZwfuU=|vTXZgdIY`Q6uSs4wt|w%jt<%y;y!rys$vC+l0X=R%pmBcY^?^&#RL!ao
zN^L;tHQo~H6Vn;&juRg*RH5!ol+QFSp7S%~;HLeagNf+z8fEspX?aw|>;3OvfEFPR
z>pHPXVr^mUdw>%b=Fq-3A1;$jOCR<0(Gm-_I)K(xPTgy?Poz!XCwljRLsnfL3k@5m
zmDcUalf(J>Ii3rODR|5c-p;j|k9IfEp@ml0KP}d28Bd|U+jJkm`RUim3Xr-$=6Rp!
z1Ni<R&qT+<+}7H{)Kt&<k6Z}zGd@F$+X`rZn+pNRO_KG#7nl$5xZEGr9t!-!_Pzfv
zYDY8EpGhXJmwbkI^^q1ISt%(m+FNfQnK4b$OIZi{)gisOw+HiD$&mO#Tp6W<N}A>`
z-#Li0M>~!$^Q<qO?$*Ci+`@ikZ^*7U>D{-)E>uDlEWhpCGCMX<b!v*A1{uU4B+$mG
zS<3Jli5Ph@)o%JTsk87v>Gwcfo5I@U@D7ntC<P`sw>lPgBk6syC_XAt6^7Ws{a5V~
zF?ckRo&^$9D(Q^M^SDmF8%fs0WTlG43>okFk1>o)`q0eWQZ9o!(9PKxLHvRPHPSP*
z&7OQVbs<1?eI-~@`(+P-eA?v(I0TY>o?MNS`dcqUo^w@}H!TJ?m}w9PWX{?mBSTgR
z=uQZ&VRKrKBAuOt{P&In)@s|Am35KZAL#vGfb~x6KUEp~#8CHc&+7BkD?V}vW&d3$
z%-zG6Gw_yg^oG2(o06#Z5CSf|Da*#qX2ymxlP?n%K@W4eu&usj?|hQDjDy^>>=OXv
zn6B3VoW2ieK%T1Ip1c7Qch3>a_xDo@;0Vn9l<I2PYW+%-5dh^k6uhl~_R+N6pW_eR
z?<Fze0|7mMB<Yby<WEjeobVam)knhn;s-1yX;C_NFHzO0IZdIbBuBvH3%2d!CdKm0
z`4#XZKD!g^7*z82x^Yj5k^%!WbsN7m#CPIuxE^=f|HM}9q9k1oh7V=Xx^8nx#@B?D
zjKqX3<kQ3TE|^-l9$1R?`ElqqCZtHU2%@N;jF5X4a_Ccv*k#W(yiwzpJrIKOd9pz*
z0RdC)_OaJOdI;EE>bxCqF_aRZL<x!T@*B;;L!{k4RQ0jEEscBP>TK{qFP8;84l<mA
zbJft+c~mr4p=jxa4J91*(=!zWU?YaN2!izMDH_(;$2eqSDe_XD4LspIEZ9h&Yp$v5
zhpw!4pIQpBzP1o{lruwUzy|rep<egnwdtBM@6<w1jeIRFm~x<<y@m9tWXJ`s)s3KD
z!vZl0iWC&Fz9Db)SS8F2cotzwsg3v3l<p}wywA&_q$2!i4OtKtw#-o%=g|#h8J>qe
zFkV&tJFq8p5jsARwBYoH=PYt%3>G|;Kz>>k_)XZ~Xubz_2)ou1MK6GvplTT^bk~bs
zS?+%iABKVEfR>->nVt=L$5yTIJ}XhKsemyJ1!Vh6&vZ5(+vXT&n)_EH{wDvD*=jZg
z&l<d@qa;n08?|TI2X!{EGFqdrl4cW4RUCk<TdJiGU0IgwW?eOVIZR4+4jp2T3uWwL
zzaWk9sgynIrOpviPsER!6N*Act<jkPM{SZ#dKQUI)#DFkif!+)jM<OS{oGRE;IrAf
zTSE$WY-5(hghzQ^uinww=!mbfJaI=I3%4P6w)N*G(_^1cgdHG;tZG@TRKl;tgT4;Q
zKVj>9rc#f<_)=OqelX4!bjn`2R2oKL4^ypoZ<j3)6^HeV(~6F9p&Drn&KkcJxrpKH
zBeKp@*|pFR>{!iE)Z~r*ifz+6a*0fgn-*^abF-p$({~j)5{_dbECY0PS^eJU$|20h
z>`NORYQ<a8<AoB;r~dWh7|UbGTQ`xGK1S$NMs=djQ42TkG2Wauj4rG*`QXH2IxcbS
zdI{eo?g^^L#vIJgEPXvb)4Ck*t>$5b7r+=+Y1>LTE-L0yH?^*I<ZC@HtWE{1pZi>j
zZ%nniVKi#4MD$rxtbngELtQVi61F;YJr8DoZdSox+~qrnk&OZJE@slH?n0u(fwq``
zBLDcgG3kDCnU&;VeL~ep>2!+5_4Br$50aM3LUZ$5*9c$%KUmsP_c^J)z(7Dmf4{VU
zIXmz6eDWS%!@K%uefxyIglnON5j;&MIFJOz=0`GbYQt41g$|p{swOkGkg{rb7r>WW
zbZNA3vx%P+;(YHH%LeUd4IfY&N^AcHTBpcCn58C-7D~?AM>$06slSY1xwNqH;7Dk`
zM8n!B^4i2&&5Jn&Di331Ssi?9vMO0D2t`r8h=8}bRPQ^4!P?~;KhCYE96|7z&Jdjl
z^$z%2Cp|NQ#JTuf0KaotMHW~GT@~O49k-*S%V&v>%bhAz3{~5*&N%i0u%8-XS;E!a
z?w9~T?xEU!09C>5PD}XTpt_BV-62JIi_~3xgj9Ptr&Jp)jPL1nKo>ieDU7Ijb&@}4
zo_3Xh@lx{lv^qB_(V7LKo@Rk8m(TAKIRZEnJDWd8eDVL}mHh(|W~D3EdRKF3X8{wg
zA&-*qQ}GwJ0%={|h(X>M!zP|jm10^*$s$Kc<=gz`!XWf?!}EBeYWjBF10^0$3$|hh
z%Duv&pqe4=zO67@()Pin+8(bB&oHg%zNneOF$xJMQF{)5y<#4=si%CtUl{e#i!YlP
z3jgell6h+7#ATtZn+5$r(}5eC_yzX`$87Nrr1&yfsj{VRrm3}yTjZWVI+$^_PuwC2
z!0R4KLAU$o$=*{SAmaam<S$Rq9hL`&e1>=R5te<gWh_Q$5#H?<?_DB=;(;d*M(UF9
zfg)^~#S`>4;1M#z85orFT{_^fi;hqkjnFY&4tDfAIDBjj+)Y$98w;2Q#>(xDYzUr{
z1|j~&7d<DdJhgc2{5FaT%^ywK0D4i#vx)I3fuI;Pl-RN+<)_d$ITSo~eReWHwYkeY
z-OA9?WKkFw+2aWqST0U(?|dRnVabkTRiBC>1emjJehgpn{;(ROpTL8s^y3ZkV3UUX
z0x89JqLhiXrPpKT6zHka@e{2g-p?xp5E$w@-p-kn!LYGeNy$_w74+HkU|}Q{eE){2
zUswWxve&V>3kEG<)4?h~wxY@RsT!U+Q{8edpZNu+pP*_*BwW5#r~G!4DU3@IhB;`(
zd2#s}5=LJ*h;WHPa(X~2)l6gJ=%T2Y{J`S!;Zf==NI#pT$!qhujRO%GerZZaAzCZ(
zt*y-ypnP#*4G%jP7Wzbg;auOVqqvdkbSt4osLC6ns04dqD@KZ*5u(U*5c;qo>22R`
zn7u59)VgdAKDG~fpH|Rg!4g_!&qyU9s6q_K@C>6>r(G_~Yt?5cp9Z&GyeK2$#kbjb
zb!B{soauw!m{znQ6hpK*$$NCM0#tJf3%AWf<`4z#fN#*iUw2M{1S>wcXV*9`WE4x_
z;!qDPFe`@}SYPu|W@0-Rv#fe|l>pY<K30FrOUTDCfj66<-rVAo{|8znU+Ar$4oU=O
z9a^+o5(4OUPef6rdm<v<?v2lWPsCrD#eXj5w<Uarcl8kglZJWWJ@HVQ_Uk@+);lX#
z!SQ7G+pX`!ay&~oz>yuX_?TAMXDrS_+i}*q3&ef=st<RajsJBM{tPI;N+?q$lI7{j
z4&;kC%V7aJt)+Q5XzO@PsucWSyO5+exQAmhf&K7JEtNn^!zG@qFjht-V_)(hz$NIr
zY;q>-a-GmZE=8g*eysGBU%lQpCFxn_w%qYUW6sl1wa1=fvDy5#VPBn>Hj@{iSKQNQ
zOfGI)y3>my+_RDHI_1RVw*PbMYog1aufJWe05tA#dhdUa6Cmjw`}a86TG(0tiINOB
zzu{edgp%urQY24W1OexU#n0-}6oFtWH%;MC%bIi%%e27Fjm*A6(DjP}^{&iyi)C!k
z@K1+IVjL;qA(z?5I^dDka_2to%cA>e_Nk(W3W0617}6>ew2QtPmEd>+))`;W<}*nd
zsWcP&<=D3Xq8LZi#iD@QQfB24-Z|qUQJZ-AXbvNCpNb|}uE7c=9Sg~ZaH=%@nd(;-
zE|0MxYZ36?VvV%PfUiC|)+&xa6aAs}i|ul65dn<*7b3S`fQUWr5dnNhxJSfBPsh&M
z$kyrC0o(8P4E+AK0@_C~+~bS!;DBxX*GJ2r(u}OBfAWx8OGih~#)ewQ#{Q?_6?VjD
z7=Bv;?IQ(IVKYx#@nD2@FDU71r|N?gUj>eGr<EOV#~?b;Zw4%{624dl(+-C0@!A`5
zj~L@9K*a8dSC({FC&-CTHiWhu9lct`N`$@0DECcV5$B_B(`^x{#z})#yqHpx_r=zb
zp--u{u{-UxCR!JZO^(M1{yuurDO5s8NVgU6T}%i4Y`W8QJa%)jV21*`(M2ucVqE{F
zlm+0qV2t-FHu@Gez+mrT0QI{EHRqOq51z1p!tjT!argaCy7}GSyt%87V5pblG`S7T
z%p6j9qfN|DQrWI+vZ}1X&F~=v*hzwy=}A^5TwiPwc1v%{iuptMFU{81Im3UQlC$Un
zSucOa;!3(p=8;+<1}`kRmRskXHl;78qG$FA|9TX%P*AXZ5oMa_1p?tDqU8^SB^TP-
zdXC*3SGs2KehAWyLXg(n4!REimu2=?K?Gh~9wU|`tHfoY6Tsmjf_$&k1eeA1meG}E
zJO|ZUXyk^CGZiP)H+gN*ROa5ryew2~W}!&Vv<lsIGj|nW|InXbCDOP+?M&GMgUK%P
zL{B2wPRpr5BdJ+x{KW#bMCm!XdS!%q(fFIh;=x+6p6vCiF?^9@?r89=1a^BQ>ro^0
zQwW0#PjY5k8p`V=e7A$5qTzuHFu-)Y44&o3z2!Fm^m~r8MBbkb0UU?D=eWI*p2HvR
zg{=-=!=2j-XdeN8KeLt(Hrn;u{3DXbq8^!$8b1~MIk|1OJMLY5q@(`Ls}Hx2{QA>3
ziVa5~u$FwJ8|wC9EX~0EVJ1?t(BF_Jc4822o=s_qix0P9E58*{8Th<*?wDMA-l~<}
z1s4>R1O3B(-xp~R$7j@zx6$?L><6c-V!uA?)3%kMvp44M>4*XWo}PAJcqcGSo2qrX
zs}DIMO{Lw*a~J~c+kH>1X)2mH&5Q^Xh<TyXCLo?-CEK2(V0OpM1v#Oq74g7~)0qp#
z_>~3g4H4lxAe|tJ#fT}N=J9>1;aOHn(7!Na*zZD*?YEih5mbCGS&e)~FbE^eO=X~e
z7LVj$dTB6s{HY|(WIS5YKzr@u_nR0wzeDK`$}*WMh{C1R*`*>11PCh~WYwtYb4zi3
zcVfrZRNx(+vr5rqB!BQs5VyQW$aqb55>ypYV%Kv>Cf=Jb(ArdW6KhI2Ka?7O5XX`<
zrYvB072BY&b=7szm?jW$*Fjj$92@4tnkT$d38>G`{I<*q#A?@GS{vCSTNaZBu?F3F
zw6Ep}7U7EKN45TeRoEvhNZ=4&_Z{g{%jMeOtdsqP%MI8r_DXsqq^wT&m5j|izpXyB
z*U)hoJd6kM_*u;}AcukF-o14Cd2j5^j_z@buj`sVXRhw`T8xSPorH?!x~38Xh&4^b
zB6vSC2j&siyCiUV-vrSE%bgU)?GwjCprpJ2os#e$Wj9(8^XU!<xn!rpf|?xo=9pUh
ztd74^Ue=+bQyfjeNpNInfv@~ye$O2JZu+;8@s<+N#M&En_9_T|u8pU=k@)nIouecS
zt|r+8tmYN;onV&MyccHQgf)|oFPAUsd>d(lW)lwzu9gmd=FUB#1m)Ae-LC-ry=OK=
z$o(QQz5VsU5#rY(`GeVJdggXEe-fJWGq2%YeMIPBSx)#CJcOAGimq+t8JtP1CrPgg
z=<&BR@k<ArjF#sDGZjo=!-Abpxg7DQW~ZR*QcGp?amIR4Y2~AeGL5ZSV*J`6Dh0!J
zYsYFuV!4$9H_4$?NWS9?hT|<K4L!rMzJ#()X9Si@YsWdc0x981hJHU;7g4^Rt?PU(
zGV==0k>ZO2gquIFM_|CEmr&{wG&owQ)1F7zyJZPsXpJTdl|X@pT%)ZK+RGOuOUrCQ
zy7mP}&|%Y@Zn#{lr190MhT89vpY3j5Bxf!ha@+#-1d?u5grfg;{{X;q4_tiEJ#gu_
zzdX3v_7~vpE=K({Js<50ku|)Acl8mBnc>{G7Z?%R9o(MUI;YOC0Zq_3h!`x#n*@3{
zgJ7}GHC!4IF-xY4X#qjaT+BFIx>N;4<Gd7DN^*bk0}^azzQXD`h&ud*pTT%{(UQ6|
z4SP5`Jj+>%{}2dm7FTM#=SSL%2(fG^1?~(OP`bCluic@_XWFduCRnx&3=4XknGG4O
zuHJ+6z#Xjk3B$#qv*x3^k%9;*uzRS=AMy{pGu*dPdB?!=hEv2u`-j38rNH>Z8wy7~
z)E>5rnADYP`}yzv8y!PJ?}QJyU_SDQCAo8eQFv99ACbKDVS`+Q?`}<fwK-=xM#QaV
z$V7#vuv7}oK(Q5-|J|ZrDWT@)Bi5mJIpgXUbpW>asMm$wqn>p83;BQBh;OqyAAzpj
z#A|q0AAz0}{t^JXkHgI^=$3VQP5omv{B(rIMLVJxlqsc$yU^8P(=5SxZ<`=zE_OIs
ztYV+E_OFAoPBqcHf%sP1#GB4FZJd=&wWb!5FO=JuwGY4Gu<th!ss)CT5sXZCApGGN
zCUY!n_p;O-Rx;hlokK&C@4DB&z#f@0$J6-*L%b5BCa9Co$uV|kNFIC|HFzT=2ZRz9
zFMDr0Y-8<0cc0F2oF%!nA*|0?y5gvKKgu;tE;Kp+STnc(+F_^B?vGV$ZtUY59f~<u
zpum`y+arg?TT#Q)@jmOlkg9V8!9H0zvl7Ds%(J(k>npW<WnO3d^rmP>qdF(X=mG@5
zb=aiuYuC4^1F*eE{kz@$!Nv*#2#E5(q5jk2eT2L65})B+eS~|30;k2)?Fr|K0+QO;
zL(#YjF3po#Z7MY>z*Dm;;*~rBcz9Won+C34_BBEjkSK$6)5gyJOr3+YtJ^wz8R%LC
zW6(QlZW$0E<;*_cAr0=v!p6Z1{ffN8vPz|}vr)JVy8y#>(lb1ez2*sERmbH@vt}!N
zeu4Cis;6v16%^U;JDKN9(zlW&HK9tr4w7jLzy~dgCOjABId}%_s+Jik;SV}6qEODd
z=P%==El(oUd)#xvhdp_DEsb+sg@pTLUK!OPLvz0g&K~3J%mC9YTkGTcVAcrNbj8>)
zI;tHb6o!IpTp|==X+P>SJO6z>OnU1`$ja$<lOq&Kb28p5jNl!%^jO(_>qRedIPt?w
zbf}SZX;hCQr3)FcJuXixBKo&27-Rx4aKQ5W5*0}LpgVcvyzD<Ds7Rl+@uPP?VOI@n
z+(3m*Cm!59=UDLc9~m`fI@f|}j@LPnIWjVFH196CiRjOc8bSJ2jcQT@ejdZMad{kD
z*eB_nRLc%!{VKc2lF#6iYrT`zli4qynW63EZbDYudkwPIs(==lk{iqtpZT3z*f4cZ
zki|3_=1UA+&VLNghgqqO-=ObtN)_#(wqv~Fz5Z!%R2@kC=mx&cVY(&CFAClA326P%
z4fF>WUjg^fjdXsQ8w$6_;Mcbm&^{u`y{zp=vL0MS|3j9I^&j#yZslm+RzUk7<sCed
zhk8%%2l9;nI4~OBiZZ^ffcCc{0r9Al2O^Pf!NB=XB7fSP#x_4qA*)-7clD8EMybB1
zxBke6Q!;IuDNXxR|3!936imCoiV`Z~zCnCvhx`#7kn`)VH9sE8JrJR_^gS5p=~#%t
z&L`^m(HQyMnGRyPH(JUR>+jc8UuY3Bl%5ylK<m^|573Iqy7(7($9OUzpE9nKW7?jB
zr)j=tTn68Qw^SEwnPpwZqT>&SU6@uXodWJOy1;omw|$Id#wFb<kVZ3s^=+7xMXW5j
zElU~kV)g^v>Jb=NmOCZx(xmhb7hliZ<(J+}dGS(bP{7_oh@^edbV~&Q9@65#ASj^!
z`Q-!v$^gn0FW~dn8RQp3{`OS`zl1(~S;6f-_Ok#ua#zYd1aJTRL+GFBa{v9+1$UhW
z$OpWt;Ncy5z^e-GO1Ve&L;3&e#RYdd0J0ulo(FsfL`m<;6))-S(Ec35zq#jq4-!DZ
z!+Yw0Gt6Bn_i*~Xf<FQKmp8p1j_h~W;O`V)@%rDIaes{LA76^Ur_*E90N3L0$`$X|
z|0Da~U6TJRcn|NR0Itd3m2wX%u*YL~sOQ({{^v{ccTfU|2pI0edj_0;>Bj7B*!{0i
z{!KUjm#Z+3yYetYo$fDP2}b_AuKcgp93Ja`m_ZB3gTE`~9<!)_r~lE6{7+f&kNfg4
ztMso}G~C`_`p?ty|GUEOXz?)j0+73USIRw@u>Vf~V=VrsOx8Ov07`$C?s3<dE8b9|
zKRa`unDI}NKknBeApPTE76Bmr<F1r@Ec{-<AH%%Y`KNh%mk0uAA3ca44(oRr3U^ws
zc*{xt9M(U~SGXS!AY<WSz!Q+Ma97Gbt{+K$G+%!X<1Tjr(Ef*E0H|_!8yI{TG6cj>
z?@GDH1o>Zw@krS}j;P)N2ha!zMLrCz0UV`wrQ8FI^1o>OXMxFw3IP$jhanU|#O|(?
zdq`6Ky~2MSzk8?>5Sw`zjQ}`S?n=4GH=5t7{O6IHhpGVKjfZZ2KzQS>lzZIJ{#MmL
z4R+iq0tozF)Zk9h6)z&)pNj6>+y5Yja8G(b6yc#`{jaRh_>J`cA*OIQ8~_ag{`!Zm
z<G(7@r~kjh`Gfp-!~1ik-FfZ-?SEKl0Gxj3k-o?8nBh-_|I|Ny4<5in{m{b)@KE2C
zat|KHN1`9E;(zR~euy8yclgjb_E(ByvHV`=KlUU(R0Z(1J@g3u^`t7t`g>LXqwnpZ
zVt~Wxp;H9laJnny9^GvJP4T~RKRwh7aJf9Rqya9MyHf7K&GBFK{!0hUoo0ZB-&yK+
zny+|0x&GArkL~tfECtB;o#lQnyZ`^l{%3alz5L%<^7ry_xgYl(ApalP_4f-6VA(&k
mWC51_T`Bjl`cEqUp`9-+4u0!40fY_-fiU2KfD(8B-~J!|>pjx|

literal 0
HcmV?d00001

diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
index 855223a92d0..6971e731d1c 100644
--- a/estate/models/estate_property.py
+++ b/estate/models/estate_property.py
@@ -57,14 +57,14 @@ def _compute_best_price(self):
                 record.state = 'offer_received'
 
     @api.onchange("garden")
-    def _onchange_partner_id(self):
+    def _onchange_garden(self):
         for record in self:
             if record.garden:
                 record.garden_area = 10
                 record.garden_orientation = 'north'
             else:
-                record.garden_area = None
-                record.garden_orientation = None
+                record.garden_area = 0
+                record.garden_orientation = False
 
     def action_property_sold(self):
         for record in self:
diff --git a/estate/tests/test_estate_property.py b/estate/tests/test_estate_property.py
index 30ed9d64712..2ac0c5e6822 100644
--- a/estate/tests/test_estate_property.py
+++ b/estate/tests/test_estate_property.py
@@ -1,7 +1,7 @@
 from odoo.tests.common import TransactionCase
 from odoo.exceptions import UserError
-from odoo.tests import tagged
-from odoo import Command
+from odoo.tests import Form, tagged
+
 
 @tagged('post_install', '-at_install')
 class EstateTestCase(TransactionCase):
@@ -82,3 +82,24 @@ def test_creation_offer_for_sold_property(self):
                 'price': 130,
                 'validity': 14,
             })
+
+    def test_enable_garden(self):
+        """Test that default values are assigned to garden area and orientation when garden is enabled"""
+
+        form = Form(self.env['estate.property'])
+        form.garden = True
+
+        self.assertEqual(form.garden_area, 10)
+        self.assertEqual(form.garden_orientation, 'north')
+
+    def test_disable_garden(self):
+        """Test that values are removed from garden area and orientation when garden is disabled"""
+
+        form = Form(self.env['estate.property'])
+        form.garden = True
+        form.garden_area = 15
+        form.garden_orientation = 'south'
+        form.garden = False
+
+        self.assertEqual(form.garden_area, 0)
+        self.assertEqual(form.garden_orientation, False)

From c04331e9e45d269994b7e78ed8b4359228f57f05 Mon Sep 17 00:00:00 2001
From: abdrahmanrashed <abdelrahman0rashed@gmail.com>
Date: Wed, 11 Jun 2025 10:15:09 +0200
Subject: [PATCH 23/23] [IMP] -refactoring based of PR feedback.

---
 estate/demo/estate_property.xml      | 15 +++-----
 estate/tests/test_estate_property.py | 53 ++++++++++------------------
 2 files changed, 24 insertions(+), 44 deletions(-)

diff --git a/estate/demo/estate_property.xml b/estate/demo/estate_property.xml
index 8359e8af293..71d03835dee 100644
--- a/estate/demo/estate_property.xml
+++ b/estate/demo/estate_property.xml
@@ -1,10 +1,9 @@
 <odoo>
     <record id="estate_property_1" model="estate.property">
         <field name="name">Big Villa</field>
-        <field name="state">new</field>
         <field name="description">A nice and big villa</field>
         <field name="postcode">12345</field>
-        <field name="date_availability">2020-02-02</field>
+        <field name="date_availability" eval="datetime.now() + timedelta(days=14)" />
         <field name="expected_price">1600000</field>
         <field name="bedrooms">6</field>
         <field name="living_area">100</field>
@@ -21,13 +20,11 @@
         <field name="state">cancelled</field>
         <field name="description">Home in a trailer park</field>
         <field name="postcode">54321</field>
-        <field name="date_availability">1970-01-01</field>
+        <field name="date_availability" eval="datetime.now() + timedelta(days=14)" />
         <field name="expected_price">100000</field>
-        <field name="selling_price">120000</field>
         <field name="bedrooms">1</field>
         <field name="living_area">10</field>
         <field name="facades">4</field>
-        <field name="garage">False</field>
         <field name="property_type_id" ref="data_property_type_residential"/>
       </record>
 
@@ -36,24 +33,22 @@
         <field name="state">offer_received</field>
         <field name="description">X2many tutorial</field>
         <field name="postcode">13579</field>
-        <field name="date_availability">2011-11-11</field>
+        <field name="date_availability" eval="datetime.now() + timedelta(days=14)" />
         <field name="expected_price">123456</field>
-        <field name="selling_price">135791</field>
         <field name="bedrooms">2</field>
         <field name="living_area">30</field>
         <field name="facades">4</field>
-        <field name="garage">False</field>
         <field name="property_type_id" ref="data_property_type_residential"/>
         <field name="offer_ids" eval="[
             Command.clear(),
             Command.create({
                 'partner_id': ref('base.res_partner_12'),
-                'price': 10000,
+                'price': 150000,
                 'validity': 14,
             }),
             Command.create({
                 'partner_id': ref('base.res_partner_2'),
-                'price': 10002,
+                'price': 150002,
                 'validity': 14,
             }),
         ]"/>
diff --git a/estate/tests/test_estate_property.py b/estate/tests/test_estate_property.py
index 2ac0c5e6822..75692d12952 100644
--- a/estate/tests/test_estate_property.py
+++ b/estate/tests/test_estate_property.py
@@ -1,6 +1,7 @@
 from odoo.tests.common import TransactionCase
 from odoo.exceptions import UserError
 from odoo.tests import Form, tagged
+from odoo import Command
 
 
 @tagged('post_install', '-at_install')
@@ -11,73 +12,57 @@ def setUpClass(cls):
         # add env on cls and many other things
         super().setUpClass()
 
-        cls.properties = cls.env['estate.property'].create([{'name': 'test_house'}])
+        cls.property = cls.env['estate.property'].create([{'name': 'test_house'}])
         cls.partner = cls.env['res.partner'].create([{
-            'id': 'test_partner',
             'name': 'test_person',
-            'company_name': 'test_company',
-            'street': 'test_street',
-            'city': 'test_city',
-            'zip': '12345',
-            'country_id': cls.env.ref('base.us').id,
-            'state_id': cls.env.ref('base.state_us_39').id,
-            'phone': '+1 555-555-5555',
-            'email': 'test@testing.example.com',
-            'tz': 'Europe/Brussels',
         }])
 
     def test_creation_area(self):
         """Test that the total_area is computed like it should."""
-        self.properties.living_area = 20
-        self.properties.garden = True
-        self.properties.garden_area = 15
-        self.properties.garden_orientation = 'north'
+        self.property.living_area = 20
+        self.property.garden = True
+        self.property.garden_area = 15
 
-        self.assertRecordValues(self.properties, [
-           {'total_area': 35},
-        ])
+        self.assertEqual(self.property.total_area, 35)
 
     def test_action_sell_without_accepted_offer(self):
         """Test that everything behaves like it should when selling an invalid property."""
 
-        self.assertRecordValues(self.properties, [
-           {'state': 'new'},
-        ])
+        self.assertEqual(self.property.state, 'new')
 
         with self.assertRaises(UserError):
-            self.properties.action_property_sold()
+            self.property.action_property_sold()
 
     def test_action_sell_with_accepted_offer(self):
         """Test that everything behaves like it should when selling a valid property."""
 
-        self.properties.offer_ids.create({
-            'property_id': self.properties.id,
+        self.property.offer_ids.create({
+            'property_id': self.property.id,
             'partner_id': self.partner.id,
             'price': 124,
             'validity': 14,
         })
-        self.properties.offer_ids.action_offer_accept()
-        self.properties.action_property_sold()
+        self.property.offer_ids.action_offer_accept()
+        self.property.action_property_sold()
 
-        self.assertRecordValues(self.properties, [
+        self.assertRecordValues(self.property, [
            {'state': 'sold'},
         ])
 
     def test_creation_offer_for_sold_property(self):
         """Test that everything behaves like it should when property is sold."""
 
-        self.properties.offer_ids.create({
-            'property_id': self.properties.id,
+        self.property.write({'offer_ids': [Command.create({
             'partner_id': self.partner.id,
             'price': 124,
             'validity': 14,
-        })
-        self.properties.offer_ids.action_offer_accept()
-        self.properties.action_property_sold()
+        })]})
+        self.property.offer_ids.action_offer_accept()
+        self.property.action_property_sold()
 
         with self.assertRaises(UserError):
-            self.properties.offer_ids.create({
-                'property_id': self.properties.id,
+            self.property.offer_ids.create({
+                'property_id': self.property.id,
                 'partner_id': self.partner.id,
                 'price': 130,
                 'validity': 14,