[FIX] estate: code formatting and rename compute method

Improved code readability by applying consistent formatting.
Renamed compute method for best_price to follow naming conventions. changed discription position after name in manifest file.

Author: danal-odoo

estate/__manifest__.py

@@ -1,15 +1,15 @@
 #!/usr/bin/env python3
 {
     "name": "Real Estate",
+    "description": """
+    Real Estate Module to Buy and Sell Your Real Estate with Ease.
+    """,
     "version": "1.0",
     "depends": ["base"],
     "author": "danal",
     "category": "Category",
     "application": True,
     "license": "LGPL-3",
-    "description": """
-    Real Estate Module to Buy and Sell Your Real Estate with Ease.
-    """,
     "data": [
         "security/ir.model.access.csv",
         "views/estate_property_views.xml",

estate/models/estate_property.py

@@ -15,7 +15,7 @@ class EstateProperty(models.Model):
         "CHECK(selling_price >= 0)", "A property selling price must be positive."
     )
 
-    name = fields.Char(required=True, default="Unknown")
+    name = fields.Char(required=True)
     description = fields.Text()
     postcode = fields.Char()
     date_avaliability = fields.Date(
@@ -29,7 +29,6 @@ class EstateProperty(models.Model):
     garage = fields.Boolean()
     garden = fields.Boolean()
     garden_area = fields.Integer()
-    last_seen = fields.Datetime("Last Seen", default=fields.Datetime.now)
     garden_orientation = fields.Selection(
         selection=[
             ("north", "North"),
@@ -61,15 +60,15 @@ class EstateProperty(models.Model):
     tag_ids = fields.Many2many("estate.property.tag", string="Property Tags")
     offer_ids = fields.One2many("estate.property.offer", "property_id", string="Offer")
     total_area = fields.Integer(compute="_compute_area")
-    best_price = fields.Integer(compute="_compute_highest")
+    best_price = fields.Integer(compute="_compute_best_price")
 
     @api.depends("living_area", "garden_area")
     def _compute_area(self):
         for record in self:
             record.total_area = record.living_area + record.garden_area
 
     @api.depends("offer_ids.price")
-    def _compute_highest(self):
+    def _compute_best_price(self):
         for record in self:
             if not record.mapped("offer_ids.price"):
                 record.best_price = 0

estate/models/estate_property_offer.py

@@ -9,7 +9,7 @@ class EstatePropertyOffer(models.Model):
     _check_offer_price = models.Constraint(
         "CHECK(price > 0)", "An offer price must be strictly positive"
     )
-    
+
     price = fields.Float()
     status = fields.Selection(
         selection=[("accepted", "Accepted"), ("refused", "Refused")], copy=False
@@ -44,6 +44,5 @@ def action_accept(self):
     def action_refuse(self):
         for record in self:
             record.status = "refused"
-            record.property_id.selling_price = 0.00
             record.property_id.customer = None
         return True