[IMP] estate: UI Enhancements, Ordering, Filters, and Constraints

. Enhanced UI with inline list views, status bar widget, color picker for tags, and visual decorations.
. Improved form behavior by hiding garden fields conditionally and making availability date optional and hidden by default.
. Controlled editing by preventing direct modification of property types from property forms.
. Enabled manual ordering using sequence fields and set default/custom ordering across models.
. Restricted actions such as offer submission based on property state and adjusted button visibility accordingly.
. Refined views and filters with editable list views, default filter for available properties, improved living area search, and stat buttons with filtered results.

Author: satp-odoo

estate/__manifest__.py

@@ -6,10 +6,10 @@
     "data": [
         "security/ir.model.access.csv",
         "views/estate_property_views.xml",
+        "views/estate_property_offer_views.xml",
         "views/estate_property_type_views.xml",
         "views/estate_property_tag_views.xml",
-        "views/estate_property_offer_views.xml",
         "views/estate_menus.xml",
     ],
-    "license": "LGPL-3"
+    "license": "LGPL-3",
 }

estate/models/estate_property.py

@@ -13,7 +13,8 @@ class EstateModel(models.Model):
     description = fields.Text()
     postcode = fields.Char()
     date_availability = fields.Date(
-        copy=False, default=fields.Date.today() + timedelta(days=90)
+        copy=False,
+        default=fields.Date.today() + timedelta(days=90),
     )
     active = fields.Boolean(default=True)
     expected_price = fields.Float(required=True)
