[ADD] real_estate: estate: enhance UI with sprinkles improvements

- Added statusbar widget and inline list views for better usability.
- Defined default ordering on estate models.
- Improved views with decorations, options, and conditional behavior.

Author: rugot-odoo

real_estate/models/real_estate_properties.py

@@ -37,7 +37,6 @@ class real_estate(models.Model):
     offer_ids = fields.One2many(
         "real.estate.property.offer", "property_id", string="Offers")
     total_area = fields.Float(compute="_compute_total", store=True)
-    total_area_square = fields.Float(compute="_compute_total_area_square", store=True)
     best_price = fields.Float(
         string="Best Offer",
         compute="_compute_best_price",
@@ -96,11 +95,6 @@ def _compute_total(self):
         for record in self:
             record.total_area = (record.living_area or 0) + (record.garden_area or 0)
 
-    @api.depends("total_area")
-    def _compute_total_area_square(self):
-        for record in self:
-            record.total_area_square = (record.total_area or 0) ** 2
-
     @api.onchange('garden')
     def _onchange_garden(self):
         if self.garden:
@@ -111,10 +105,12 @@ def _onchange_garden(self):
             self.garden_orientation = False
 
     @api.ondelete(at_uninstall=False)
-    def _unlink_if_accepted_offer(self):
-        accepted_offer = self.offer_ids.filtered_domain([('status', '=', 'accepted')])
-        if accepted_offer:
-            raise UserError("Can't delete an active record!")
+    def _check_property_delete(self):
+        for record in self:
+            if record.stage not in ('new', 'cancelled'):
+                raise UserError(
+                    "You can only delete properties in New or Cancelled state."
+                )
 
     # @api.depends('create_date')
     # def _compute_create_date_ist(self):

real_estate/models/real_estate_properties_maintenance_request.py

@@ -19,10 +19,10 @@ class real_estate_properties_maintenance_request(models.Model):
     @api.onchange('status')
     def _check_cost_on_accepted_status(self):
         if self.status == 'approved' and self.cost <= 0:
-            raise UserError(" Maintenance Request Approved cost must be greater than 0")
+            raise UserError("Approved cost must be greater than 0")
 
     @api.ondelete(at_uninstall=False)
     def _unlink_if_maintenance_request_not_done(self):
         maintenace_request = self.filtered_domain([('status', '!=', 'done')])
         if maintenace_request:
-            raise UserError("Can't delete an active Maintenance Request Record.")
+            raise UserError("Can't delete an active Maintenance Request Record!")

real_estate/models/real_estate_property_offer.py

@@ -7,9 +7,14 @@
 class real_estate_property_offer(models.Model):
     _name = 'real.estate.property.offer'
     _description = 'Real Estate Property Offer'
+    _order = "price desc"
 
     price = fields.Float(required=True)
-    property_id = fields.Many2one('real.estate', string='Property', ondelete='cascade')
+    property_id = fields.Many2one('real.estate', string='Property', ondelete='restrict')
+    property_type_id = fields.Many2one(
+        related="property_id.property_type_id",
+        store=True
+    )
     status = fields.Selection([
         ('accepted', 'Accepted'),
         ('refused', 'Refused'),
@@ -27,6 +32,23 @@ class real_estate_property_offer(models.Model):
         'The offer price must be strictly positive.',
     )
 
+    @api.model
+    def create(self, vals):
+        # property_id = vals.get('property_id')
+        # price = vals.get('price')
+        # if property_id and price:
+        #     property_rec = self.env['estate.property'].browse(property_id)
+        #     if property_rec.offer_ids:
+        #         max_offer = max(property_rec.offer_ids.mapped('price'))
+        #         if price < max_offer:
+        #             raise UserError(
+        #                 "The offer must be higher than existing offers."
+        #             )
+        offer = super().create(vals)
+        if offer.property_id and offer.property_id.stage == 'new':
+            offer.property_id.stage = 'offer_received'
+        return offer
+
     @api.depends('create_date', 'validity')
     def _compute_date_deadline(self):
         for offer in self:
@@ -40,6 +62,12 @@ def _inverse_date_deadline(self):
             if offer.create_date and offer.date_deadline:
                 offer.validity = (offer.date_deadline - offer.create_date.date()).days
 
+    @api.ondelete(at_uninstall=False)
+    def _unlink_if_accepted_offer(self):
+        accepted_offer = self.filtered_domain([('status', '=', 'accepted')])
+        if accepted_offer:
+            raise UserError("Can't delete an active record!")
+
     def action_accept(self):
         # accepted_offer = self.search([
         #     ('property_id', '=', self.property_id.id),
@@ -51,14 +79,24 @@ def action_accept(self):
         accepted_offer = self.property_id.offer_ids.filtered_domain([
             ('status', '=', 'accepted')
         ])
+        maintenace_request = self.property_id.maintenance_request_ids.filtered_domain([('status', '!=', 'done')])
         if accepted_offer:
             raise UserError(
-                "Only one offer can be accepted for a property."
-            )
+                "Only one offer can be accepted for a property.")
+        if maintenace_request:
+            raise UserError("Property cannot be sold , there is any maintenance request not done")
         self.status = 'accepted'
-        self.property_id.selling_price = self.price
-        self.property_id.stage = 'sold'
-        self.property_id.buyer_id = self.partner_id.id
+        self.property_id.write({
+            'selling_price': self.price,
+            'stage': 'sold',
+            'buyer_id': self.partner_id.id,
+        })
+        refused_offer = self.property_id.offer_ids.filtered_domain([
+            ('id', '!=', 'self.id'),
+            ('status', '!=', 'accepted')
+        ])
+        for refuse in refused_offer:
+            refuse.status = 'refused'
 
     def action_refuse(self):
         self.status = 'refused'

real_estate/models/real_estate_property_type.py

@@ -4,14 +4,28 @@
 class RealEstateTag(models.Model):
     _name = 'real.estate.property.type'
     _description = 'Real Estate Property Type'
+    _order = "sequence, name desc"
 
     name = fields.Char(required=True)
+    sequence = fields.Integer(string="Sequence", default=10)
     property_ids = fields.One2many(
         'real.estate',
         'property_type_id',
         string='Properties'
     )
+    offer_ids = fields.One2many(
+        "real.estate.property.offer",
+        "property_type_id",
+        string="Offers"
+    )
     _unique_type_name = models.Constraint(
         'UNIQUE(name)',
         'The property type name must be unique.',
     )
+    offer_count = fields.Integer(
+        compute="_compute_offer_count"
+    )
+
+    def _compute_offer_count(self):
+        for record in self:
+            record.offer_count = len(record.offer_ids)

real_estate/models/real_estate_tag.py

@@ -1,11 +1,18 @@
+from random import randint
+
 from odoo import fields, models
 
 
 class RealEstateTag(models.Model):
     _name = 'real.estate.tag'
     _description = 'Real Estate Tag'
+    _order = "name desc"
+
+    def _get_default_color(self):
+        return randint(1, 11)
 
     name = fields.Char(required=True)
+    color = fields.Integer(default=_get_default_color)
     _unique_tag_name = models.Constraint(
         'UNIQUE(name)',
         'The property tag name must be unique.',

real_estate/views/real_estate_properties_offer_views.xml

@@ -4,24 +4,25 @@
         <field name="name">real.estate.property.offer.view.tree</field>
         <field name="model">real.estate.property.offer</field>
         <field name="arch" type="xml">
-            <list>
+            <list decoration-danger="status == 'refused'"
+                  decoration-success="status == 'accepted'">
                 <field name="price"/>
                 <field name="partner_id"/>
                 <field name="validity"/>
                 <field name="date_deadline"/>
                 <button
-                    name="action_accept"
-                    type="object"
-                    icon="fa-check"
-                    title="accept"
-                    invisible="status == 'refused'"
+                        name="action_accept"
+                        type="object"
+                        icon="fa-check"
+                        title="accept"
+                        invisible="status == 'refused'"
                 />
                 <button
-                    name="action_refuse"
-                    type="object"
-                    icon="fa-times"
-                    title="refused"
-                    invisible="status == 'accepted'"
+                        name="action_refuse"
+                        type="object"
+                        icon="fa-times"
+                        title="refused"
+                        invisible="status == 'refused'"
                 />
                 <field name="status" readonly="1"/>
             </list>
@@ -39,6 +40,7 @@
                         <field name="partner_id"/>
                         <field name="status"/>
                         <field name="property_id"/>
+                        <field name="property_type_id"/>
                         <field name="validity"/>
                         <field name="date_deadline"/>
                     </group>
@@ -53,4 +55,11 @@
         <field name="view_mode">list,form</field>
     </record>
 
+    <record id="real_estate_property_offer_action_view_state_button" model="ir.actions.act_window">
+        <field name="name">Property Offer</field>
+        <field name="res_model">real.estate.property.offer</field>
+        <field name="view_mode">list</field>
+        <field name="domain">[('property_type_id', '=', active_id)]</field>
+    </record>
+
 </odoo>

real_estate/views/real_estate_properties_views.xml

@@ -75,7 +75,6 @@
                                 <group string="Area Details">
                                     <field name="living_area"/>
                                     <field name="total_area"/>
-                                    <field name="total_area_square"/>
                                 </group>
                                 <group string="Interior Details">
                                     <field name="bedrooms"/>
@@ -140,7 +139,7 @@
                         string="Garden orientation"
                         name="name"
                         domain="['|',('garden_orientation', '=' , 'north'),
-                    ('garden_orientation', '=' , 'south')]"
+                                ('garden_orientation', '=' , 'south')]"
                 />
                 <group>
                     <filter

