[IMP] estate: Improve UI, ordering, filters, and constraints

• Introduced inline list view for property types.
• Added status bar widget for property state visualization.
• Set default and manual ordering for multiple models.
• Included sequence field for property types to enable manual ordering.
• Prevented property type editing directly from the property form.
• Enabled color picker for property tags.
• Adjusted button visibility based on property state.
• Made garden-related fields invisible when no garden exists.
• Restricted offer submissions when property is in specific states.
• Made list views for offers and tags editable.
• Set availability date as optional and hidden by default.
• Applied visual decorations for various property and offer states.
• Implemented default filter to display available properties.
• Enhanced search filter for living area.
• Added stat button to property types to monitor associated offers.
• Filtered stat button results to display only relevant offers.

Author: pusu-odoo

estate/__init__.py

@@ -1 +1 @@
-from . import models
+from . import models

estate/__manifest__.py

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

estate/models/__init__.py

@@ -1,4 +1,4 @@
 from . import estate_property
 from . import estate_property_type
 from . import estate_property_tag
-from . import estate_property_offer
+from . import estate_property_offer

estate/models/estate_property.py

@@ -1,13 +1,14 @@
+from datetime import timedelta
 from odoo import models, fields, api
 from odoo.exceptions import ValidationError
 from odoo.exceptions import UserError
 from odoo.tools import float_is_zero, float_compare
-from datetime import timedelta
 
 
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Estate Property"
+    _order = "id desc"
 
     name = fields.Char(string="Name", required=True)
     description = fields.Text(string="Description")
@@ -56,6 +57,8 @@ class EstateProperty(models.Model):
             ("sold", "Sold"),
             ("cancelled", "Cancelled"),
         ],
+        compute="_compute_state",
+        store=True,
     )
     salesman_id = fields.Many2one(
         "res.users", string="Salesman", default=lambda self: self.env.user
@@ -92,6 +95,14 @@ def _compute_best_price(self):
         for property in self:
             property.best_price = max(property.offer_ids.mapped("price"), default=0.0)
 
+    @api.depends("offer_ids")
+    def _compute_state(self):
+        for record in self:
+            if record.offer_ids and record.selling_price == 0.0:
+                record.state = "offer received"
+            else:
+                record.state = "new"
+
     @api.onchange("garden")
     def _onchange_garden(self):
         for property in self:

estate/models/estate_property_offer.py

@@ -1,11 +1,12 @@
+from datetime import timedelta
 from odoo import models, fields, api
 from odoo.exceptions import UserError
-from datetime import timedelta
 
 
 class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
     _description = "Estate Property Offer"
+    _order = "price desc"
 
     price = fields.Float(string="Price")
     status = fields.Selection(
@@ -16,7 +17,7 @@ class EstatePropertyOffer(models.Model):
         ],
     )
     validity = fields.Integer(
-        string="Validity (days)",
+        string="Validity(days)",
         default=7,
         help="Validity of the offer in days, after that it will be refused automatically.",
     )
@@ -28,6 +29,12 @@ class EstatePropertyOffer(models.Model):
     )
     partner_id = fields.Many2one("res.partner", string="Partner", required=True)
     property_id = fields.Many2one("estate.property", string="Property", required=True)
+    property_type_id = fields.Many2one(
+        "estate.property.type",
+        string="Property Type",
+        related="property_id.property_type_id",
+        store=True,
+    )
 
     @api.depends("validity")
     def _compute_date_deadline(self):
@@ -76,7 +83,9 @@ def action_set_accepted(self):
 
     def action_set_refused(self):
         for offer in self:
-            if offer.status == "accepted":
+            if offer.status is False:
+                offer.status = "refused"
+            elif offer.status == "accepted":
                 offer.status = "refused"
                 offer.property_id.state = "offer received"
                 offer.property_id.buyer_id = False

estate/models/estate_property_tag.py

