[IMP] estate: fixed state button layout.

added onchange on offer_ids if  len(offer_ids) is zero then will change
state to 'new'.
fixed state button layout was in header first shifted to context menu.
added sold button disabled when there is no offer accepted and
if offer is accepted then  will show in highlight color.

Author: danal-odoo

estate/models/estate_property.py

@@ -47,7 +47,7 @@ class EstateProperty(models.Model):
         copy=False,
         default="new",
     )
-    property_type_id = fields.Many2one("estate.property.type", string="Property Type")
+    property_type_id = fields.Many2one("estate.property.type", string="Property Type", required=True)
     customer = fields.Many2one("res.partner", string="Customer", copy=False)
     salesperson = fields.Many2one(
         "res.users", string="Salesperson", default=lambda self: self.env.user
@@ -86,6 +86,11 @@ def _onchange_garden(self):
         else:
             self.garden_area = 0
             self.garden_orientation = None
+    
+    @api.onchange("offer_ids")
+    def _onchange_offer_ids(self):
+        if len(self.offer_ids) == 0:
+            self.state = "new"
 
     def action_sold_property(self):
         for record in self:

estate/views/estate_property_type_views.xml

@@ -16,15 +16,23 @@
         <field name="model">estate.property.type</field>
         <field name="arch" type="xml">
             <form name="Property Type List">
-                <header>
-                    <div class="oe_stat_button">
-                        <button type="action" name="%(estate.action_estate_property_offer)d" icon="fa-list">
-                            Offers:
-                            <field name="offer_count"/>
+                <sheet>
+                    <div class="oe_button_box" name="button_box">
+                        <button class="oe_stat_button" 
+                                type="action" 
+                                name="%(estate.action_estate_property_offer)d" 
+                                icon="fa-bars" 
+                                invisible="offer_count == 0">
+                            <div class="o_stat_info">
+                                <span class="o_stat_value">
+                                    <field name="offer_count"/>
+                                </span>
+                                <span class="o_stat_text">
+                                    Offers
+                                </span>
+                            </div>
                         </button>
                     </div>
-                </header>
-                <sheet>
                     <h1>
                         <field name="name"/>
                     </h1>

estate/views/estate_property_views.xml

@@ -18,27 +18,29 @@
         <field name="arch" type="xml">
             <form string="Property">
                 <header>
-                    <button name="action_sold_property" type="object" string="Sold" invisible="state in ('sold','cancelled')"/>
-                    <button name="action_cancel_offer" type="object" string="Cancel" invisible="state in ('sold','cancelled')"/>
+                    <button name="action_sold_property" type="object" string="Sold" class="oe_highlight" invisible="state in ('sold','cancelled','new','offer_received')"/>
+                    <button name="action_sold_property" type="object" string="Sold" class="disabled" invisible="state in ('sold','cancelled','offer_accepted')"/>
+                    <button name="action_cancel_offer" type="object" string="Cancel" invisible="state in ('sold','cancelled')" 
+                            confirm="Are you sure to Cancel this Property? You cannot revert this back."/>
                     <field name="state" widget="statusbar"/>
-                </header>
+                </header>   
                 <sheet>
-                    <h1>
+                    <h1>    
                         <field name="name" string="Title"/>
                     </h1>
                     <field name="tag_ids" widget="many2many_tags" options="{'color_field': 'color'}"/>
                     <hr/>
                     <group>
                         <group>
                             <field name="postcode"/>    
-                            <field name="date_avaliability" string="Avaliable From"/>
-                        </group>
-                        <group>
-                            <field name="property_type_id" options="{'no_create': true}"/>
                             <field name="expected_price"/>
                             <field name="selling_price"/>
                             <field name="best_price" string="Best Offer"/>
                         </group>
+                        <group>
+                            <field name="property_type_id" options="{'no_create': true}"/>
+                            <field name="date_avaliability" string="Avaliable From"/>
+                        </group>
                     </group>
                     <notebook>
                         <page string="Description">
@@ -49,6 +51,8 @@
                                 <field name="living_area" string="Living Area(sqm)"/>
                                 <field name="facades"/>
                                 <field name="garage"/>
+                            </group>
+                            <group>
                                 <field name="garden"/>
                                 <field name="garden_area" string="Garden Area(sqm)" invisible="not garden"/>
                                 <field name="garden_orientation" invisible="not garden"/>

estate_account/models/estate_property.py

@@ -8,8 +8,6 @@ class EstatePropertyInherit(models.Model):
 
     def action_sold_property(self):
         for record in self:
-            if record.selling_price == 0:
-                raise UserError("Cannot sell Property without any accepted offer")
             self.env["account.move"].create(
                 {
                     "partner_id": record.customer.id,