real_estate/views/real_estate_property_type.xml

@@ -6,6 +6,7 @@
         <field name="arch" type="xml">
             <list>
                 <field name="name"/>
+                <field name="sequence" widget="handle"/>
             </list>
         </field>
     </record>
@@ -16,14 +17,25 @@
         <field name="arch" type="xml">
             <form>
                 <sheet>
+                    <div class="oe_button_box" name="button_box">
+                        <button name="%(real_estate_property_offer_action_view_state_button)d"
+                                type="action"
+                                icon="fa-tag">
+                            <div class="o_form_field o_stat_info">
+                                <span class="o_stat_value">
+                                    <field name="offer_count" string="Offers" widget="statinfo"/>
+                                </span>
+                            </div>
+                        </button>
+                    </div>
                     <group>
                         <field name="name"/>
                     </group>
                     <notebook>
                         <page string="Property Details">
                             <group>
                                 <field name="property_ids">
-                                    <list editable="bottom">
+                                    <list editable="bottom" default_order="expected_price desc">
                                         <field name="name"/>
                                         <field name="expected_price"/>
                                         <field name="stage"/>

real_estate/views/real_estate_tag_views.xml

@@ -6,6 +6,7 @@
         <field name="arch" type="xml">
             <list>
                 <field name="name"/>
+                 <field name="color"/>
             </list>
         </field>
     </record>
@@ -18,11 +19,13 @@
                 <sheet>
                     <group>
                         <field name="name"/>
+                        <field name="color" widget="color_picker"/>
                     </group>
                 </sheet>
             </form>
         </field>
     </record>
+
     <record id="real_estate_tag_action_view" model="ir.actions.act_window">
         <field name="name">Property Tag</field>
         <field name="res_model">real.estate.tag</field>