@@ -4,10 +4,11 @@
 class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "Estate Property Tag"
-
-    name = fields.Char(string="Tag Name", required=True)
-    description = fields.Text(string="Description")
-
     _sql_constraints = [
         ("unique_name", "UNIQUE(name)", "A tag with same name is already exists."),
     ]
+    _order = "name asc"
+
+    name = fields.Char(string="Tag Name", required=True)
+    description = fields.Text(string="Description")
+    color = fields.Integer(string="Color")

estate/models/estate_property_type.py

@@ -1,12 +1,28 @@
-from odoo import models, fields
+from odoo import models, fields, api
 
 
 class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "Estate Property Type"
-
-    name = fields.Char(required=True)
-
     _sql_constraints = [
         ("unique_name", "UNIQUE(name)", "A type with same name is already exists."),
     ]
+    _order = "sequence,name"
+
+    name = fields.Char(required=True)
+    property_ids = fields.One2many(
+        comodel_name="estate.property",
+        inverse_name="property_type_id",
+        string="Properties",
+    )
+    offer_ids = fields.One2many("estate.property.offer", "property_type_id")
+    sequence = fields.Integer(
+        "Sequence", default=1, help="Used to order types. Lower is better."
+    )
+
+    offer_count = fields.Integer("Offers", compute="_compute_offer_count")
+
+    @api.depends("offer_ids")
+    def _compute_offer_count(self):
+        for record in self:
+            record.offer_count = len(record.offer_ids)

estate/views/estate_property_offer_views.xml

@@ -1,17 +1,26 @@
 <?xml version="1.0" encoding="utf-8"?>
 
 <odoo>
+    <record id="estate_property_offer_action" model="ir.actions.act_window">
+        <field name="name">Property Offer</field>
+        <field name="res_model">estate.property.offer</field>
+        <field name="view_mode">list,form</field>
+        <field name="domain">[('property_type_id', '=', active_id)]</field>
+    </record>
+
     <record id="estate_property_offer_view" 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>
+            <list editable="bottom" decoration-success="status == 'accepted'" decoration-danger="status == 'refused'">
                 <field name="price"/>
                 <field name="status"/>
                 <field name="partner_id"/>
                 <field name="validity"/>
-                <button name="action_set_accepted" string="Accepted" type="object" icon="fa-check"/>
-                <button name="action_set_refused" string="Refused" type="object" icon="fa-times"/>   
+                <button name="action_set_accepted" string="Accepted" type="object" icon="fa-check" 
+                invisible="status != False"/>
+                <button name="action_set_refused" string="Refused" type="object" icon="fa-times" 
+                invisible="status != False"/>   
                 <field name="date_deadline"/>
             </list>
         </field>

estate/views/estate_property_tag_views.xml

@@ -17,7 +17,7 @@
         <field name="name">estate.property.tag.list</field>
         <field name="model">estate.property.tag</field>
         <field name="arch" type="xml">
-            <list string="Property Types">
+            <list string="Property Types" editable="bottom">
                 <field name="name"/>
                 <field name="description"/>
             </list>

estate/views/estate_property_type_views.xml

@@ -18,6 +18,7 @@
         <field name="model">estate.property.type</field>
         <field name="arch" type="xml">
             <list string="Property Types">
+                <field name="sequence" widget="handle"/>
                 <field name="name"/>
             </list>
         </field>
@@ -30,9 +31,28 @@
         <field name="arch" type="xml">
             <form string="Property Type">
                 <sheet>
-                    <group>
+                    <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"/>
-                    </group>
+                    </h1>
+                    <notebook>
+                        <page string="Properties">
+                            <field name="property_ids">
+                                <list>
+                                    <field name="name"/>
+                                    <field name="expected_price"/>
+                                    <field name="state"/>
+                                </list>
+                            </field>
+                        </page>
+                    </notebook>
                 </sheet>
             </form>
         </field>