@@ -25,7 +26,7 @@ class EstateModel(models.Model):
     garden = fields.Boolean()
     garden_area = fields.Integer()
     state = fields.Selection(
-        string="state",
+        string="State",
         default="new",
         required=True,
         copy=False,
@@ -53,7 +54,7 @@ class EstateModel(models.Model):
     tag_ids = fields.Many2many("estate.property.tag")
     offer_ids = fields.One2many("estate.property.offer", inverse_name="property_id")
     total_area = fields.Float(compute="_compute_total_area", store=True)
-    best_price = fields.Float(compute="_compute_best_price", store=True)
+    best_price = fields.Float(compute="_compute_best_price", store=True, default=0.0)
 
     _sql_constraints = [
         (
@@ -93,7 +94,13 @@ def _compute_total_area(self):
     @api.depends("offer_ids.price")
     def _compute_best_price(self):
         for record in self:
-            record.best_price = max(record.offer_ids.mapped("price"), default=0.0)
+            self.write(
+                {
+                    "best_price": max(record.offer_ids.mapped("price"), default=0.0),
+                }
+            )
+        if record.best_price > 0.0:
+            record.state = "offer_received"
 
     @api.onchange("garden")
     def _set_garden_default_values(self):
@@ -103,15 +110,14 @@ def _set_garden_default_values(self):
             self.write({"garden_area": 0, "garden_orientation": ""})
 
     def action_property_sold(self):
-        if self.state in ["sold", "cancelled"]:
-            raise UserError(f"Property is already {self.state}")
         if float_is_zero(self.selling_price, precision_rounding=0.01):
             raise UserError(
                 "Atleast one offer must be accepted before selling the property"
             )
         self.state = "sold"
 
     def action_property_cancelled(self):
-        if self.state in ["sold", "cancelled"]:
-            raise UserError(f"Property is already {self.state}")
         self.state = "cancelled"
+        for offer in self.offer_ids:
+            if not offer.status:
+                offer.status = "refused"

estate/models/estate_property_offer.py

@@ -14,11 +14,18 @@ class EstateOfferModel(models.Model):
     )
     partner_id = fields.Many2one("res.partner", required=True)
     property_id = fields.Many2one("estate.property", required=True)
+    property_type_id = fields.Many2one(
+        "estate.property.type", related="property_id.property_type_id", required=True
+    )
     validity = fields.Integer(
         default=7,
         help="offer validity period in days; the offer will be automatically refused when this expires.",
     )
-    date_deadline = fields.Date(default=fields.Date.today() + timedelta(days=7))
+    date_deadline = fields.Date(
+        compute="_compute_date_deadline",
+        inverse="_inverse_date_deadline",
+        default=fields.Date.today() + timedelta(days=7),
+    )
 
     _sql_constraints = [
         (
@@ -44,22 +51,20 @@ def _inverse_date_deadline(self):
             )
 
     def action_offer_confirm(self):
-        for record in self:
-            if record.status in ["accepted", "refused"]:
-                raise UserError(f"Offer is already {record.status}")
-            # breakpoint()
-            for offer in record.property_id.offer_ids:
-                if offer.id == record.id:
-                    record.status = "accepted"
-                    record.property_id.write(
-                        {
-                            "state": "offer_accepted",
-                            "buyer": record.partner_id,
-                            "selling_price": record.price,
-                        }
-                    )
-                else:
-                    offer.status = "refused"
+        if self.status in ["accepted", "refused"]:
+            raise UserError(f"Offer is already {self.status}")
+        for offer in self.property_id.offer_ids:
+            if offer.id == self.id:
+                self.status = "accepted"
+                self.property_id.write(
+                    {
+                        "state": "offer_accepted",
+                        "buyer": self.partner_id,
+                        "selling_price": self.price,
+                    }
+                )
+            else:
+                offer.status = "refused"
 
     def action_offer_cancel(self):
         for record in self:

estate/models/estate_property_tag.py

@@ -7,6 +7,7 @@ class EstateTagModel(models.Model):
     _order = "name"
 
     name = fields.Char(required=True)
+    color = fields.Integer(string="Color", default=0)
 
     _sql_constraints = [
         ("check_property_tag_name", "UNIQUE(name)", "Property Tag Name must be unique")

estate/models/estate_property_type.py

@@ -1,4 +1,4 @@
-from odoo import fields, models
+from odoo import api, fields, models
 
 
 class EstateTypeModel(models.Model):
@@ -8,7 +8,15 @@ class EstateTypeModel(models.Model):
 
     name = fields.Char(required=True)
     property_ids = fields.One2many("estate.property", "property_type_id")
-    sequence = fields.Integer("Sequence", default=1, help="Used to change the sequence of Property Types")
+    sequence = fields.Integer(
+        "Sequence", default=1, help="Used to change the sequence of Property Types"
+    )
+    offer_ids = fields.One2many(
+        "estate.property.offer", inverse_name="property_type_id"
+    )
+    offer_count = fields.Integer(
+        default=0, readonly=True, compute="_compute_offers_by_type"
+    )
 
     _sql_constraints = [
         (
@@ -17,3 +25,7 @@ class EstateTypeModel(models.Model):
             "Property Type Name must be unique",
         )
     ]
+
+    @api.depends("offer_ids")
+    def _compute_offers_by_type(self):
+        self.offer_count = len(self.offer_ids)

estate/views/estate_menus.xml

@@ -1,14 +1,19 @@
 <?xml version="1.0" encoding="utf-8"?>
 
 <odoo>
+
     <menuitem id="estate_menu_root" name="Real Estate">
+
         <menuitem id="estate_advertisements_menu" name="Advertisements">
             <menuitem id="estate_property_adv_menu_action" action="action_estate_property" name="Property" />
         </menuitem>
+
         <menuitem id="estate_settings_menu" name="Settings">
             <menuitem id="estate_property_menu_action" action="action_estate_property" name="Property" />
             <menuitem id="estate_property_type_menu_action" action="action_estate_property_type" name="Property Types" />
             <menuitem id="estate_property_tag_menu_action" action="action_estate_property_tag" name="Property Tags" />
         </menuitem>
+        
     </menuitem>
+
 </odoo>

estate/views/estate_property_offer_views.xml

@@ -1,29 +1,34 @@
 <?xml version="1.0"?>
 
 <odoo>
-    <record id="action_estate_property_offer" model="ir.actions.act_window">
+
+    <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="view_mode">list,form</field>
+        <field name="view_mode">list</field>
+        <field name="domain">[('property_type_id', '=', active_id)]</field>
         <field name="help" type="html">
             <p class="o_view_nocontent_smiling_face">
                 Create the first property Type
             </p>
         </field>
     </record>
+
     <record id="list_view_estate_property_offer" model="ir.ui.view">
         <field name="name">Estate Properties Offer list</field>
         <field name="model">estate.property.offer</field>
         <field name="arch" type="xml">
-            <list>
+            <list editable="bottom" decoration-danger="status == 'refused'" decoration-success="status == 'accepted'">
                 <field name="price" string="Price" />
                 <field name="partner_id" string="Partner" />
+                <field name="property_type_id" string="Type" />
                 <field name="validity" string="Validity" />
                 <field name="date_deadline" string="Deadline" />
-                <button name="action_offer_confirm" type="object" icon="fa-check" invisible="status == 'refused'" />
-                <button name="action_offer_cancel" type="object" icon="fa-times" invisible="status == 'refused'" />
-                <field name="status" string="Status" />
+                <button name="action_offer_confirm" type="object" icon="fa-check" invisible="status == 'refused' or status == 'accepted'" />
+                <button name="action_offer_cancel" type="object" icon="fa-times" invisible="status in ['refused', 'accepted']" />
+                <field name="status" string="Status" readonly="True"/>
             </list>
         </field>
     </record>
+
 </odoo>

estate/views/estate_property_tag_views.xml

@@ -1,14 +1,27 @@
 <?xml version="1.0"?>
 
 <odoo>
+
     <record id="action_estate_property_tag" model="ir.actions.act_window">
         <field name="name">Property Tag</field>
         <field name="res_model">estate.property.tag</field>
-        <field name="view_mode">list,form</field>
+        <field name="view_mode">list</field>
         <field name="help" type="html">
             <p class="o_view_nocontent_smiling_face">
                 Create the first property Tag
             </p>
         </field>
     </record>
+
+    <record id="list_estate_property_tag" model="ir.ui.view">
+        <field name="name">Property Tag List</field>
+        <field name="model">estate.property.tag</field>
+        <field name="arch" type="xml">
+            <list editable="bottom">
+                <field name="name" string="Tag Name"/>
+                <field name="color" string="Color" options="{'color_field' : 'color'}"/>
+            </list>
+        </field>
+    </record>
+    
 </odoo>

estate/views/estate_property_type_views.xml

@@ -2,16 +2,6 @@
 
 <odoo>
 
-    <record id="estate_property_type_view_list" model="ir.ui.view">
-        <field name="name">estate.property.type.view.list</field>
-        <field name="model">estate.property.type</field>
-        <field name="arch" type="xml">
-            <list name="estate_property_type_list" string="Estate Property Type">
-                <field name="sequence" widget="handle" />
-                <field name="name"></field>
-            </list>
-        </field>
-    </record>
     <record id="action_estate_property_type" model="ir.actions.act_window">
         <field name="name">Property Types</field>
         <field name="res_model">estate.property.type</field>
@@ -22,12 +12,35 @@
             </p>
         </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="Estate Property Type">
+                <field name="sequence" widget="handle" />
+                <field name="name" string="Property Type"/>
+            </list>
+        </field>
+    </record>
+
     <record id="action_estate_property_type_form" model="ir.ui.view">
         <field name="name">Properties by Types</field>
         <field name="model">estate.property.type</field>
         <field name="arch" type="xml">
             <form>
                 <sheet>
+                    <div class="oe_button_box" name='button_box'>
+                        <button class="oe_stat_button" type='action'
+                                name='estate.estate_property_offer_action'
+                                icon="fa-th-list"
+                                invisible="offer_count == 0">
+                            <field name='offer_count' string='Offers' widget='statinfo'/>
+                        </button>
+                    </div>
+                    <h1>
+                        <field name="name" string="Type Name" placeholder="Property Type Name" />
+                    </h1>
                     <notebook>
                         <page name="properties" string="Properties">
                             <field name="property_ids">
@@ -43,4 +56,5 @@
             </form>
         </field>
     </record>
+
 </odoo>