[CLN] estate: remove some unnecessary code

Removed some unnecessary code but
- TODO double check validation stuff and overwrite this commit

Author: robal-odoo

estate/models/estate_property_offer.py

@@ -47,11 +47,8 @@ def create(self, vals):
 
     # Public methods
     def action_accept(self) -> bool:
-        # TODO validation
+        # TODO double check validation
         for record in self:
-            if record.status == "accepted":
-                raise UserError(_("This offer has already been accepted."))
-
             if record.property_id.offer_ids.filtered(lambda r: r.status == "accepted"):
                 raise UserError(_("Cannot accept this offer because another offer has already been accepted for the property."))
 
@@ -62,13 +59,8 @@ def action_accept(self) -> bool:
         return True
 
     def action_refuse(self) -> bool:
-        # TODO validation
+        # TODO double check validation
         for record in self:
-            if record.status == "accepted":
-                record.property_id.selling_price = 0
-                record.property_id.buyer_id = None
-                record.property_id.state = "offer_received"
-
             record.status = "refused"
         return True
 

estate/views/estate_property_views.xml

@@ -59,7 +59,7 @@
                             </group>
                         </page>
                         <page string="Offers">
-                            <!-- TODO: backend validation ? -->
+                            <!-- TODO: double check backend validation ? -->
                             <field name="offer_ids" readonly="state in ['offer_accepted', 'sold', 'cancelled']"/>
                         </page>
                         <page string="Other Info">